Skip to content

Add inherited policies for Gateway and Backends#3198

Merged
k8s-ci-robot merged 1 commit into
kubernetes-sigs:mainfrom
gauravkghildiyal:gw-and-backend-inherited-policies
Aug 9, 2024
Merged

Add inherited policies for Gateway and Backends#3198
k8s-ci-robot merged 1 commit into
kubernetes-sigs:mainfrom
gauravkghildiyal:gw-and-backend-inherited-policies

Conversation

@gauravkghildiyal
Copy link
Copy Markdown
Member

@gauravkghildiyal gauravkghildiyal commented Jul 16, 2024

What type of PR is this?
/kind feature

What this PR does / why we need it:

Add inherited policies for Gateway and Backends. Refer test cases for sample outputs

/hold

Does this PR introduce a user-facing change?:

Describing Gateways and Backends will include a separate field showing InheritedPolicies

@k8s-ci-robot k8s-ci-robot added do-not-merge/hold Indicates that a PR should not merge because someone has issued a /hold command. release-note Denotes a PR that will be considered when it comes time to generate release notes. kind/feature Categorizes issue or PR as related to a new feature. cncf-cla: yes Indicates the PR's author has signed the CNCF CLA. size/XL Denotes a PR that changes 500-999 lines, ignoring generated files. labels Jul 16, 2024
@k8s-ci-robot
Copy link
Copy Markdown
Contributor

[APPROVALNOTIFIER] This PR is APPROVED

This pull-request has been approved by: gauravkghildiyal

The full list of commands accepted by this bot can be found here.

The pull request process is described here

Details Needs approval from an approver in each of these files:

Approvers can indicate their approval by writing /approve in a comment
Approvers can cancel approval by writing /approve cancel in a comment

@k8s-ci-robot k8s-ci-robot added area/gwctl approved Indicates a PR has been approved by an approver from all required OWNERS files. labels Jul 16, 2024
@k8s-ci-robot k8s-ci-robot requested review from robscott and thockin July 16, 2024 08:06
@k8s-ci-robot k8s-ci-robot added the needs-rebase Indicates a PR cannot be merged because it has merge conflicts with HEAD. label Jul 22, 2024
Comment thread gwctl/pkg/printer/backends_test.go Outdated
},
"spec": map[string]interface{}{
"default": map[string]interface{}{
"key2": "value-parent-2",
Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Worth adding an example of a default "key1": "value-parent-1" here perhaps, to highlight the difference between an override vs a default inherited from the gateway?

Suggested change
"key2": "value-parent-2",
"key1": "value-parent-1",
"key2": "value-parent-2",

Copy link
Copy Markdown
Member Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thanks! Done.

@gauravkghildiyal gauravkghildiyal force-pushed the gw-and-backend-inherited-policies branch from 940d66a to 1a1fbbc Compare July 23, 2024 20:08
@k8s-ci-robot k8s-ci-robot added size/L Denotes a PR that changes 100-499 lines, ignoring generated files. and removed size/XL Denotes a PR that changes 500-999 lines, ignoring generated files. needs-rebase Indicates a PR cannot be merged because it has merge conflicts with HEAD. labels Jul 23, 2024
@youngnick
Copy link
Copy Markdown
Contributor

/lgtm

@k8s-ci-robot k8s-ci-robot added the lgtm "Looks good to me", indicates that a PR is ready to be merged. label Aug 9, 2024
@gauravkghildiyal
Copy link
Copy Markdown
Member Author

/unhold

@k8s-ci-robot k8s-ci-robot removed the do-not-merge/hold Indicates that a PR should not merge because someone has issued a /hold command. label Aug 9, 2024
@k8s-ci-robot k8s-ci-robot merged commit 2df90ed into kubernetes-sigs:main Aug 9, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

approved Indicates a PR has been approved by an approver from all required OWNERS files. area/gwctl cncf-cla: yes Indicates the PR's author has signed the CNCF CLA. kind/feature Categorizes issue or PR as related to a new feature. lgtm "Looks good to me", indicates that a PR is ready to be merged. release-note Denotes a PR that will be considered when it comes time to generate release notes. size/L Denotes a PR that changes 100-499 lines, ignoring generated files.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants