Change in vdsm[master]: properties: py3: properties.py and properties_test.py compli...

2016-10-07 Thread nsoffer
Nir Soffer has posted comments on this change. Change subject: properties: py3: properties.py and properties_test.py compliance .. Patch Set 3: Your new patches replace this patch, you can abandon it now. -- To view, visit

Change in vdsm[master]: properties: py3: properties.py and properties_test.py compli...

2016-09-12 Thread nsoffer
Nir Soffer has posted comments on this change. Change subject: properties: py3: properties.py and properties_test.py compliance .. Patch Set 3: (1 comment) https://gerrit.ovirt.org/#/c/63230/3/tests/properties_test.py File

Change in vdsm[master]: properties: py3: properties.py and properties_test.py compli...

2016-09-12 Thread nsoffer
Nir Soffer has posted comments on this change. Change subject: properties: py3: properties.py and properties_test.py compliance .. Patch Set 3: You can remove the module from the blacklist in the last patch, or just send

Change in vdsm[master]: properties: py3: properties.py and properties_test.py compli...

2016-09-12 Thread nsoffer
Nir Soffer has posted comments on this change. Change subject: properties: py3: properties.py and properties_test.py compliance .. Patch Set 3: (5 comments) https://gerrit.ovirt.org/#/c/63230/3/lib/vdsm/properties.py File

Change in vdsm[master]: properties: py3: properties.py and properties_test.py compli...

2016-09-12 Thread nsoffer
Nir Soffer has posted comments on this change. Change subject: properties: py3: properties.py and properties_test.py compliance .. Patch Set 3: Looking in the failures when running this under python 3, we have 3 issues: 1.

Change in vdsm[master]: properties: py3: properties.py and properties_test.py compli...

2016-09-12 Thread piotr . kliczewski
Piotr Kliczewski has posted comments on this change. Change subject: properties: py3: properties.py and properties_test.py compliance .. Patch Set 3: (2 comments) https://gerrit.ovirt.org/#/c/63230/3//COMMIT_MSG Commit

Change in vdsm[master]: properties: py3: properties.py and properties_test.py compli...

2016-09-03 Thread automation
gerrit-hooks has posted comments on this change. Change subject: properties: py3: properties.py and properties_test.py compliance .. Patch Set 3: * Update tracker: IGNORE, no Bug-Url found * Check Bug-Url::WARN, no bug url

Change in vdsm[master]: properties: py3: properties.py and properties_test.py compli...

2016-09-03 Thread Jenkins CI
Jenkins CI has posted comments on this change. Change subject: properties: py3: properties.py and properties_test.py compliance .. Patch Set 2: Continuous-Integration+1 Propagate review hook: Continuous Integration value

Change in vdsm[master]: properties: py3: properties.py and properties_test.py compli...

2016-09-03 Thread automation
gerrit-hooks has posted comments on this change. Change subject: properties: py3: properties.py and properties_test.py compliance .. Patch Set 2: * Update tracker: IGNORE, no Bug-Url found * Check Bug-Url::WARN, no bug url

Change in vdsm[master]: properties: py3: properties.py and properties_test.py compli...

2016-09-03 Thread automation
gerrit-hooks has posted comments on this change. Change subject: properties: py3: properties.py and properties_test.py compliance .. Patch Set 1: * Update tracker: IGNORE, no Bug-Url found * Check Bug-Url::WARN, no bug url