Change in vdsm[master]: safelease: Unbreak safelease on systemd

2015-09-15 Thread danken
Dan Kenigsberg has submitted this change and it was merged. Change subject: safelease: Unbreak safelease on systemd .. safelease: Unbreak safelease on systemd spmprotect.sh depends on Vdsm pid file for fencing Vdsm when the SPM

Change in vdsm[master]: safelease: Unbreak safelease on systemd

2015-09-15 Thread automation
automat...@ovirt.org has posted comments on this change. Change subject: safelease: Unbreak safelease on systemd .. Patch Set 5: * Update tracker::#1222564::OK * Set MODIFIED::bug 1222564#1222564IGNORE, not oVirt prod b

Change in vdsm[master]: safelease: Unbreak safelease on systemd

2015-09-14 Thread alitke
Adam Litke has posted comments on this change. Change subject: safelease: Unbreak safelease on systemd .. Patch Set 4: Code-Review+2 -- To view, visit https://gerrit.ovirt.org/45963 To unsubscribe, visit https://gerrit.ovirt.o

Change in vdsm[master]: safelease: Unbreak safelease on systemd

2015-09-12 Thread amureini
Allon Mureinik has posted comments on this change. Change subject: safelease: Unbreak safelease on systemd .. Patch Set 4: Code-Review+1 -- To view, visit https://gerrit.ovirt.org/45963 To unsubscribe, visit https://gerrit.ovi

Change in vdsm[master]: safelease: Unbreak safelease on systemd

2015-09-11 Thread nsoffer
Nir Soffer has posted comments on this change. Change subject: safelease: Unbreak safelease on systemd .. Patch Set 4: Verified+1 Verify that spmprotect.sh can send signals to vdsm, and vdsm received these signals. This is no

Change in vdsm[master]: safelease: Unbreak safelease on systemd

2015-09-11 Thread automation
automat...@ovirt.org has posted comments on this change. Change subject: safelease: Unbreak safelease on systemd .. Patch Set 4: * Update tracker::#1222564::OK * Check Bug-Url::OK * Check Public Bug::#1222564::OK, public bug *

Change in vdsm[master]: safelease: Unbreak safelease on systemd

2015-09-10 Thread nsoffer
Nir Soffer has posted comments on this change. Change subject: safelease: Unbreak safelease on systemd .. Patch Set 3: (2 comments) https://gerrit.ovirt.org/#/c/45963/3//COMMIT_MSG Commit Message: Line 7: safelease: Unbreak s

Change in vdsm[master]: safelease: Unbreak safelease on systemd

2015-09-10 Thread ahino
Ala Hino has posted comments on this change. Change subject: safelease: Unbreak safelease on systemd .. Patch Set 3: Code-Review+1 (2 comments) https://gerrit.ovirt.org/#/c/45963/3//COMMIT_MSG Commit Message: Line 11: try tri

Change in vdsm[master]: safelease: Unbreak safelease on systemd

2015-09-10 Thread ybronhei
Yaniv Bronhaim has posted comments on this change. Change subject: safelease: Unbreak safelease on systemd .. Patch Set 3: Code-Review+1 (1 comment) https://gerrit.ovirt.org/#/c/45963/3//COMMIT_MSG Commit Message: Line 13: sy

Change in vdsm[master]: safelease: Unbreak safelease on systemd

2015-09-09 Thread nsoffer
Nir Soffer has posted comments on this change. Change subject: safelease: Unbreak safelease on systemd .. Patch Set 3: This version removes trailing whitespace. -- To view, visit https://gerrit.ovirt.org/45963 To unsubscribe,

Change in vdsm[master]: safelease: Unbreak safelease on systemd

2015-09-09 Thread automation
automat...@ovirt.org has posted comments on this change. Change subject: safelease: Unbreak safelease on systemd .. Patch Set 3: * Update tracker::#1222564::OK * Check Bug-Url::OK * Check Public Bug::#1222564::OK, public bug *

Change in vdsm[master]: safelease: Unbreak safelease on systemd

2015-09-09 Thread nsoffer
Nir Soffer has posted comments on this change. Change subject: safelease: Unbreak safelease on systemd .. Patch Set 1: This version replace the pid file with command line argument and separate the unrelated argument parsing re

Change in vdsm[master]: safelease: Unbreak safelease on systemd

2015-09-09 Thread automation
automat...@ovirt.org has posted comments on this change. Change subject: safelease: Unbreak safelease on systemd .. Patch Set 2: * Update tracker::#1222564::OK * Check Bug-Url::OK * Check Public Bug::#1222564::OK, public bug *

Change in vdsm[master]: safelease: Unbreak safelease on systemd

2015-09-09 Thread nsoffer
Nir Soffer has posted comments on this change. Change subject: safelease: Unbreak safelease on systemd .. Patch Set 1: (1 comment) https://gerrit.ovirt.org/#/c/45963/1/vdsm/vdsm File vdsm/vdsm: Line 151: pid = str(os.

Change in vdsm[master]: safelease: Unbreak safelease on systemd

2015-09-09 Thread nsoffer
Nir Soffer has posted comments on this change. Change subject: safelease: Unbreak safelease on systemd .. Patch Set 1: Verified+1 Works - but the timeouts used by safelease helper scripts are not good. 1. spmprotect.sh send SI

Change in vdsm[master]: safelease: Unbreak safelease on systemd

2015-09-09 Thread Yaniv Bronhaim
Yaniv Bronhaim has posted comments on this change. Change subject: safelease: Unbreak safelease on systemd .. Patch Set 1: Code-Review-1 -- To view, visit https://gerrit.ovirt.org/45963 To unsubscribe, visit https://gerrit.ovi

Change in vdsm[master]: safelease: Unbreak safelease on systemd

2015-09-09 Thread automation
automat...@ovirt.org has posted comments on this change. Change subject: safelease: Unbreak safelease on systemd .. Patch Set 1: * Update tracker::#1222564::OK * Check Bug-Url::OK * Check Public Bug::#1222564::OK, public bug *

Change in vdsm[master]: safelease: Unbreak safelease on systemd

2015-09-09 Thread nsoffer
Nir Soffer has uploaded a new change for review. Change subject: safelease: Unbreak safelease on systemd .. safelease: Unbreak safelease on systemd spmprotect.sh depends on Vdsm pid file for fencing Vdsm when the SPM lease canno