Skip to content

Conversation

@crazy-max
Copy link
Member

@crazy-max crazy-max commented Nov 3, 2025

closes #3477

@crazy-max crazy-max marked this pull request as ready for review November 13, 2025 14:04
Copy link
Member

@thaJeztah thaJeztah left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM, but left a suggestion

- worker: "docker@27.5"
pkg: ./tests
- worker: "docker+containerd@26.1" # same as docker, but with containerd snapshotter
- worker: "docker+containerd@27.5" # same as docker, but with containerd snapshotter
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Wondering if we should include a 25.0 somewhere; 26.x and 27.x are EOL, but 25.0 is maintained by mirantis and AWS

Copy link
Member Author

@crazy-max crazy-max Nov 13, 2025

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

We chose to keep n-2 for our integration tests to check for regressions with past stables like we do for BuildKit:

- v0.26.0
- v0.25.2
- v0.24.0

But also to check for issues with default Docker version used on GitHub Hosted runners: https://github.com/crazy-max/ghaction-dump-context/actions/runs/19068023635/job/54463406479#step:3:638

Maybe we could add 25 if you think this can be useful.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants