-
Notifications
You must be signed in to change notification settings - Fork 3.4k
k8s: Remove deprecated identity status field #13053
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
k8s: Remove deprecated identity status field #13053
Conversation
Following the deprecation notice from cilium#11275, remove the field. Signed-off-by: Chris Tarazi <chris@isovalent.com>
test-me-please Edit: net-next hit #13071 |
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 🥇
Just want to check if any docs (e.g. spec, upgrade note, etc) is required, or this change will be just transparent to users.
The change should be transparent as the aforementioned PR migrates its usage to be under the purview of CiliumEndpoint. I did a quick scan over the docs and don't see any mention of this field. Thanks for the tip. |
retest-net-next |
Code owner approval and full CI have passed. Marking ready to merge. (Not sure why maintainer's little helper didn't detect this PR as ready.) |
Following the deprecation notice from
#11275, remove the field.