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@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