-   Notifications  You must be signed in to change notification settings 
- Fork 0
Open
Labels
Description
CI Failure Details
CI Run Link: https://github.com/coder/coder/actions/runs/18040492821/job/51337933495
 Commit: 0a6ba5d51a02603b5d1ccf9ea3c47f9555ee2a20 - feat: add endpoint to list aibridge interceptions (#19929)
 Job: test-go-pg (ubuntu-latest)
 Date: 2025-09-26 14:28:38Z
Test Failure
=== FAIL: agent/agentcontainers TestAPI/FileWatcher (0.00s) api_test.go:1629:	Error Trace:	/home/runner/work/coder/coder/agent/agentcontainers/api_test.go:1629	Error:	Should be false	Test:	TestAPI/FileWatcher	Messages:	dirty flag should be cleared on the devcontainer after container recreation Assignment Analysis
Git blame for test function: ./agent/agentcontainers/api_test.go:1510
git blame -L 1510,1530 ./agent/agentcontainers/api_test.go 268a50c193 (Mathias Fredriksson 2025-04-29 11:53:58 +0300) t.Run("FileWatcher", func(t *testing.T) { Last significant modification: Mathias Fredriksson (mafredri@gmail.com)
Root Cause Assessment
- Test Type: Unit test failure in agentcontainers package
- Issue: Assertion failure - dirty flag not being cleared after container recreation
- Error Pattern: Clean assertion failure, no data races or panics detected
- Code Impact: The failing commit touches AIBridge functionality, not agentcontainers, suggesting this is a flaky test unrelated to the code changes
Error Analysis
The test is verifying that a "dirty" flag is properly cleared on a devcontainer after container recreation. The assertion Should be false failed, indicating the dirty flag remained true when it should have been false. This suggests a timing issue or state management problem in the devcontainer lifecycle.