Skip to content

helm: add more addons for gateway-addons-helm#3541

Merged
arkodg merged 9 commits intoenvoyproxy:mainfrom
shawnh2:more-helm-addons
Jun 14, 2024
Merged

helm: add more addons for gateway-addons-helm#3541
arkodg merged 9 commits intoenvoyproxy:mainfrom
shawnh2:more-helm-addons

Conversation

@shawnh2
Copy link
Copy Markdown
Contributor

@shawnh2 shawnh2 commented Jun 5, 2024

What type of PR is this?

What this PR does / why we need it:

Which issue(s) this PR fixes:

xref #3355

Signed-off-by: shawnh2 <[email protected]>
@shawnh2
Copy link
Copy Markdown
Contributor Author

shawnh2 commented Jun 5, 2024

will update the observability doc, makefile and clean the helm-values under example within a follow-up PR.

@codecov
Copy link
Copy Markdown

codecov bot commented Jun 5, 2024

Codecov Report

All modified and coverable lines are covered by tests ✅

Project coverage is 68.18%. Comparing base (8a4b13d) to head (fde22af).

Additional details and impacted files
@@            Coverage Diff             @@
##             main    #3541      +/-   ##
==========================================
+ Coverage   68.17%   68.18%   +0.01%     
==========================================
  Files         168      168              
  Lines       20484    20484              
==========================================
+ Hits        13964    13967       +3     
+ Misses       5511     5509       -2     
+ Partials     1009     1008       -1     

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

@shawnh2 shawnh2 marked this pull request as ready for review June 11, 2024 14:23
@shawnh2 shawnh2 requested a review from a team as a code owner June 11, 2024 14:23
zirain
zirain previously approved these changes Jun 11, 2024
Signed-off-by: shawnh2 <[email protected]>
Signed-off-by: shawnh2 <[email protected]>
@@ -1,5 +1,12 @@
tags:
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.

curious why we need this ?

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

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

to provide users more options to play with, also indicates which component is all about metrics or logging etc.

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.

Copy link
Copy Markdown
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 !

@shawnh2 shawnh2 requested a review from zirain June 14, 2024 02:48
@shawnh2 shawnh2 requested review from a team June 14, 2024 03:12
@arkodg arkodg merged commit 6425d03 into envoyproxy:main Jun 14, 2024
@shawnh2 shawnh2 deleted the more-helm-addons branch June 15, 2024 05:38
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants