Skip to content
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

Mark ManagedClusterServicePrincipalProfile.Secret as a secret reference #3026

Merged
merged 12 commits into from
Jun 18, 2023

Conversation

super-harsh
Copy link
Collaborator

What this PR does / why we need it:

Marked ContainerService/ManagedCluster field ManagedClusterServicePrincipalProfile.Secret to be a secret reference.

@codecov-commenter
Copy link

codecov-commenter commented May 30, 2023

Codecov Report

Merging #3026 (61923cf) into main (c6211a9) will increase coverage by 0.00%.
The diff coverage is 28.70%.

@@           Coverage Diff           @@
##             main    #3026   +/-   ##
=======================================
  Coverage   54.29%   54.29%           
=======================================
  Files        1400     1400           
  Lines      599548   599598   +50     
=======================================
+ Hits       325508   325579   +71     
+ Misses     220665   220614   -51     
- Partials    53375    53405   +30     
Impacted Files Coverage Δ
.../v1api20230201storage/managed_cluster_types_gen.go 55.17% <ø> (ø)
...service/v1api20210501/managed_cluster_types_gen.go 53.31% <14.28%> (-0.07%) ⬇️
...service/v1api20230201/managed_cluster_types_gen.go 45.57% <14.28%> (+0.01%) ⬆️
.../v1api20230202preview/managed_cluster_types_gen.go 50.99% <14.28%> (-0.06%) ⬇️
...ervice/v1beta20210501/managed_cluster_types_gen.go 51.66% <14.28%> (-0.06%) ⬇️
.../v1api20210501storage/managed_cluster_types_gen.go 60.83% <20.00%> (-0.13%) ⬇️
...0230202previewstorage/managed_cluster_types_gen.go 65.56% <20.00%> (-0.09%) ⬇️
...v1beta20210501storage/managed_cluster_types_gen.go 64.49% <20.00%> (-0.15%) ⬇️
...2/internal/controllers/controller_resources_gen.go 85.73% <77.27%> (-0.11%) ⬇️

... and 37 files with indirect coverage changes

….0.md

Co-authored-by: Bevan Arps <bevan.arps@microsoft.com>
@matthchr matthchr added the breaking A change which likely be breaking label May 31, 2023
@@ -860,6 +860,9 @@ objectModelConfiguration:
ManagedClusterAgentPoolProfileProperties:
ProximityPlacementGroupID:
$armReference: false # This is true in the next version and we have custom conversion to make round-tripping work
ManagedClusterServicePrincipalProfile:
Secret:
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

We should fix this upstream in the AKS Swagger specifications as well.

@super-harsh super-harsh merged commit 66c9ca7 into main Jun 18, 2023
@super-harsh super-harsh deleted the fix/managedcluster-props branch June 18, 2023 22:08
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
breaking A change which likely be breaking
Projects
Development

Successfully merging this pull request may close these issues.

4 participants