-
Notifications
You must be signed in to change notification settings - Fork 540
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
chore: update vault test matrix #2221
Conversation
Reverting the Vault SDK bump for now since it looks like we need to update TFVP. I think the failures are related to hashicorp/vault#25744 |
2f7a185
to
662ee2c
Compare
Also reverting the go version bump. It seems to be breaking our tests? |
Looks like the Secrets Sync Config test is failing due to the feature flag not being enabled. We should probably skip that test in CI since it can only be run on servers where the feature has been enabled (whoops!). We can add a skip on this line either in this PR or a follow up PR:
|
@vinay-gopalan Thanks! Done in 48a48c9 |
No description provided.