Bug 2049304

Summary: katello-rhsm-consumer script subscription-manager version detection depends on subscription-manager rpm being installed
Product: Red Hat Satellite Reporter: Martin Kolman <mkolman>
Component: InstallationAssignee: satellite6-bugs <satellite6-bugs>
Status: CLOSED ERRATA QA Contact: Omkar Khatavkar <okhatavk>
Severity: medium Docs Contact:
Priority: unspecified    
Version: 6.10.0CC: cdonnell, egolov, gtalreja, mkolman, pcreech, zhunting
Target Milestone: 6.11.0Keywords: Triaged
Target Release: Unused   
Hardware: Unspecified   
OS: Unspecified   
Whiteboard:
Fixed In Version: puppet-foreman_proxy_content-20.2.0 Doc Type: If docs needed, set a value
Doc Text:
Story Points: ---
Clone Of: Environment:
Last Closed: 2022-07-05 14:32:43 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 Martin Kolman 2022-02-01 21:35:46 UTC
Description of problem:
The subman version detection code in the katello-rhsm-consumer script depends on the subscription-manager being installed from an RPM and an RPM database actually being available.

Code in questions:


# configure rhsm
  # the config command was introduced in rhsm 0.96.6
  # fallback left for older versions
  if test ${RHSM_VERSION[0]:-0} -gt 0 -o ${RHSM_VERSION[1]:-0} -gt 96 -o \( ${RHSM_VERSION[1]:-0} -eq 96 -a ${RHSM_VERSION[2]:-0} -gt 6 \); then
    subscription-manager config \
      --server.hostname="$KATELLO_SERVER" \
      --server.prefix="$PREFIX" \
      --server.port="$PORT" \
      --rhsm.repo_ca_cert="%(ca_cert_dir)s$KATELLO_SERVER_CA_CERT" \
      --rhsm.baseurl="$BASEURL"

    # Older versions of subscription manager may not recognize
    # report_package_profile and package_profile_on_trans options.
    # So set them separately and redirect out & error to /dev/null
    # to fail silently.
    subscription-manager config --rhsm.package_profile_on_trans=1 > /dev/null 2>&1 || true
    subscription-manager config --rhsm.report_package_profile=1 > /dev/null 2>&1 || true
  else
    sed -i "s/^hostname\s*=.*/hostname = $KATELLO_SERVER/g" $CFG
    sed -i "s/^port\s*=.*/port = $PORT/g" $CFG
    sed -i "s|^prefix\s*=.*|prefix = $PREFIX|g" $CFG
    sed -i "s|^repo_ca_cert\s*=.*|repo_ca_cert = %(ca_cert_dir)s$KATELLO_SERVER_CA_CERT|g" $CFG
    sed -i "s|^baseurl\s*=.*|baseurl=$BASEURL|g" $CFG
  fi


This might not always work, for example if the given system is not RPM based or possibly is a minimal immutable system without a RPM database present.

Parsing the versions out of the subscription-manager command output should be more robust.


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


How reproducible:
always

Steps to Reproduce:
1. boot into a system that has subscription-manager command present but subscription-manager RPM is nor marked as installed (for example any RHEL 9 installation boot.iso)
2. download and run the Satellite bootstrap script

Actual results:
Version detection fails and the bootsrap script will enter the sub-optimal fallback branch.

Expected results:
Version detection works even if subscription-manager RPM is no marked as installed, fallback branch not used.

Additional info:
The RHEL 9 installation image (AKA the "boot.iso") is a minimal image that doesn't contain a local RPM database. 

(Installation image size is important as it directly drives customer storage, RAM and data transfer requirements for RHEL installations.)

The Anaconda installer runs the Bootstrap script during a registration attempt to a Satellite instance, the version check fails due to missing RPM database and the bootstrap script falls back to the sed based rhsm.config manipulation method.
This directly triggers RHSM bug 2049296 and requires Anaconda side workaround to still work correctly.

Comment 2 Evgeni Golov 2022-02-07 13:16:40 UTC
Hi Martin,

is there a way to get the sub-man version in machine readable way? parsing the (rather verbose) output of "subscription-manager version" doesn't sound robust at all to me.

Comment 4 Evgeni Golov 2022-02-07 15:20:43 UTC
Created redmine issue https://projects.theforeman.org/issues/34406 from this bug

Comment 8 errata-xmlrpc 2022-07-05 14:32:43 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 (Moderate: Satellite 6.11 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-2022:5498

Comment 9 Red Hat Bugzilla 2023-09-15 01:51:32 UTC
The needinfo request[s] on this closed bug have been removed as they have been unresolved for 365 days