-
Notifications
You must be signed in to change notification settings - Fork 571
New issue
Have a question about this project? # for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “#”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? # to your account
Remove legacy Policy
type
#3370
base: master
Are you sure you want to change the base?
Conversation
This has not been used for many many years
@kyessenov can you verify we don't actually need this for something? |
s/1.24/1.6/ - istio/istio#22602 @kyessenov bump! |
I thought the same but we actually used it internally in Envoy until 1.24
IIUC
…On Thu, Dec 12, 2024 at 2:49 PM Craig Box ***@***.***> wrote:
s/1.24/1.6/ - istio/istio#22602
<istio/istio#22602>
@kyessenov <https://github.com/kyessenov> bump!
—
Reply to this email directly, view it on GitHub
<#3370 (comment)>, or
unsubscribe
<https://github.com/notifications/unsubscribe-auth/AAEYGXLH5JKCQBEBNPWCY732FIHGXAVCNFSM6AAAAABSTPX2LOVHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMZDKNBQGE3DINRZHE>
.
You are receiving this because you were mentioned.Message ID:
***@***.***>
|
PR needs rebase. Instructions for interacting with me using PR comments are available here. If you have questions or suggestions related to my behavior, please file an issue against the kubernetes-sigs/prow repository. |
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.
Finally an alpha API that goes away !
This has not been used since Istio 1.24. We should consider cleaning up others like tcp_cluster_rewrite/v2alpha1 and jwt_auth/v2alpha1 as well