Skip to content

Conversation

zhaohuabing
Copy link
Member

@zhaohuabing zhaohuabing commented Nov 26, 2024

Fixes #4769

This PR uses port name as the sectionName for the targetRefs of the BackendTLSPolicy.

Release Notes: Yes

@zhaohuabing zhaohuabing requested a review from a team as a code owner November 26, 2024 07:39
@zhaohuabing zhaohuabing marked this pull request as draft November 26, 2024 07:39
@zhaohuabing zhaohuabing changed the title fix btlsp section name fix btlsp section name doesn't support service name Nov 26, 2024
@zhaohuabing zhaohuabing changed the title fix btlsp section name doesn't support service name fix: btlsp section name doesn't support port name Nov 26, 2024
@zhaohuabing zhaohuabing force-pushed the fix-btlsp-section-name branch from 153fd00 to 454d292 Compare November 26, 2024 07:50
Copy link

codecov bot commented Nov 26, 2024

Codecov Report

Attention: Patch coverage is 80.00000% with 5 lines in your changes missing coverage. Please review.

Project coverage is 66.31%. Comparing base (8cba958) to head (1d5abcd).
Report is 2 commits behind head on main.

Files with missing lines Patch % Lines
internal/gatewayapi/resource/resource.go 0.00% 5 Missing ⚠️
Additional details and impacted files
@@            Coverage Diff             @@
##             main    #4784      +/-   ##
==========================================
+ Coverage   66.27%   66.31%   +0.04%     
==========================================
  Files         209      209              
  Lines       31901    31912      +11     
==========================================
+ Hits        21142    21163      +21     
+ Misses       9510     9503       -7     
+ Partials     1249     1246       -3     

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

@zhaohuabing zhaohuabing force-pushed the fix-btlsp-section-name branch from 454d292 to 97289e5 Compare November 26, 2024 08:10
@zhaohuabing zhaohuabing marked this pull request as ready for review November 26, 2024 08:10
Signed-off-by: Huabing Zhao <zhaohuabing@gmail.com>
Signed-off-by: Huabing Zhao <zhaohuabing@gmail.com>
Signed-off-by: Huabing Zhao <zhaohuabing@gmail.com>
@zhaohuabing zhaohuabing force-pushed the fix-btlsp-section-name branch from 19a4814 to 1536c2a Compare November 26, 2024 11:32
Signed-off-by: Huabing Zhao <zhaohuabing@gmail.com>
@zhaohuabing zhaohuabing requested a review from arkodg November 27, 2024 02:11
Signed-off-by: Huabing Zhao <zhaohuabing@gmail.com>
Signed-off-by: Huabing Zhao <zhaohuabing@gmail.com>
Signed-off-by: Huabing Zhao <zhaohuabing@gmail.com>
Signed-off-by: Huabing Zhao <zhaohuabing@gmail.com>
Signed-off-by: Huabing Zhao <zhaohuabing@gmail.com>
@zhaohuabing zhaohuabing force-pushed the fix-btlsp-section-name branch from 299e3fe to 1c829ce Compare December 2, 2024 14:32
arkodg
arkodg previously approved these changes Dec 4, 2024
Copy link
Contributor

@arkodg arkodg left a comment

Choose a reason for hiding this comment

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

LGTM thanks !

@arkodg arkodg requested review from a team December 5, 2024 01:13
zirain
zirain previously approved these changes Dec 5, 2024
Signed-off-by: Huabing Zhao <zhaohuabing@gmail.com>
@zhaohuabing zhaohuabing dismissed stale reviews from zirain and arkodg via 1d5abcd December 6, 2024 09:59
@zhaohuabing zhaohuabing requested review from arkodg and zirain December 6, 2024 09:59
@zirain zirain merged commit b9f9a9f into envoyproxy:main Dec 6, 2024
24 checks passed
@zhaohuabing zhaohuabing deleted the fix-btlsp-section-name branch December 9, 2024 03:22
zhaohuabing added a commit to zhaohuabing/gateway that referenced this pull request Dec 13, 2024
* fix btlsp section name

Signed-off-by: Huabing Zhao <zhaohuabing@gmail.com>

* add release note

Signed-off-by: Huabing Zhao <zhaohuabing@gmail.com>

* optmize service search

Signed-off-by: Huabing Zhao <zhaohuabing@gmail.com>

* address comment

Signed-off-by: Huabing Zhao <zhaohuabing@gmail.com>

* fix test

Signed-off-by: Huabing Zhao <zhaohuabing@gmail.com>

* fix test

Signed-off-by: Huabing Zhao <zhaohuabing@gmail.com>

* address comment

Signed-off-by: Huabing Zhao <zhaohuabing@gmail.com>

* fix e2e

Signed-off-by: Huabing Zhao <zhaohuabing@gmail.com>

---------

Signed-off-by: Huabing Zhao <zhaohuabing@gmail.com>
(cherry picked from commit b9f9a9f)
Signed-off-by: Huabing Zhao <zhaohuabing@gmail.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
3 participants