Skip to content

fix(jangar): relax torghut status timeout#5564

Merged
gregkonush merged 1 commit intomainfrom
codex/swarm-jangar-control-plane-verify
May 5, 2026
Merged

fix(jangar): relax torghut status timeout#5564
gregkonush merged 1 commit intomainfrom
codex/swarm-jangar-control-plane-verify

Conversation

@gregkonush
Copy link
Copy Markdown
Member

Summary

  • Replaces the hardcoded 2s Jangar Torghut status fetch timeout with a configurable JANGAR_TORGHUT_STATUS_TIMEOUT_MS setting.
  • Defaults the timeout to 5000ms and bounds overrides at 30000ms so the empirical-services gate can tolerate the observed Knative status latency without hanging indefinitely.
  • Adds regression coverage for a slow successful Torghut status response and for a configured timeout degradation.
  • Documents the new rollout-safety setting in the Jangar README.

Related Issues

None

Testing

  • PASS bun install --frozen-lockfile --ignore-scripts
  • PASS bun run --cwd services/jangar pretest
  • PASS bun run --cwd services/jangar test -- src/server/__tests__/control-plane-config.test.ts src/server/__tests__/control-plane-empirical-services.test.ts
  • PASS bun run --cwd services/jangar tsc
  • PASS bun run --cwd services/jangar lint
  • PASS bun run --cwd services/jangar lint:oxlint with existing warnings only
  • PASS bun run --cwd services/jangar lint:oxlint:type with existing warnings only
  • PASS bunx oxfmt --check services/jangar/src/server/control-plane-config.ts services/jangar/src/server/control-plane-empirical-services.ts services/jangar/src/server/__tests__/control-plane-config.test.ts services/jangar/src/server/__tests__/control-plane-empirical-services.test.ts
  • PASS git diff --check

Breaking Changes

None

Checklist

  • Testing section documents the exact validation performed (or N/A with justification).
  • Screenshots and Breaking Changes sections are handled appropriately (removed or filled in).
  • Documentation, release notes, and follow-ups are updated or tracked.

@gregkonush
Copy link
Copy Markdown
Member Author

gregkonush commented May 5, 2026

Status: merged and rollout verified.

Evidence:

  • PR fix(jangar): relax torghut status timeout #5564 merged as 6ae22d3f26ff284b91746f6276f9381780d823d9 after all required checks passed.
  • Local validation passed: bun install --frozen-lockfile --ignore-scripts, bun run --cwd services/jangar pretest, targeted Bun tests, tsc, lint, lint:oxlint, lint:oxlint:type, bunx oxfmt --check, and git diff --check.
  • GitHub checks were green or skipped: agents-ci / integration passed in 15m51s, agents-ci / validate passed, jangar-ci / lint-and-typecheck / run passed, title/commit/routing checks passed.
  • The fix was built and promoted; it was later superseded by follow-up Jangar images, with current rollout verified at image 49e27b93.

Decision: complete. Current Jangar rollout is Synced/Healthy through promotion PR #5574 and post-deploy verification run 25405337063.

@gregkonush gregkonush merged commit 6ae22d3 into main May 5, 2026
19 checks passed
@gregkonush gregkonush deleted the codex/swarm-jangar-control-plane-verify branch May 5, 2026 21:22
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant