Bug 2182028 - Foreman-maintain health check should be run before configuring Custom SSL certs. [NEEDINFO]
Summary: Foreman-maintain health check should be run before configuring Custom SSL certs.
Keywords:
Status: NEW
Alias: None
Product: Red Hat Satellite
Classification: Red Hat
Component: Foreman Maintain
Version: 6.10.0
Hardware: x86_64
OS: Linux
unspecified
high
Target Milestone: Unspecified
Assignee: Vedashree Deshpande
QA Contact: Satellite QE Team
URL:
Whiteboard:
Depends On:
Blocks:
TreeView+ depends on / blocked
 
Reported: 2023-03-27 10:43 UTC by Vedashree Deshpande
Modified: 2023-08-10 12:47 UTC (History)
3 users (show)

Fixed In Version:
Doc Type: If docs needed, set a value
Doc Text:
Clone Of:
Environment:
Last Closed:
Target Upstream Version:
Embargoed:
ehelms: needinfo? (ekohlvan)


Attachments (Terms of Use)


Links
System ID Private Priority Status Summary Last Updated
Red Hat Issue Tracker SAT-16754 0 None None None 2023-03-27 10:46:08 UTC

Description Vedashree Deshpande 2023-03-27 10:43:05 UTC
Document URL: 
https://access.redhat.com/documentation/en-us/red_hat_satellite/6.10/html/installing_satellite_server_from_a_connected_network/performing-additional-configuration#configuring-satellite-custom-server-certificate_satellite

same for 6.11 and 6.12

Section Number and Name: 
In section 4.12

Describe the issue: 
Overall health check should be run to ensure hostname and other services etc are all in place before replacing the custom SSL certificates. 

Suggestions for improvement: 
Add a note to run #foreman-maintain health check before renewal of custom SSL certs. 

for 6.11 and 6.12 respectively. 

Additional information: 
If the hostname is not changed and is not uniform across all config files, installer fails and the only option is to rebuild from Scratch.

Comment 2 Ewoud Kohl van Wijngaarden 2023-03-30 15:19:12 UTC
(In reply to Vedashree Deshpande from comment #0)
> Overall health check should be run to ensure hostname and other services etc
> are all in place before replacing the custom SSL certificates. 

This is not something we'll do, because the health check may fail due to expired certificates. The procedure to replace custom SSL certificates is then supposed to resolve that situation.

Since foreman-installer 2.3.0 (IIRC that was Satellite 6.9) we do verify the certificates, regardless of which options were passed so they should always be checked.
 
> Additional information: 
> If the hostname is not changed and is not uniform across all config files,
> installer fails and the only option is to rebuild from Scratch.

I don't understand this. Why is this? It should never be needed to rebuild from scratch because of incorrect configs: the installer is supposed to always write out the desired config, regardless of the current system state.

I was going to close this as WONTFIX, but I'd first like to understand what made you think a rebuild from Scratch would be needed.


Note You need to log in before you can comment on or make changes to this bug.