Skip to content

Expose root command of envoy-gateway#5407

Merged
arkodg merged 2 commits intoenvoyproxy:mainfrom
mathetake:cmdroot
Mar 4, 2025
Merged

Expose root command of envoy-gateway#5407
arkodg merged 2 commits intoenvoyproxy:mainfrom
mathetake:cmdroot

Conversation

@mathetake
Copy link
Copy Markdown
Member

@mathetake mathetake commented Mar 4, 2025

What type of PR is this?
Refactors the envoy-gateway CLI packages around main program.

What this PR does / why we need it:
Refactors the envoy-gateway CLI packages around main program so that envoyproxy/ai-gateway project can utilize the standalone mode as part of its CLI.

Release Notes: No

Signed-off-by: Takeshi Yoneda <t.y.mathetake@gmail.com>
arkodg
arkodg previously approved these changes Mar 4, 2025
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.

SGTM thanks

@arkodg arkodg requested review from a team March 4, 2025 22:34
zirain
zirain previously approved these changes Mar 4, 2025
@mathetake
Copy link
Copy Markdown
Member Author

oh wait sorry lint

Signed-off-by: Takeshi Yoneda <t.y.mathetake@gmail.com>
@mathetake mathetake dismissed stale reviews from zirain and arkodg via 4d6a32b March 4, 2025 22:37
@codecov
Copy link
Copy Markdown

codecov bot commented Mar 4, 2025

Codecov Report

Attention: Patch coverage is 75.00000% with 1 line in your changes missing coverage. Please review.

Project coverage is 65.01%. Comparing base (eb5ab1d) to head (4d6a32b).
Report is 1 commits behind head on main.

Files with missing lines Patch % Lines
internal/cmd/versions.go 0.00% 1 Missing ⚠️
Additional details and impacted files
@@            Coverage Diff             @@
##             main    #5407      +/-   ##
==========================================
- Coverage   65.12%   65.01%   -0.11%     
==========================================
  Files         213      212       -1     
  Lines       33592    33579      -13     
==========================================
- Hits        21878    21833      -45     
- Misses      10389    10416      +27     
- Partials     1325     1330       +5     

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

@mathetake
Copy link
Copy Markdown
Member Author

looks like there's a flaky little guy

@arkodg arkodg merged commit 779c0a6 into envoyproxy:main Mar 4, 2025
24 of 25 checks passed
@mathetake mathetake deleted the cmdroot branch March 4, 2025 23:39
mathetake added a commit to envoyproxy/ai-gateway that referenced this pull request Mar 5, 2025
**Commit Message**

The CLI's standalone mode needs to use Envoy Gateway's latest patch [1]
that allows us to use it inside our CLI. This upgrades the EG dependency
version to the latest. It is not compatible with the latest cel-go, so
this also pins it to 0.22 until the upstream k8s.io/apiserver dependency
[2] issue is resolved. This commit is separated from the standalone
implementation commit as the upgrade results in a CRD generation which
is not relevant to the change.

1: envoyproxy/gateway#5407
2: https://github.com/kubernetes/apiserver/issues/116

**Related Issues/PRs (if applicable)**

This is a preparation for #458

Signed-off-by: Takeshi Yoneda <t.y.mathetake@gmail.com>
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