Bug #32725
closedkatello-change-hostname should handle --no-enable-puppet properly
Description
Cloned from https://bugzilla.redhat.com/show_bug.cgi?id=1967974
Description of problem:
katello-change-hostname will error out if @scenario_answers['puppet'] is false (https://github.com/theforeman/foreman-packaging/blob/rpm/1.24/packages/katello/katello/hostname-change.rb#L170). We should skip `delete_puppet_certs` if this is the case.
Version-Release number of selected component (if applicable):
Katello 3.14
How reproducible:
Always
Steps to Reproduce:
1. Remove the puppet configuration from katello-answers.yaml
2. Run katello-change-hostname
3.
Actual results:
deleting old certs
/usr/share/katello/hostname-change.rb:217:in `delete_puppet_certs': undefined method `[]' for nil:NilClass (NoMethodError)
from /usr/share/katello/hostname-change.rb:488:in `run'
from /sbin/katello-change-hostname:23:in `<main>'
Expected results:
hostname changed successfully
Additional info:
Updated by Ian Ballou over 3 years ago
- Subject changed from katello-change-hostname should handle --no-enable-puppet properly to katello-change-hostname should handle --no-enable-puppet properly
- Pull request https://github.com/theforeman/foreman-packaging/pull/6749 added
Updated by The Foreman Bot over 3 years ago
- Status changed from New to Ready For Testing
- Assignee set to Ewoud Kohl van Wijngaarden
Updated by Ewoud Kohl van Wijngaarden over 3 years ago
- Status changed from Ready For Testing to Closed
Applied in changeset foreman-packaging|ee14227ebe203d54e5640c67d7d4925c60b9b27a.
Updated by The Foreman Bot over 3 years ago
- Pull request https://github.com/theforeman/foreman-packaging/pull/6774 added