Change in vdsm[master]: spec: Require python-ioprocess-0.16.1-1

[automation]

gerrit-hooks has posted comments on this change.

Change subject: spec: Require python-ioprocess-0.16.1-1
..


Patch Set 3:

* #1339780::Update tracker: OK
* Set MODIFIED::bug 1339780#1339780OK

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

Gerrit-MessageType: comment
Gerrit-Change-Id: Ib93b5649981ef423f88d354334bd7d3364f52a0b
Gerrit-PatchSet: 3
Gerrit-Project: vdsm
Gerrit-Branch: master
Gerrit-Owner: Allon Mureinik 
Gerrit-Reviewer: Allon Mureinik 
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[master]: spec: Require python-ioprocess-0.16.1-1

[nsoffer]

Nir Soffer has submitted this change and it was merged.

Change subject: spec: Require python-ioprocess-0.16.1-1
..


spec: Require python-ioprocess-0.16.1-1

ioprocess-0.16.1-1 solves a bug where ioprocess keeps a file open
after truncating or touching it (see the Related-To bug).

Vdsm requires ioprocess via the noarch package python-ioprocess, and
this patch bumps the requirement to include a fix for the
aforementioned issue.

Change-Id: Ib93b5649981ef423f88d354334bd7d3364f52a0b
Related-To: https://bugzilla.redhat.com/1339777
Bug-Url: https://bugzilla.redhat.com/1339780
Signed-off-by: Allon Mureinik 
Reviewed-on: https://gerrit.ovirt.org/60858
Reviewed-by: Nir Soffer 
Continuous-Integration: Jenkins CI
---
M vdsm.spec.in
1 file changed, 3 insertions(+), 3 deletions(-)

Approvals:
  Nir Soffer: Looks good to me, approved
  Jenkins CI: Passed CI tests
  Allon Mureinik: Verified



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

Gerrit-MessageType: merged
Gerrit-Change-Id: Ib93b5649981ef423f88d354334bd7d3364f52a0b
Gerrit-PatchSet: 3
Gerrit-Project: vdsm
Gerrit-Branch: master
Gerrit-Owner: Allon Mureinik 
Gerrit-Reviewer: Allon Mureinik 
Gerrit-Reviewer: Jenkins CI
Gerrit-Reviewer: Nir Soffer 
Gerrit-Reviewer: gerrit-hooks 
___
vdsm-patches mailing list
vdsm-patches@lists.fedorahosted.org
https://lists.fedorahosted.org/admin/lists/vdsm-patches@lists.fedorahosted.org

Change in vdsm[master]: spec: Require python-ioprocess-0.16.1-1

[amureini]

Allon Mureinik has posted comments on this change.

Change subject: spec: Require python-ioprocess-0.16.1-1
..


Patch Set 2: Verified+1

CI passes, as do the builds I've tried locally. Nir, can you merge please?

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

Gerrit-MessageType: comment
Gerrit-Change-Id: Ib93b5649981ef423f88d354334bd7d3364f52a0b
Gerrit-PatchSet: 2
Gerrit-Project: vdsm
Gerrit-Branch: master
Gerrit-Owner: Allon Mureinik 
Gerrit-Reviewer: Allon Mureinik 
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[master]: spec: Require python-ioprocess-0.16.1-1

[Jenkins CI]

Jenkins CI has posted comments on this change.

Change subject: spec: Require python-ioprocess-0.16.1-1
..


Patch Set 2: Continuous-Integration+1

Propagate review hook: Continuous Integration value inherited from patch 1

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

Gerrit-MessageType: comment
Gerrit-Change-Id: Ib93b5649981ef423f88d354334bd7d3364f52a0b
Gerrit-PatchSet: 2
Gerrit-Project: vdsm
Gerrit-Branch: master
Gerrit-Owner: Allon Mureinik 
Gerrit-Reviewer: Allon Mureinik 
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[master]: spec: Require python-ioprocess-0.16.1-1

[nsoffer]

Nir Soffer has posted comments on this change.

Change subject: spec: Require python-ioprocess-0.16.1-1
..


Patch Set 2: Code-Review+2

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

Gerrit-MessageType: comment
Gerrit-Change-Id: Ib93b5649981ef423f88d354334bd7d3364f52a0b
Gerrit-PatchSet: 2
Gerrit-Project: vdsm
Gerrit-Branch: master
Gerrit-Owner: Allon Mureinik 
Gerrit-Reviewer: Allon Mureinik 
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[master]: spec: Require python-ioprocess-0.16.1-1

[amureini]

Allon Mureinik has posted comments on this change.

Change subject: spec: Require python-ioprocess-0.16.1-1
..


Patch Set 2:

And now with the right Bug-Url...

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

Gerrit-MessageType: comment
Gerrit-Change-Id: Ib93b5649981ef423f88d354334bd7d3364f52a0b
Gerrit-PatchSet: 2
Gerrit-Project: vdsm
Gerrit-Branch: master
Gerrit-Owner: Allon Mureinik 
Gerrit-Reviewer: Allon Mureinik 
Gerrit-Reviewer: Jenkins CI
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[master]: spec: Require python-ioprocess-0.16.1-1

[automation]

gerrit-hooks has posted comments on this change.

Change subject: spec: Require python-ioprocess-0.16.1-1
..


Patch Set 2:

* #1339780::Update tracker: OK
* Check Bug-Url::OK
* Check Public Bug::#1339780::OK, public bug
* Check Product::#1339780::OK, Correct classification oVirt
* Check TM::SKIP, not in a monitored branch (ovirt-3.6 ovirt-4.0)
* Check merged to previous::IGNORE, Not in stable branch (['ovirt-3.6', 
'ovirt-4.0'])

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

Gerrit-MessageType: comment
Gerrit-Change-Id: Ib93b5649981ef423f88d354334bd7d3364f52a0b
Gerrit-PatchSet: 2
Gerrit-Project: vdsm
Gerrit-Branch: master
Gerrit-Owner: Allon Mureinik 
Gerrit-Reviewer: Allon Mureinik 
Gerrit-Reviewer: Jenkins CI
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[master]: spec: Require python-ioprocess-0.16.1-1

[amureini]

Allon Mureinik has posted comments on this change.

Change subject: spec: Require python-ioprocess-0.16.1-1
..


Patch Set 1:

This should be available in all o/s we support:

F23: http://koji.fedoraproject.org/koji/buildinfo?buildID=773414
F24: http://koji.fedoraproject.org/koji/buildinfo?buildID=773408
CentOS 7: https://cbs.centos.org/koji/buildinfo?buildID=11371

Pending CI run to mark as verified

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

Gerrit-MessageType: comment
Gerrit-Change-Id: Ib93b5649981ef423f88d354334bd7d3364f52a0b
Gerrit-PatchSet: 1
Gerrit-Project: vdsm
Gerrit-Branch: master
Gerrit-Owner: Allon Mureinik 
Gerrit-Reviewer: Allon Mureinik 
Gerrit-Reviewer: Jenkins CI
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[master]: spec: Require python-ioprocess-0.16.1-1

[amureini]

Allon Mureinik has uploaded a new change for review.

Change subject: spec: Require python-ioprocess-0.16.1-1
..

spec: Require python-ioprocess-0.16.1-1

ioprocess-0.16.1-1 solves a bug where ioprocess keeps a file open
after truncating or touching it (see the Related-To bug).

Vdsm requires ioprocess via the noarch package python-ioprocess, and
this patch bumps the requirement to include a fix for the
aforementioned issue.

Change-Id: Ib93b5649981ef423f88d354334bd7d3364f52a0b
Related-To: https://bugzilla.redhat.com/1339777
Bug-Url: https://bugzilla.redhat.com/??
Signed-off-by: Allon Mureinik 
---
M vdsm.spec.in
1 file changed, 3 insertions(+), 3 deletions(-)


  git pull ssh://gerrit.ovirt.org:29418/vdsm refs/changes/58/60858/1

diff --git a/vdsm.spec.in b/vdsm.spec.in
index 6cac13a..e8de2df 100644
--- a/vdsm.spec.in
+++ b/vdsm.spec.in
@@ -79,7 +79,7 @@
 BuildRequires: PyYAML
 BuildRequires: python-cpopen >= 1.4
 BuildRequires: python-inotify
-BuildRequires: python-ioprocess >= 0.15.0-4
+BuildRequires: python-ioprocess >= 0.16.1-1
 BuildRequires: python-pthreading
 BuildRequires: qemu-img
 BuildRequires: rpm-python
@@ -136,7 +136,7 @@
 Requires: python-inotify
 Requires: python-argparse
 Requires: python-cpopen >= 1.4
-Requires: python-ioprocess >= 0.15.0-4
+Requires: python-ioprocess >= 0.16.1-1
 Requires: python-pthreading >= 0.1.3-3
 Requires: python-six >= 1.9.0
 Requires: python-requests
@@ -337,7 +337,7 @@
 BuildArch:  noarch
 Requires:   python-cpopen >= 1.4
 Requires:   m2crypto
-Requires:   python-ioprocess >= 0.15.0-4
+Requires:   python-ioprocess >= 0.16.1-1
 
 %description python
 Shared libraries between the various VDSM packages.


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

Gerrit-MessageType: newchange
Gerrit-Change-Id: Ib93b5649981ef423f88d354334bd7d3364f52a0b
Gerrit-PatchSet: 1
Gerrit-Project: vdsm
Gerrit-Branch: master
Gerrit-Owner: Allon Mureinik 
___
vdsm-patches mailing list
vdsm-patches@lists.fedorahosted.org
https://lists.fedorahosted.org/admin/lists/vdsm-patches@lists.fedorahosted.org

Change in vdsm[master]: spec: Require python-ioprocess-0.16.1-1

[automation]

gerrit-hooks has posted comments on this change.

Change subject: spec: Require python-ioprocess-0.16.1-1
..


Patch Set 1:

* 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-4.0'])

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

Gerrit-MessageType: comment
Gerrit-Change-Id: Ib93b5649981ef423f88d354334bd7d3364f52a0b
Gerrit-PatchSet: 1
Gerrit-Project: vdsm
Gerrit-Branch: master
Gerrit-Owner: Allon Mureinik 
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