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@redhat.com Gerrit-Reviewer: Alon Bar-Lev alonbl@redhat.com Gerrit-Reviewer: Barak Azulay bazulay@redhat.com Gerrit-Reviewer: Dan Kenigsberg danken@redhat.com Gerrit-Reviewer: Doron Fediuck dfediuck@redhat.com Gerrit-Reviewer: Douglas Schilling Landgraf dougsland@redhat.com Gerrit-Reviewer: Eyal Edri eedri@redhat.com Gerrit-Reviewer: Juan Hernandez juan.hernandez@redhat.com Gerrit-Reviewer: oVirt Jenkins CI Server