Bug 1825841

Summary: Updating rubygem-foreman_maintain package using "satellite-maintain packages" command should not execute installer at the end
Product: Red Hat Satellite Reporter: Ashish Humbe <ahumbe>
Component: Satellite MaintainAssignee: Kavita <kgaikwad>
Status: CLOSED ERRATA QA Contact: Jameer Pathan <jpathan>
Severity: medium Docs Contact:
Priority: medium    
Version: 6.7.0CC: apatel, bkearney, kgaikwad
Target Milestone: 6.8.0Keywords: Triaged
Target Release: Unused   
Hardware: x86_64   
OS: Linux   
Whiteboard:
Fixed In Version: rubygem-foreman_maintain-0.6.4 Doc Type: If docs needed, set a value
Doc Text:
Story Points: ---
Clone Of: Environment:
Last Closed: 2020-10-27 12:38:21 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 Ashish Humbe 2020-04-20 11:33:28 UTC
Description of problem:
Customers are expected to do any package install/update on satellite server via satellite-maintain command, in the Satellite upgrade guide we are suggesting customers to use "yum" command to update rubygem-foreman_maintain package, this package is whitelisted in foreman-protector plugin so it is allowed to update via yum. 

When we try to update the rubygem-foreman_maintain package using satellite-maintain command it executes satellite-installer command and that causes unexpected service downtime. 

Version-Release number of selected component (if applicable):
Satellite 6.x 

How reproducible:
Always 

Steps to Reproduce:
1. satellite-maintain packages install rubygem-foreman_maintain

which stops satellite services and execute the installer command. 

Actual results:

# satellite-maintain packages install rubygem-foreman_maintain
Running preparation steps required to run the next scenarios
================================================================================
Check if tooling for package locking is installed:                    [OK]
--------------------------------------------------------------------------------
 
 
Running install packages in unlocked session
================================================================================
Confirm installer run is allowed:
WARNING: This script runs satellite-installer after the yum execution
to ensure the Satellite is in a consistent state.
As a result some of your services may be restarted.
 
Do you want to proceed?, [y(yes), q(quit)] y
                                                                      [OK]      
--------------------------------------------------------------------------------
Unlock packages:                                                      [OK]
--------------------------------------------------------------------------------
Install packages: Loaded plugins: product-id, search-disabled-repos, subscription-manager
.
.
.
.
Updated:
  rubygem-foreman_maintain.noarch 1:0.5.3-1.el7sat                                                                                                                          
 
Complete!
                                                    [OK]
--------------------------------------------------------------------------------
Running satellite-installer --upgrade --disable-system-checks: Upgrading, to monitor the progress on all related services, please do:
  foreman-tail | tee upgrade-$(date +%Y-%m-%d-%H%M).log
Upgrade Step: stop_services...
Running Stop Services
================================================================================



Expected results:
We should allow and promote customers use satellite-maintain to update the rubygem-foreman_maintain package without running any extra installer related steps. 

Additional info:

Comment 3 Bryan Kearney 2020-05-14 12:04:00 UTC
Moving this bug to POST for triage into Satellite 6 since the upstream issue https://projects.theforeman.org/issues/29511 has been resolved.

Comment 4 Jameer Pathan 2020-07-27 07:30:31 UTC
Verified:

Verified with:
- Satellite 6.8.0 snap 9
- rubygem-foreman_maintain-0.6.8-1.el7sat.noarch

Test step:
- foreman-maintain packages install rubygem-foreman_maintain

Observation:
- foreman-maintain package got updated and satellite-installer was not executed after package update.

Comment 7 errata-xmlrpc 2020-10-27 12:38:21 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 (Satellite 6.8 Satellite Maintenance 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/RHBA-2020:4365