Skip to content

fix(ci): wait for snap gateway readiness - #3131

Closed
drew wants to merge 1 commit into
mainfrom
codex/fix-snap-canary-readiness/drew
Closed

fix(ci): wait for snap gateway readiness#3131
drew wants to merge 1 commit into
mainfrom
codex/fix-snap-canary-readiness/drew

Conversation

@drew

@drew drew commented Sep 2, 2026

Copy link
Copy Markdown
Collaborator

Summary

Wait for the Snap-packaged gateway to accept status requests before declaring the Ubuntu Snap canary failed. The service can report active before its listener is ready, which caused an intermittent connection-refused failure.

The same Release Dev commit demonstrated the race: run 33562715502 failed the Ubuntu Snap check while run 33562871938 passed it.

Related Issue

No issue required: this is an obvious localized release-canary reliability fix.

Changes

  • Poll openshell status for up to 60 seconds after gateway registration.
  • Preserve a hard failure when readiness never arrives.
  • Print Snap service state and gateway logs on timeout.

Testing

  • mise run pre-commit passes
  • Unit tests added/updated (not applicable: workflow-only change)
  • E2E tests added/updated (not applicable: canary is the E2E smoke test)

Checklist

  • Follows Conventional Commits
  • Commits are signed off (DCO)
  • Architecture docs updated (not applicable: no architecture change)

Signed-off-by: Drew Newberry <anewberry@nvidia.com>
@drew
drew requested review from a team, derekwaynecarr, mrunalp and sjenning as code owners September 2, 2026 14:59
@elezar

elezar commented Sep 2, 2026

Copy link
Copy Markdown
Member

@drew could you comment on the difference between this and #2866?

@drew

drew commented Sep 2, 2026

Copy link
Copy Markdown
Collaborator Author

I have a scheduled automation that started reviewing flaky tests and opening prs to address findings. This PR is one of those. We can close this and prefer yours.

@drew drew closed this Sep 2, 2026
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.

3 participants