Bug 1155721 - Add ppc64el support to virt-install --location (Debian/Ubuntu)
Summary: Add ppc64el support to virt-install --location (Debian/Ubuntu)
Keywords:
Status: CLOSED UPSTREAM
Alias: None
Product: Virtualization Tools
Classification: Community
Component: virt-manager
Version: unspecified
Hardware: ppc64le
OS: Linux
unspecified
medium
Target Milestone: ---
Assignee: Cole Robinson
QA Contact:
URL:
Whiteboard:
Depends On:
Blocks:
TreeView+ depends on / blocked
 
Reported: 2014-10-22 17:33 UTC by Mauricio Faria de Oliveira
Modified: 2015-03-28 20:23 UTC (History)
4 users (show)

Fixed In Version:
Doc Type: Bug Fix
Doc Text:
Clone Of:
Environment:
Last Closed: 2015-03-27 17:11:41 UTC
Embargoed:


Attachments (Terms of Use)
[PATCH] Add ppc64el support to virt-install --location (Debian/Ubuntu) (1.75 KB, application/mbox)
2014-10-22 17:33 UTC, Mauricio Faria de Oliveira
no flags Details

Description Mauricio Faria de Oliveira 2014-10-22 17:33:41 UTC
Created attachment 949507 [details]
[PATCH] Add ppc64el support to virt-install --location (Debian/Ubuntu)

Description of problem:

The installation of Debian/Ubuntu for ppc64el (a.k.a. ppc64le on Fedora/RHEL)
fails with 'virt-install --location', as the arch dir and kernel filename
are not correct for this arch/distros.

This patch fixes it by adding checks for 'treeArch' and 'hvm_kernel_paths'.


Steps to Reproduce:

With either Debian or Ubuntu, see the attempted URL includes a wrong arch dir (i386 != ppc64el) and wrong kernel filename (linux != vmlinux).

	$ URL='http://d-i.debian.org/daily-images/ppc64el/'

	$ virt-install \
	  --name guest-ppc64el \
	  --vcpus 1 \
	  --memory 512 \
	  --disk size=1 \
	  --network default \
	  --location $URL

	WARNING  No 'console' seen in --extra-args, a 'console=ttyS0' kernel argument is likely required to see text install output from the guest.

	Starting install...
	Retrieving file MANIFEST...                                                                                                        |  758 B  00:00:00 !!! 
	ERROR    Couldn't find hvm kernel for Debian tree.

	$ cat ~/.cache/virt-manager/virt-install.log
	[...] DEBUG (urlfetcher:142) HTTP hasFile: didn't find http://d-i.debian.org/daily-images/ppc64el/daily/netboot/debian-installer/i386/linux: HTTP Error 404: Not Found



	$ URL='http://ports.ubuntu.com/ubuntu-ports/dists/trusty-updates/main/installer-ppc64el/'
	
	$ virt-install \
	  --name guest-ppc64el \
	  --vcpus 1 \
	  --memory 512 \
	  --disk size=1 \
	  --network default \
	  --location $URL

	WARNING  No 'console' seen in --extra-args, a 'console=ttyS0' kernel argument is likely required to see text install output from the guest.

	Starting install...
	Retrieving file MANIFEST...                                                                                                        |  804 B  00:00:00 !!! 
	Retrieving file MANIFEST...                                                                                                        |  804 B  00:00:00 !!! 
	ERROR    Couldn't find hvm kernel for Ubuntu tree.

	$ cat ~/.cache/virt-manager/virt-install.log
	DEBUG (urlfetcher:142) HTTP hasFile: didn't find http://ports.ubuntu.com/ubuntu-ports/dists/trusty-updates/main/installer-ppc64el/current/images/netboot/ubuntu-installer/i386/linux: HTTP Error 404: Not Found

Comment 1 Cole Robinson 2015-03-27 17:11:41 UTC
Thanks for the patch, pushed now. Sorry for the long delay:

commit 0dc334fa58a921492e4d8fe1a1cffd1accc021cb
Author: Mauricio Faria de Oliveira <mauricfo.ibm.com>
Date:   Wed Oct 22 15:17:40 2014 -0200

    Add ppc64el support for 'virt-install --location' (Debian/Ubuntu)

Comment 2 Mauricio Faria de Oliveira 2015-03-28 20:23:33 UTC
Thanks, Cole!
No problem at all. :)


Note You need to log in before you can comment on or make changes to this bug.