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

Add support for 2024-09-01 version of AKS API #4419

Merged
merged 1 commit into from
Nov 6, 2024

Conversation

matthchr
Copy link
Member

@matthchr matthchr commented Nov 6, 2024

What this PR does

  • 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.
  • Update the rest-api-specs submodule, to get access to the hot off the presses 2024-09-01 version of the containerservice API.
  • Add new StripLargeDescriptions pipeline stage, which can be used to strip descriptions off CRD types to reduce their size.
  • Add support for 2024-09-01 version of AKS API.
  • Add support for MaintenanceConfiguration.

Fixes #4164.
Fixes #4327.

  • this PR contains documentation
  • this PR contains tests
  • this PR contains YAML Samples

Copy link
Member

@theunrepentantgeek theunrepentantgeek left a 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.

@matthchr
Copy link
Member Author

matthchr commented Nov 6, 2024

I've changed how I stripped descriptions to make it a configuration option in objectModelConfiguration and done it per resource version, and only stripped some of the older versions documentation. Agree that's better - I had missed that it would impact our reference docs, which as you say we definitely don't want to lose.

- 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.
@matthchr
Copy link
Member Author

matthchr commented Nov 6, 2024

/ok-to-test sha=43d8b2a

@matthchr matthchr added this pull request to the merge queue Nov 6, 2024
Merged via the queue into Azure:main with commit 4b0c23e Nov 6, 2024
8 checks passed
@matthchr matthchr deleted the feature/update-mc branch November 6, 2024 23:06
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
2 participants