Skip to content

tests: replace aaa/bbb node names with descriptive builder names#3470

Merged
tonistiigi merged 1 commit intodocker:masterfrom
yzewei:tests_fix
Feb 23, 2026
Merged

tests: replace aaa/bbb node names with descriptive builder names#3470
tonistiigi merged 1 commit intodocker:masterfrom
yzewei:tests_fix

Conversation

@yzewei
Copy link
Contributor

@yzewei yzewei commented Oct 15, 2025

The current buildx driver tests use generic placeholder node names like "aaa", "bbb", and "ccc".
This makes the tests less readable and harder to understand for new contributors.

This patch updates all node names in driver_test.go to semantic, descriptive names
reflecting their platform, e.g., "builder-amd64", "builder-riscv64", "builder-armv8", etc.

No test logic is changed; the patch only improves readability and maintainability
of the tests.

@tonistiigi
Copy link
Member

PTAL the ci

@yzewei
Copy link
Contributor Author

yzewei commented Oct 24, 2025

fix this format problem

@tonistiigi
Copy link
Member

@yzewei sorry, needs rebase.

Signed-off-by: yzewei <yangzewei@loongson.cn>
@crazy-max
Copy link
Member

Rebased myself

@crazy-max crazy-max added this to the v0.32.0 milestone Feb 10, 2026
@tonistiigi tonistiigi merged commit 5683e9f into docker:master Feb 23, 2026
159 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants