-
Notifications
You must be signed in to change notification settings - Fork 205
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
Add support for 2024-09-01 version of AKS API #4419
Conversation
v2/tools/generator/internal/codegen/pipeline/strip_descriptions.go
Outdated
Show resolved
Hide resolved
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.
I'm not a fan of stripping the descriptions entirely, especially since it will impact on our public docs too - might we potentially target just the older or preview versions?
Otherwise looks good.
I've changed how I stripped descriptions to make it a configuration option in |
- Fix issue in RepairSkippingProperties which could cause an issue when a type is reused among multiple resources, such as in a containerservice ManagedCluster and AgentPool. - Add new StripDocumentation pipeline stage, which can be used to strip descriptions off CRD types to reduce their size. - We don't want to do this in perpetuity, see Azure#4418 which tracks removing some old versions of ManagedCluster so we can drop this in a future release. - Add support for 2024-09-01 version of AKS API. - Add support for MaintenanceConfiguration.
e3a0f49
to
43d8b2a
Compare
/ok-to-test sha=43d8b2a |
What this PR does
RepairSkippingProperties
which could cause an issue when a type is reused among multiple resources, such as in a containerservice ManagedCluster and AgentPool.StripLargeDescriptions
pipeline stage, which can be used to strip descriptions off CRD types to reduce their size.Fixes #4164.
Fixes #4327.