Bug 1679482

Summary:	Use a better icon for RHEL 7+ guests
Product:	Red Hat Enterprise Linux 7	Reporter:	Pino Toscano <ptoscano>
Component:	libguestfs	Assignee:	Richard W.M. Jones <rjones>
Status:	CLOSED ERRATA	QA Contact:	YongkuiGuo <yoguo>
Severity:	unspecified	Docs Contact:
Priority:	unspecified
Version:	7.6	CC:	juzhou, linl, ptoscano, yoguo
Target Milestone:	rc
Target Release:	---
Hardware:	Unspecified
OS:	Unspecified
Whiteboard:
Fixed In Version:	libguestfs-1.40.2-2.el7	Doc Type:	If docs needed, set a value
Doc Text:		Story Points:	---
Clone Of:
Clones:	1679483 (view as bug list)		Environment:
Last Closed:	2019-08-06 12:44:47 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 Pino Toscano 2019-02-21 08:42:54 UTC

Description of problem:
When extracting the icon of RHEL 7 and greater (using the inspect_get_icon API), libguestfs uses the file: /usr/share/pixmaps/fedora-logo-sprite.png.
It turned out that this file is not a good choice, as (especially starting with RHEL 8) it is not a proper "logo", but more like a banner.

After talking with Ray Strode (who maintains redhat-logos), the better option for this is to use one of the system-logo-icon.png icons in the hicolor icon theme, for example:
/usr/share/icons/hicolor/96x96/apps/system-logo-icon.png

Steps to Reproduce:
1. get the icon of a RHEL 7 guest, for example using virt-inspector, or in virt-manager (after installing python-libguestfs)
2. look at the icon

Actual results:
- in RHEL 7 guests, it is a shadowman
- in RHEL 8 guests, it is a RHEL "banner"

Expected results:
- the same shadowman in RHEL 7 & 8

Additional info:
Patch proposed upstream for this:
https://www.redhat.com/archives/libguestfs/2019-February/msg00025.html

Comment 2 Pino Toscano 2019-03-26 08:17:11 UTC

Fixed upstream with
https://github.com/libguestfs/libguestfs/commit/c648052690a4a07a59c741308e0e8497d6f18057
which is in libguestfs >= 1.41.1.

Comment 4 YongkuiGuo 2019-03-27 10:05:05 UTC

Verified with package:
libguestfs-1.40.2-2.el7.x86_64

Steps:

1. On rhel7.6 host
# virt-inspector -a rhel7.6.raw | virt-inspector --xpath 'string(//icon)' | base64 -i -d | display -
# virt-inspector -a rhel8.0.raw | virt-inspector --xpath 'string(//icon)' | base64 -i -d | display -

Both are the shadowman. Verified this bug.

Comment 6 errata-xmlrpc 2019-08-06 12:44:47 UTC

Since the problem described in this bug report should be
resolved in a recent advisory, it has been closed with a
resolution of ERRATA.

For information on the advisory, and where to find the updated
files, follow the link below.

If the solution does not work for you, open a new bug report.

https://access.redhat.com/errata/RHEA-2019:2096