Change in vdsm[master]: bootstrap: survive missing ntpq

2012-08-30 Thread danken
Dan Kenigsberg has posted comments on this change. Change subject: bootstrap: survive missing ntpq .. Patch Set 1: I would prefer that you didn't submit this (1 inline comment)

Change in vdsm[master]: bootstrap: survive missing ntpq

2012-08-30 Thread Alon Bar-Lev
Alon Bar-Lev has posted comments on this change. Change subject: bootstrap: survive missing ntpq .. Patch Set 1: (1 inline comment) File vdsm_reg/deployUtil.py.in Line 693:

Change in vdsm[master]: bootstrap: survive missing ntpq

2012-08-30 Thread apahim
Amador Pahim has posted comments on this change. Change subject: bootstrap: survive missing ntpq .. Patch Set 3: Looks good to me, but someone else must approve -- To view, visit http://gerrit.ovirt.org/7597 To unsubscribe,

Change in vdsm[master]: bootstrap: survive missing ntpq

2012-08-30 Thread danken
Dan Kenigsberg has posted comments on this change. Change subject: bootstrap: survive missing ntpq .. Patch Set 3: (1 inline comment) File vdsm_reg/deployUtil.py.in Line

Change in vdsm[master]: bootstrap: survive missing ntpq

2012-08-30 Thread apahim
Amador Pahim has posted comments on this change. Change subject: bootstrap: survive missing ntpq .. Patch Set 3: No score Indeed. No need to the 1st if. -- To view, visit http://gerrit.ovirt.org/7597 To unsubscribe, visit

Change in vdsm[master]: bootstrap: survive missing ntpq

2012-08-30 Thread Alon Bar-Lev
Alon Bar-Lev has posted comments on this change. Change subject: bootstrap: survive missing ntpq .. Patch Set 3: (1 inline comment) File vdsm_reg/deployUtil.py.in Line 685:

Change in vdsm[master]: bootstrap: survive missing ntpq

2012-08-30 Thread Gerrit Code Review
oVirt Jenkins CI Server has posted comments on this change. Change subject: bootstrap: survive missing ntpq .. Patch Set 4: Build Successful http://jenkins.ovirt.info/job/patch_vdsm_unit_tests/777/ : SUCCESS -- To view,

Change in vdsm[master]: bootstrap: survive missing ntpq

2012-08-30 Thread apahim
Amador Pahim has posted comments on this change. Change subject: bootstrap: survive missing ntpq .. Patch Set 4: Looks good to me, but someone else must approve -- To view, visit http://gerrit.ovirt.org/7597 To unsubscribe,

Change in vdsm[master]: bootstrap: survive missing ntpq

2012-08-30 Thread dougsland
Douglas Schilling Landgraf has posted comments on this change. Change subject: bootstrap: survive missing ntpq .. Patch Set 4: Looks good to me, but someone else must approve -- To view, visit http://gerrit.ovirt.org/7597 To

Change in vdsm[master]: bootstrap: survive missing ntpq

2012-08-30 Thread bazulay
Barak Azulay has posted comments on this change. Change subject: bootstrap: survive missing ntpq .. Patch Set 4: Looks good to me, but someone else must approve -- To view, visit http://gerrit.ovirt.org/7597 To unsubscribe,

Change in vdsm[master]: bootstrap: survive missing ntpq

2012-08-30 Thread danken
Dan Kenigsberg has posted comments on this change. Change subject: bootstrap: survive missing ntpq .. Patch Set 4: Looks good to me, approved -- To view, visit http://gerrit.ovirt.org/7597 To unsubscribe, visit

Change in vdsm[master]: bootstrap: survive missing ntpq

2012-08-30 Thread danken
Dan Kenigsberg has submitted this change and it was merged. Change subject: bootstrap: survive missing ntpq .. bootstrap: survive missing ntpq Minimal fedora installation does not have ntpq. Change-Id:

Change in vdsm[master]: bootstrap: survive missing ntpq

2012-08-30 Thread danken
Dan Kenigsberg has posted comments on this change. Change subject: bootstrap: survive missing ntpq .. Patch Set 3: (1 inline comment) File vdsm_reg/deployUtil.py.in Line

Change in vdsm[master]: bootstrap: survive missing ntpq

2012-08-29 Thread Alon Bar-Lev
Alon Bar-Lev has uploaded a new change for review. Change subject: bootstrap: survive missing ntpq .. bootstrap: survive missing ntpq Minimal fedora installation does not have ntpq. Change-Id:

Change in vdsm[master]: bootstrap: survive missing ntpq

2012-08-29 Thread Alon Bar-Lev
Alon Bar-Lev has posted comments on this change. Change subject: bootstrap: survive missing ntpq .. Patch Set 1: Verified -- To view, visit http://gerrit.ovirt.org/7597 To unsubscribe, visit http://gerrit.ovirt.org/settings

Change in vdsm[master]: bootstrap: survive missing ntpq

2012-08-29 Thread dougsland
Douglas Schilling Landgraf has posted comments on this change. Change subject: bootstrap: survive missing ntpq .. Patch Set 1: I would prefer that you didn't submit this The vdsm.spec requires ntp, which provides ntpq. Any

Change in vdsm[master]: bootstrap: survive missing ntpq

2012-08-29 Thread Alon Bar-Lev
Alon Bar-Lev has posted comments on this change. Change subject: bootstrap: survive missing ntpq .. Patch Set 1: bootstrap is something completely different. It is sent from the engine pre to any other operation. As time set

Change in vdsm[master]: bootstrap: survive missing ntpq

2012-08-29 Thread Gerrit Code Review
oVirt Jenkins CI Server has posted comments on this change. Change subject: bootstrap: survive missing ntpq .. Patch Set 1: Build Successful http://jenkins.ovirt.info/job/patch_vdsm_unit_tests/764/ : SUCCESS -- To view,

Change in vdsm[master]: bootstrap: survive missing ntpq

2012-08-29 Thread dougsland
Douglas Schilling Landgraf has posted comments on this change. Change subject: bootstrap: survive missing ntpq .. Patch Set 1: Looks good to me, but someone else must approve Hi, I see the point and agree with your comments.