Skip to content

Replace contour-authserver in e2e with gRPC server running in test suite#7597

Draft
tsaarni wants to merge 1 commit into
projectcontour:mainfrom
Nordix:e2e-grpc-ext-auth-local-server
Draft

Replace contour-authserver in e2e with gRPC server running in test suite#7597
tsaarni wants to merge 1 commit into
projectcontour:mainfrom
Nordix:e2e-grpc-ext-auth-local-server

Conversation

@tsaarni

@tsaarni tsaarni commented Jun 15, 2026

Copy link
Copy Markdown
Member

This PR proposes dropping contour-authserver as a dependency for the e2e test suite and replacing it with a test server stub that runs directly within the test case.

This approach makes the external authorization tests self-contained, allowing the authorization server to be easily programmed to return specific responses or assert request parameters right inside the test case.

This is similar to the HTTP-based authorization server I mentioned here: #7418 (comment)

configure Envoy to connect to the host via a Service and EndpointSlice, and then set up an HTTP server as part of e2e test suite, giving the test case full control to run assertions.

image

Additionally, contour-authserver has not been maintained for a long time, with its last release in 2023. It might be better to archive that project altogether.

@tsaarni tsaarni added the release-note/none-required Marks a PR as not requiring a release note. Should only be used for very small changes. label Jun 15, 2026
Signed-off-by: Tero Saarni <tero.saarni@est.tech>
@tsaarni tsaarni force-pushed the e2e-grpc-ext-auth-local-server branch from 4e588a0 to 7425508 Compare June 15, 2026 21:06
@codecov

codecov Bot commented Jun 15, 2026

Copy link
Copy Markdown

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 82.35%. Comparing base (34d2ab1) to head (7425508).

Additional details and impacted files

Impacted file tree graph

@@           Coverage Diff           @@
##             main    #7597   +/-   ##
=======================================
  Coverage   82.35%   82.35%           
=======================================
  Files         130      130           
  Lines       15873    15873           
=======================================
  Hits        13073    13073           
  Misses       2513     2513           
  Partials      287      287           
🚀 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.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

release-note/none-required Marks a PR as not requiring a release note. Should only be used for very small changes.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant