Change in vdsm[master]: require specific pep8 version

2016-05-11 Thread ybronhei
Yaniv Bronhaim has abandoned this change. Change subject: require specific pep8 version .. Abandoned -- To view, visit https://gerrit.ovirt.org/56148 To unsubscribe, visit https://gerrit.ovirt.org/settings

Change in vdsm[master]: require specific pep8 version

2016-05-09 Thread nsoffer
Nir Soffer has posted comments on this change. Change subject: require specific pep8 version .. Patch Set 1: Code-Review-1 This is already solved in master, installing using pip. -- To view, visit

Change in vdsm[master]: require specific pep8 version

2016-04-26 Thread dcaro
David Caro has posted comments on this change. Change subject: require specific pep8 version .. Patch Set 1: (1 comment) https://gerrit.ovirt.org/#/c/56148/1/automation/check-patch.packages.el7 File

Change in vdsm[master]: require specific pep8 version

2016-04-24 Thread ybronhei
Yaniv Bronhaim has posted comments on this change. Change subject: require specific pep8 version .. Patch Set 1: (1 comment) https://gerrit.ovirt.org/#/c/56148/1/automation/check-patch.packages.el7 File

Change in vdsm[master]: require specific pep8 version

2016-04-19 Thread dcaro
David Caro has posted comments on this change. Change subject: require specific pep8 version .. Patch Set 1: Code-Review-1 (1 comment) https://gerrit.ovirt.org/#/c/56148/1/automation/check-patch.packages.el7 File

Change in vdsm[master]: require specific pep8 version

2016-04-19 Thread dcaro
David Caro has posted comments on this change. Change subject: require specific pep8 version .. Patch Set 1: Unfortunately, it does not seem to work (running locally), debugging -- To view, visit

Change in vdsm[master]: require specific pep8 version

2016-04-19 Thread dcaro
David Caro has posted comments on this change. Change subject: require specific pep8 version .. Patch Set 1: (1 comment) https://gerrit.ovirt.org/#/c/56148/1/automation/check-patch.packages.el7 File

Change in vdsm[master]: require specific pep8 version

2016-04-19 Thread ybronhei
Yaniv Bronhaim has posted comments on this change. Change subject: require specific pep8 version .. Patch Set 1: (1 comment) https://gerrit.ovirt.org/#/c/56148/1/automation/check-patch.packages.el7 File

Change in vdsm[master]: require specific pep8 version

2016-04-19 Thread dcaro
David Caro has posted comments on this change. Change subject: require specific pep8 version .. Patch Set 1: Code-Review+1 (1 comment) https://gerrit.ovirt.org/#/c/56148/1/automation/check-patch.packages.el7 File

Change in vdsm[master]: require specific pep8 version

2016-04-19 Thread ybronhei
Yaniv Bronhaim has posted comments on this change. Change subject: require specific pep8 version .. Patch Set 1: (1 comment) https://gerrit.ovirt.org/#/c/56148/1/automation/check-patch.packages.el7 File

Change in vdsm[master]: require specific pep8 version

2016-04-19 Thread ybronhei
Yaniv Bronhaim has posted comments on this change. Change subject: require specific pep8 version .. Patch Set 1: the problem is with older versions - http://jenkins.ovirt.org/job/vdsm_master_check-patch-el7-x86_64/837/console

Change in vdsm[master]: require specific pep8 version

2016-04-18 Thread danken
Dan Kenigsberg has posted comments on this change. Change subject: require specific pep8 version .. Patch Set 1: can you explain why? what happens if a newer version exits in the repository? -- To view, visit

Change in vdsm[master]: require specific pep8 version

2016-04-17 Thread ybronhei
Yaniv Bronhaim has posted comments on this change. Change subject: require specific pep8 version .. Patch Set 1: Verified+1 if Ci passes I guess where good .. no ? -- To view, visit https://gerrit.ovirt.org/56148 To

Change in vdsm[master]: require specific pep8 version

2016-04-14 Thread automation
gerrit-hooks has posted comments on this change. Change subject: require specific pep8 version .. 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]: require specific pep8 version

2016-04-14 Thread ybronhei
Yaniv Bronhaim has uploaded a new change for review. Change subject: require specific pep8 version .. require specific pep8 version Change-Id: Ie47f108c2065d5bd0e4e0e987121f91db4868397 Signed-off-by: Yaniv Bronhaim