Skip to content
This repository has been archived by the owner on Apr 7, 2020. It is now read-only.

Make validator AWS support all offered core shoot versions #525

Merged
merged 2 commits into from
Jan 7, 2020

Conversation

timuthy
Copy link
Contributor

@timuthy timuthy commented Dec 31, 2019

What this PR does / why we need it:
Previously, the validator-aws only supported v1beta1.shoots.core.gardener.cloud which is not sufficient because Gardener also offeres a v1alpha1 version. The validator now works with the internal shoot version, so that validation for all offered external versions are covered out of the box.

Special notes for your reviewer:
The Validatingwebhookconfiguration for gardener-extensions-validator-aws now also requires a minimum Kubernetes version of 1.15.x. because of the introduced objectSelector field.
Gardener version v0.34.0 will rely on the k8s code base 1.16.0 which on the other hand can't dispatch admission requests for webhook configurations w/o an objectSelector.

See: kubernetes/kubernetes#78505

Release note:

Validating AWS shoots of version `v1beta1` as well as `v1alpha1` (group `core.gardener.cloud`) is now supported by the webhook `gardener-extensions-validator-aws`.
The `Validatingwebhookconfiguration` for `gardener-extensions-validator-aws` now requires a minimum Kubernetes version of 1.15.x.

@timuthy timuthy requested a review from a team as a code owner December 31, 2019 13:04
@gardener-robot-ci-1 gardener-robot-ci-1 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 Dec 31, 2019
@timuthy timuthy force-pushed the enhancement.validator-aws branch from 48bf113 to 63c8e76 Compare December 31, 2019 15:58
@gardener-robot-ci-2 gardener-robot-ci-2 added reviewed/ok-to-test Has 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 Dec 31, 2019
@timuthy timuthy force-pushed the enhancement.validator-aws branch from 63c8e76 to ee2f42b Compare January 1, 2020 19:27
@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 1, 2020
@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 1, 2020
Copy link
Contributor

@rfranzke rfranzke left a comment

Choose a reason for hiding this comment

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

/lgtm, nice PR!

@rfranzke rfranzke merged commit b479f32 into gardener-attic:master Jan 7, 2020
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
needs/ok-to-test Needs approval for testing (check PR in detail before setting this label because PR is run on CI/CD)
Projects
None yet
Development

Successfully merging this pull request may close these issues.

5 participants