Bug #15775
katello-certs-check should print absolute paths to certificates
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
History
#1
Updated by Brad Buckingham almost 7 years ago
- Bugzilla link set to 1358935
#2
Updated by The Foreman Bot almost 7 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 7 years ago
- Status changed from Ready For Testing to Closed
- % Done changed from 0 to 100
Applied in changeset katello-installer|b57b21e46f51490bd7d5e1f5f5d18b03fa494ed2.
#4
Updated by Justin Sherrill almost 7 years ago
- Legacy Backlogs Release (now unused) set to 171
- Difficulty set to easy
#5
Updated by Eric Helms over 6 years ago
- Legacy Backlogs Release (now unused) changed from 171 to 162
Fixes #15775 - Use absolute paths instead of relative. (#379)
Switching to absolute paths to avoid issues with validate_absolute_path
during upgrades.
Fixes #15775