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 beta annotations to apps #90457
add beta annotations to apps #90457
Conversation
@@ -56,6 +56,10 @@ type ScaleStatus struct { | |||
} | |||
|
|||
// +k8s:deepcopy-gen:interfaces=k8s.io/apimachinery/pkg/runtime.Object | |||
// +k8s:prerelease-lifecycle-gen:introduced=1.6 | |||
// +k8s:prerelease-lifecycle-gen:deprecated=1.8 | |||
// +k8s:prerelease-lifecycle-gen:removed=1.22 |
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.
all apps/v1beta1 and apps/v1beta2 types are unserved as of 1.18 #85903
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.
all apps/v1beta1 and apps/v1beta2 types are unserved as of 1.18 #85903
Look at me, being too generous
This PR may require API review. If so, when the changes are ready, complete the pre-review checklist and request an API review. Status of requested reviews is tracked in the API Review project. |
b95bc4a
to
ac2f03d
Compare
/lgtm |
b14ccd2
to
ac2f03d
Compare
// +k8s:prerelease-lifecycle-gen:introduced=1.6 | ||
// +k8s:prerelease-lifecycle-gen:deprecated=1.8 | ||
// +k8s:prerelease-lifecycle-gen:removed=1.18 | ||
// +k8s:prerelease-lifecycle-gen:replacement=apps,v1,DeploymentRollback |
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.
no replacement for this API, support was removed in later versions
/lgtm one nit https://github.com/kubernetes/kubernetes/pull/90457/files#r416015663, follow up to drop that is fine |
[APPROVALNOTIFIER] This PR is APPROVED This pull-request has been approved by: deads2k, liggitt The full list of commands accepted by this bot can be found here. The pull request process is described here
Needs approval from an approver in each of these files:
Approvers can indicate their approval by writing |
This adds the lifecycle annotation tags matching https://github.com/kubernetes/enhancements/blob/master/keps/sig-architecture/1635-prevent-permabeta/README.md
The clock is set for removal of beta levels in 1.22.
/kind cleanup
/priority important-soon
@kubernetes/sig-apps-misc @kubernetes/sig-api-machinery-misc