Bug 2164757

Summary: Require rubygem(foreman_google) for Satellite installer
Product: Red Hat Satellite Reporter: Leos Stejskal <lstejska>
Component: PackagingAssignee: Leos Stejskal <lstejska>
Status: CLOSED ERRATA QA Contact: Gaurav Talreja <gtalreja>
Severity: high Docs Contact:
Priority: unspecified    
Version: 6.13.0CC: ehelms, gtalreja, pcreech, rlavi, zhunting
Target Milestone: 6.13.0Keywords: Triaged
Target Release: Unused   
Hardware: Unspecified   
OS: Unspecified   
Whiteboard:
Fixed In Version: Doc Type: If docs needed, set a value
Doc Text:
Story Points: ---
Clone Of: Environment:
Last Closed: 2023-05-03 13:24:53 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:
Bug Depends On:    
Bug Blocks: 2163425    

Description Leos Stejskal 2023-01-26 11:52:24 UTC
Description of problem:

Require foreman-google for Satellite installer

Version-Release number of selected component (if applicable):
6.13

How reproducible:
always

Comment 1 Ron Lavi 2023-02-01 07:50:59 UTC
https://gitlab.cee.redhat.com/satellite/satellite-packaging/-/merge_requests/6821 got merged,
moving to POST

Comment 2 Gaurav Talreja 2023-02-03 12:19:32 UTC
Tested on Satellite 6.13.0 Snap 9.0

foreman_google isn't included as the requirement of the satellite package, with other CR packages. Maybe this is the reason for the https://bugzilla.redhat.com/show_bug.cgi?id=2163425#c7, but I am not sure if it is the cause. Can anyone confirm this?

# rpm -q satellite --requires
ansible-collection-redhat-satellite_operations
foreman >= 3.5.0
foreman-dynflow-sidekiq
foreman-ec2
foreman-libvirt
foreman-openstack
foreman-ovirt
foreman-service
foreman-vmware
...

Comment 4 Gaurav Talreja 2023-02-13 15:23:45 UTC
Tested on Satellite 6.13.0 Snap 10.0

Steps:
1. # rpm -q satellite --requires
ansible-collection-redhat-satellite_operations
foreman >= 3.5.0
foreman-dynflow-sidekiq
foreman-ec2
foreman-libvirt
foreman-openstack
foreman-ovirt
foreman-service
foreman-vmware
katello >= 4.7.0
puppet-foreman_scap_client
rpmlib(CompressedFileNames) <= 3.0.4-1
rpmlib(FileDigests) <= 4.6.0-1
rpmlib(PayloadFilesHavePrefix) <= 4.0-1
rpmlib(PayloadIsXz) <= 5.2-1
rubygem(foreman_ansible)
rubygem(foreman_azure_rm)
rubygem(foreman_bootdisk)
rubygem(foreman_discovery)
rubygem(foreman_google)
rubygem(foreman_hooks)
rubygem(foreman_remote_execution)
rubygem(foreman_templates)
rubygem(foreman_theme_satellite)
rubygem(foreman_virt_who_configure)
satellite-cli
satellite-common

Observation:
rubygem(foreman_google) is included, which wasn't in snap 9 as per comment 2, and foreman-google is not there with another foreman-* CR's.
Can you please confirm if this is expected and are we good to verify this BZ?


Thanks, 
Gaurav

Comment 5 Eric Helms 2023-02-14 14:16:10 UTC
The title of the BZ is a bit misleading, the plugin is rubygem(foreman_google) which you show as being there so this BZ is good to verify.

Comment 6 Gaurav Talreja 2023-02-14 14:54:52 UTC
Eric, Thank you for confirming. I updated the BZ title. 

Please update if it still appears misleading, moving this to VERIFIED.

Comment 9 errata-xmlrpc 2023-05-03 13:24:53 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 (Important: Satellite 6.13 Release), 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/RHSA-2023:2097