Bug #20921
katello-change-hostname tries to change the wrong default proxy if default proxy id has multiple digits
Description
k-c-h currently uses the following to get the ID of the default proxy:
default_capsule_id = `hammer -u #{@options[:username]} -p #{@options[:password]} capsule list | grep #{old_hostname} | awk '{ print $1 }'`[0]
this only works for IDs that have a single digit, otherwise the `[0]` cuts all the remaining digits silently.
While I see no reason for the default proxy to be anything than ID 1, the code should cope for any number.
Associated revisions
History
#1
Updated by The Foreman Bot almost 5 years ago
- Status changed from New to Ready For Testing
- Pull request https://github.com/Katello/katello-packaging/pull/527 added
#2
Updated by Evgeni Golov almost 5 years ago
- Status changed from Ready For Testing to Closed
- % Done changed from 0 to 100
Applied in changeset katello-packaging|f7db11e5e7019a5a264c44cd77d581253776ba0d.
#3
Updated by Justin Sherrill almost 5 years ago
- Legacy Backlogs Release (now unused) set to 250
Fixes #20921 - use String.chomp, not slice to remove newlines