Change in vdsm[master]: Using six.moves configparser to support ConfigParser in python3

2015-11-19 Thread nsoffer
Nir Soffer has posted comments on this change.

Change subject: Using six.moves configparser to support ConfigParser in python3
..


Patch Set 2: Code-Review+2

Thanks, can you test this?

-- 
To view, visit https://gerrit.ovirt.org/48780
To unsubscribe, visit https://gerrit.ovirt.org/settings

Gerrit-MessageType: comment
Gerrit-Change-Id: Ib7deb264d9a11a763f1b95ca52433188e45f8b1a
Gerrit-PatchSet: 2
Gerrit-Project: vdsm
Gerrit-Branch: master
Gerrit-Owner: Yaniv Bronhaim 
Gerrit-Reviewer: Dan Kenigsberg 
Gerrit-Reviewer: Jenkins CI
Gerrit-Reviewer: Nir Soffer 
Gerrit-Reviewer: Yaniv Bronhaim 
Gerrit-Reviewer: gerrit-hooks 
Gerrit-HasComments: No
___
vdsm-patches mailing list
vdsm-patches@lists.fedorahosted.org
https://lists.fedorahosted.org/mailman/listinfo/vdsm-patches


Change in vdsm[master]: Using six.moves configparser to support ConfigParser in python3

2015-11-19 Thread automation
gerrit-hooks has posted comments on this change.

Change subject: Using six.moves configparser to support ConfigParser in python3
..


Patch Set 3:

* Update tracker: IGNORE, no Bug-Url found
* Set MODIFIED::IGNORE, no Bug-Url found.

-- 
To view, visit https://gerrit.ovirt.org/48780
To unsubscribe, visit https://gerrit.ovirt.org/settings

Gerrit-MessageType: comment
Gerrit-Change-Id: Ib7deb264d9a11a763f1b95ca52433188e45f8b1a
Gerrit-PatchSet: 3
Gerrit-Project: vdsm
Gerrit-Branch: master
Gerrit-Owner: Yaniv Bronhaim 
Gerrit-Reviewer: Dan Kenigsberg 
Gerrit-Reviewer: Jenkins CI
Gerrit-Reviewer: Nir Soffer 
Gerrit-Reviewer: Yaniv Bronhaim 
Gerrit-Reviewer: gerrit-hooks 
Gerrit-HasComments: No
___
vdsm-patches mailing list
vdsm-patches@lists.fedorahosted.org
https://lists.fedorahosted.org/mailman/listinfo/vdsm-patches


Change in vdsm[master]: Using six.moves configparser to support ConfigParser in python3

2015-11-19 Thread nsoffer
Nir Soffer has submitted this change and it was merged.

Change subject: Using six.moves configparser to support ConfigParser in python3
..


Using six.moves configparser to support ConfigParser in python3

Change-Id: Ib7deb264d9a11a763f1b95ca52433188e45f8b1a
Signed-off-by: Yaniv Bronhaim 
Reviewed-on: https://gerrit.ovirt.org/48780
Continuous-Integration: Jenkins CI
Reviewed-by: Nir Soffer 
---
M lib/vdsm/config.py.in
M lib/vdsm/tool/configfile.py
M tests/momPolicyTests.py
M tests/momTests.py
M tests/testlib.py
M vdsm/get-conf-item
M vdsm/network/configurators/__init__.py
M vdsm/set-conf-item
8 files changed, 17 insertions(+), 17 deletions(-)

Approvals:
  Nir Soffer: Looks good to me, approved
  Yaniv Bronhaim: Verified
  Jenkins CI: Passed CI tests



-- 
To view, visit https://gerrit.ovirt.org/48780
To unsubscribe, visit https://gerrit.ovirt.org/settings

Gerrit-MessageType: merged
Gerrit-Change-Id: Ib7deb264d9a11a763f1b95ca52433188e45f8b1a
Gerrit-PatchSet: 3
Gerrit-Project: vdsm
Gerrit-Branch: master
Gerrit-Owner: Yaniv Bronhaim 
Gerrit-Reviewer: Dan Kenigsberg 
Gerrit-Reviewer: Jenkins CI
Gerrit-Reviewer: Nir Soffer 
Gerrit-Reviewer: Yaniv Bronhaim 
Gerrit-Reviewer: gerrit-hooks 
___
vdsm-patches mailing list
vdsm-patches@lists.fedorahosted.org
https://lists.fedorahosted.org/mailman/listinfo/vdsm-patches


Change in vdsm[master]: Using six.moves configparser to support ConfigParser in python3

2015-11-19 Thread ybronhei
Yaniv Bronhaim has posted comments on this change.

Change subject: Using six.moves configparser to support ConfigParser in python3
..


Patch Set 2: Verified+1

-- 
To view, visit https://gerrit.ovirt.org/48780
To unsubscribe, visit https://gerrit.ovirt.org/settings

Gerrit-MessageType: comment
Gerrit-Change-Id: Ib7deb264d9a11a763f1b95ca52433188e45f8b1a
Gerrit-PatchSet: 2
Gerrit-Project: vdsm
Gerrit-Branch: master
Gerrit-Owner: Yaniv Bronhaim 
Gerrit-Reviewer: Jenkins CI
Gerrit-Reviewer: Yaniv Bronhaim 
Gerrit-Reviewer: gerrit-hooks 
Gerrit-HasComments: No
___
vdsm-patches mailing list
vdsm-patches@lists.fedorahosted.org
https://lists.fedorahosted.org/mailman/listinfo/vdsm-patches


Change in vdsm[master]: Using six.moves configparser to support ConfigParser in python3

2015-11-19 Thread automation
gerrit-hooks has posted comments on this change.

Change subject: Using six.moves configparser to support ConfigParser in python3
..


Patch Set 2:

* Update tracker: IGNORE, no Bug-Url found
* Check Bug-Url::WARN, no bug url found, make sure header matches 'Bug-Url: ' 
and is a valid url.
* Check merged to previous::IGNORE, Not in stable branch (['ovirt-3.6', 
'ovirt-3.5', 'ovirt-3.4', 'ovirt-3.3'])

-- 
To view, visit https://gerrit.ovirt.org/48780
To unsubscribe, visit https://gerrit.ovirt.org/settings

Gerrit-MessageType: comment
Gerrit-Change-Id: Ib7deb264d9a11a763f1b95ca52433188e45f8b1a
Gerrit-PatchSet: 2
Gerrit-Project: vdsm
Gerrit-Branch: master
Gerrit-Owner: Yaniv Bronhaim 
Gerrit-Reviewer: Jenkins CI
Gerrit-Reviewer: gerrit-hooks 
Gerrit-HasComments: No
___
vdsm-patches mailing list
vdsm-patches@lists.fedorahosted.org
https://lists.fedorahosted.org/mailman/listinfo/vdsm-patches