Change in vdsm[master]: bootstrap: sync logging between two phases

Alon Bar-Lev alonbl at redhat.com
Sat Aug 11 21:00:25 UTC 2012


Alon Bar-Lev has posted comments on this change.

Change subject: bootstrap: sync logging between two phases
......................................................................


Patch Set 1: (2 inline comments)

....................................................
File vds_bootstrap/vds_bootstrap_complete.py
Line 45:     print ("<BSTRAP component='INIT' status='FAIL' message='%s'/>" % message)
Every time we finish a stage the script folder is removed, as no scripts are stored in different location. Actually this test can be removed from both stages, as if we use tarball we can relay on whole content to be transferred.

....................................................
File vds_bootstrap/vds_bootstrap.py
Line 59:     LOGDIR="/tmp"
In this case, ok.

--
To view, visit http://gerrit.ovirt.org/7089
To unsubscribe, visit http://gerrit.ovirt.org/settings

Gerrit-MessageType: comment
Gerrit-Change-Id: Iac3faeeaa9e85d92e1d8bcec6517745a665a3da8
Gerrit-PatchSet: 1
Gerrit-Project: vdsm
Gerrit-Branch: master
Gerrit-Owner: Alon Bar-Lev <alonbl at redhat.com>
Gerrit-Reviewer: Alon Bar-Lev <alonbl at redhat.com>
Gerrit-Reviewer: Barak Azulay <bazulay at redhat.com>
Gerrit-Reviewer: Dan Kenigsberg <danken at redhat.com>
Gerrit-Reviewer: Doron Fediuck <dfediuck at redhat.com>
Gerrit-Reviewer: Douglas Schilling Landgraf <dougsland at redhat.com>
Gerrit-Reviewer: Eyal Edri <eedri at redhat.com>
Gerrit-Reviewer: Juan Hernandez <juan.hernandez at redhat.com>


More information about the vdsm-patches mailing list