docker / cli

The Docker CLI
Apache License 2.0
4.87k stars 1.91k forks source link

CI: remove "experimental" checks, as they may be redundant #5140

Open thaJeztah opened 3 months ago

thaJeztah commented 3 months ago

Description

While working on https://github.com/docker/cli/pull/5139, I noticed that CI was green even if we didn't enable experimental on the daemon.

I was curious how the experimental and non-experimental e2e tests differed, and after a cursory glance, it may be only for a single test; https://github.com/docker/cli/blob/591bd174241555b26fb5e92de5d7a9da094523c1/e2e/image/build_test.go#L128

If that's correct, then it would mean we run all e2e tests twice, but only for a single test; we can cut the number of checks in half if we don't do so;