Skip to content

e2e: add a egress test on http request to non 80 port - #1054

Merged
haiyanmeng merged 1 commit into
agent-substrate:mainfrom
yufan-su:e2e-http-non-80-port
Aug 24, 2026
Merged

e2e: add a egress test on http request to non 80 port#1054
haiyanmeng merged 1 commit into
agent-substrate:mainfrom
yufan-su:e2e-http-non-80-port

Conversation

@yufan-su

@yufan-su yufan-su commented Aug 19, 2026

Copy link
Copy Markdown
Collaborator

This PR adds TestActorEgressNonStandardPort to the networking suite: an Actor fetches an in-cluster HTTP origin on :8080 through the egress gateway, and we assert both a 200 and an atenet-egress access-log line carrying authority=:8080 with this Actor's SPIFFE SAN.

Changes

  • internal/e2e/suites/networking/networking_test.go — the test itself, reusing createAndResumeActor, fetchThroughEgressActor and assertEgressGatewayConnect unchanged.

  • Tests pass

#1017

Comment thread internal/e2e/fixtures/egressprobe/httptarget.yaml.tmpl Outdated
Comment thread internal/e2e/suites/networking/httptarget_test.go Outdated
@haiyanmeng

Copy link
Copy Markdown
Collaborator

yufan-su thanks for the work.

Can you take a look across this PR, #1119, and the existing test code, to figure out a solution minimizing the duplication in our test code?

@yufan-su

Copy link
Copy Markdown
Collaborator Author

I will rebase this e2e on top of #1119 and expect reduce the overall change size to < 100 lines

@yufan-su
yufan-su force-pushed the e2e-http-non-80-port branch from cef6662 to 3e445b6 Compare August 24, 2026 20:50
@yufan-su
yufan-su requested a review from haiyanmeng August 24, 2026 20:52
@haiyanmeng
haiyanmeng merged commit 8cb13f2 into agent-substrate:main Aug 24, 2026
10 of 11 checks passed
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