Project

General

Profile

Bug #15775

katello-certs-check should print absolute paths to certificates

Added by Jake Callahan almost 3 years ago. Updated 10 months ago.

Status:
Closed
Priority:
Normal
Assignee:
Category:
Installer
Target version:
Difficulty:
easy
Triaged:
Bugzilla link:
Team Backlog:
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 almost 3 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 almost 3 years ago

  • Bugzilla link set to 1358935

#2 Updated by The Foreman Bot almost 3 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 almost 3 years ago

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

#4 Updated by Justin Sherrill almost 3 years ago

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

#5 Updated by Eric Helms over 2 years ago

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

Also available in: Atom PDF