api

api copied to clipboard

Updated/new PR for https://github.com/istio/istio/issues/19292

Hi @rolandkool. Thanks for your PR.

I'm waiting for a istio member to verify that this patch is reasonable to test. If it is, they should reply with /ok-to-test on its own line. Until that is done, I will not automatically test new commits in this PR, but the usual testing commands by org members will still work. Regular contributors should join the org to skip this step.

Once the patch is verified, the new status will be reflected by the ok-to-test label.

I understand the commands that are listed here.

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/test-infra repository.

@rolandkool Can you add a bit explaining why this PR is important to you? Some additional context regarding the request would help in getting the necessary approvals. Thanks!

Original motivation was because of multiple CRDs with similar name making tabcompletion useless. This has been fixed. However, other Istio resources that are used often also have a shortname, so for convenience, I would like to see one for authorizationpolicies as well. Networkpolicies also has a shortname (netpol), so if ap is too short, something like authzpol or authpol will work as well.

@rolandkool: 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/test-infra repository.

@rolandkool any plans to rebase and get this merged?

It's frustrating typing out authorizationpolicy each time when interacting with the resource. Shortname ap would be awesome. The other CRDs (peerauthentication and requestauthentication) have short names set to pa and ra.