esp-v2 icon indicating copy to clipboard operation
esp-v2 copied to clipboard

Enforce the default http rule

Open dafudeng opened this issue 3 years ago • 2 comments

Add the default http rule of a rpc endpoint to the additional_bindings field of any new http rules of that rpc endpoint. Otherwise, once a new http rule is set, the default http rule will no longer work, which is not expected.

dafudeng avatar Aug 16 '22 14:08 dafudeng

[APPROVALNOTIFIER] This PR is APPROVED

This pull-request has been approved by: dafudeng, nareddyt, TAOXUY

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

The pull request process is described here

Needs approval from an approver in each of these files:
  • ~~OWNERS~~ [TAOXUY,dafudeng,nareddyt]

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

google-oss-prow[bot] avatar Aug 22 '22 19:08 google-oss-prow[bot]

@dafudeng: The following tests failed, say /retest to rerun all failed tests or /retest-required to rerun all mandatory failed tests:

Test name Commit Details Required Rerun command
ESPv2-gke-e2e-grpc-interop-managed 8f5edf2fd6f3e5da298492c8eff669eca64e9401 link true /test ESPv2-gke-e2e-grpc-interop-managed
ESPv2-cloud-run-e2e-cloud-run-grpc-echo 8f5edf2fd6f3e5da298492c8eff669eca64e9401 link true /test ESPv2-cloud-run-e2e-cloud-run-grpc-echo
ESPv2-gke-e2e-http-bookstore-managed 8f5edf2fd6f3e5da298492c8eff669eca64e9401 link true /test ESPv2-gke-e2e-http-bookstore-managed
ESPv2-gke-e2e-http-bookstore-managed-using-sa-cred 8f5edf2fd6f3e5da298492c8eff669eca64e9401 link true /test ESPv2-gke-e2e-http-bookstore-managed-using-sa-cred
ESPv2-gke-e2e-grpc-echo-managed 8f5edf2fd6f3e5da298492c8eff669eca64e9401 link true /test ESPv2-gke-e2e-grpc-echo-managed
ESPv2-presubmit-asan 8f5edf2fd6f3e5da298492c8eff669eca64e9401 link true /test ESPv2-presubmit-asan
ESPv2-anthos-cloud-run-e2e-anthos-cloud-run-http-bookstore 8f5edf2fd6f3e5da298492c8eff669eca64e9401 link true /test ESPv2-anthos-cloud-run-e2e-anthos-cloud-run-http-bookstore

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. I understand the commands that are listed here.

google-oss-prow[bot] avatar Aug 22 '22 19:08 google-oss-prow[bot]