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

Support for Kubernetes v1.23 #480

Merged
merged 3 commits into from
Jan 25, 2022
Merged

Conversation

rfranzke
Copy link
Member

@rfranzke rfranzke commented Jan 24, 2022

How to categorize this PR?

/area open-source usability
/kind enhancement
/platform aws
/exp intermediate
/topology garden seed shoot
/merge squash

What this PR does / why we need it:
This PR adds support for Kubernetes 1.23 to the extension.

Which issue(s) this PR fixes:
Part of gardener/gardener#5102

Special notes for your reviewer:

  • Depends on Revendor gardener/gardener@v1.39 #478, hence, PR is in draft state.
  • I have successfully validated the functionality as follows:
    • ✅ Create new clusters with versions < 1.23
    • ✅ Create new clusters with version = 1.23
    • ✅ Upgrade old clusters from version 1.22 to version 1.23
    • ✅ Delete clusters with versions < 1.23
    • ✅ Delete clusters with version = 1.23

Release note:

The AWS extension does now support shoot clusters with Kubernetes version 1.23. You should consider the [Kubernetes release notes](https://github.com/kubernetes/kubernetes/blob/master/CHANGELOG/CHANGELOG-1.23.md) before upgrading to 1.23. 

@gardener-robot gardener-robot added area/open-source Open Source (community, enablement, contributions, conferences, CNCF, etc.) related area/usability Usability related exp/intermediate Issue that requires some project experience kind/enhancement Enhancement, improvement, extension merge/squash Should be merged via 'Squash and merge' platform/aws Amazon web services platform/infrastructure topology/garden Affects Garden clusters topology/seed Affects Seed clusters topology/shoot Affects Shoot clusters needs/review Needs review size/m Size of pull request is medium (see gardener-robot robot/bots/size.py) labels Jan 24, 2022
@gardener-robot-ci-3 gardener-robot-ci-3 added reviewed/ok-to-test Has approval for testing (check PR in detail before setting this label because PR is run on CI/CD) needs/ok-to-test Needs approval for testing (check PR in detail before setting this label because PR is run on CI/CD) and removed reviewed/ok-to-test Has approval for testing (check PR in detail before setting this label because PR is run on CI/CD) labels Jan 24, 2022
@gardener-robot-ci-1 gardener-robot-ci-1 added the reviewed/ok-to-test Has approval for testing (check PR in detail before setting this label because PR is run on CI/CD) label Jan 24, 2022
@gardener-robot-ci-2 gardener-robot-ci-2 removed the reviewed/ok-to-test Has approval for testing (check PR in detail before setting this label because PR is run on CI/CD) label Jan 24, 2022
```feature dependency github.com/gardener/cloud-provider-aws $d3fd5b0532b64f7e0981775c51bac17400260e08
`k8s.io/legacy-cloud-providers` is now updated to `v1.23.2`.
```

```improvement developer github.com/gardener/cloud-provider-aws $a238662f9b18f333281b6eb2cab51fe0572aaa6f
The alpine version has been updated to `v3.13.7`.
```

```feature dependency github.com/gardener/cloud-provider-aws $d2b414faa94cb46a05e14f8dc5f0bd0f0100663a
`k8s.io/legacy-cloud-providers` is now updated to `v0.22.6`.
```

```feature dependency github.com/gardener/cloud-provider-aws $234a445f1961b5c7ede08ed1ad5ee834430ec2f2
`k8s.io/legacy-cloud-providers` is now updated to `v0.21.9`.
```

```feature dependency github.com/gardener/cloud-provider-aws $6b6e8acb12a93ff6a3baa681fa4de3e093b51ded
`k8s.io/legacy-cloud-providers` is now updated to `v0.20.15`.
```
@rfranzke
Copy link
Member Author

/ready

@gardener-robot gardener-robot marked this pull request as ready for review January 25, 2022 08:15
@gardener-robot-ci-3 gardener-robot-ci-3 added the reviewed/ok-to-test Has approval for testing (check PR in detail before setting this label because PR is run on CI/CD) label Jan 25, 2022
@gardener-robot gardener-robot requested review from a team as code owners January 25, 2022 08:15
@gardener-robot-ci-3 gardener-robot-ci-3 removed the reviewed/ok-to-test Has approval for testing (check PR in detail before setting this label because PR is run on CI/CD) label Jan 25, 2022
Copy link
Member

@dkistner dkistner left a comment

Choose a reason for hiding this comment

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

/lgtm

@gardener-robot gardener-robot added the reviewed/lgtm Has approval for merging label Jan 25, 2022
@gardener-robot gardener-robot removed the needs/review Needs review label Jan 25, 2022
@dkistner dkistner added this to the v1.33 milestone Jan 25, 2022
@dkistner
Copy link
Member

/reviewed ok-to-test

@gardener-robot gardener-robot added the reviewed/ok-to-test Has approval for testing (check PR in detail before setting this label because PR is run on CI/CD) label Jan 25, 2022
@dkistner dkistner merged commit c586b26 into gardener:master Jan 25, 2022
@rfranzke rfranzke deleted the kubernetes-v1.23 branch January 25, 2022 08:54
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
area/open-source Open Source (community, enablement, contributions, conferences, CNCF, etc.) related area/usability Usability related exp/intermediate Issue that requires some project experience kind/enhancement Enhancement, improvement, extension merge/squash Should be merged via 'Squash and merge' needs/ok-to-test Needs approval for testing (check PR in detail before setting this label because PR is run on CI/CD) platform/aws Amazon web services platform/infrastructure reviewed/lgtm Has approval for merging reviewed/ok-to-test Has approval for testing (check PR in detail before setting this label because PR is run on CI/CD) size/m Size of pull request is medium (see gardener-robot robot/bots/size.py) topology/garden Affects Garden clusters topology/seed Affects Seed clusters topology/shoot Affects Shoot clusters
Projects
None yet
Development

Successfully merging this pull request may close these issues.

6 participants