search

hashicorp / terraform-provider-vault

Terraform Vault provider
https://www.terraform.io/docs/providers/vault/
Mozilla Public License 2.0
451 stars 535 forks source link

chore: update vault test matrix #2221

Closed fairclothjm closed 2 months ago

fairclothjm commented 2 months ago

Reverting the Vault SDK bump for now since it looks like we need to update TFVP. I think the failures are related to https://github.com/hashicorp/vault/pull/25744

fairclothjm commented 2 months ago

Also reverting the go version bump. It seems to be breaking our tests?

vinay-gopalan commented 2 months ago

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:

testutil.SkipTestEnvUnset(t, "VAULT_SECRETS_SYNC_ENABLED")
fairclothjm commented 2 months ago

@vinay-gopalan Thanks! Done in https://github.com/hashicorp/terraform-provider-vault/pull/2221/commits/48a48c9e1b31be916807308a5129a266ff2fbe1f