Skip to content

feat(helm): Add management of extensionApis#5741

Merged
arkodg merged 1 commit intoenvoyproxy:mainfrom
NeCTAR-RC:helm-extensionapi
Apr 18, 2025
Merged

feat(helm): Add management of extensionApis#5741
arkodg merged 1 commit intoenvoyproxy:mainfrom
NeCTAR-RC:helm-extensionapi

Conversation

@sorrison
Copy link
Contributor

What this PR does / why we need it:
Adds the ability to enable extension APIs with the helm chart

@sorrison sorrison requested a review from a team as a code owner April 16, 2025 01:55
@sorrison sorrison force-pushed the helm-extensionapi branch from 3c49e48 to 1f02aa9 Compare April 16, 2025 01:56
Copy link
Contributor

Choose a reason for hiding this comment

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

can this be added in L97 instead ? under config.envoyGateway ?

Copy link
Contributor Author

Choose a reason for hiding this comment

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

ok I've done that but I think it has unintended consequences. At least when I added all the stuff from make generate but still learning all this stuff

Signed-off-by: Sam Morrison <sorrison@gmail.com>
@sorrison sorrison force-pushed the helm-extensionapi branch from 1f02aa9 to d53114f Compare April 17, 2025 06:28
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 April 17, 2025 17:48
@codecov
Copy link

codecov bot commented Apr 18, 2025

Codecov Report

All modified and coverable lines are covered by tests ✅

Project coverage is 65.18%. Comparing base (096cb8d) to head (d53114f).
Report is 14 commits behind head on main.

Additional details and impacted files
@@            Coverage Diff             @@
##             main    #5741      +/-   ##
==========================================
- Coverage   65.19%   65.18%   -0.02%     
==========================================
  Files         214      214              
  Lines       34321    34444     +123     
==========================================
+ Hits        22377    22451      +74     
- Misses      10591    10629      +38     
- Partials     1353     1364      +11     

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

🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.

@arkodg arkodg merged commit 6863a17 into envoyproxy:main Apr 18, 2025
28 checks passed
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.

3 participants