-
Notifications
You must be signed in to change notification settings - Fork 209
New issue
Have a question about this project? # for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “#”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? # 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.