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

danken at redhat.com danken at redhat.com
Tue Aug 14 22:04:51 UTC 2012


Dan Kenigsberg has posted comments on this change.

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


Patch Set 3: Looks good to me, but someone else must approve

(1 inline comment)

....................................................
File vds_bootstrap/vds_bootstrap_complete.py
Line 23: import sys
Line 24: import getopt
Line 25: import logging
Line 26: import tempfile
Line 27: from time import strftime
we try to have a newline between standard modules and ours.
Line 28: import deployUtil
Line 29: 
Line 30: VDSM_CONF_FILE = '/etc/vdsm/vdsm.conf'
Line 31: 


--
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: 3
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>
Gerrit-Reviewer: oVirt Jenkins CI Server


More information about the vdsm-patches mailing list