Bug 2165107 - Unclear error message with empty --certs-state
Summary: Unclear error message with empty --certs-state
Keywords:
Status: VERIFIED
Alias: None
Product: Red Hat Satellite
Classification: Red Hat
Component: Installer
Version: 6.13.0
Hardware: Unspecified
OS: Unspecified
unspecified
medium
Target Milestone: 6.14.0
Assignee: Eric Helms
QA Contact: Radek Mynar
URL:
Whiteboard:
Depends On:
Blocks:
TreeView+ depends on / blocked
 
Reported: 2023-01-27 17:36 UTC by Radek Mynar
Modified: 2023-06-30 18:54 UTC (History)
2 users (show)

Fixed In Version:
Doc Type: If docs needed, set a value
Doc Text:
Clone Of:
Environment:
Last Closed:
Target Upstream Version:
Embargoed:

Attachments (Terms of Use)


Links
System ID Private Priority Status Summary Last Updated
Github theforeman puppet-katello pull 460 0 None open Allow but filter out empty strings in build_dn 2023-02-01 14:35:08 UTC
Red Hat Issue Tracker SAT-17439 0 None None None 2023-05-25 12:54:42 UTC

Description Radek Mynar 2023-01-27 17:36:19 UTC 
Description of problem:
When creating new certificates using satellite-installer and passing empty "state" parameter an Evaluation Error is shown

Version-Release number of selected component (if applicable):
-

How reproducible:
always

Steps to Reproduce:
a)
1. satellite-installer --certs-regenerate true --certs-update-all --certs-update-server --certs-update-server-ca --certs-org-unit "FooOrg"  --certs-state ""
b)
1. edit /etc/foreman-installer/scenarios.d/satellite-answers.yaml and set certs -> state: ''   
2. satellite-installer --certs-regenerate true --certs-update-all --certs-update-server --certs-update-server-ca

Actual results:
satellite-installer fails with following error
<timestamp> [ERROR ] [configure] Evaluation Error: Error while evaluating a Function Call, 'katello::build_dn' parameter 'options' index 3 index 1 expects a value of type Undef or String[1], got String (file: /usr/share/foreman-installer/modules/katello/manifests/application.pp, line: 40, column: 24) on node <node-fqdn>

Expected results:
Reasonable error should be shown.

Additional info:
After fix (if this gets fixed) the installer should never cause inconsistency between certificates and config files - 'hammer ping' should pass (see bug 1964037)
Comment 1 Brad Buckingham 2023-01-30 14:14:28 UTC 
Is this a regression from prior release?

Thanks!
Comment 2 Radek Mynar 2023-01-31 12:52:39 UTC 
This was most probably introduced after fix of bug 1964037, so I suppose it is a regression.
Comment 3 Radek Mynar 2023-06-07 12:06:36 UTC 
I tested various combinations (empty string/valid string) of state and org_unit. There was no error message. No combination led to failing 'hammer ping'.

VERIFIED with Satellite 6.14 SNAP2.0 @RHEL8.8
Note You need to log in before you can comment on or make changes to this bug.