envoyproxy/gateway

Run e2e test with a binary built with `-race` flag

Open

#5,539 opened on Mar 18, 2025

View on GitHub
 (0 comments) (0 reactions) (0 assignees)Go (802 forks)auto 404
area/cihelp wanted

Repository metrics

Stars
 (2,871 stars)
PR merge metrics
 (PR metrics pending)

Description

Description:

I have discovered quite a few race conditions around the main routine of envoy-gateway program #5538 #5505 #5522 in the last few days, and I believe there are still more to be discovered due to the lack of the test around the main program.

Critical orchestration of many goroutines happens in the main routine, so I believe this is really important to make this project more robust.

I recommend that we write comprehensive main program unit tests for all the EG configuration (not partially like i did in AI Gateway but this must include Kubernetes provider) or run e2e/integration tests with the binary built with -race, which I think is easier to test k8s provider.

Contributor guide

Research direction
Investigate how to compile the binary with the race flag using Go's test framework, and identify the existing e2e test suite to integrate this flag for detecting race conditions.
Tech stack
go
Domain
backend
Issue type
Test
DifficultyEstimated implementation difficulty for a new contributor, from 1 for very small changes to 5 for expert-level work.
3
Estimated timeA rough time range for an experienced contributor to investigate, implement, test, and prepare a pull request.
Half day
Activity statusHow available the issue appears right now: fresh, active, stale, blocked, or waiting on maintainer input.
Fresh
ClarityHow clearly the issue explains the expected change, acceptance criteria, and next step.
Mostly clear
Prerequisites
GoGo test race flag
Newbie friendlinessA 1-100 score estimating how approachable this issue is for first-time contributors.
60