Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

[Release-1.21] Get node name from metadata if AWS cloud provider is enabled #2166

Merged

Conversation

galal-hussein
Copy link
Contributor

@galal-hussein galal-hussein commented Nov 24, 2021

The AWS cloud provider expects the node name of a node to have a
specific form. If the user sets the hostname of the of the underlying
machine to a custom name, then the node will fail to bootstrap because
the node will not have the expected name.

With this change, the node name is fetched from the metadata endpoint to
ensure it has the proper value. If this fails, then it will fallback to
the hostname command.

Signed-off-by: Donnie Adams [email protected]

Backport for PR #2163

…#2163)

The AWS cloud provider expects the node name of a node to have a
specific form. If the user sets the hostname of the of the underlying
machine to a custom name, then the node will fail to bootstrap because
the node will not have the expected name.

With this change, the node name is fetched from the metadata endpoint to
ensure it has the proper value. If this fails, then it will fallback to
the hostname command.

Signed-off-by: Donnie Adams <[email protected]>
@galal-hussein galal-hussein requested a review from a team as a code owner November 24, 2021 20:29
@galal-hussein galal-hussein merged commit 62bca4d into rancher:release-1.21 Nov 24, 2021
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.

3 participants