Skip to content

SVCPLAN-3648: register_ping_check - optionally add prefix/suffix or o…#12

Merged
jakerundall merged 1 commit intomainfrom
rundall/SVCPLAN-3648/optional_hostname_suffix_for_ping_registration
Jul 8, 2025
Merged

SVCPLAN-3648: register_ping_check - optionally add prefix/suffix or o…#12
jakerundall merged 1 commit intomainfrom
rundall/SVCPLAN-3648/optional_hostname_suffix_for_ping_registration

Conversation

@jakerundall
Copy link
Copy Markdown
Contributor

…verride domain name

The hostname set in the OS of some cluster nodes corresponds to a private IP that is not pingable by nodes with ping checks based on exported resources.

This change allows overriding the default networking.fqdn fact that is normally passed as an exported resource to fix this issue.

E.g., allows adding a '-mgmt' suffix to the hostname and forcing an 'internal' domain name.

…verride domain name

The hostname set in the OS of some cluster nodes corresponds to a private IP that
is not pingable by nodes with ping checks based on exported resources.

This change allows overriding the default networking.fqdn fact that is normally
passed as an exported resource to fix this issue.

E.g., allows adding a '-mgmt' suffix to the hostname and forcing an 'internal'
domain name.
@jakerundall jakerundall requested a review from bsper2 June 30, 2025 19:34
@jakerundall jakerundall self-assigned this Jun 30, 2025
@jakerundall
Copy link
Copy Markdown
Contributor Author

Tested on mg with an internal node (mg001) and a "public" node (mgtesthn1). Both nodes would normally register themselves for ping via a hostname that is not pingable by the Puppet server, but now I can override the hostname that is registered with their internal '-mgmt' address, which IS pingable by the Puppet server.

Copy link
Copy Markdown
Member

@bsper2 bsper2 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Looks good

@jakerundall jakerundall merged commit 2606a29 into main Jul 8, 2025
4 checks passed
@jakerundall jakerundall deleted the rundall/SVCPLAN-3648/optional_hostname_suffix_for_ping_registration branch July 8, 2025 18:51
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants