Skip to content

infra: fix DeleteAllOf rbac#4459

Merged
zirain merged 2 commits intoenvoyproxy:mainfrom
zirain:fix/DeleteAllOf
Oct 17, 2024
Merged

infra: fix DeleteAllOf rbac#4459
zirain merged 2 commits intoenvoyproxy:mainfrom
zirain:fix/DeleteAllOf

Conversation

@zirain
Copy link
Copy Markdown
Member

@zirain zirain commented Oct 16, 2024

following up: #4430

e2e tracked in: #4453

Signed-off-by: zirain <zirain2009@gmail.com>
@zirain zirain requested a review from a team as a code owner October 16, 2024 16:24
@codecov
Copy link
Copy Markdown

codecov bot commented Oct 16, 2024

Codecov Report

Attention: Patch coverage is 57.14286% with 3 lines in your changes missing coverage. Please review.

Project coverage is 65.82%. Comparing base (958df48) to head (daa2795).
Report is 9 commits behind head on main.

Files with missing lines Patch % Lines
...ternal/infrastructure/kubernetes/infra_resource.go 57.14% 3 Missing ⚠️
Additional details and impacted files
@@            Coverage Diff             @@
##             main    #4459      +/-   ##
==========================================
+ Coverage   65.80%   65.82%   +0.02%     
==========================================
  Files         200      200              
  Lines       24277    24284       +7     
==========================================
+ Hits        15976    15986      +10     
  Misses       7159     7159              
+ Partials     1142     1139       -3     

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

Signed-off-by: zirain <zirain2009@gmail.com>
@arkodg arkodg requested review from a team October 16, 2024 23:53
@zirain zirain merged commit aab669d into envoyproxy:main Oct 17, 2024
@zirain zirain deleted the fix/DeleteAllOf branch October 17, 2024 02:46
@arkodg
Copy link
Copy Markdown
Contributor

arkodg commented Oct 19, 2024

should we backport this @zirain ?

@zirain
Copy link
Copy Markdown
Member Author

zirain commented Oct 19, 2024

both or not

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants