Skip to content

chore(make): fix go.mod.tidy.examples target#7386

Closed
maxbrunet wants to merge 1 commit intoenvoyproxy:mainfrom
maxbrunet:chore/make/fix-go.mod.tidy.examples
Closed

chore(make): fix go.mod.tidy.examples target#7386
maxbrunet wants to merge 1 commit intoenvoyproxy:mainfrom
maxbrunet:chore/make/fix-go.mod.tidy.examples

Conversation

@maxbrunet
Copy link
Copy Markdown
Contributor

@maxbrunet maxbrunet commented Oct 31, 2025

What type of PR is this?

Chore

What this PR does / why we need it:

Fixes:

$ make go.mod.tidy.examples 
[...]
~/Documents/github.com/maxbrunet/envoy-gateway/examples/extension-server ~/Documents/github.com/maxbrunet/envoy-gateway
go: finding module for package sigs.k8s.io/gateway-api-inference-extension/api/v1alpha2
go: github.com/exampleorg/envoygateway-extension/internal/extensionserver imports
        sigs.k8s.io/gateway-api-inference-extension/api/v1alpha2: module sigs.k8s.io/gateway-api-inference-extension@latest found (v1.1.0), but does not contain package sigs.k8s.io/gateway-api-inference-extension/api/v1alpha2
[...]

Which issue(s) this PR fixes:

Reverts #7282

Release Notes: No

@maxbrunet maxbrunet requested a review from a team as a code owner October 31, 2025 00:29
Signed-off-by: Maxime Brunet <max@brnt.mx>
@maxbrunet maxbrunet force-pushed the chore/make/fix-go.mod.tidy.examples branch from 701ddac to 438d702 Compare October 31, 2025 00:29
@codecov
Copy link
Copy Markdown

codecov bot commented Oct 31, 2025

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 72.30%. Comparing base (e5c1bb3) to head (438d702).
⚠️ Report is 10 commits behind head on main.

Additional details and impacted files
@@           Coverage Diff           @@
##             main    #7386   +/-   ##
=======================================
  Coverage   72.30%   72.30%           
=======================================
  Files         231      231           
  Lines       33939    33939           
=======================================
+ Hits        24539    24541    +2     
+ Misses       7632     7630    -2     
  Partials     1768     1768           

☔ 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.
  • 📦 JS Bundle Analysis: Save yourself from yourself by tracking and limiting bundle sizes in JS merges.

@arkodg
Copy link
Copy Markdown
Contributor

arkodg commented Oct 31, 2025

can we stick to v1.1.0 and update the examples from v1alpha2 to v1

@maxbrunet
Copy link
Copy Markdown
Contributor Author

can we stick to v1.1.0 and update the examples from v1alpha2 to v1

I will not be able to dive into the details immediately and guarantee things are working, especially that there are no automated tests, so I would rather someone else to do it, the goal is to do the minimum to fix all the make targets

arkodg pushed a commit to arkodg/gateway that referenced this pull request Oct 31, 2025
Relates to envoyproxy#7386

Signed-off-by: Arko Dasgupta <arko@tetrate.io>
@arkodg
Copy link
Copy Markdown
Contributor

arkodg commented Oct 31, 2025

np @maxbrunet, raised #7389

arkodg added a commit that referenced this pull request Oct 31, 2025
Relates to #7386

Signed-off-by: Arko Dasgupta <arko@tetrate.io>
@maxbrunet maxbrunet closed this Oct 31, 2025
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.

2 participants