-
Notifications
You must be signed in to change notification settings - Fork 451
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
Expand output for gwctl describe httproutes
#3037
Comments
No worries @gauravkghildiyal and thank you for creating the issue. |
Hey @tdn21, did you get a chance to look into this? |
1 similar comment
Hey @tdn21, did you get a chance to look into this? |
Hello @gauravkghildiyal, apologies for the delay. I've checked the expected output and identified the required changes. I'll most likely submit a draft PR today or tomorrow. (draft because i'd like to get some initial feedback on |
Hi @gauravkghildiyal, I have a couple of questions about inherited policies. Please bear with me as some of them might seem basic due to my limited understanding of policies.
|
Those are all valid and good questions, @tdn21. Thanks for requesting clarification.
Essentially:
Essentially, the inherited policy for a resource is effectively the aggregation of all policies that are inheritable by resources in it's hierarchy. You do not need to include these changes in your PR, but just so you don't get confused, we'll later on re-use the same "filtering for inherited policies" function within the EffectivePolicies calculation as well. So essentially, EffectivePolicies will be the merging of all inherited policies. |
Refer https://gateway-api.sigs.k8s.io/geps/gep-2722/ and #2761 for details
The text was updated successfully, but these errors were encountered: