feat: add --runtime-id to integration tests, add merge queue CI#14
Merged
lostmygithubaccount merged 4 commits intomainfrom Mar 3, 2026
Merged
feat: add --runtime-id to integration tests, add merge queue CI#14lostmygithubaccount merged 4 commits intomainfrom
lostmygithubaccount merged 4 commits intomainfrom
Conversation
- Add --runtime-id argument to all three integration test scripts (default: "ascend-tools"); resolves runtime by ID, falls back to first runtime in the list if not found - Add .github/workflows/integration.yml (merge_group trigger only) - Fix flaky pause/resume tests: increase health wait loops, add run_flow_with_retry to rest.py, fix sleep-before-check ordering Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com>
- Add manual trigger (workflow_dispatch) to integration workflow - Add merge_group trigger to CI workflow so checks run in the queue Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com>
CI (lint/test) runs on PRs already — no need to re-run in the merge queue. Only integration tests run there. Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com>
- Remove health=null wait loop after pause from all three test suites. We already assert paused=true (synchronous). The health field clearing is async pod shutdown timing, not something the SDK controls. - Add merge_group trigger to ci.yml so the check/check required status actually runs in the merge queue (was hanging forever without it). Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com>
lostmygithubaccount
removed this pull request from the merge queue due to a manual request
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
1 participant
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Summary
--runtime-idargument to all three integration test scripts (default:ascend-tools); resolves runtime by ID vialist_runtimes(id=...), falls back to first runtime if not found.github/workflows/integration.yml— runs integration tests onmerge_groupevents onlyrun_flow_with_retrytorest.py, fix sleep-before-check ordering inintegration.shAfter merging, enable the merge queue on the repo and add the required secrets (
APP_DEV_ASCEND_SERVICE_ACCOUNT_ID,APP_DEV_ASCEND_SERVICE_ACCOUNT_KEY,APP_DEV_ASCEND_INSTANCE_API_URL).🤖 Generated with Claude Code