envoyproxy / gateway

Manages Envoy Proxy as a Standalone or Kubernetes-based Application Gateway
https://gateway.envoyproxy.io
Apache License 2.0
1.63k stars 352 forks source link

fix: Route with multiple parents has incorrect namespace in parentRef status #4592

Closed zhaohuabing closed 2 weeks ago

zhaohuabing commented 2 weeks ago

Fixes: https://github.com/envoyproxy/gateway/issues/4127

Release Notes: Yes

zhaohuabing commented 2 weeks ago

@guydc this is worth cherry-picking into v1.1.3. It happens whenever a xRoute targets multple Gateways in the same ns.

codecov[bot] commented 2 weeks ago

Codecov Report

All modified and coverable lines are covered by tests :white_check_mark:

Project coverage is 65.44%. Comparing base (6e2587d) to head (d4b0bb3). Report is 2 commits behind head on main.

Additional details and impacted files ```diff @@ Coverage Diff @@ ## main #4592 +/- ## ========================================== + Coverage 65.41% 65.44% +0.02% ========================================== Files 211 211 Lines 31887 31904 +17 ========================================== + Hits 20859 20878 +19 Misses 9782 9782 + Partials 1246 1244 -2 ```

:umbrella: View full report in Codecov by Sentry.
:loudspeaker: Have feedback on the report? Share it here.