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
CORS-2845: azure: Enable storage account encryption #7520
CORS-2845: azure: Enable storage account encryption #7520
Conversation
@rna-afk: This pull request references CORS-2845 which is a valid jira issue. Warning: The referenced jira issue has an invalid target version for the target branch this PR targets: expected the story to target the "4.15.0" version, but no target version was set. In response to this:
Instructions for interacting with me using PR comments are available here. If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository. |
It looks like there is an issue with generating the terraform variables. Other than that, this looks straight forward and good. |
276d06f
to
ce453f0
Compare
@gpei can someone from the QE team test this? I'm having some permission problems for creating certs. |
ce453f0
to
3716647
Compare
3716647
to
c28ff2f
Compare
c28ff2f
to
2cfe84f
Compare
de7869c
to
22dd607
Compare
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
This looks good to me.
[APPROVALNOTIFIER] This PR is APPROVED This pull-request has been approved by: jhixson74 The full list of commands accepted by this bot can be found here. The pull request process is described here
Needs approval from an approver in each of these files:
Approvers can indicate their approval by writing |
a7d57e8
to
03bb78e
Compare
/lgtm |
/hold looks like the azure job is failing |
f3189d7
to
18b65f7
Compare
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM but I have not tested it.
18b65f7
to
46185c5
Compare
/lgtm |
46185c5
to
ac03e4f
Compare
/retest |
Added a field to accept the customer managed key vault id and user assigned identity required for the encryption of storage account. This would require a few changes to the storage account as azure requires[1] the account tier to be Premium and kind to be StorageV2 which is default and we do not set it. [1] - https://registry.terraform.io/providers/hashicorp/azurerm/latest/docs/resources/storage_account#customer_managed_key
Added test cases for new field customerManagedKey
ac03e4f
to
6077dd1
Compare
/lgtm |
/jira refresh |
@jhixson74: This pull request references CORS-2845 which is a valid jira issue. In response to this:
Instructions for interacting with me using PR comments are available here. If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository. |
/retest-required |
/retest |
1 similar comment
/retest |
@rna-afk: The following test failed, say
Full PR test history. Your PR dashboard. Instructions for interacting with me using PR comments are available here. If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository. I understand the commands that are listed here. |
5744fea
into
openshift:master
CORS-2845: azure: Enable storage account encryption
CORS-2845: azure: Enable storage account encryption
[ART PR BUILD NOTIFIER] This PR has been included in build ose-installer-altinfra-container-v4.15.0-202311302048.p0.g5744fea.assembly.stream for distgit ose-installer-altinfra. |
Added a field to accept the customer managed key vault id and
user assigned identity required for the encryption of storage
account.