Change in vdsm[master]: osutils: Start the osutils module

2016-08-11 Thread automation
gerrit-hooks has posted comments on this change. Change subject: osutils: Start the osutils module .. Patch Set 7: * Update tracker: IGNORE, no Bug-Url found * Set MODIFIED::IGNORE, no Bug-Url found. -- To view, visit

Change in vdsm[master]: osutils: Start the osutils module

2016-08-11 Thread alitke
Adam Litke has submitted this change and it was merged. Change subject: osutils: Start the osutils module .. osutils: Start the osutils module We use to have code calling os.close() using utils.NoIntrCall. If os.close(fd) was

Change in vdsm[master]: osutils: Start the osutils module

2016-08-10 Thread nsoffer
Nir Soffer has posted comments on this change. Change subject: osutils: Start the osutils module .. Patch Set 6: -Code-Review Continuous-Integration+1 -- To view, visit https://gerrit.ovirt.org/61142 To unsubscribe, visit

Change in vdsm[master]: osutils: Start the osutils module

2016-08-10 Thread nsoffer
Nir Soffer has posted comments on this change. Change subject: osutils: Start the osutils module .. Patch Set 6: Code-Review+1 CI seems to be sick, tests pass locally. -- To view, visit https://gerrit.ovirt.org/61142 To

Change in vdsm[master]: osutils: Start the osutils module

2016-08-10 Thread automation
gerrit-hooks has posted comments on this change. Change subject: osutils: Start the osutils module .. Patch Set 6: * Update tracker: IGNORE, no Bug-Url found * Check Bug-Url::WARN, no bug url found, make sure header matches

Change in vdsm[master]: osutils: Start the osutils module

2016-08-08 Thread piotr . kliczewski
Piotr Kliczewski has posted comments on this change. Change subject: osutils: Start the osutils module .. Patch Set 5: Code-Review+1 -- To view, visit https://gerrit.ovirt.org/61142 To unsubscribe, visit

Change in vdsm[master]: osutils: Start the osutils module

2016-08-05 Thread automation
gerrit-hooks has posted comments on this change. Change subject: osutils: Start the osutils module .. Patch Set 5: * Update tracker: IGNORE, no Bug-Url found * Check Bug-Url::WARN, no bug url found, make sure header matches

Change in vdsm[master]: osutils: Start the osutils module

2016-07-22 Thread alitke
Adam Litke has posted comments on this change. Change subject: osutils: Start the osutils module .. Patch Set 4: Code-Review+2 (1 comment) https://gerrit.ovirt.org/#/c/61142/4/tests/osutils_test.py File tests/osutils_test.py:

Change in vdsm[master]: osutils: Start the osutils module

2016-07-22 Thread alitke
Adam Litke has posted comments on this change. Change subject: osutils: Start the osutils module .. Patch Set 4: (1 comment) https://gerrit.ovirt.org/#/c/61142/4/tests/osutils_test.py File tests/osutils_test.py: Line 27:

Change in vdsm[master]: osutils: Start the osutils module

2016-07-21 Thread nsoffer
Nir Soffer has posted comments on this change. Change subject: osutils: Start the osutils module .. Patch Set 4: Verified+1 -- To view, visit https://gerrit.ovirt.org/61142 To unsubscribe, visit

Change in vdsm[master]: osutils: Start the osutils module

2016-07-21 Thread nsoffer
Nir Soffer has posted comments on this change. Change subject: osutils: Start the osutils module .. Patch Set 1: (1 comment) https://gerrit.ovirt.org/#/c/61142/1/tests/osutils_test.py File tests/osutils_test.py: Line 33:

Change in vdsm[master]: osutils: Start the osutils module

2016-07-21 Thread automation
gerrit-hooks has posted comments on this change. Change subject: osutils: Start the osutils module .. Patch Set 4: * Update tracker: IGNORE, no Bug-Url found * Check Bug-Url::WARN, no bug url found, make sure header matches

Change in vdsm[master]: osutils: Start the osutils module

2016-07-21 Thread nsoffer
Nir Soffer has posted comments on this change. Change subject: osutils: Start the osutils module .. Patch Set 1: (1 comment) https://gerrit.ovirt.org/#/c/61142/1/lib/vdsm/osutils.py File lib/vdsm/osutils.py: Line 37:

Change in vdsm[master]: osutils: Start the osutils module

2016-07-21 Thread ybronhei
Yaniv Bronhaim has posted comments on this change. Change subject: osutils: Start the osutils module .. Patch Set 1: (2 comments) https://gerrit.ovirt.org/#/c/61142/1/lib/vdsm/osutils.py File lib/vdsm/osutils.py: Line 37:

Change in vdsm[master]: osutils: Start the osutils module

2016-07-21 Thread fromani
Francesco Romani has posted comments on this change. Change subject: osutils: Start the osutils module .. Patch Set 3: Code-Review+1 -- To view, visit https://gerrit.ovirt.org/61142 To unsubscribe, visit

Change in vdsm[master]: osutils: Start the osutils module

2016-07-20 Thread nsoffer
Nir Soffer has posted comments on this change. Change subject: osutils: Start the osutils module .. Patch Set 3: Verified+1 -- To view, visit https://gerrit.ovirt.org/61142 To unsubscribe, visit

Change in vdsm[master]: osutils: Start the osutils module

2016-07-20 Thread automation
gerrit-hooks has posted comments on this change. Change subject: osutils: Start the osutils module .. Patch Set 3: * Update tracker: IGNORE, no Bug-Url found * Check Bug-Url::WARN, no bug url found, make sure header matches

Change in vdsm[master]: osutils: Start the osutils module

2016-07-20 Thread nsoffer
Nir Soffer has posted comments on this change. Change subject: osutils: Start the osutils module .. Patch Set 1: (3 comments) https://gerrit.ovirt.org/#/c/61142/1/lib/vdsm/osutils.py File lib/vdsm/osutils.py: Line 37:

Change in vdsm[master]: osutils: Start the osutils module

2016-07-20 Thread automation
gerrit-hooks has posted comments on this change. Change subject: osutils: Start the osutils module .. Patch Set 2: * Update tracker: IGNORE, no Bug-Url found * Check Bug-Url::WARN, no bug url found, make sure header matches

Change in vdsm[master]: osutils: Start the osutils module

2016-07-20 Thread ybronhei
Yaniv Bronhaim has posted comments on this change. Change subject: osutils: Start the osutils module .. Patch Set 1: (3 comments) https://gerrit.ovirt.org/#/c/61142/1/lib/vdsm/osutils.py File lib/vdsm/osutils.py: Line 37:

Change in vdsm[master]: osutils: Start the osutils module

2016-07-20 Thread automation
gerrit-hooks has posted comments on this change. Change subject: osutils: Start the osutils module .. Patch Set 1: * Update tracker: IGNORE, no Bug-Url found * Check Bug-Url::WARN, no bug url found, make sure header matches

Change in vdsm[master]: osutils: Start the osutils module

2016-07-20 Thread nsoffer
Nir Soffer has posted comments on this change. Change subject: osutils: Start the osutils module .. Patch Set 1: Verified+1 Verified with the tests. -- To view, visit https://gerrit.ovirt.org/61142 To unsubscribe, visit

Change in vdsm[master]: osutils: Start the osutils module

2016-07-20 Thread nsoffer
Nir Soffer has uploaded a new change for review. Change subject: osutils: Start the osutils module .. osutils: Start the osutils module We use to have code calling os.close() using utils.NoIntrCall. If os.close(fd) was