Skip to content

chore(release): let the migrate race shard stay silent for an hour on CircleCI - #1066

Closed
devin-ai-integration[bot] wants to merge 1 commit into
mainfrom
hotfix/0.10.0-race-output-timeout
Closed

devin-ai-integration[bot] wants to merge 1 commit into
mainfrom
hotfix/0.10.0-race-output-timeout

Conversation

@devin-ai-integration

Copy link
Copy Markdown
Contributor

What and why

Hotfix from main for the 0.10.0 release, completing #1065. The first CircleCI build with the dedicated migrate race shard (develop 7969673) still failed: tests/migrate alone on a 4-vCPU large node reached panic: test timed out after 30m0s (FAIL github.com/Open-MBEE/OpenSysML/tests/migrate 1800.088s), which is why #1065 gave that shard a 55-minute go test -timeout. But the step also has no_output_timeout: 30m, and go test prints a package's -v output only once the package finishes (that node emitted its whole log in one chunk at the end), so a run between 30 and 55 minutes is killed by CircleCI for silence before it can pass. The release workflow's publish jobs all require Go race tests, so this is what stands between main and a green rehearsal.

  • go-race-test's Run Go race tests step: no_output_timeout 30m → 60m, with the reason beside it (the repository already documents the same buffering for the REPL gate step).

No test, flag or package change.

How it was verified

  • circleci config validate .circleci/config.yml passes.
  • CircleCI evidence: develop pipeline 3993, Go race tests node 4 (shard migrate): 31.2 min, output delivered in a single chunk at the end, panic: test timed out after 30m0s. Locally the package alone takes 1423 s under -race on 8 vCPU.
  • The proof is the CircleCI release_rehearsal on main after merge; GitHub PR CI does not run this step.

Checklist

  • make test and make lint pass locally (CI configuration change; the Go suite is unchanged)
  • Tests added or updated for the change (n/a: CI configuration)
  • Documentation extended where it already covers the surface (n/a)
  • Changelog entry added as changes/unreleased/<slug>.<section>.md (n/a: CI-only, no user-visible change; chore(release): give tests/migrate its own race shard so the 0.10.0 release workflow can pass #1065 already records the shard)
  • baselines regenerated and make docs-counts run if a gate count moved (n/a)
  • No internal work-item labels (waves, slices, F4, K5) in the body, docs, or changelog

Link to Devin session: https://nasa-jpl-demo.devinenterprise.com/sessions/3db6a090aa214667b2a66d372c6e36b1
Open in Devin Desktop: https://nasa-jpl-demo.devinenterprise.com/desktop/session/3db6a090aa214667b2a66d372c6e36b1?variant=devin
Requested by: @HuiJun

go test buffers a package's -v output until it finishes, so a 55-minute package under a 30-minute no_output_timeout is killed for silence before it can pass.

Co-Authored-By: jason.han <hanhuijun@gmail.com>
@devin-ai-integration

Copy link
Copy Markdown
Contributor Author

I'll fix CI failures and address comments from users with write access. I'll skip comments containing "(aside)".

  • Disable automatic comment, CI, and merge conflict monitoring

@devin-ai-integration
devin-ai-integration Bot changed the base branch from develop to main October 11, 2026 00:48
@devin-ai-integration
devin-ai-integration Bot marked this pull request as ready for review October 11, 2026 01:22

@devin-ai-integration devin-ai-integration Bot left a comment

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

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

🔍 Devin Review: 1 flag

Not posted on this PR by your GitHub settings — view it in Devin Review. (Configure)

Devin Review

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