Skip to content

chore(ci): harness CI timeout #117

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom
Merged
mfazekas merged 1 commit into from
Jan 16, 2026
Merged

chore(ci): harness CI timeout #117

mfazekas merged 1 commit into from
Jan 16, 2026

Conversation

@mfazekas
Copy link
Collaborator

@mfazekas mfazekas commented Jan 12, 2026 •
edited
Loading

Summary

Re-enables harness CI tests with retry mechanism to handle flaky WebSocket connections.

Changes:

  • Removed if: false that was disabling harness tests
  • Added retry logic (3 attempts) for flaky harness connections
  • Added wait step for simulator to be fully ready before tests
  • Added debug logging on failure to capture simulator logs

Test plan

  • CI harness tests pass (may need 1-3 attempts due to flaky connection)

@mfazekas mfazekas marked this pull request as ready for review January 13, 2026 10:23
@mfazekas mfazekas changed the title fix: debug harness CI timeout fix: harness CI timeout Jan 14, 2026
@mfazekas mfazekas force-pushed the fix/harness-ci-timeout branch from fe97adc to e459d76 Compare January 16, 2026 07:24
@mfazekas mfazekas changed the title fix: harness CI timeout chore(ci): harness CI timeout Jan 16, 2026
@mfazekas mfazekas enabled auto-merge (squash) January 16, 2026 08:18
@mfazekas mfazekas requested a review from HayesGordon January 16, 2026 08:18
HayesGordon
HayesGordon approved these changes Jan 16, 2026
View reviewed changes
Copy link
Contributor

@HayesGordon HayesGordon left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM! Thanks

@mfazekas mfazekas merged commit 65bc6f1 into main Jan 16, 2026
8 checks passed
@mfazekas mfazekas deleted the fix/harness-ci-timeout branch January 16, 2026 10:09
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@HayesGordon HayesGordon HayesGordon approved these changes

Assignees

No one assigned

Labels

None yet

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

3 participants

@mfazekas @HayesGordon