terraform-provider-ibm icon indicating copy to clipboard operation
terraform-provider-ibm copied to clipboard

Create key with policy override resource

Open Sidd-kp opened this issue 3 years ago • 1 comments

Community Note

  • Please vote on this pull request by adding a 👍 reaction to the original pull request comment to help the community and maintainers prioritize this request
  • Please do not leave "+1" or other comments that do not add relevant new information or questions, they generate extra noise for pull request followers and do not help prioritize the request

Relates OR Closes #0000

Output from acceptance testing:

$ make testacc TESTARGS='-run=TestAccXXX'

...

Sidd-kp avatar Sep 01 '22 21:09 Sidd-kp

Dependencies: CreateKeyWithPolicyOverrides SDK support Set Key Policy SDK support for enabled parameter.

Sidd-kp avatar Sep 01 '22 21:09 Sidd-kp

FOUND SECRETS: File: ibm/service/kms/resource_ibm_kms_key_with_policy_overrides_test.go Line: 23 Type: Base64 High Entropy String failure

Can u address this please !!

harshit777 avatar Oct 14 '22 08:10 harshit777

Also can attach the evidences to the resource provisioning !

harshit777 avatar Oct 14 '22 08:10 harshit777

Screenshot 2022-10-17 at 3 21 05 PM

Sidd-kp avatar Oct 17 '22 09:10 Sidd-kp

Screenshot 2022-10-28 at 11 04 50 AM All test results Note: I have commented out the following test case as it requires a paid account:

{
          Config: testAccCheckIBMKmsResourceRootkeyWithCOSConfig(instanceName, resourceName, keyName, cosInstanceName, bucketName),
          Check: resource.ComposeTestCheckFunc(
	          resource.TestCheckResourceAttr("ibm_kms_key.test", "key_name", keyName),
          ),
},

Sidd-kp avatar Oct 28 '22 05:10 Sidd-kp

Screenshot 2022-10-28 at 11 24 00 AM

Sidd-kp avatar Oct 28 '22 05:10 Sidd-kp

Root Key, Standard Key, Imported Root Key, Imported Standard Key Screenshot 2022-10-28 at 12 41 16 PM Screenshot 2022-10-28 at 12 44 14 PM

Sidd-kp avatar Oct 28 '22 07:10 Sidd-kp

Screenshot 2022-10-28 at 11 04 50 AM

All test results Note: I have commented out the following test case as it requires a paid account:

{
          Config: testAccCheckIBMKmsResourceRootkeyWithCOSConfig(instanceName, resourceName, keyName, cosInstanceName, bucketName),
          Check: resource.ComposeTestCheckFunc(
	          resource.TestCheckResourceAttr("ibm_kms_key.test", "key_name", keyName),
          ),
},

Nice! Work, LGTM, one suggestion instead of commenting we can use t.Skip() inside the function to skip it while testing, in that case we will not need to comment it !

harshit777 avatar Oct 28 '22 09:10 harshit777

fix conflicts

hkantare avatar Nov 11 '22 08:11 hkantare

Screenshot 2022-11-14 at 11 19 17 AM 4 of 5 of the test cases are passing. The last one requires a paid account. Screenshot 2022-11-14 at 11 19 53 AM Screenshot 2022-11-14 at 11 20 06 AM

Sidd-kp avatar Nov 14 '22 05:11 Sidd-kp