(chore): use native go instead of curl pod in non agentgateway tests

[1Shubham7]

Migrating all the non agentgateway tests from curl pods to native go http client

Description

part of #13053

--- PASS: TestKgateway (56.26s)
    --- PASS: TestKgateway/ExtAuth (45.10s)
        --- PASS: TestKgateway/ExtAuth/TestExtAuthPolicy (1.28s)
            --- PASS: TestKgateway/ExtAuth/TestExtAuthPolicy/request_allowed_with_allow_header (0.01s)
            --- PASS: TestKgateway/ExtAuth/TestExtAuthPolicy/request_denied_without_allow_header (0.00s)
            --- PASS: TestKgateway/ExtAuth/TestExtAuthPolicy/request_denied_with_deny_header (0.00s)
            --- PASS: TestKgateway/ExtAuth/TestExtAuthPolicy/request_allowed_on_insecure_route (0.00s)
        --- PASS: TestKgateway/ExtAuth/TestRouteTargetedExtAuthPolicy (1.26s)
            --- PASS: TestKgateway/ExtAuth/TestRouteTargetedExtAuthPolicy/request_allowed_by_default (0.00s)
            --- PASS: TestKgateway/ExtAuth/TestRouteTargetedExtAuthPolicy/request_allowed_on_insecure_route (0.00s)
            --- PASS: TestKgateway/ExtAuth/TestRouteTargetedExtAuthPolicy/request_allowed_with_allow_header_on_secured_route (0.00s)
            --- PASS: TestKgateway/ExtAuth/TestRouteTargetedExtAuthPolicy/request_denied_without_header_on_secured_route (0.00s)

Change Type

/kind fix

Changelog

NONE

Additional Notes

• Not migrating session_persistence - It needs the raw response to extract cookies/headers and then later assert those.

[Copilot]

Pull request overview

This PR updates the ExtAuth e2e test suite to stop relying on a dedicated curl pod and instead use the shared Go-based gateway client used by agentgateway tests.

Changes:

• Remove the curl pod manifest and its associated resources from the ExtAuth e2e test setup.
• Initialize common.BaseGateway in the ExtAuth suite using SetupBaseGateway, and use BaseGateway.Send for HTTP requests instead of AssertEventualCurlResponse with a curl pod.
• Update readiness checks to wait on the super-gateway and ext-authz pods directly, rather than the curl pod.

---

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

[npolshakova]

Couple notes:

1. The changes should be at the suite level not at the per-feature test level:

   kgateway/test/e2e/tests/agent_gateway_test.go

   Line 55 in da4c3eb

   common.SetupBaseConfig(ctx, t, testInstallation, filepath.Join("manifests", "agent-gateway-base.yaml"))

2. Please update the PR description to be correctly formatted based on the template (description, release note should be NONE for non-user changes, etc.)

[1Shubham7]

got it @npolshakova ! since I just had changed one test, it would mess up other tests so that was a temp measure : )

also, what did you mean by non-user changes?

[npolshakova]

The PR description needs to follow the PR template. See the contributing guide and design doc for changelog management for more info:

• https://github.com/kgateway-dev/kgateway/blob/6a749bc9026244a429f3788f8c462cd0b45343b7/devel/contributing/pull-requests.md
• https://github.com/kgateway-dev/kgateway/blob/6a749bc9026244a429f3788f8c462cd0b45343b7/design/10381-adopt-kubernetes-approach-for-changelog-management.md

Use NONE for non-user facing changes.