10:58 a.m.
Yaniv Bronhaim has uploaded a new change for review.
Change subject: Raise python-cpopen requirement to 1.4
......................................................................
Raise python-cpopen requirement to 1.4
Change-Id: I4c5d62281cfba20eed51f9960065b192f7a209d3
Bug-Url: https://bugzilla.redhat.com/show_bug.cgi?id=1264667
Signed-off-by: Yaniv Bronhaim <ybronhei(a)redhat.com>
---
M vdsm.spec.in
1 file changed, 3 insertions(+), 3 deletions(-)
git pull ssh://gerrit.ovirt.org:29418/vdsm refs/changes/24/46724/1
diff --git a/vdsm.spec.in b/vdsm.spec.in
index bced975..e63936e 100644
--- a/vdsm.spec.in
+++ b/vdsm.spec.in
@@ -88,7 +88,7 @@
BuildRequires: openssl
BuildRequires: policycoreutils-python
BuildRequires: psmisc
-BuildRequires: python-cpopen >= 1.3
+BuildRequires: python-cpopen >= 1.4
BuildRequires: python-inotify
BuildRequires: python-ioprocess >= 0.14
BuildRequires: python-pthreading
@@ -121,7 +121,7 @@
Requires: python-netaddr
Requires: python-inotify
Requires: python-argparse
-Requires: python-cpopen >= 1.3
+Requires: python-cpopen >= 1.4
Requires: python-ioprocess >= 0.14
Requires: python-pthreading >= 0.1.3-3
Requires: python-six
@@ -298,7 +298,7 @@
%package python
Summary: VDSM python libraries
Requires: %{name}-infra = %{version}-%{release}
-Requires: python-cpopen >= 1.2.3-5
+Requires: python-cpopen >= 1.4
Requires: m2crypto
Requires: python-ioprocess >= 0.14
--
To view, visit https://gerrit.ovirt.org/46724
To unsubscribe, visit https://gerrit.ovirt.org/settings
Gerrit-MessageType: newchange
Gerrit-Change-Id: I4c5d62281cfba20eed51f9960065b192f7a209d3
Gerrit-PatchSet: 1
Gerrit-Project: vdsm
Gerrit-Branch: master
Gerrit-Owner: Yaniv Bronhaim <ybronhei(a)redhat.com>
10:58 a.m.
automation(a)ovirt.org has posted comments on this change.
Change subject: Raise python-cpopen requirement to 1.4
......................................................................
Patch Set 1:
* Update tracker::#1264667::OK
* Check Bug-Url::OK
* Check Public Bug::#1264667::OK, public bug
* Check Product::#1264667::OK, Correct product Red Hat Enterprise Virtualization Manager
* Check TR::SKIP, not in a monitored branch (ovirt-3.5 ovirt-3.4 ovirt-3.3 ovirt-3.2)
* Check merged to previous::IGNORE, Not in stable branch (['ovirt-3.5',
'ovirt-3.4', 'ovirt-3.3'])
--
To view, visit https://gerrit.ovirt.org/46724
To unsubscribe, visit https://gerrit.ovirt.org/settings
Gerrit-MessageType: comment
Gerrit-Change-Id: I4c5d62281cfba20eed51f9960065b192f7a209d3
Gerrit-PatchSet: 1
Gerrit-Project: vdsm
Gerrit-Branch: master
Gerrit-Owner: Yaniv Bronhaim <ybronhei(a)redhat.com>
Gerrit-Reviewer: Jenkins CI
Gerrit-Reviewer: automation(a)ovirt.org
Gerrit-HasComments: No
6:26 a.m.
Yaniv Bronhaim has posted comments on this change.
Change subject: Raise python-cpopen requirement to 1.4
......................................................................
Patch Set 1: Verified+1
--
To view, visit https://gerrit.ovirt.org/46724
To unsubscribe, visit https://gerrit.ovirt.org/settings
Gerrit-MessageType: comment
Gerrit-Change-Id: I4c5d62281cfba20eed51f9960065b192f7a209d3
Gerrit-PatchSet: 1
Gerrit-Project: vdsm
Gerrit-Branch: master
Gerrit-Owner: Yaniv Bronhaim <ybronhei(a)redhat.com>
Gerrit-Reviewer: Jenkins CI
Gerrit-Reviewer: Yaniv Bronhaim <ybronhei(a)redhat.com>
Gerrit-Reviewer: automation(a)ovirt.org
Gerrit-HasComments: No
6:27 a.m.
Yaniv Bronhaim has posted comments on this change.
Change subject: Raise python-cpopen requirement to 1.4
......................................................................
Patch Set 1:
lets move forward with that - it requires also backport to 3.6 and 3.5
--
To view, visit https://gerrit.ovirt.org/46724
To unsubscribe, visit https://gerrit.ovirt.org/settings
Gerrit-MessageType: comment
Gerrit-Change-Id: I4c5d62281cfba20eed51f9960065b192f7a209d3
Gerrit-PatchSet: 1
Gerrit-Project: vdsm
Gerrit-Branch: master
Gerrit-Owner: Yaniv Bronhaim <ybronhei(a)redhat.com>
Gerrit-Reviewer: Jenkins CI
Gerrit-Reviewer: Yaniv Bronhaim <ybronhei(a)redhat.com>
Gerrit-Reviewer: automation(a)ovirt.org
Gerrit-HasComments: No
8:13 a.m.
Yeela Kaplan has posted comments on this change.
Change subject: Raise python-cpopen requirement to 1.4
......................................................................
Patch Set 1: Code-Review+1
--
To view, visit https://gerrit.ovirt.org/46724
To unsubscribe, visit https://gerrit.ovirt.org/settings
Gerrit-MessageType: comment
Gerrit-Change-Id: I4c5d62281cfba20eed51f9960065b192f7a209d3
Gerrit-PatchSet: 1
Gerrit-Project: vdsm
Gerrit-Branch: master
Gerrit-Owner: Yaniv Bronhaim <ybronhei(a)redhat.com>
Gerrit-Reviewer: Jenkins CI
Gerrit-Reviewer: Yaniv Bronhaim <ybronhei(a)redhat.com>
Gerrit-Reviewer: Yeela Kaplan <ykaplan(a)redhat.com>
Gerrit-Reviewer: automation(a)ovirt.org
Gerrit-HasComments: No
6:26 a.m.
Yaniv Bronhaim has posted comments on this change.
Change subject: Raise python-cpopen requirement to 1.4
......................................................................
Patch Set 1:
the failure is not related. the patch can get in
--
To view, visit https://gerrit.ovirt.org/46724
To unsubscribe, visit https://gerrit.ovirt.org/settings
Gerrit-MessageType: comment
Gerrit-Change-Id: I4c5d62281cfba20eed51f9960065b192f7a209d3
Gerrit-PatchSet: 1
Gerrit-Project: vdsm
Gerrit-Branch: master
Gerrit-Owner: Yaniv Bronhaim <ybronhei(a)redhat.com>
Gerrit-Reviewer: Jenkins CI
Gerrit-Reviewer: Yaniv Bronhaim <ybronhei(a)redhat.com>
Gerrit-Reviewer: Yeela Kaplan <ykaplan(a)redhat.com>
Gerrit-Reviewer: automation(a)ovirt.org
Gerrit-HasComments: No
6:50 a.m.
Dan Kenigsberg has posted comments on this change.
Change subject: Raise python-cpopen requirement to 1.4
......................................................................
Patch Set 1: Continuous-Integration+1
11:35:05 ERROR: Tests mkimage.mkIsoFs creating an image and checking its content
11:35:05 ----------------------------------------------------------------------
11:35:05 Traceback (most recent call last):
11:35:05 File
"/home/jenkins/workspace/vdsm_master_check-patch-fc22-x86_64/vdsm/tests/testlib.py",
line 65, in wrapper
11:35:05 return f(self, *args)
11:35:05 File
"/home/jenkins/workspace/vdsm_master_check-patch-fc22-x86_64/vdsm/tests/mkimageTests.py",
line 189, in test_mkIsoFs
11:35:05 m.mount(mntOpts='loop')
11:35:05 File
"/home/jenkins/workspace/vdsm_master_check-patch-fc22-x86_64/vdsm/vdsm/storage/mount.py",
line 225, in mount
11:35:05 return self._runcmd(cmd, timeout)
11:35:05 File
"/home/jenkins/workspace/vdsm_master_check-patch-fc22-x86_64/vdsm/vdsm/storage/mount.py",
line 241, in _runcmd
11:35:05 raise MountError(rc, ";".join((out, err)))
11:35:05 MountError: (32, ';mount:
/tmp/tmphrqTSx/vmId_iso.08c0f2a2cfab1ab66fad14104c1659a8.img: failed to setup loop device:
No such file or directory\n')
--
To view, visit https://gerrit.ovirt.org/46724
To unsubscribe, visit https://gerrit.ovirt.org/settings
Gerrit-MessageType: comment
Gerrit-Change-Id: I4c5d62281cfba20eed51f9960065b192f7a209d3
Gerrit-PatchSet: 1
Gerrit-Project: vdsm
Gerrit-Branch: master
Gerrit-Owner: Yaniv Bronhaim <ybronhei(a)redhat.com>
Gerrit-Reviewer: Dan Kenigsberg <danken(a)redhat.com>
Gerrit-Reviewer: Jenkins CI
Gerrit-Reviewer: Yaniv Bronhaim <ybronhei(a)redhat.com>
Gerrit-Reviewer: Yeela Kaplan <ykaplan(a)redhat.com>
Gerrit-Reviewer: automation(a)ovirt.org
Gerrit-HasComments: No
6:50 a.m.
Dan Kenigsberg has posted comments on this change.
Change subject: Raise python-cpopen requirement to 1.4
......................................................................
Patch Set 1: Code-Review+2
--
To view, visit https://gerrit.ovirt.org/46724
To unsubscribe, visit https://gerrit.ovirt.org/settings
Gerrit-MessageType: comment
Gerrit-Change-Id: I4c5d62281cfba20eed51f9960065b192f7a209d3
Gerrit-PatchSet: 1
Gerrit-Project: vdsm
Gerrit-Branch: master
Gerrit-Owner: Yaniv Bronhaim <ybronhei(a)redhat.com>
Gerrit-Reviewer: Dan Kenigsberg <danken(a)redhat.com>
Gerrit-Reviewer: Jenkins CI
Gerrit-Reviewer: Yaniv Bronhaim <ybronhei(a)redhat.com>
Gerrit-Reviewer: Yeela Kaplan <ykaplan(a)redhat.com>
Gerrit-Reviewer: automation(a)ovirt.org
Gerrit-HasComments: No
6:50 a.m.
Dan Kenigsberg has submitted this change and it was merged.
Change subject: Raise python-cpopen requirement to 1.4
......................................................................
Raise python-cpopen requirement to 1.4
Change-Id: I4c5d62281cfba20eed51f9960065b192f7a209d3
Bug-Url: https://bugzilla.redhat.com/show_bug.cgi?id=1264667
Signed-off-by: Yaniv Bronhaim <ybronhei(a)redhat.com>
Reviewed-on: https://gerrit.ovirt.org/46724
Reviewed-by: Yeela Kaplan <ykaplan(a)redhat.com>
Continuous-Integration: Dan Kenigsberg <danken(a)redhat.com>
Reviewed-by: Dan Kenigsberg <danken(a)redhat.com>
---
M vdsm.spec.in
1 file changed, 3 insertions(+), 3 deletions(-)
Approvals:
Yeela Kaplan: Looks good to me, but someone else must approve
Yaniv Bronhaim: Verified
Dan Kenigsberg: Looks good to me, approved; Passed CI tests
Objections:
Jenkins CI: Failed CI tests
--
To view, visit https://gerrit.ovirt.org/46724
To unsubscribe, visit https://gerrit.ovirt.org/settings
Gerrit-MessageType: merged
Gerrit-Change-Id: I4c5d62281cfba20eed51f9960065b192f7a209d3
Gerrit-PatchSet: 2
Gerrit-Project: vdsm
Gerrit-Branch: master
Gerrit-Owner: Yaniv Bronhaim <ybronhei(a)redhat.com>
Gerrit-Reviewer: Dan Kenigsberg <danken(a)redhat.com>
Gerrit-Reviewer: Jenkins CI
Gerrit-Reviewer: Yaniv Bronhaim <ybronhei(a)redhat.com>
Gerrit-Reviewer: Yeela Kaplan <ykaplan(a)redhat.com>
Gerrit-Reviewer: automation(a)ovirt.org
6:50 a.m.
automation(a)ovirt.org has posted comments on this change.
Change subject: Raise python-cpopen requirement to 1.4
......................................................................
Patch Set 2:
* Update tracker::#1264667::OK
* Set MODIFIED::bug 1264667::::#1264667::::IGNORE, not oVirt prod but Red Hat Enterprise
Virtualization Manager
--
To view, visit https://gerrit.ovirt.org/46724
To unsubscribe, visit https://gerrit.ovirt.org/settings
Gerrit-MessageType: comment
Gerrit-Change-Id: I4c5d62281cfba20eed51f9960065b192f7a209d3
Gerrit-PatchSet: 2
Gerrit-Project: vdsm
Gerrit-Branch: master
Gerrit-Owner: Yaniv Bronhaim <ybronhei(a)redhat.com>
Gerrit-Reviewer: Dan Kenigsberg <danken(a)redhat.com>
Gerrit-Reviewer: Jenkins CI
Gerrit-Reviewer: Yaniv Bronhaim <ybronhei(a)redhat.com>
Gerrit-Reviewer: Yeela Kaplan <ykaplan(a)redhat.com>
Gerrit-Reviewer: automation(a)ovirt.org
Gerrit-HasComments: No
3158
days inactive
3197
days old
vdsm-patches@lists.fedorahosted.org
9 comments
4 participants
participants (4)
-
automation＠ovirt.org -
Dan Kenigsberg -
ybronhei＠redhat.com -
ykaplan＠redhat.com