Change in vdsm[master]: NetReload: ConfigWriter, ifup/down to configurator

2013-04-10 Thread danken
Dan Kenigsberg has submitted this change and it was merged. Change subject: NetReload: ConfigWriter, ifup/down to configurator .. NetReload: ConfigWriter, ifup/down to configurator NetworkReloaded mandates that the configuratio

Change in vdsm[master]: NetReload: ConfigWriter, ifup/down to configurator

2013-04-10 Thread danken
Dan Kenigsberg has posted comments on this change. Change subject: NetReload: ConfigWriter, ifup/down to configurator .. Patch Set 13: Looks good to me, approved (1 inline comment) .

Change in vdsm[master]: NetReload: ConfigWriter, ifup/down to configurator

2013-04-09 Thread Gerrit Code Review
oVirt Jenkins CI Server has posted comments on this change. Change subject: NetReload: ConfigWriter, ifup/down to configurator .. Patch Set 13: Build Successful http://jenkins.ovirt.org/job/vdsm_pep8_gerrit/1882/ : SUCCESS h

Change in vdsm[master]: NetReload: ConfigWriter, ifup/down to configurator

2013-04-09 Thread asegurap
Antoni Segura Puimedon has posted comments on this change. Change subject: NetReload: ConfigWriter, ifup/down to configurator .. Patch Set 13: Verified Sorry. I had forgotten the __init__.py in vdsm.spec.in. Fixed and verified

Change in vdsm[master]: NetReload: ConfigWriter, ifup/down to configurator

2013-04-09 Thread Gerrit Code Review
oVirt Jenkins CI Server has posted comments on this change. Change subject: NetReload: ConfigWriter, ifup/down to configurator .. Patch Set 13: Build Started http://jenkins.ovirt.org/job/vdsm_pep8_gerrit/1882/ (2/2) -- To view

Change in vdsm[master]: NetReload: ConfigWriter, ifup/down to configurator

2013-04-09 Thread Gerrit Code Review
oVirt Jenkins CI Server has posted comments on this change. Change subject: NetReload: ConfigWriter, ifup/down to configurator .. Patch Set 13: Build Started http://jenkins.ovirt.org/job/vdsm_unit_tests_gerrit/1933/ (1/2) -- T

Change in vdsm[master]: NetReload: ConfigWriter, ifup/down to configurator

2013-04-09 Thread danken
Dan Kenigsberg has posted comments on this change. Change subject: NetReload: ConfigWriter, ifup/down to configurator .. Patch Set 12: Looks good to me, approved Thanks a bunch for noticing these two other ConfigWriter users!

Change in vdsm[master]: NetReload: ConfigWriter, ifup/down to configurator

2013-04-09 Thread wudxw
Mark Wu has posted comments on this change. Change subject: NetReload: ConfigWriter, ifup/down to configurator .. Patch Set 12: Looks good to me, but someone else must approve -- To view, visit http://gerrit.ovirt.org/13182 To

Change in vdsm[master]: NetReload: ConfigWriter, ifup/down to configurator

2013-04-08 Thread Gerrit Code Review
oVirt Jenkins CI Server has posted comments on this change. Change subject: NetReload: ConfigWriter, ifup/down to configurator .. Patch Set 12: Build Successful http://jenkins.ovirt.org/job/vdsm_pep8_gerrit/1873/ : SUCCESS h

Change in vdsm[master]: NetReload: ConfigWriter, ifup/down to configurator

2013-04-08 Thread Gerrit Code Review
oVirt Jenkins CI Server has posted comments on this change. Change subject: NetReload: ConfigWriter, ifup/down to configurator .. Patch Set 12: Build Started http://jenkins.ovirt.org/job/vdsm_unit_tests_gerrit/1924/ (1/2) -- T

Change in vdsm[master]: NetReload: ConfigWriter, ifup/down to configurator

2013-04-08 Thread Gerrit Code Review
oVirt Jenkins CI Server has posted comments on this change. Change subject: NetReload: ConfigWriter, ifup/down to configurator .. Patch Set 12: Build Started http://jenkins.ovirt.org/job/vdsm_pep8_gerrit/1873/ (2/2) -- To view

Change in vdsm[master]: NetReload: ConfigWriter, ifup/down to configurator

2013-04-08 Thread asegurap
Antoni Segura Puimedon has posted comments on this change. Change subject: NetReload: ConfigWriter, ifup/down to configurator .. Patch Set 10: (1 inline comment) File configu

Change in vdsm[master]: NetReload: ConfigWriter, ifup/down to configurator

2013-04-08 Thread wudxw
Mark Wu has posted comments on this change. Change subject: NetReload: ConfigWriter, ifup/down to configurator .. Patch Set 10: (1 inline comment) sorry for not finding that in previous review!

Change in vdsm[master]: NetReload: ConfigWriter, ifup/down to configurator

2013-04-08 Thread Gerrit Code Review
oVirt Jenkins CI Server has posted comments on this change. Change subject: NetReload: ConfigWriter, ifup/down to configurator .. Patch Set 11: Build Successful http://jenkins.ovirt.org/job/vdsm_pep8_gerrit/1871/ : SUCCESS h

Change in vdsm[master]: NetReload: ConfigWriter, ifup/down to configurator

2013-04-08 Thread asegurap
Antoni Segura Puimedon has posted comments on this change. Change subject: NetReload: ConfigWriter, ifup/down to configurator .. Patch Set 11: Thanks Mark! I has totally overlooked configwriter being used in clientIF.py and th

Change in vdsm[master]: NetReload: ConfigWriter, ifup/down to configurator

2013-04-08 Thread Gerrit Code Review
oVirt Jenkins CI Server has posted comments on this change. Change subject: NetReload: ConfigWriter, ifup/down to configurator .. Patch Set 11: Build Started http://jenkins.ovirt.org/job/vdsm_pep8_gerrit/1871/ (2/2) -- To view

Change in vdsm[master]: NetReload: ConfigWriter, ifup/down to configurator

2013-04-08 Thread Gerrit Code Review
oVirt Jenkins CI Server has posted comments on this change. Change subject: NetReload: ConfigWriter, ifup/down to configurator .. Patch Set 11: Build Started http://jenkins.ovirt.org/job/vdsm_unit_tests_gerrit/1922/ (1/2) -- T

Change in vdsm[master]: NetReload: ConfigWriter, ifup/down to configurator

2013-04-08 Thread wudxw
Mark Wu has posted comments on this change. Change subject: NetReload: ConfigWriter, ifup/down to configurator .. Patch Set 10: I would prefer that you didn't submit this ConfigWriter is moved into netcf.ifcfg. The import in cl

Change in vdsm[master]: NetReload: ConfigWriter, ifup/down to configurator

2013-04-08 Thread danken
Dan Kenigsberg has posted comments on this change. Change subject: NetReload: ConfigWriter, ifup/down to configurator .. Patch Set 10: Looks good to me, approved -- To view, visit http://gerrit.ovirt.org/13182 To unsubscribe, v

Change in vdsm[master]: NetReload: ConfigWriter, ifup/down to configurator

2013-04-08 Thread Gerrit Code Review
oVirt Jenkins CI Server has posted comments on this change. Change subject: NetReload: ConfigWriter, ifup/down to configurator .. Patch Set 10: Build Successful http://jenkins.ovirt.org/job/vdsm_pep8_gerrit/1863/ : SUCCESS h

Change in vdsm[master]: NetReload: ConfigWriter, ifup/down to configurator

2013-04-08 Thread Gerrit Code Review
oVirt Jenkins CI Server has posted comments on this change. Change subject: NetReload: ConfigWriter, ifup/down to configurator .. Patch Set 10: Build Started http://jenkins.ovirt.org/job/vdsm_pep8_gerrit/1863/ (2/2) -- To view

Change in vdsm[master]: NetReload: ConfigWriter, ifup/down to configurator

2013-04-08 Thread Gerrit Code Review
oVirt Jenkins CI Server has posted comments on this change. Change subject: NetReload: ConfigWriter, ifup/down to configurator .. Patch Set 10: Build Started http://jenkins.ovirt.org/job/vdsm_unit_tests_gerrit/1914/ (1/2) -- T

Change in vdsm[master]: NetReload: ConfigWriter, ifup/down to configurator

2013-04-08 Thread danken
Dan Kenigsberg has posted comments on this change. Change subject: NetReload: ConfigWriter, ifup/down to configurator .. Patch Set 9: I would prefer that you didn't submit this (1 inline comment) Moving the code to a new modul

Change in vdsm[master]: NetReload: ConfigWriter, ifup/down to configurator

2013-03-29 Thread wudxw
Mark Wu has posted comments on this change. Change subject: NetReload: ConfigWriter, ifup/down to configurator .. Patch Set 9: Looks good to me, but someone else must approve (1 inline comment)

Change in vdsm[master]: NetReload: ConfigWriter, ifup/down to configurator

2013-03-28 Thread Gerrit Code Review
oVirt Jenkins CI Server has posted comments on this change. Change subject: NetReload: ConfigWriter, ifup/down to configurator .. Patch Set 9: Build Successful http://jenkins.ovirt.org/job/vdsm_pep8_gerrit/1749/ : SUCCESS ht

Change in vdsm[master]: NetReload: ConfigWriter, ifup/down to configurator

2013-03-28 Thread Gerrit Code Review
oVirt Jenkins CI Server has posted comments on this change. Change subject: NetReload: ConfigWriter, ifup/down to configurator .. Patch Set 9: Build Started http://jenkins.ovirt.org/job/vdsm_pep8_gerrit/1749/ (2/2) -- To view,

Change in vdsm[master]: NetReload: ConfigWriter, ifup/down to configurator

2013-03-28 Thread Gerrit Code Review
oVirt Jenkins CI Server has posted comments on this change. Change subject: NetReload: ConfigWriter, ifup/down to configurator .. Patch Set 9: Build Started http://jenkins.ovirt.org/job/vdsm_unit_tests_gerrit/1800/ (1/2) -- To

Change in vdsm[master]: NetReload: ConfigWriter, ifup/down to configurator

2013-03-22 Thread asegurap
Antoni Segura Puimedon has posted comments on this change. Change subject: NetReload: ConfigWriter, ifup/down to configurator .. Patch Set 8: Verified -- To view, visit http://gerrit.ovirt.org/13182 To unsubscribe, visit http:/

Change in vdsm[master]: NetReload: ConfigWriter, ifup/down to configurator

2013-03-21 Thread Gerrit Code Review
oVirt Jenkins CI Server has posted comments on this change. Change subject: NetReload: ConfigWriter, ifup/down to configurator .. Patch Set 8: Build Successful http://jenkins.ovirt.org/job/vdsm_pep8_gerrit/1692/ : SUCCESS ht

Change in vdsm[master]: NetReload: ConfigWriter, ifup/down to configurator

2013-03-21 Thread Gerrit Code Review
oVirt Jenkins CI Server has posted comments on this change. Change subject: NetReload: ConfigWriter, ifup/down to configurator .. Patch Set 8: Build Started http://jenkins.ovirt.org/job/vdsm_pep8_gerrit/1692/ (2/2) -- To view,

Change in vdsm[master]: NetReload: ConfigWriter, ifup/down to configurator

2013-03-21 Thread Gerrit Code Review
oVirt Jenkins CI Server has posted comments on this change. Change subject: NetReload: ConfigWriter, ifup/down to configurator .. Patch Set 8: Build Started http://jenkins.ovirt.org/job/vdsm_unit_tests_gerrit/1742/ (1/2) -- To

Change in vdsm[master]: NetReload: ConfigWriter, ifup/down to configurator

2013-03-21 Thread asegurap
Antoni Segura Puimedon has posted comments on this change. Change subject: NetReload: ConfigWriter, ifup/down to configurator .. Patch Set 7: Verified -- To view, visit http://gerrit.ovirt.org/13182 To unsubscribe, visit http:/

Change in vdsm[master]: NetReload: ConfigWriter, ifup/down to configurator

2013-03-21 Thread Gerrit Code Review
oVirt Jenkins CI Server has posted comments on this change. Change subject: NetReload: ConfigWriter, ifup/down to configurator .. Patch Set 7: Build Successful http://jenkins.ovirt.org/job/vdsm_pep8_gerrit/1688/ : SUCCESS ht

Change in vdsm[master]: NetReload: ConfigWriter, ifup/down to configurator

2013-03-21 Thread Gerrit Code Review
oVirt Jenkins CI Server has posted comments on this change. Change subject: NetReload: ConfigWriter, ifup/down to configurator .. Patch Set 7: Build Started http://jenkins.ovirt.org/job/vdsm_pep8_gerrit/1688/ (1/2) -- To view,

Change in vdsm[master]: NetReload: ConfigWriter, ifup/down to configurator

2013-03-21 Thread Gerrit Code Review
oVirt Jenkins CI Server has posted comments on this change. Change subject: NetReload: ConfigWriter, ifup/down to configurator .. Patch Set 7: Build Started http://jenkins.ovirt.org/job/vdsm_unit_tests_gerrit/1738/ (2/2) -- To

Change in vdsm[master]: NetReload: ConfigWriter, ifup/down to configurator

2013-03-21 Thread Gerrit Code Review
oVirt Jenkins CI Server has posted comments on this change. Change subject: NetReload: ConfigWriter, ifup/down to configurator .. Patch Set 6: Fails; I would prefer that you didn't submit this Build Failed http://jenkins.ovir

Change in vdsm[master]: NetReload: ConfigWriter, ifup/down to configurator

2013-03-21 Thread Gerrit Code Review
oVirt Jenkins CI Server has posted comments on this change. Change subject: NetReload: ConfigWriter, ifup/down to configurator .. Patch Set 6: Build Started http://jenkins.ovirt.org/job/vdsm_pep8_gerrit/1687/ (1/2) -- To view,

Change in vdsm[master]: NetReload: ConfigWriter, ifup/down to configurator

2013-03-21 Thread Gerrit Code Review
oVirt Jenkins CI Server has posted comments on this change. Change subject: NetReload: ConfigWriter, ifup/down to configurator .. Patch Set 6: Build Started http://jenkins.ovirt.org/job/vdsm_unit_tests_gerrit/1737/ (2/2) -- To

Change in vdsm[master]: NetReload: ConfigWriter, ifup/down to configurator

2013-03-21 Thread Gerrit Code Review
oVirt Jenkins CI Server has posted comments on this change. Change subject: NetReload: ConfigWriter, ifup/down to configurator .. Patch Set 5: Fails Build Failed http://jenkins.ovirt.org/job/vdsm_pep8_gerrit/1685/ : SUCCESS

Change in vdsm[master]: NetReload: ConfigWriter, ifup/down to configurator

2013-03-21 Thread Gerrit Code Review
oVirt Jenkins CI Server has posted comments on this change. Change subject: NetReload: ConfigWriter, ifup/down to configurator .. Patch Set 5: Build Started http://jenkins.ovirt.org/job/vdsm_unit_tests_gerrit/1735/ (1/2) -- To

Change in vdsm[master]: NetReload: ConfigWriter, ifup/down to configurator

2013-03-21 Thread Gerrit Code Review
oVirt Jenkins CI Server has posted comments on this change. Change subject: NetReload: ConfigWriter, ifup/down to configurator .. Patch Set 5: Build Started http://jenkins.ovirt.org/job/vdsm_pep8_gerrit/1685/ (2/2) -- To view,

Change in vdsm[master]: NetReload: ConfigWriter, ifup/down to configurator

2013-03-20 Thread Gerrit Code Review
oVirt Jenkins CI Server has posted comments on this change. Change subject: NetReload: ConfigWriter, ifup/down to configurator .. Patch Set 4: Fails Build Failed http://jenkins.ovirt.org/job/vdsm_pep8_gerrit/1670/ : SUCCESS

Change in vdsm[master]: NetReload: ConfigWriter, ifup/down to configurator

2013-03-20 Thread Gerrit Code Review
oVirt Jenkins CI Server has posted comments on this change. Change subject: NetReload: ConfigWriter, ifup/down to configurator .. Patch Set 4: Build Started http://jenkins.ovirt.org/job/vdsm_unit_tests_gerrit/1720/ (2/2) -- To

Change in vdsm[master]: NetReload: ConfigWriter, ifup/down to configurator

2013-03-20 Thread Gerrit Code Review
oVirt Jenkins CI Server has posted comments on this change. Change subject: NetReload: ConfigWriter, ifup/down to configurator .. Patch Set 4: Build Started http://jenkins.ovirt.org/job/vdsm_pep8_gerrit/1670/ (1/2) -- To view,

Change in vdsm[master]: NetReload: ConfigWriter ifup/down to configurator

2013-03-20 Thread Gerrit Code Review
oVirt Jenkins CI Server has posted comments on this change. Change subject: NetReload: ConfigWriter ifup/down to configurator .. Patch Set 3: Fails Build Failed http://jenkins.ovirt.org/job/vdsm_pep8_gerrit/1667/ : SUCCESS h

Change in vdsm[master]: NetReload: ConfigWriter ifup/down to configurator

2013-03-20 Thread Gerrit Code Review
oVirt Jenkins CI Server has posted comments on this change. Change subject: NetReload: ConfigWriter ifup/down to configurator .. Patch Set 3: Build Started http://jenkins.ovirt.org/job/vdsm_unit_tests_gerrit/1716/ (1/2) -- To

Change in vdsm[master]: NetReload: ConfigWriter ifup/down to configurator

2013-03-20 Thread Gerrit Code Review
oVirt Jenkins CI Server has posted comments on this change. Change subject: NetReload: ConfigWriter ifup/down to configurator .. Patch Set 3: Build Started http://jenkins.ovirt.org/job/vdsm_pep8_gerrit/1667/ (2/2) -- To view,

Change in vdsm[master]: NetReload: ConfigWriter ifup/down to configurator

2013-03-20 Thread Gerrit Code Review
oVirt Jenkins CI Server has posted comments on this change. Change subject: NetReload: ConfigWriter ifup/down to configurator .. Patch Set 2: Fails Build Failed http://jenkins.ovirt.org/job/vdsm_pep8_gerrit/1663/ : SUCCESS h

Change in vdsm[master]: NetReload: ConfigWriter ifup/down to configurator

2013-03-20 Thread Gerrit Code Review
oVirt Jenkins CI Server has posted comments on this change. Change subject: NetReload: ConfigWriter ifup/down to configurator .. Patch Set 2: No score Build Started http://jenkins.ovirt.org/job/vdsm_unit_tests_gerrit/1709/ (2/2

Change in vdsm[master]: NetReload: ConfigWriter ifup/down to configurator

2013-03-20 Thread Gerrit Code Review
oVirt Jenkins CI Server has posted comments on this change. Change subject: NetReload: ConfigWriter ifup/down to configurator .. Patch Set 2: Fails Build Failed http://jenkins.ovirt.org/job/vdsm_pep8_gerrit/1663/ : SUCCESS h

Change in vdsm[master]: NetReload: ConfigWriter ifup/down to configurator

2013-03-20 Thread Gerrit Code Review
oVirt Jenkins CI Server has posted comments on this change. Change subject: NetReload: ConfigWriter ifup/down to configurator .. Patch Set 2: Build Started http://jenkins.ovirt.org/job/vdsm_pep8_gerrit/1663/ (2/2) -- To view,

Change in vdsm[master]: NetReload: ConfigWriter ifup/down to configurator

2013-03-20 Thread Gerrit Code Review
oVirt Jenkins CI Server has posted comments on this change. Change subject: NetReload: ConfigWriter ifup/down to configurator .. Patch Set 2: Build Started http://jenkins.ovirt.org/job/vdsm_unit_tests_gerrit/1708/ (1/2) -- To

Change in vdsm[master]: NetReload: ConfigWriter ifup/down to configurator

2013-03-19 Thread Gerrit Code Review
oVirt Jenkins CI Server has posted comments on this change. Change subject: NetReload: ConfigWriter ifup/down to configurator .. Patch Set 1: Fails Build Failed http://jenkins.ovirt.org/job/vdsm_pep8_gerrit/1657/ : SUCCESS h

Change in vdsm[master]: NetReload: ConfigWriter ifup/down to configurator

2013-03-19 Thread asegurap
Antoni Segura Puimedon has posted comments on this change. Change subject: NetReload: ConfigWriter ifup/down to configurator .. Patch Set 1: Verified -- To view, visit http://gerrit.ovirt.org/13182 To unsubscribe, visit http://

Change in vdsm[master]: NetReload: ConfigWriter ifup/down to configurator

2013-03-19 Thread asegurap
Antoni Segura Puimedon has uploaded a new change for review. Change subject: NetReload: ConfigWriter ifup/down to configurator .. NetReload: ConfigWriter ifup/down to configurator NetworkReloaded mandates that the configuration

Change in vdsm[master]: NetReload: ConfigWriter ifup/down to configurator

2013-03-19 Thread Gerrit Code Review
oVirt Jenkins CI Server has posted comments on this change. Change subject: NetReload: ConfigWriter ifup/down to configurator .. Patch Set 1: Build Started http://jenkins.ovirt.org/job/vdsm_unit_tests_gerrit/1702/ (2/2) -- To

Change in vdsm[master]: NetReload: ConfigWriter ifup/down to configurator

2013-03-19 Thread Gerrit Code Review
oVirt Jenkins CI Server has posted comments on this change. Change subject: NetReload: ConfigWriter ifup/down to configurator .. Patch Set 1: Build Started http://jenkins.ovirt.org/job/vdsm_pep8_gerrit/1657/ (1/2) -- To view,