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

Store Cluster Name in Physical Storage #26878

Merged
merged 2 commits into from
May 13, 2024
Merged

Store Cluster Name in Physical Storage #26878

merged 2 commits into from
May 13, 2024

Conversation

ltcarbonell
Copy link
Contributor

@ltcarbonell ltcarbonell commented May 8, 2024

Vault currently displays duplicate and blank cluster labels when deployments are made with k8s.

This is the result of a race condition during the activation of the telemetry system. Telemetry is initialized before Vault is unsealed, therefore it is not able to pull the cluster name and information that is stored in encrypted storage.

To solve this, it was suggested that we store the cluster name in unencrypted storage, so that it can be accessed prior to Vault being unsealed. Vault will then attempt to pull from encrypted storage first, and if it is unable to pull out a valid name, it will then use the value stored in unencrypted storage.

@github-actions github-actions bot added the hashicorp-contributed-pr If the PR is HashiCorp (i.e. not-community) contributed label May 8, 2024
@ltcarbonell ltcarbonell requested a review from marcboudreau May 8, 2024 14:42
Copy link

github-actions bot commented May 8, 2024

CI Results:
All Go tests succeeded! ✅

Copy link

github-actions bot commented May 8, 2024

Build Results:
All builds succeeded! ✅

@ltcarbonell ltcarbonell merged commit 33d9365 into main May 13, 2024
85 checks passed
@ltcarbonell ltcarbonell deleted the blank-metrics branch May 13, 2024 18:36
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
hashicorp-contributed-pr If the PR is HashiCorp (i.e. not-community) contributed pr/no-milestone
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants