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 a script to auto add Kubernetes patch versions #4660

Open
wants to merge 9 commits into
base: master
Choose a base branch
from

Conversation

gaopenghigh
Copy link
Collaborator

What type of PR is this?

/kind feature

What this PR does / why we need it:

From AKS v20240627, we start to support all Kubernetes patch versions if a minor version is supported, that means we need to update VHD cache more frequently than before. This PR adds a script to help automate adding new patch versions. Now we just need to run make auto-add-k8s-patch-versions, if everything is good, then make generate.

To keep the latest 2 patch versions always cached in VHD, we actually cache 3 patch versions so latest 2 still in cache even if RP side haven't been updated.

This PR also contains the result of the script: 1.30.0 is added back to cache.

Which issue(s) this PR fixes:

Fixes #

Requirements:

Special notes for your reviewer:

Release note:

none

@coveralls
Copy link

coveralls commented Jul 17, 2024

Pull Request Test Coverage Report for Build 10070699361

Details

  • 0 of 0 changed or added relevant lines in 0 files are covered.
  • No unchanged relevant lines lost coverage.
  • Overall coverage remained the same at 71.021%

Totals Coverage Status
Change from base Build 10066056764: 0.0%
Covered Lines: 2566
Relevant Lines: 3613

💛 - Coveralls

@ShiqianTao
Copy link
Contributor

Need to rebase master, but there are conflicts

@gaopenghigh
Copy link
Collaborator Author

Need to rebase master, but there are conflicts

Thank you @ShiqianTao , rebased.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants