Change in vdsm[ovirt-3.3]: spec: replace requires vdsm-python-cpopen

[Douglas Schilling Landgraf]

Douglas Schilling Landgraf has uploaded a new change for review.

Change subject: spec: replace requires vdsm-python-cpopen
......................................................................

spec: replace requires vdsm-python-cpopen

python-cpopen obsoletes vdsm-python-cpopen

Change-Id: Iee2c64d489745db5d0e4a8491d7d8b16dfb1d21b
Signed-off-by: Douglas Schilling Landgraf <dougsland(a)redhat.com&gt;
---
M vdsm.spec.in
1 file changed, 2 insertions(+), 2 deletions(-)


  git pull ssh://gerrit.ovirt.org:29418/vdsm refs/changes/42/23942/1

diff --git a/vdsm.spec.in b/vdsm.spec.in
index a1fa38c..1435e0a 100644
--- a/vdsm.spec.in
+++ b/vdsm.spec.in
@@ -242,7 +242,7 @@
 
 Requires: %{name}-python = %{version}-%{release}
 Requires: %{name}-xmlrpc = %{version}-%{release}
-Requires: %{name}-python-cpopen
+Requires: python-cpopen
 
 %description cli
 Call VDSM commands from the command line. Used for testing and debugging.
@@ -252,7 +252,7 @@
 BuildArch:      noarch
 
 Requires: %{name}-python = %{version}-%{release}
-Requires: %{name}-python-cpopen
+Requires: python-cpopen
 
 
 %description xmlrpc


-- 
To view, visit http://gerrit.ovirt.org/23942
To unsubscribe, visit http://gerrit.ovirt.org/settings

Gerrit-MessageType: newchange
Gerrit-Change-Id: Iee2c64d489745db5d0e4a8491d7d8b16dfb1d21b
Gerrit-PatchSet: 1
Gerrit-Project: vdsm
Gerrit-Branch: ovirt-3.3
Gerrit-Owner: Douglas Schilling Landgraf <dougsland(a)redhat.com&gt;

[Douglas Schilling Landgraf]

Douglas Schilling Landgraf has posted comments on this change.

Change subject: spec: replace requires vdsm-python-cpopen
......................................................................


Patch Set 1:

Need to verify.

-- 
To view, visit http://gerrit.ovirt.org/23942
To unsubscribe, visit http://gerrit.ovirt.org/settings

Gerrit-MessageType: comment
Gerrit-Change-Id: Iee2c64d489745db5d0e4a8491d7d8b16dfb1d21b
Gerrit-PatchSet: 1
Gerrit-Project: vdsm
Gerrit-Branch: ovirt-3.3
Gerrit-Owner: Douglas Schilling Landgraf <dougsland(a)redhat.com&gt;
Gerrit-Reviewer: Dan Kenigsberg <danken(a)redhat.com&gt;
Gerrit-Reviewer: Douglas Schilling Landgraf <dougsland(a)redhat.com&gt;
Gerrit-Reviewer: Sandro Bonazzola <sbonazzo(a)redhat.com&gt;
Gerrit-Reviewer: Yaniv Bronhaim <ybronhei(a)redhat.com&gt;
Gerrit-HasComments: No

[Dan Kenigsberg]

Dan Kenigsberg has submitted this change and it was merged.

Change subject: spec: replace requires vdsm-python-cpopen
......................................................................


spec: replace requires vdsm-python-cpopen

Due dependency problems and yum bug [1] we are
obsoleting vdsm-python-cpopen with python-cpopen to
avoid installation problem with VDSM.

[1]
http://yum.baseurl.org/gitweb?p=yum.git;a=commitdiff;h=1bbf40af8548d2d3fb...

Change-Id: Iee2c64d489745db5d0e4a8491d7d8b16dfb1d21b
Signed-off-by: Douglas Schilling Landgraf <dougsland(a)redhat.com&gt;
Reviewed-on: http://gerrit.ovirt.org/23942
Reviewed-by: Sandro Bonazzola <sbonazzo(a)redhat.com&gt;
Reviewed-by: Dan Kenigsberg <danken(a)redhat.com&gt;
---
M vdsm.spec.in
1 file changed, 2 insertions(+), 2 deletions(-)

Approvals:
  Douglas Schilling Landgraf: Verified
  Sandro Bonazzola: Looks good to me, but someone else must approve
  Dan Kenigsberg: Looks good to me, approved



-- 
To view, visit http://gerrit.ovirt.org/23942
To unsubscribe, visit http://gerrit.ovirt.org/settings

Gerrit-MessageType: merged
Gerrit-Change-Id: Iee2c64d489745db5d0e4a8491d7d8b16dfb1d21b
Gerrit-PatchSet: 3
Gerrit-Project: vdsm
Gerrit-Branch: ovirt-3.3
Gerrit-Owner: Douglas Schilling Landgraf <dougsland(a)redhat.com&gt;
Gerrit-Reviewer: Barak Azulay <bazulay(a)redhat.com&gt;
Gerrit-Reviewer: Dan Kenigsberg <danken(a)redhat.com&gt;
Gerrit-Reviewer: Douglas Schilling Landgraf <dougsland(a)redhat.com&gt;
Gerrit-Reviewer: Sandro Bonazzola <sbonazzo(a)redhat.com&gt;
Gerrit-Reviewer: Yaniv Bronhaim <ybronhei(a)redhat.com&gt;
Gerrit-Reviewer: oVirt Jenkins CI Server

[Jenkins CI RO]

oVirt Jenkins CI Server has posted comments on this change.

Change subject: spec: replace requires vdsm-python-cpopen
......................................................................


Patch Set 3:

Build Successful 

http://jenkins.ovirt.org/job/vdsm_3.3_install_rpm_sanity_gerrit/357/ : SUCCESS

-- 
To view, visit http://gerrit.ovirt.org/23942
To unsubscribe, visit http://gerrit.ovirt.org/settings

Gerrit-MessageType: comment
Gerrit-Change-Id: Iee2c64d489745db5d0e4a8491d7d8b16dfb1d21b
Gerrit-PatchSet: 3
Gerrit-Project: vdsm
Gerrit-Branch: ovirt-3.3
Gerrit-Owner: Douglas Schilling Landgraf <dougsland(a)redhat.com&gt;
Gerrit-Reviewer: Barak Azulay <bazulay(a)redhat.com&gt;
Gerrit-Reviewer: Dan Kenigsberg <danken(a)redhat.com&gt;
Gerrit-Reviewer: Douglas Schilling Landgraf <dougsland(a)redhat.com&gt;
Gerrit-Reviewer: Sandro Bonazzola <sbonazzo(a)redhat.com&gt;
Gerrit-Reviewer: Yaniv Bronhaim <ybronhei(a)redhat.com&gt;
Gerrit-Reviewer: oVirt Jenkins CI Server
Gerrit-HasComments: No

[Jenkins CI RO]

oVirt Jenkins CI Server has posted comments on this change.

Change subject: spec: replace requires vdsm-python-cpopen
......................................................................


Patch Set 2:

Build Failed 

http://jenkins.ovirt.org/job/vdsm_3.3_install_rpm_sanity_gerrit/350/ : ABORTED

http://jenkins.ovirt.org/job/vdsm_3.4_install_rpm_sanity_gerrit/8/ : SUCCESS

-- 
To view, visit http://gerrit.ovirt.org/23942
To unsubscribe, visit http://gerrit.ovirt.org/settings

Gerrit-MessageType: comment
Gerrit-Change-Id: Iee2c64d489745db5d0e4a8491d7d8b16dfb1d21b
Gerrit-PatchSet: 2
Gerrit-Project: vdsm
Gerrit-Branch: ovirt-3.3
Gerrit-Owner: Douglas Schilling Landgraf <dougsland(a)redhat.com&gt;
Gerrit-Reviewer: Barak Azulay <bazulay(a)redhat.com&gt;
Gerrit-Reviewer: Dan Kenigsberg <danken(a)redhat.com&gt;
Gerrit-Reviewer: Douglas Schilling Landgraf <dougsland(a)redhat.com&gt;
Gerrit-Reviewer: Sandro Bonazzola <sbonazzo(a)redhat.com&gt;
Gerrit-Reviewer: Yaniv Bronhaim <ybronhei(a)redhat.com&gt;
Gerrit-Reviewer: oVirt Jenkins CI Server
Gerrit-HasComments: No

[Dan Kenigsberg]

Dan Kenigsberg has posted comments on this change.

Change subject: spec: replace requires vdsm-python-cpopen
......................................................................


Patch Set 2:

Merging now, to save time for rebuilding ovirt-3.3.3's vdsm.

-- 
To view, visit http://gerrit.ovirt.org/23942
To unsubscribe, visit http://gerrit.ovirt.org/settings

Gerrit-MessageType: comment
Gerrit-Change-Id: Iee2c64d489745db5d0e4a8491d7d8b16dfb1d21b
Gerrit-PatchSet: 2
Gerrit-Project: vdsm
Gerrit-Branch: ovirt-3.3
Gerrit-Owner: Douglas Schilling Landgraf <dougsland(a)redhat.com&gt;
Gerrit-Reviewer: Barak Azulay <bazulay(a)redhat.com&gt;
Gerrit-Reviewer: Dan Kenigsberg <danken(a)redhat.com&gt;
Gerrit-Reviewer: Douglas Schilling Landgraf <dougsland(a)redhat.com&gt;
Gerrit-Reviewer: Sandro Bonazzola <sbonazzo(a)redhat.com&gt;
Gerrit-Reviewer: Yaniv Bronhaim <ybronhei(a)redhat.com&gt;
Gerrit-Reviewer: oVirt Jenkins CI Server
Gerrit-HasComments: No

[Dan Kenigsberg]

Dan Kenigsberg has posted comments on this change.

Change subject: spec: replace requires vdsm-python-cpopen
......................................................................


Patch Set 2: Code-Review+2

-- 
To view, visit http://gerrit.ovirt.org/23942
To unsubscribe, visit http://gerrit.ovirt.org/settings

Gerrit-MessageType: comment
Gerrit-Change-Id: Iee2c64d489745db5d0e4a8491d7d8b16dfb1d21b
Gerrit-PatchSet: 2
Gerrit-Project: vdsm
Gerrit-Branch: ovirt-3.3
Gerrit-Owner: Douglas Schilling Landgraf <dougsland(a)redhat.com&gt;
Gerrit-Reviewer: Barak Azulay <bazulay(a)redhat.com&gt;
Gerrit-Reviewer: Dan Kenigsberg <danken(a)redhat.com&gt;
Gerrit-Reviewer: Douglas Schilling Landgraf <dougsland(a)redhat.com&gt;
Gerrit-Reviewer: Sandro Bonazzola <sbonazzo(a)redhat.com&gt;
Gerrit-Reviewer: Yaniv Bronhaim <ybronhei(a)redhat.com&gt;
Gerrit-Reviewer: oVirt Jenkins CI Server
Gerrit-HasComments: No

[Jenkins CI RO]

oVirt Jenkins CI Server has posted comments on this change.

Change subject: spec: replace requires vdsm-python-cpopen
......................................................................


Patch Set 2:

Build Successful 

http://jenkins.ovirt.org/job/vdsm_3.3_install_rpm_sanity_gerrit/349/ : SUCCESS

http://jenkins.ovirt.org/job/vdsm_3.4_install_rpm_sanity_gerrit/7/ : SUCCESS

-- 
To view, visit http://gerrit.ovirt.org/23942
To unsubscribe, visit http://gerrit.ovirt.org/settings

Gerrit-MessageType: comment
Gerrit-Change-Id: Iee2c64d489745db5d0e4a8491d7d8b16dfb1d21b
Gerrit-PatchSet: 2
Gerrit-Project: vdsm
Gerrit-Branch: ovirt-3.3
Gerrit-Owner: Douglas Schilling Landgraf <dougsland(a)redhat.com&gt;
Gerrit-Reviewer: Dan Kenigsberg <danken(a)redhat.com&gt;
Gerrit-Reviewer: Douglas Schilling Landgraf <dougsland(a)redhat.com&gt;
Gerrit-Reviewer: Sandro Bonazzola <sbonazzo(a)redhat.com&gt;
Gerrit-Reviewer: Yaniv Bronhaim <ybronhei(a)redhat.com&gt;
Gerrit-Reviewer: oVirt Jenkins CI Server
Gerrit-HasComments: No

[Douglas Schilling Landgraf]

Douglas Schilling Landgraf has posted comments on this change.

Change subject: spec: replace requires vdsm-python-cpopen
......................................................................


Patch Set 1: Verified+1

On a clean Fedora 19 install:
sudo yum localinstall http://ovirt.org/releases/ovirt-release-fedora.noarch.rpm


# yum repolist
repo id                                                              repo name            
                                                                      status
fedora/19/x86_64                                                     Fedora 19 - x86_64   
                                                                      36,253
ovirt-3.3.2/19                                                       oVirt 3.3.2 release  
                                                                         152
ovirt-stable/19                                                      Older Stable builds
of the oVirt project                                                       406
updates/19/x86_64                                                    Fedora 19 - x86_64 -
Updates                                                                17,384
repolist: 54,195


(/me build locally the vdsm with this patch and there is no conflict)

# yum install ./noarch/vdsm-xmlrpc-4.13.3.1-9.git7086012.fc19.noarch.rpm
./x86_64/vdsm-4.13.3.1-9.git7086012.fc19.x86_64.rpm
./x86_64/vdsm-python-4.13.3.1-9.git7086012.fc19.x86_64.rpm

-- 
To view, visit http://gerrit.ovirt.org/23942
To unsubscribe, visit http://gerrit.ovirt.org/settings

Gerrit-MessageType: comment
Gerrit-Change-Id: Iee2c64d489745db5d0e4a8491d7d8b16dfb1d21b
Gerrit-PatchSet: 1
Gerrit-Project: vdsm
Gerrit-Branch: ovirt-3.3
Gerrit-Owner: Douglas Schilling Landgraf <dougsland(a)redhat.com&gt;
Gerrit-Reviewer: Dan Kenigsberg <danken(a)redhat.com&gt;
Gerrit-Reviewer: Douglas Schilling Landgraf <dougsland(a)redhat.com&gt;
Gerrit-Reviewer: Sandro Bonazzola <sbonazzo(a)redhat.com&gt;
Gerrit-Reviewer: Yaniv Bronhaim <ybronhei(a)redhat.com&gt;
Gerrit-Reviewer: oVirt Jenkins CI Server
Gerrit-HasComments: No

[Dan Kenigsberg]

Dan Kenigsberg has posted comments on this change.

Change subject: spec: replace requires vdsm-python-cpopen
......................................................................


Patch Set 1:

Specifically, you need to verify that with this change only, you could `yum install vdsm`
from a yum repo created of the ovirt-3.3 branch.

Please improve the commit message and explain why the removal of vdsm-python-cpopen was
needed.

-- 
To view, visit http://gerrit.ovirt.org/23942
To unsubscribe, visit http://gerrit.ovirt.org/settings

Gerrit-MessageType: comment
Gerrit-Change-Id: Iee2c64d489745db5d0e4a8491d7d8b16dfb1d21b
Gerrit-PatchSet: 1
Gerrit-Project: vdsm
Gerrit-Branch: ovirt-3.3
Gerrit-Owner: Douglas Schilling Landgraf <dougsland(a)redhat.com&gt;
Gerrit-Reviewer: Dan Kenigsberg <danken(a)redhat.com&gt;
Gerrit-Reviewer: Douglas Schilling Landgraf <dougsland(a)redhat.com&gt;
Gerrit-Reviewer: Sandro Bonazzola <sbonazzo(a)redhat.com&gt;
Gerrit-Reviewer: Yaniv Bronhaim <ybronhei(a)redhat.com&gt;
Gerrit-Reviewer: oVirt Jenkins CI Server
Gerrit-HasComments: No

[Jenkins CI RO]

oVirt Jenkins CI Server has posted comments on this change.

Change subject: spec: replace requires vdsm-python-cpopen
......................................................................


Patch Set 1:

Build Successful 

http://jenkins.ovirt.org/job/vdsm_3.3_install_rpm_sanity_gerrit/348/ : SUCCESS

http://jenkins.ovirt.org/job/vdsm_3.4_install_rpm_sanity_gerrit/6/ : SUCCESS

-- 
To view, visit http://gerrit.ovirt.org/23942
To unsubscribe, visit http://gerrit.ovirt.org/settings

Gerrit-MessageType: comment
Gerrit-Change-Id: Iee2c64d489745db5d0e4a8491d7d8b16dfb1d21b
Gerrit-PatchSet: 1
Gerrit-Project: vdsm
Gerrit-Branch: ovirt-3.3
Gerrit-Owner: Douglas Schilling Landgraf <dougsland(a)redhat.com&gt;
Gerrit-Reviewer: Dan Kenigsberg <danken(a)redhat.com&gt;
Gerrit-Reviewer: Douglas Schilling Landgraf <dougsland(a)redhat.com&gt;
Gerrit-Reviewer: Sandro Bonazzola <sbonazzo(a)redhat.com&gt;
Gerrit-Reviewer: Yaniv Bronhaim <ybronhei(a)redhat.com&gt;
Gerrit-Reviewer: oVirt Jenkins CI Server
Gerrit-HasComments: No

[Sandro Bonazzola]

Sandro Bonazzola has posted comments on this change.

Change subject: spec: replace requires vdsm-python-cpopen
......................................................................


Patch Set 1: Code-Review+1

-- 
To view, visit http://gerrit.ovirt.org/23942
To unsubscribe, visit http://gerrit.ovirt.org/settings

Gerrit-MessageType: comment
Gerrit-Change-Id: Iee2c64d489745db5d0e4a8491d7d8b16dfb1d21b
Gerrit-PatchSet: 1
Gerrit-Project: vdsm
Gerrit-Branch: ovirt-3.3
Gerrit-Owner: Douglas Schilling Landgraf <dougsland(a)redhat.com&gt;
Gerrit-Reviewer: Dan Kenigsberg <danken(a)redhat.com&gt;
Gerrit-Reviewer: Douglas Schilling Landgraf <dougsland(a)redhat.com&gt;
Gerrit-Reviewer: Sandro Bonazzola <sbonazzo(a)redhat.com&gt;
Gerrit-Reviewer: Yaniv Bronhaim <ybronhei(a)redhat.com&gt;
Gerrit-HasComments: No