Change in vdsm[ovirt-4.0]: Adding python-setuptools to required ci packages

2016-07-18 Thread ybronhei
Yaniv Bronhaim has abandoned this change.

Change subject: Adding python-setuptools to required ci packages
..


Abandoned

this is not needed over f23

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

Gerrit-MessageType: abandon
Gerrit-Change-Id: Ibb72f3e15091bd103718cc1354c135b7b79b7f50
Gerrit-PatchSet: 4
Gerrit-Project: vdsm
Gerrit-Branch: ovirt-4.0
Gerrit-Owner: Yaniv Bronhaim 
Gerrit-Reviewer: Allon Mureinik 
Gerrit-Reviewer: Dan Kenigsberg 
Gerrit-Reviewer: Francesco Romani 
Gerrit-Reviewer: Nir Soffer 
Gerrit-Reviewer: Yaniv Bronhaim 
___
vdsm-patches mailing list
vdsm-patches@lists.fedorahosted.org
https://lists.fedorahosted.org/admin/lists/vdsm-patches@lists.fedorahosted.org


Change in vdsm[ovirt-4.0]: Adding python-setuptools to required ci packages

2016-07-18 Thread automation
gerrit-hooks has posted comments on this change.

Change subject: Adding python-setuptools to required ci packages
..


Patch Set 4:

* Update tracker: IGNORE, no Bug-Url found

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

Gerrit-MessageType: comment
Gerrit-Change-Id: Ibb72f3e15091bd103718cc1354c135b7b79b7f50
Gerrit-PatchSet: 4
Gerrit-Project: vdsm
Gerrit-Branch: ovirt-4.0
Gerrit-Owner: Yaniv Bronhaim 
Gerrit-Reviewer: Allon Mureinik 
Gerrit-Reviewer: Dan Kenigsberg 
Gerrit-Reviewer: Francesco Romani 
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/admin/lists/vdsm-patches@lists.fedorahosted.org


Change in vdsm[ovirt-4.0]: Adding python-setuptools to required ci packages

2016-07-18 Thread ybronhei
Yaniv Bronhaim has posted comments on this change.

Change subject: Adding python-setuptools to required ci packages
..


Patch Set 4:

this is not needed if we continue to run over f23 in ci

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

Gerrit-MessageType: comment
Gerrit-Change-Id: Ibb72f3e15091bd103718cc1354c135b7b79b7f50
Gerrit-PatchSet: 4
Gerrit-Project: vdsm
Gerrit-Branch: ovirt-4.0
Gerrit-Owner: Yaniv Bronhaim 
Gerrit-Reviewer: Allon Mureinik 
Gerrit-Reviewer: Dan Kenigsberg 
Gerrit-Reviewer: Francesco Romani 
Gerrit-Reviewer: Nir Soffer 
Gerrit-Reviewer: Yaniv Bronhaim 
Gerrit-HasComments: No
___
vdsm-patches mailing list
vdsm-patches@lists.fedorahosted.org
https://lists.fedorahosted.org/admin/lists/vdsm-patches@lists.fedorahosted.org


Change in vdsm[ovirt-4.0]: Adding python-setuptools to required ci packages

2016-07-18 Thread ybronhei
Yaniv Bronhaim has posted comments on this change.

Change subject: Adding python-setuptools to required ci packages
..


Patch Set 4: Verified+1

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

Gerrit-MessageType: comment
Gerrit-Change-Id: Ibb72f3e15091bd103718cc1354c135b7b79b7f50
Gerrit-PatchSet: 4
Gerrit-Project: vdsm
Gerrit-Branch: ovirt-4.0
Gerrit-Owner: Yaniv Bronhaim 
Gerrit-Reviewer: Allon Mureinik 
Gerrit-Reviewer: Dan Kenigsberg 
Gerrit-Reviewer: Francesco Romani 
Gerrit-Reviewer: Nir Soffer 
Gerrit-Reviewer: Yaniv Bronhaim 
Gerrit-HasComments: No
___
vdsm-patches mailing list
vdsm-patches@lists.fedorahosted.org
https://lists.fedorahosted.org/admin/lists/vdsm-patches@lists.fedorahosted.org


Change in vdsm[ovirt-4.0]: Adding python-setuptools to required ci packages

2016-07-18 Thread fromani
Francesco Romani has posted comments on this change.

Change subject: Adding python-setuptools to required ci packages
..


Patch Set 4:

we'll also need to switch from yum-builddep to dnf, but this is material for 
another patch I'll post soon.

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

Gerrit-MessageType: comment
Gerrit-Change-Id: Ibb72f3e15091bd103718cc1354c135b7b79b7f50
Gerrit-PatchSet: 4
Gerrit-Project: vdsm
Gerrit-Branch: ovirt-4.0
Gerrit-Owner: Yaniv Bronhaim 
Gerrit-Reviewer: Allon Mureinik 
Gerrit-Reviewer: Dan Kenigsberg 
Gerrit-Reviewer: Francesco Romani 
Gerrit-Reviewer: Nir Soffer 
Gerrit-HasComments: No
___
vdsm-patches mailing list
vdsm-patches@lists.fedorahosted.org
https://lists.fedorahosted.org/admin/lists/vdsm-patches@lists.fedorahosted.org


Change in vdsm[ovirt-4.0]: Adding python-setuptools to required ci packages

2016-07-18 Thread fromani
Francesco Romani has posted comments on this change.

Change subject: Adding python-setuptools to required ci packages
..


Patch Set 4: Code-Review+2

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

Gerrit-MessageType: comment
Gerrit-Change-Id: Ibb72f3e15091bd103718cc1354c135b7b79b7f50
Gerrit-PatchSet: 4
Gerrit-Project: vdsm
Gerrit-Branch: ovirt-4.0
Gerrit-Owner: Yaniv Bronhaim 
Gerrit-Reviewer: Allon Mureinik 
Gerrit-Reviewer: Dan Kenigsberg 
Gerrit-Reviewer: Francesco Romani 
Gerrit-Reviewer: Nir Soffer 
Gerrit-HasComments: No
___
vdsm-patches mailing list
vdsm-patches@lists.fedorahosted.org
https://lists.fedorahosted.org/admin/lists/vdsm-patches@lists.fedorahosted.org


Change in vdsm[ovirt-4.0]: Adding python-setuptools to required ci packages

2016-07-18 Thread fromani
Francesco Romani has posted comments on this change.

Change subject: Adding python-setuptools to required ci packages
..


Patch Set 4: Continuous-Integration+1

we can't really trust the fc24 CI score without this patch

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

Gerrit-MessageType: comment
Gerrit-Change-Id: Ibb72f3e15091bd103718cc1354c135b7b79b7f50
Gerrit-PatchSet: 4
Gerrit-Project: vdsm
Gerrit-Branch: ovirt-4.0
Gerrit-Owner: Yaniv Bronhaim 
Gerrit-Reviewer: Allon Mureinik 
Gerrit-Reviewer: Dan Kenigsberg 
Gerrit-Reviewer: Francesco Romani 
Gerrit-Reviewer: Jenkins CI
Gerrit-Reviewer: Nir Soffer 
Gerrit-HasComments: No
___
vdsm-patches mailing list
vdsm-patches@lists.fedorahosted.org
https://lists.fedorahosted.org/admin/lists/vdsm-patches@lists.fedorahosted.org


Change in vdsm[ovirt-4.0]: Adding python-setuptools to required ci packages

2016-07-18 Thread fromani
Francesco Romani has posted comments on this change.

Change subject: Adding python-setuptools to required ci packages
..


Patch Set 4:

no change to production code -> no BZ needed

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

Gerrit-MessageType: comment
Gerrit-Change-Id: Ibb72f3e15091bd103718cc1354c135b7b79b7f50
Gerrit-PatchSet: 4
Gerrit-Project: vdsm
Gerrit-Branch: ovirt-4.0
Gerrit-Owner: Yaniv Bronhaim 
Gerrit-Reviewer: Allon Mureinik 
Gerrit-Reviewer: Dan Kenigsberg 
Gerrit-Reviewer: Francesco Romani 
Gerrit-Reviewer: Jenkins CI
Gerrit-Reviewer: Nir Soffer 
Gerrit-Reviewer: gerrit-hooks 
Gerrit-HasComments: No
___
vdsm-patches mailing list
vdsm-patches@lists.fedorahosted.org
https://lists.fedorahosted.org/admin/lists/vdsm-patches@lists.fedorahosted.org


Change in vdsm[ovirt-4.0]: Adding python-setuptools to required ci packages

2016-07-17 Thread amureini
Allon Mureinik has posted comments on this change.

Change subject: Adding python-setuptools to required ci packages
..


Patch Set 4: Code-Review+1

"Assumption is the mother of all.."

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

Gerrit-MessageType: comment
Gerrit-Change-Id: Ibb72f3e15091bd103718cc1354c135b7b79b7f50
Gerrit-PatchSet: 4
Gerrit-Project: vdsm
Gerrit-Branch: ovirt-4.0
Gerrit-Owner: Yaniv Bronhaim 
Gerrit-Reviewer: Allon Mureinik 
Gerrit-Reviewer: Dan Kenigsberg 
Gerrit-Reviewer: Francesco Romani 
Gerrit-Reviewer: Jenkins CI
Gerrit-Reviewer: Nir Soffer 
Gerrit-Reviewer: gerrit-hooks 
Gerrit-HasComments: No
___
vdsm-patches mailing list
vdsm-patches@lists.fedorahosted.org
https://lists.fedorahosted.org/admin/lists/vdsm-patches@lists.fedorahosted.org


Change in vdsm[ovirt-4.0]: Adding python-setuptools to required ci packages

2016-07-17 Thread automation
gerrit-hooks has posted comments on this change.

Change subject: Adding python-setuptools to required ci packages
..


Patch Set 4:

* Update tracker: IGNORE, no Bug-Url found

* Check Bug-Url::ERROR, At least one bug-url is required for the stable branch
* Check merged to previous::OK, change not open on any previous branch

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

Gerrit-MessageType: comment
Gerrit-Change-Id: Ibb72f3e15091bd103718cc1354c135b7b79b7f50
Gerrit-PatchSet: 4
Gerrit-Project: vdsm
Gerrit-Branch: ovirt-4.0
Gerrit-Owner: Yaniv Bronhaim 
Gerrit-Reviewer: Dan Kenigsberg 
Gerrit-Reviewer: Francesco Romani 
Gerrit-Reviewer: Jenkins CI
Gerrit-Reviewer: Nir Soffer 
Gerrit-Reviewer: gerrit-hooks 
Gerrit-HasComments: No
___
vdsm-patches mailing list
vdsm-patches@lists.fedorahosted.org
https://lists.fedorahosted.org/admin/lists/vdsm-patches@lists.fedorahosted.org


Change in vdsm[ovirt-4.0]: Adding python-setuptools to required ci packages

2016-07-17 Thread nsoffer
Nir Soffer has posted comments on this change.

Change subject: Adding python-setuptools to required ci packages
..


Patch Set 3: Code-Review+1

The dd issue is fixed in https://gerrit.ovirt.org/60857

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

Gerrit-MessageType: comment
Gerrit-Change-Id: Ibb72f3e15091bd103718cc1354c135b7b79b7f50
Gerrit-PatchSet: 3
Gerrit-Project: vdsm
Gerrit-Branch: ovirt-4.0
Gerrit-Owner: Yaniv Bronhaim 
Gerrit-Reviewer: Dan Kenigsberg 
Gerrit-Reviewer: Francesco Romani 
Gerrit-Reviewer: Jenkins CI
Gerrit-Reviewer: Nir Soffer 
Gerrit-Reviewer: gerrit-hooks 
Gerrit-HasComments: No
___
vdsm-patches mailing list
vdsm-patches@lists.fedorahosted.org
https://lists.fedorahosted.org/admin/lists/vdsm-patches@lists.fedorahosted.org


Change in vdsm[ovirt-4.0]: Adding python-setuptools to required ci packages

2016-07-17 Thread danken
Dan Kenigsberg has posted comments on this change.

Change subject: Adding python-setuptools to required ci packages
..


Patch Set 3:

seems unrelated?

11:51:52 ERROR: test_path_ok (storage_check_test.TestDirectioChecker)
11:51:52 --
11:51:52 Traceback (most recent call last):
11:51:52   File 
"/home/jenkins/workspace/vdsm_4.0_check-patch-fc24-x86_64/vdsm/tests/storage_check_test.py",
 line 77, in test_path_ok
11:51:52 delay = result.delay()
11:51:52   File 
"/home/jenkins/workspace/vdsm_4.0_check-patch-fc24-x86_64/vdsm/lib/vdsm/storage/check.py",
 line 377, in delay
11:51:52 "no match: %r" % stats)
11:51:52 MiscFileReadException: Internal file read failure: 
('/var/tmp/tmpW7Scnc', "no match: bytearray(b'4 bytes copied, 0.000834647 s, 
4.8 kB/s')")

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

Gerrit-MessageType: comment
Gerrit-Change-Id: Ibb72f3e15091bd103718cc1354c135b7b79b7f50
Gerrit-PatchSet: 3
Gerrit-Project: vdsm
Gerrit-Branch: ovirt-4.0
Gerrit-Owner: Yaniv Bronhaim 
Gerrit-Reviewer: Dan Kenigsberg 
Gerrit-Reviewer: Francesco Romani 
Gerrit-Reviewer: Jenkins CI
Gerrit-Reviewer: Nir Soffer 
Gerrit-Reviewer: gerrit-hooks 
Gerrit-HasComments: No
___
vdsm-patches mailing list
vdsm-patches@lists.fedorahosted.org
https://lists.fedorahosted.org/admin/lists/vdsm-patches@lists.fedorahosted.org


Change in vdsm[ovirt-4.0]: Adding python-setuptools to required ci packages

2016-07-17 Thread automation
gerrit-hooks has posted comments on this change.

Change subject: Adding python-setuptools to required ci packages
..


Patch Set 3: Verified-1

* Update tracker: IGNORE, no Bug-Url found

* Check Bug-Url::ERROR, At least one bug-url is required for the stable branch
* Check merged to previous::WARN, Still open on branches master

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

Gerrit-MessageType: comment
Gerrit-Change-Id: Ibb72f3e15091bd103718cc1354c135b7b79b7f50
Gerrit-PatchSet: 3
Gerrit-Project: vdsm
Gerrit-Branch: ovirt-4.0
Gerrit-Owner: Yaniv Bronhaim 
Gerrit-Reviewer: Dan Kenigsberg 
Gerrit-Reviewer: Francesco Romani 
Gerrit-Reviewer: Jenkins CI
Gerrit-Reviewer: Nir Soffer 
Gerrit-Reviewer: gerrit-hooks 
Gerrit-HasComments: No
___
vdsm-patches mailing list
vdsm-patches@lists.fedorahosted.org
https://lists.fedorahosted.org/admin/lists/vdsm-patches@lists.fedorahosted.org


Change in vdsm[ovirt-4.0]: Adding python-setuptools to required ci packages

2016-07-17 Thread automation
gerrit-hooks has posted comments on this change.

Change subject: Adding python-setuptools to required ci packages
..


Patch Set 2:

* Update tracker: IGNORE, no Bug-Url found

* Check Bug-Url::ERROR, At least one bug-url is required for the stable branch
* Check merged to previous::WARN, Still open on branches master

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

Gerrit-MessageType: comment
Gerrit-Change-Id: Ibb72f3e15091bd103718cc1354c135b7b79b7f50
Gerrit-PatchSet: 2
Gerrit-Project: vdsm
Gerrit-Branch: ovirt-4.0
Gerrit-Owner: Yaniv Bronhaim 
Gerrit-Reviewer: Dan Kenigsberg 
Gerrit-Reviewer: Francesco Romani 
Gerrit-Reviewer: Jenkins CI
Gerrit-Reviewer: Nir Soffer 
Gerrit-Reviewer: gerrit-hooks 
Gerrit-HasComments: No
___
vdsm-patches mailing list
vdsm-patches@lists.fedorahosted.org
https://lists.fedorahosted.org/admin/lists/vdsm-patches@lists.fedorahosted.org


Change in vdsm[ovirt-4.0]: Adding python-setuptools to required ci packages

2016-07-17 Thread automation
gerrit-hooks has posted comments on this change.

Change subject: Adding python-setuptools to required ci packages
..


Patch Set 1: Verified-1

* Update tracker: IGNORE, no Bug-Url found

* Check Bug-Url::ERROR, At least one bug-url is required for the stable branch
* Check merged to previous::WARN, Still missing on branches master

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

Gerrit-MessageType: comment
Gerrit-Change-Id: Ibb72f3e15091bd103718cc1354c135b7b79b7f50
Gerrit-PatchSet: 1
Gerrit-Project: vdsm
Gerrit-Branch: ovirt-4.0
Gerrit-Owner: Yaniv Bronhaim 
Gerrit-Reviewer: gerrit-hooks 
Gerrit-HasComments: No
___
vdsm-patches mailing list
vdsm-patches@lists.fedorahosted.org
https://lists.fedorahosted.org/admin/lists/vdsm-patches@lists.fedorahosted.org


Change in vdsm[ovirt-4.0]: Adding python-setuptools to required ci packages

2016-07-17 Thread ybronhei
Yaniv Bronhaim has uploaded a new change for review.

Change subject: Adding python-setuptools to required ci packages
..

Adding python-setuptools to required ci packages

Change-Id: Ibb72f3e15091bd103718cc1354c135b7b79b7f50
Signed-off-by: Yaniv Bronhaim 
---
M automation/check-patch.packages.fc23
1 file changed, 1 insertion(+), 0 deletions(-)


  git pull ssh://gerrit.ovirt.org:29418/vdsm refs/changes/47/60847/1

diff --git a/automation/check-patch.packages.fc23 
b/automation/check-patch.packages.fc23
index 1e45923..d4b6612 100644
--- a/automation/check-patch.packages.fc23
+++ b/automation/check-patch.packages.fc23
@@ -17,6 +17,7 @@
 python-netaddr
 python-nose
 python-pthreading
+python-setuptools
 python-six
 python3-netaddr
 python3-nose


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

Gerrit-MessageType: newchange
Gerrit-Change-Id: Ibb72f3e15091bd103718cc1354c135b7b79b7f50
Gerrit-PatchSet: 1
Gerrit-Project: vdsm
Gerrit-Branch: ovirt-4.0
Gerrit-Owner: Yaniv Bronhaim 
___
vdsm-patches mailing list
vdsm-patches@lists.fedorahosted.org
https://lists.fedorahosted.org/admin/lists/vdsm-patches@lists.fedorahosted.org