Project

General

Profile

Actions

Bug #32725

closed

katello-change-hostname should handle --no-enable-puppet properly

Added by Ian Ballou almost 3 years ago. Updated over 2 years ago.

Status:
Closed
Priority:
Normal
Category:
RPMs
Target version:
Fixed in Releases:
Found in Releases:

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:

Actions #1

Updated by Ian Ballou almost 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
Actions #2

Updated by The Foreman Bot almost 3 years ago

  • Status changed from New to Ready For Testing
  • Assignee set to Ewoud Kohl van Wijngaarden
Actions #3

Updated by The Foreman Bot almost 3 years ago

  • Fixed in Releases 3.0.0 added
Actions #4

Updated by Ewoud Kohl van Wijngaarden almost 3 years ago

  • Status changed from Ready For Testing to Closed
Actions #5

Updated by The Foreman Bot almost 3 years ago

  • Pull request https://github.com/theforeman/foreman-packaging/pull/6774 added
Actions #6

Updated by Evgeni Golov almost 3 years ago

  • Fixed in Releases 2.5.1 added
Actions #7

Updated by Evgeni Golov almost 3 years ago

  • Target version set to 2.5.1
Actions #8

Updated by Amit Upadhye over 2 years ago

  • Category set to RPMs
Actions

Also available in: Atom PDF