-
Notifications
You must be signed in to change notification settings - Fork 451
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 Resource "ShootExtensionStatus" in core.gardener.cloud #3878
Add Resource "ShootExtensionStatus" in core.gardener.cloud #3878
Conversation
3b580c1
to
61315e4
Compare
61315e4
to
d6ba111
Compare
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.
Do you also want to include necessary storage changes for pkg/registry/core
?
What about calling the resource |
d6ba111
to
540e1ce
Compare
I added validation and storage implementation in a second commit. I am a bit torn between naming the plural of the resource "shootextensionstatus" or "shootextensionstatus" - see here. On the one hand, EDIT: plural being used is "shootextensionstatuses" |
540e1ce
to
38a8746
Compare
38a8746
to
286ff0d
Compare
@danielfoehrKn The pull request was assigned to you under |
b483cb6
to
5093f61
Compare
/needs rebase |
5093f61
to
63a571f
Compare
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.
/lgtm
@timuthy - any final opinions?
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.
/lgtm
Thank you
…3878) * Add ShootExtensionStatus in core.gardener.cloud/v1alpha1 * Add Gardener APIServer storage configuration for ShootExtensionStatus
…3878) * Add ShootExtensionStatus in core.gardener.cloud/v1alpha1 * Add Gardener APIServer storage configuration for ShootExtensionStatus
**How to categorize this PR?**
/area quality
/kind enhancement
What this PR does / why we need it:
Adds a new resource in the
core.gardener.cloud
resource group with the nameShootExtensionStatus
.This resource is intended to hold the information of the
ProviderStatus
field from multiple Gardener Extension resources.It is modeled after the
ShootState
resource. Hence, there is only aSpec
(not Status), as no spec is actually reconciled.Which issue(s) this PR fixes:
Part of #3873
Special notes for your reviewer:
This PR only contains the type definitions & client generations.
Release note: