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

openstack: Fix a crash in the bootstrapper during node join discovery #3375

Merged
merged 1 commit into from
Sep 26, 2024

Conversation

3u13r
Copy link
Member

@3u13r 3u13r commented Sep 24, 2024

Context

The bootstrapper still likes to query any nodes in addition to the LB endpoint when searching for join endpoints.

Openstack has a catalog discovery feature that gets URL endpoints during authentication (see: https://github.com/jtopjian/gophercloud/blob/master/openstack/endpoint_location.go#L63). Since we did not set the region we received more endpoints than expected and some even invalid ones.

Proposed change(s)

  • Passthrough openstack region name via IMDS to instance to use it in the cloud API client.

Checklist

  • Add labels (e.g., for changelog category)
  • Is PR title adequate for changelog?
  • Link to Milestone

@3u13r 3u13r added the bug fix Fixing a bug label Sep 24, 2024
@3u13r 3u13r added this to the v2.18.0 milestone Sep 24, 2024
Copy link

netlify bot commented Sep 24, 2024

Deploy Preview for constellation-docs canceled.

Name Link
🔨 Latest commit 09875af
🔍 Latest deploy log https://app.netlify.com/sites/constellation-docs/deploys/66f2b10e0a93dd00070fd0a5

Copy link
Contributor

Coverage report

Package Old New Trend
internal/cloud/openstack 86.60% 84.40% ↘️

Copy link
Contributor

@burgerdev burgerdev left a comment

Choose a reason for hiding this comment

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

I don't see where the actual crash is being fixed or handled - the bootstrapper should be able to join the cluster even if the metadata API does not return or returns wrong addresses, shouldn't it?

@3u13r
Copy link
Member Author

3u13r commented Sep 25, 2024

I also not remember the actual crash location. I'll spin up a cluster to document this more thoroughly.

Copy link
Contributor

@burgerdev burgerdev left a comment

Choose a reason for hiding this comment

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

I tested the branch and the change lgtm. Let's follow up on the resilience topic.

@3u13r 3u13r merged commit 882d602 into main Sep 26, 2024
18 checks passed
@3u13r 3u13r deleted the fix/openstack/set-region-in-cloud-client branch September 26, 2024 09:08
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug fix Fixing a bug
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants