Skip to content

fix: sync delegated runners after allocation#110

Merged
vincentkoc merged 1 commit into
mainfrom
fix/provider-router-visibility
May 15, 2026
Merged

fix: sync delegated runners after allocation#110
vincentkoc merged 1 commit into
mainfrom
fix/provider-router-visibility

Conversation

@vincentkoc
Copy link
Copy Markdown
Member

Summary

  • refresh the coordinator external-runner portal after successful delegated warmup/run allocations
  • add a regression test for Blacksmith JSON-list runner sync
  • add a clearer Blacksmith outage hint that points users to coordinator-backed providers

Verification

  • go test ./internal/cli -run 'TestSyncExternalRunnersBestEffortPostsBlacksmithJSONList|TestCoordinatorExternalRunnersFromBlacksmithListView' -count=1\n- go test ./internal/providers/blacksmith -run 'TestBlacksmithWarmupFailure|TestBlacksmithWarmupArgs' -count=1\n- git diff --check\n\n## Notes\n- Broad go test -timeout=30s ./internal/cli ./internal/providers/blacksmith timed out in unrelated TestInitProjectWritesExpectedFiles while waiting on git repo discovery; the targeted regressions above passed.

@vincentkoc vincentkoc marked this pull request as ready for review May 15, 2026 18:44
@vincentkoc vincentkoc merged commit 6777ae2 into main May 15, 2026
10 checks passed
@vincentkoc vincentkoc deleted the fix/provider-router-visibility branch May 15, 2026 18:47
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