Bug 1187266

Summary: remove virt-manager as dependency from /distribution/virt/install test
Product: [Retired] Beaker Reporter: Jan Stancek <jstancek>
Component: testsAssignee: Dan Callaghan <dcallagh>
Status: CLOSED CURRENTRELEASE QA Contact: tools-bugs <tools-bugs>
Severity: unspecified Docs Contact:
Priority: unspecified    
Version: 19CC: aigao, asaha, bpeck, dcallagh, dowang, dzickus, jburke
Target Milestone: 19.3Keywords: Patch, TestCaseNeeded
Target Release: ---   
Hardware: Unspecified   
OS: Unspecified   
Whiteboard:
Fixed In Version: Doc Type: Bug Fix
Doc Text:
Story Points: ---
Clone Of: Environment:
Last Closed: 2015-03-17 06:34:26 UTC Type: Bug
Regression: --- Mount Type: ---
Documentation: --- CRM:
Verified Versions: Category: ---
oVirt Team: --- RHEL 7.3 requirements from Atomic Host:
Cloudforms Team: --- Target Upstream Version:
Embargoed:

Description Jan Stancek 2015-01-29 16:38:26 UTC
Description of problem:
virt-manager doesn't seem to be used for anything in /virt/install test and it's causing trouble for RHEL7 powerppc, because this package is not going to be shipped as part of RHEL. It pulls in couple other packages, which also are not going to be shipped with RHEL.

It seems this dependency can be removed entirely, as test is not interacting with virt-manager in any way. Furthermore it appears as redundant, at least on RHEL7, where @virtualization-client package group contains it as well:

Removing it from Makefile allows powerpc to use virt-manager repo (to install virt-install) and at the same time it doesn't pull in virt-manager and its dependencies.

Group: Virtualization Client
 Group-Id: virtualization-client
 Description: Clients for installing and managing virtualization instances.
 Mandatory Packages:
    gnome-boxes
    virt-install
    virt-manager
    virt-viewer
 Default Packages:
    virt-top
 Optional Packages:
   libguestfs-tools
   libguestfs-tools-c

Proposed patch:
        @echo "Requires:        mkisofs" >> $(METADATA)
        @echo "Requires:        openssh-server" >> $(METADATA)
        @echo "Requires:        python-virtinst" >> $(METADATA)
-       @echo "Requires:        virt-manager" >> $(METADATA)
        @echo "Requires:        virt-viewer" >> $(METADATA)
        @echo "Requires:        virt-install" >> $(METADATA)
        @echo "Requires:        vnc" >> $(METADATA)

Comment 3 Nick Coghlan 2015-01-30 04:53:36 UTC
+1 from me.

While we'll announce this upstream in the Beaker 20 release notes, it can be included and a new version of the task published before then.

Comment 4 Dan Callaghan 2015-01-30 04:58:45 UTC
http://gerrit.beaker-project.org/3961

Comment 5 Dan Callaghan 2015-01-30 05:22:10 UTC
Tagged as /distribution/virt/install 4.0-88.

Comment 10 Dan Callaghan 2015-03-17 06:34:26 UTC
Beaker 19.3 has been released.