-
Notifications
You must be signed in to change notification settings - Fork 1.3k
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
[AKS] Mark "--enable-pod-security-policy" deprecated #5542
Conversation
5ed1ef0
to
f22db45
Compare
AKS |
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.
To clarify, the deprecated option has no corresponding replacement in CLI. According to the doc, the alternative is to use kubectl to add labels directly to the cluster namespace.
@bingosummer If you want to release a new extension version to release this change, please also upgrade the version defined in |
@zhoxing-ms Thanks! The fix doesn't need a release. It can be released with other PR's changes later. |
f22db45
to
760470c
Compare
@zhoxing-ms Thanks for reviews! Resolved the comments. |
After removing help, CI raises the below error. Will add them back.
|
760470c
to
c2f52cc
Compare
Mark PodSecurityPolicy deprecated because PodSecurityPolicy was deprecated in Kubernetes v1.21, and removed from Kubernetes in v1.25.