Skip to content

Stabilize network policy event capture in concurrent tests - #36779

Merged
copyberry[bot] merged 1 commit into
mainfrom
copyberry/codex-internal-to-codex-oss/20d6160d26f56ad9fd86afd5107cb4313a5fce73
Aug 3, 2026
Merged

copyberry[bot] merged 1 commit into
mainfrom
copyberry/codex-internal-to-codex-oss/20d6160d26f56ad9fd86afd5107cb4313a5fce73

Conversation

@copyberry

@copyberry copyberry Bot commented Aug 3, 2026 •

Copy link
Copy Markdown

Stabilize network policy event capture in concurrent tests

Why

Concurrent tests without a tracing subscriber can cache the network policy
callsite as disabled, causing event-capture tests to miss events.

What changed

  • Attach the test event collector directly to the captured future.
  • Rebuild the tracing interest cache before running that future while keeping
    tracing out of its single-subscriber fast path.

## Why

Concurrent tests without a tracing subscriber can cache the network policy
callsite as disabled, causing event-capture tests to miss events.

## What changed

- Attach the test event collector directly to the captured future.
- Rebuild the tracing interest cache before running that future while keeping
  tracing out of its single-subscriber fast path.

GitOrigin-RevId: 20d6160d26f56ad9fd86afd5107cb4313a5fce73
@copyberry
copyberry Bot force-pushed the copyberry/codex-internal-to-codex-oss/20d6160d26f56ad9fd86afd5107cb4313a5fce73 branch from fd295d3 to 136f75e Compare August 3, 2026 18:25
@copyberry
copyberry Bot merged commit 136f75e into main Aug 3, 2026
30 of 32 checks passed
@copyberry
copyberry Bot deleted the copyberry/codex-internal-to-codex-oss/20d6160d26f56ad9fd86afd5107cb4313a5fce73 branch August 3, 2026 18:26
@github-actions github-actions Bot locked and limited conversation to collaborators Aug 3, 2026

This branch was previously deployed

1 inactive deployment
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants