Skip to content

Comments

Upgrade infra-e2e to fix Docker 29 containerd compatibility#13708

Merged
wu-sheng merged 2 commits intomasterfrom
fix/upgrade-infra-e2e-docker29
Feb 21, 2026
Merged

Upgrade infra-e2e to fix Docker 29 containerd compatibility#13708
wu-sheng merged 2 commits intomasterfrom
fix/upgrade-infra-e2e-docker29

Conversation

@wu-sheng
Copy link
Member

@wu-sheng wu-sheng commented Feb 21, 2026

Fix E2E test failures caused by Docker 29 on GitHub Actions runners

  • Explain briefly why the bug exists and how to fix it.

GitHub Actions runners upgraded to Docker 29, which:

  1. Enables containerd image store by default. containerd v2.1.5 lowered the default file descriptor limit from 1,048,576 to 1,024, causing applications like Elasticsearch to crash at startup.
  2. Raised the minimum Docker API version to 1.44, while the current infra-e2e uses API v1.41.

The fix is in skywalking-infra-e2e 8c21e43, which disables the containerd image store and negotiates the Docker API version internally before running docker compose.

This PR:

  1. Reverts the previous workaround (88ca648) that added containerd disable steps directly in the workflow.
  2. Bumps the infra-e2e commit to 8c21e43 which handles the fix internally.

wu-sheng and others added 2 commits February 21, 2026 17:37
Upgrade skywalking-infra-e2e to 8c21e43 which disables containerd
image store and negotiates Docker API version internally, fixing
E2E test failures on GitHub Actions runners with Docker 29.

Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
@wu-sheng wu-sheng requested a review from kezhenxu94 February 21, 2026 10:07
@wu-sheng wu-sheng added this to the Agent Plugin Test Tool milestone Feb 21, 2026
@wu-sheng wu-sheng added the test Test requirements about performance, feature or before release. label Feb 21, 2026
@wu-sheng wu-sheng merged commit b39805b into master Feb 21, 2026
178 checks passed
@wu-sheng wu-sheng deleted the fix/upgrade-infra-e2e-docker29 branch February 21, 2026 10:24
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

test Test requirements about performance, feature or before release.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants