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
containerservice
- add support for customCATrustCertificates
#22032
containerservice
- add support for customCATrustCertificates
#22032
Conversation
Thanks for this PR @Vegoo89. Would you mind rebasing since we upgraded the SDK to an even later version? |
Will do when I am back from vacation in two weeks :) |
cbc2bb2
to
d6952b5
Compare
@stephybun please take a look, thanks! |
containerservice
- upgrade sdk to 2023-03-02-preview
& add support for customCATrustCertificates
containerservice
- add support for customCATrustCertificates
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.
Overall this is looking good @Vegoo89, there's a bit of tidying up to do but once that's done and if the tests pass this should be good to go.
internal/services/containers/kubernetes_cluster_resource_test.go
Outdated
Show resolved
Hide resolved
internal/services/containers/kubernetes_cluster_resource_test.go
Outdated
Show resolved
Hide resolved
@stephybun changes have been implemented (hopefully its fine). I left comments for two conversations. |
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.
Thanks @Vegoo89 LGTM 🐢
Closes #21284
This PR adds support for
customCATrustCertificates
inManagedClusterSecurityProfile
blockMicrosoft documentation:
https://learn.microsoft.com/en-us/azure/templates/microsoft.containerservice/2023-03-02-preview/managedclusters?pivots=deployment-language-bicep#managedclustersecurityprofile