Change in vdsm[master]: clusterlock: Fix double close on EINTR

2016-08-11 Thread alitke
Adam Litke has submitted this change and it was merged. Change subject: clusterlock: Fix double close on EINTR .. clusterlock: Fix double close on EINTR Use new osutil.close_fd() to close lockfile properly without retrying if

Change in vdsm[master]: clusterlock: Fix double close on EINTR

2016-08-11 Thread automation
gerrit-hooks has posted comments on this change. Change subject: clusterlock: Fix double close on EINTR .. Patch Set 8: * Update tracker: IGNORE, no Bug-Url found * Set MODIFIED::IGNORE, no Bug-Url found. -- To view, visit

Change in vdsm[master]: clusterlock: Fix double close on EINTR

2016-08-10 Thread nsoffer
Nir Soffer has posted comments on this change. Change subject: clusterlock: Fix double close on EINTR .. Patch Set 7: Continuous-Integration+1 CI seems to be sick, tests pass locally. -- To view, visit

Change in vdsm[master]: clusterlock: Fix double close on EINTR

2016-08-10 Thread nsoffer
Nir Soffer has posted comments on this change. Change subject: clusterlock: Fix double close on EINTR .. Patch Set 7: Verified+1 -- To view, visit https://gerrit.ovirt.org/60539 To unsubscribe, visit

Change in vdsm[master]: clusterlock: Fix double close on EINTR

2016-08-10 Thread automation
gerrit-hooks has posted comments on this change. Change subject: clusterlock: Fix double close on EINTR .. Patch Set 7: * Update tracker: IGNORE, no Bug-Url found * Check Bug-Url::WARN, no bug url found, make sure header

Change in vdsm[master]: clusterlock: Fix double close on EINTR

2016-08-05 Thread automation
gerrit-hooks has posted comments on this change. Change subject: clusterlock: Fix double close on EINTR .. Patch Set 6: * Update tracker: IGNORE, no Bug-Url found * Check Bug-Url::WARN, no bug url found, make sure header

Change in vdsm[master]: clusterlock: Fix double close on EINTR

2016-07-22 Thread alitke
Adam Litke has posted comments on this change. Change subject: clusterlock: Fix double close on EINTR .. Patch Set 5: Code-Review+2 -- To view, visit https://gerrit.ovirt.org/60539 To unsubscribe, visit

Change in vdsm[master]: clusterlock: Fix double close on EINTR

2016-07-21 Thread automation
gerrit-hooks has posted comments on this change. Change subject: clusterlock: Fix double close on EINTR .. Patch Set 5: * Update tracker: IGNORE, no Bug-Url found * Check Bug-Url::WARN, no bug url found, make sure header

Change in vdsm[master]: clusterlock: Fix double close on EINTR

2016-07-20 Thread automation
gerrit-hooks has posted comments on this change. Change subject: clusterlock: Fix double close on EINTR .. Patch Set 4: * Update tracker: IGNORE, no Bug-Url found * Check Bug-Url::WARN, no bug url found, make sure header

Change in vdsm[master]: clusterlock: Fix double close on EINTR

2016-07-20 Thread automation
gerrit-hooks has posted comments on this change. Change subject: clusterlock: Fix double close on EINTR .. Patch Set 3: * Update tracker: IGNORE, no Bug-Url found * Check Bug-Url::WARN, no bug url found, make sure header

Change in vdsm[master]: clusterlock: Fix double close on EINTR

2016-07-20 Thread automation
gerrit-hooks has posted comments on this change. Change subject: clusterlock: Fix double close on EINTR .. Patch Set 2: * Update tracker: IGNORE, no Bug-Url found * Check Bug-Url::WARN, no bug url found, make sure header

Change in vdsm[master]: clusterlock: Fix double close on EINTR

2016-07-20 Thread nsoffer
Nir Soffer has posted comments on this change. Change subject: clusterlock: Fix double close on EINTR .. Patch Set 1: (2 comments) https://gerrit.ovirt.org/#/c/60539/1/lib/vdsm/storage/clusterlock.py File

Change in vdsm[master]: clusterlock: Fix double close on EINTR

2016-07-20 Thread alitke
Adam Litke has posted comments on this change. Change subject: clusterlock: Fix double close on EINTR .. Patch Set 1: (2 comments) https://gerrit.ovirt.org/#/c/60539/1/lib/vdsm/storage/clusterlock.py File

Change in vdsm[master]: clusterlock: Fix double close on EINTR

2016-07-11 Thread automation
gerrit-hooks has posted comments on this change. Change subject: clusterlock: Fix double close on EINTR .. Patch Set 1: * Update tracker: IGNORE, no Bug-Url found * Check Bug-Url::WARN, no bug url found, make sure header

Change in vdsm[master]: clusterlock: Fix double close on EINTR

2016-07-11 Thread nsoffer
Nir Soffer has uploaded a new change for review. Change subject: clusterlock: Fix double close on EINTR .. clusterlock: Fix double close on EINTR If os.close(fd) was interrupted by signal, we would try to close again - this may