From 75e06c732e8e3fe9bcd1a38d57fd0c391815ec0c Mon Sep 17 00:00:00 2001 From: Alessio Buraggina Date: Wed, 27 Mar 2024 00:15:01 -0400 Subject: [PATCH] Skip external_vpc KMS acceptance test in VCR. --- .../terraform/services/kms/resource_kms_crypto_key_test.go | 6 ++++++ 1 file changed, 6 insertions(+) diff --git a/mmv1/third_party/terraform/services/kms/resource_kms_crypto_key_test.go b/mmv1/third_party/terraform/services/kms/resource_kms_crypto_key_test.go index 3bc66d5fc7f5..558aee66e084 100644 --- a/mmv1/third_party/terraform/services/kms/resource_kms_crypto_key_test.go +++ b/mmv1/third_party/terraform/services/kms/resource_kms_crypto_key_test.go @@ -576,6 +576,12 @@ func TestAccKmsCryptoKeyVersion_externalProtectionLevelOptions(t *testing.T) { } func TestAccKmsCryptoKeyVersion_externalProtectionLevelOptionsVpc(t *testing.T) { + // This test relies on manual steps to set up the EkmConnection used for the + // CryptoKeyVersion creation, which means we can't spin up a temporary project. + // We also can't use bootstrapped keys because that would defeat the purpose of + // this key creation test, so we skip this test for VCR to avoid KMS resource + // accumulation in the TF test project (since KMS resources can't be deleted). + acctest.SkipIfVcr(t) t.Parallel() projectId := envvar.GetTestProjectFromEnv()