-
Notifications
You must be signed in to change notification settings - Fork 4.3k
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
Backport leader status telemetry [VAULT-672] #10147
Conversation
Thank you for your submission! We require that all contributors sign our Contributor License Agreement ("CLA") before we can accept the contribution. Read and sign the agreement Learn more about why HashiCorp requires a CLA and what the CLA includes 1 out of 2 committers have signed the CLA.
Hridoy Roy seems not to be a GitHub user. Have you signed the CLA already but the status is still pending? Recheck it. |
… into VAULT-672-backport
61f427c
to
ed98852
Compare
PR Update: CI tests were not running because of vendor issues. Needed to run go mod vendor and clean go.mod and go.sum in order for the tests to pass (go.mod somehow pulled in a conflicting metricsutil package). Go mod vendor made this a large change. Glancing through the vendor changes, they all seem pretty mild and CI does seem to pass, but I'm re-requesting reviews just to be on the safe side. Thank you! |
Backport Source: https://github.com/hashicorp/vault-enterprise/pull/1501/files
Telemetry: #10137