Skip to content

test: honor AgentTeams worker runtime env#1028

Open
RerankerGuo wants to merge 1 commit into
agentscope-ai:mainfrom
RerankerGuo:fix/test-worker-runtime-env
Open

test: honor AgentTeams worker runtime env#1028
RerankerGuo wants to merge 1 commit into
agentscope-ai:mainfrom
RerankerGuo:fix/test-worker-runtime-env

Conversation

@RerankerGuo

Copy link
Copy Markdown
Contributor

Summary

  • Make tests 02, 06, and 15 prefer AGENTTEAMS_DEFAULT_WORKER_RUNTIME.
  • Preserve the legacy variable as a fallback for direct local test invocations.
  • Keep openclaw as the final default.

Root cause

The integration matrix now exports AGENTTEAMS_DEFAULT_WORKER_RUNTIME, but these three worker-creating tests still read only HICLAW_DEFAULT_WORKER_RUNTIME. Copaw and Hermes matrix jobs therefore explicitly created OpenClaw workers, so their reported runtime coverage did not match the configured matrix dimension.

Verification

  • bash -n passes for all three scripts.
  • Shell assertions cover new-variable precedence, legacy fallback, and the default value in every file.
  • git diff --check

This is test-only and does not change runtime behavior.

@github-actions

Copy link
Copy Markdown
Contributor

❌ Integration Tests Failed (controller-cr-2 / mgr=copaw / wk=copaw)

Commit: f2cafd7
Workflow run: #1656

Test Results
No test output captured.
Debug Log (tail)
No debug logs available.

📦 Download full debug logs & test artifacts

@github-actions

Copy link
Copy Markdown
Contributor

❌ Integration Tests Failed (llm-interaction / mgr=copaw / wk=copaw)

Commit: f2cafd7
Workflow run: #1656

Test Results
No test output captured.
Debug Log (tail)
No debug logs available.

📦 Download full debug logs & test artifacts

@github-actions

Copy link
Copy Markdown
Contributor

❌ Integration Tests Failed (llm-interaction-2 / mgr=openclaw / wk=openclaw)

Commit: f2cafd7
Workflow run: #1656

Test Results
No test output captured.
Debug Log (tail)
No debug logs available.

📦 Download full debug logs & test artifacts

@github-actions

Copy link
Copy Markdown
Contributor

❌ Integration Tests Failed (llm-interaction-2 / mgr=copaw / wk=hermes)

Commit: f2cafd7
Workflow run: #1656

Test Results
No test output captured.
Debug Log (tail)
No debug logs available.

📦 Download full debug logs & test artifacts

@github-actions

Copy link
Copy Markdown
Contributor

❌ Integration Tests Failed (llm-interaction / mgr=copaw / wk=hermes)

Commit: f2cafd7
Workflow run: #1656

Test Results
No test output captured.
Debug Log (tail)
No debug logs available.

📦 Download full debug logs & test artifacts

@github-actions

Copy link
Copy Markdown
Contributor

❌ Integration Tests Failed (controller-cr / mgr=copaw / wk=hermes)

Commit: f2cafd7
Workflow run: #1656

Test Results
No test output captured.
Debug Log (tail)
No debug logs available.

📦 Download full debug logs & test artifacts

@github-actions

Copy link
Copy Markdown
Contributor

❌ Integration Tests Failed (llm-interaction-2 / mgr=copaw / wk=copaw)

Commit: f2cafd7
Workflow run: #1656

Test Results
No test output captured.
Debug Log (tail)
No debug logs available.

📦 Download full debug logs & test artifacts

@github-actions

Copy link
Copy Markdown
Contributor

❌ Integration Tests Failed (controller-cr / mgr=copaw / wk=copaw)

Commit: f2cafd7
Workflow run: #1656

Test Results
No test output captured.
Debug Log (tail)
No debug logs available.

📦 Download full debug logs & test artifacts

@github-actions

Copy link
Copy Markdown
Contributor

❌ Integration Tests Failed (controller-cr / mgr=openclaw / wk=openclaw)

Commit: f2cafd7
Workflow run: #1656

Test Results
No test output captured.
Debug Log (tail)
No debug logs available.

📦 Download full debug logs & test artifacts

@github-actions

Copy link
Copy Markdown
Contributor

❌ Integration Tests Failed (llm-interaction / mgr=openclaw / wk=openclaw)

Commit: f2cafd7
Workflow run: #1656

Test Results
No test output captured.
Debug Log (tail)
No debug logs available.

📦 Download full debug logs & test artifacts

@RerankerGuo

Copy link
Copy Markdown
Contributor Author

CI note: all 10 integration failures on run #1656 are blocked by the base workflow secret-name regression tracked and fixed in #1026. The base workflow exits during embedded install before the changes in these test scripts execute. This PR has 9 passing build and translation checks; local shell validation for the three runtime assignments is documented in the PR body.

@RerankerGuo

Copy link
Copy Markdown
Contributor Author

Fresh independent evidence after the repository secret recovered: draft #1041's base-branch run 29489698196 exported AGENTTEAMS_DEFAULT_WORKER_RUNTIME=hermes, but unpatched test-06 logged Worker Bob runtime matches test matrix (got: 'openclaw', want: 'openclaw').

That confirms the legacy-only lookup masks the Hermes matrix dimension exactly as described here. #1028 makes tests 02, 06, and 15 prefer the canonical runtime variable while retaining the legacy fallback. The current red checks on this PR are the older pre-test secret failure; no patch change is needed.

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