Project

General

Profile

Bug #15775

katello-certs-check should print absolute paths to certificates

Added by Jake Callahan about 4 years ago. Updated about 2 years ago.

Status:
Closed
Priority:
Normal
Assignee:
Category:
Installer
Target version:
Difficulty:
easy
Triaged:
Bugzilla link:
Fixed in Releases:
Found in Releases:

Description

Description of problem:
The script currently outputs installer suggestions. However, it will use whatever relative path the user passes in as an argument. This caused an error during an upgrade from 6.1.9 to 6.2.
"rhsm-qe-3.rhq.lab.eng.bos.redhat.com.crt" is not an absolute path. at /usr/share/katello-installer-base/modules/certs/manifests/init.pp:115 on node rhsm-qe-3.rhq.lab.eng.bos.redhat.com

How reproducible:
Always

Steps to Reproduce:
1. run the katello-certs-check script while passing in relative paths to the cert files.

Actual results:
katello-installer --certs-server-cert "rhsm-qe-3.rhq.lab.eng.bos.redhat.com.crt"\
--certs-server-cert-req "rhsm-qe-3.rhq.lab.eng.bos.redhat.com.crt.req"\
--certs-server-key "rhsm-qe-3.rhq.lab.eng.bos.redhat.com.key"\
--certs-server-ca-cert "cacert.crt"\
--certs-update-server --certs-update-server-ca

Expected results:
satellite-installer --scenario satellite\
--certs-server-cert "/root/sat_cert/rhsm-qe-3.rhq.lab.eng.bos.redhat.com.crt"\
--certs-server-cert-req "/root/sat_cert/rhsm-qe-3.rhq.lab.eng.bos.redhat.com.crt.req"\
--certs-server-key "/root/sat_cert/rhsm-qe-3.rhq.lab.eng.bos.redhat.com.key"\
--certs-server-ca-cert "/root/sat_cert/cacert.crt"

Associated revisions

Revision b57b21e4 (diff)
Added by Jacob Callahan about 4 years ago

Fixes #15775 - Use absolute paths instead of relative. (#379)

Switching to absolute paths to avoid issues with validate_absolute_path
during upgrades.
Fixes #15775

History

#1 Updated by Brad Buckingham about 4 years ago

  • Bugzilla link set to 1358935

#2 Updated by The Foreman Bot about 4 years ago

  • Status changed from New to Ready For Testing
  • Pull request https://github.com/Katello/katello-installer/pull/379 added

#3 Updated by Jacob Callahan about 4 years ago

  • Status changed from Ready For Testing to Closed
  • % Done changed from 0 to 100

#4 Updated by Justin Sherrill about 4 years ago

  • Legacy Backlogs Release (now unused) set to 171
  • Difficulty set to easy

#5 Updated by Eric Helms almost 4 years ago

  • Legacy Backlogs Release (now unused) changed from 171 to 162

Also available in: Atom PDF