Bug #31253
closedCheck for verifying syntax "/etc/foreman-installer/custom-hiera.yaml" file in foreman-maintain upgrade check command failed with misguiding error "Check if checkpoint_segments configuration exists on the system"
Description
Cloned from https://bugzilla.redhat.com/show_bug.cgi?id=1894149
Description of problem:
Upgrading from 6.7 with /etc/foreman-installer/custom-hiera.yaml :
s YAML file lets you set your own custom configuration in Hiera for the <=============================================
- installer puppet modules that might not be exposed to users directly through
- installer arguments.
~~~~~~~~~~~~~~~
Version-Release number of selected component (if applicable):
satellite-6.8.0-0.7.beta.el7sat.noarch
satellite-installer-6.8.0.8-1.el7sat.noarch
How reproducible:
100%
1. Have syntax issue in Sat 6.7 with /etc/foreman-installer/custom-hiera.yaml :
~~~~~~~~~~~~~
s YAML file lets you set your own custom configuration in Hiera for the <=============================================
- installer puppet modules that might not be exposed to users directly through
- installer arguments.
~~~~~~~~~~~~~
2. Run upgrade to 6.8
Actual results:
2. is supposed to fail with below misguiding error to another Bug
~~~~~~~~~~~~~~
Check if checkpoint_segments configuration exists on the system: [FAIL]
undefined method `key?' for #<String:0x0000000139c960>
~~~~~~~~~~~~~~~
Expected results:
1.) Foreman-maintain command should check for syntax issue in /etc/foreman-installer/custom-hiera.yaml should show error related to syntax and not to any other misguiding error
Additional info:
Updated by The Foreman Bot over 4 years ago
- Status changed from New to Ready For Testing
- Pull request https://github.com/theforeman/foreman_maintain/pull/415 added
Updated by Anonymous over 4 years ago
- Status changed from Ready For Testing to Closed
Applied in changeset foreman_maintain|2102e55ae129ecd605a07050e83a4b1015ffe870.
Updated by The Foreman Bot over 4 years ago
- Pull request https://github.com/theforeman/foreman_maintain/pull/424 added