Project

General

Profile

Bug #31253

Check 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"

Added by Suraj Patil 3 months ago. Updated about 2 months ago.

Status:
Closed
Priority:
Normal
Assignee:
Category:
-
Target version:
-
Difficulty:
Triaged:
No
Bugzilla link:

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 <=============================================
  1. installer puppet modules that might not be exposed to users directly through
  2. 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%

Steps to Reproduce:
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 <=============================================
  1. installer puppet modules that might not be exposed to users directly through
  2. 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:

Associated revisions

Revision 2102e55a (diff)
Added by Suraj Patil about 2 months ago

Fixes #31253 - Validate if custom-hiera is yaml (#415)

History

#1 Updated by The Foreman Bot 3 months ago

  • Status changed from New to Ready For Testing
  • Pull request https://github.com/theforeman/foreman_maintain/pull/415 added

#2 Updated by Anonymous about 2 months ago

  • Status changed from Ready For Testing to Closed

#3 Updated by The Foreman Bot about 2 months ago

  • Pull request https://github.com/theforeman/foreman_maintain/pull/424 added

Also available in: Atom PDF