Commit Graph

549 Commits

Author SHA1 Message Date
Paweł Gronowski
e4ea49e01e Merge pull request #53161 from renovate-bot/renovate/docker-actions
chore(deps): update docker github actions
2026-07-24 11:35:11 +02:00
Sebastiaan van Stijn
2df6b4baf7 Merge pull request #53160 from renovate-bot/renovate/actions-setup-go-6.x
chore(deps): update actions/setup-go action to v6.5.0
2026-07-24 01:51:10 +02:00
Mend Renovate
c9a9cbec8d chore(deps): update docker github actions
Signed-off-by: Mend Renovate <bot@renovateapp.com>
2026-07-23 23:46:24 +00:00
Mend Renovate
ce7e83073a chore(deps): update actions/setup-go action to v6.5.0
Signed-off-by: Mend Renovate <bot@renovateapp.com>
2026-07-23 22:37:42 +00:00
Mend Renovate
9604abe9fe chore(deps): update actions/labeler action to v6.2.0
Signed-off-by: Mend Renovate <bot@renovateapp.com>
2026-07-23 22:37:36 +00:00
Sebastiaan van Stijn
c8a9183e52 Merge pull request #53154 from renovate-bot/renovate/actions-checkout-7.x
chore(deps): update actions/checkout action to v7.0.1
2026-07-24 00:35:43 +02:00
Mend Renovate
53547468cd chore(deps): update actions/cache action to v5.1.0
Signed-off-by: Mend Renovate <bot@renovateapp.com>
2026-07-23 19:06:39 +00:00
Mend Renovate
bcc4d4049a chore(deps): update actions/checkout action to v7.0.1
Signed-off-by: Mend Renovate <bot@renovateapp.com>
2026-07-23 16:59:13 +00:00
Paweł Gronowski
152b399864 renovate: Group Docker GitHub Actions updates
Group actions published by the docker organization so Renovate opens
one update pull request instead of a separate pull request for each
action.

Signed-off-by: Paweł Gronowski <pawel.gronowski@docker.com>
2026-07-22 18:24:41 +02:00
Paweł Gronowski
f47a26a0e0 Merge pull request #53080 from vvoland/ci-userns
gha/test: Add userns integration mode
2026-07-17 17:19:15 +02:00
Paweł Gronowski
a5f9e7549b github: Remove obsolete code owners
Signed-off-by: Paweł Gronowski <pawel.gronowski@docker.com>
2026-07-16 20:32:22 +02:00
Paweł Gronowski
a446d73c33 gha/test: Add userns integration mode
User namespace remapping currently has no full integration-suite CI
coverage. Add a graphdriver mode that starts dockerd with
DOCKER_REMAP_ROOT=default.

Skip tests that require privileged or host namespaces, and avoid the
host network optimization in TestUpdatePidsLimit, because dockerd
rejects those combinations when remapping is enabled.

Signed-off-by: Paweł Gronowski <pawel.gronowski@docker.com>
2026-07-16 20:06:34 +02:00
Paweł Gronowski
804d0cdf27 Merge pull request #53071 from vvoland/issue-template
github: Add flaky test issue template
2026-07-16 16:33:49 +02:00
Paweł Gronowski
efa16e45ee gha: Fix pr-review channel publish URL
The agent lied to me about the previous one

Signed-off-by: Paweł Gronowski <pawel.gronowski@docker.com>
2026-07-14 16:01:18 +02:00
Paweł Gronowski
6bab67d1ad github: Add flaky test issue template
Signed-off-by: Paweł Gronowski <pawel.gronowski@docker.com>
2026-07-14 15:24:48 +02:00
Sebastiaan van Stijn
d9a0b729b3 Merge pull request #53034 from vvoland/gha-pr-review
gha: Setup AI reviews using Docker Agentic Platform
2026-07-14 14:24:26 +02:00
Sebastiaan van Stijn
3a41c21f63 Merge pull request #52318 from AkihiroSuda/ci-rootless-systemd
rootless: keep the cgroup mount for containers with --net=host; CI: add back rootless + systemd
2026-07-14 11:24:51 +02:00
Akihiro Suda
554e83f818 CI: add back ubuntu-24.04 + rootless + systemd
Signed-off-by: Akihiro Suda <akihiro.suda.cz@hco.ntt.co.jp>
2026-07-10 04:49:48 +09:00
Paweł Gronowski
1c73a37fd9 gha: Restore AI review request trigger
Publish review requests to the Docker's Agentic Platform reviewer agent.

Signed-off-by: Paweł Gronowski <pawel.gronowski@docker.com>
2026-07-09 14:31:32 +02:00
Paweł Gronowski
f8672f9897 gha/sync-release-branch: Adjust dry_run description
Signed-off-by: Paweł Gronowski <pawel.gronowski@docker.com>
2026-07-09 13:09:38 +02:00
Paweł Gronowski
6ca7e8f1fc gha/sync-release-branch: Add -r to xargs to handle empty tag list
Without -r, xargs invokes sync-branch with no arguments when the tags
file is empty (all tags already merged), causing sync-branch to exit
with a usage error.

Signed-off-by: Paweł Gronowski <pawel.gronowski@docker.com>
2026-07-09 13:09:38 +02:00
Paweł Gronowski
0abcfdca46 gha/sync-release-branch: Configure git author regardless of dry-run
sync-branch calls git merge --continue, which creates a commit and
therefore requires a git author identity even in dry-run mode.

Skipping the "Configure git author" step on dry runs caused those runs
to fail with a git identity error before any merges were attempted.

Signed-off-by: Paweł Gronowski <pawel.gronowski@docker.com>
2026-07-09 13:09:38 +02:00
Paweł Gronowski
4025d46230 gha/sync-release-branch: Fix master brach comparison
The guard compared the lowercase shell variable $branch (undefined,
expands to empty string) against "master " with a trailing space, so
the branch protection never fired.

Signed-off-by: Paweł Gronowski <pawel.gronowski@docker.com>
2026-07-09 13:09:37 +02:00
Sebastiaan van Stijn
44c855a73e update to go1.26.5
go1.26.5 (released 2026-07-07) includes security fixes to the crypto/tls
and os packages, as well as bug fixes to the compiler, the runtime, the
go command, and the net, os, and syscall packages. See the Go 1.26.5
milestone on our issue tracker for details;

- https://github.com/golang/go/issues?q=milestone%3AGo1.26.5+label%3ACherryPickApproved
- full diff: https://github.com/golang/go/compare/go1.26.4...go1.26.5

From the security announcement:

We have just released Go versions 1.26.5 and 1.25.12, minor point releases.

These releases include 2 security fixes following the security policy:

- os: Root escape via symlink plus trailing slash

  On Unix systems, opening a file in an os.Root improperly
  followed symlinks to locations outside of the Root when
  the final path component of the a path is a symbolic link
  and the path ends in /.

  For example, root.Open("symlink/") would open "symlink"
  even when "symlink" is a symbolic link pointing outside of the root.

  On Unix, openat(fd, path, O_NOFOLLOW) will follow symlinks
  in path when path ends in a /. Root failed to account for
  this behavior, permitting paths with a trailing / to escape.
  It now properly sanitizes the path parameter provided to openat.

  hanks to Mundur for reporting this issue.

  This is CVE-2026-39822 and Go issue https://go.dev/issue/79005.

- crypto/tls: Encrypted Client Hello privacy leak

  he Encrypted Client Hello implementation would leak the pre-shared key
  dentities during the handshake, allowing a passive network observer who can
  ollect handshakes to de-anonymize the hostname of the server, even when ECH was
  eing used.

  Thanks to Coia Prant (github.com/rbqvq) for reporting this issue.

  This is CVE-2026-42505 and Go issue https://go.dev/issue/79282.

Signed-off-by: Sebastiaan van Stijn <github@gone.nl>
2026-07-07 23:59:17 +02:00
Paweł Gronowski
2f5f9d08df Merge pull request #52995 from vvoland/gha-flaky-command
test-integration-flaky: Add /flaky-check=Test1,Test2 PR body directive
2026-07-06 11:33:36 +02:00
Paweł Gronowski
64ea51f2fb test-integration-flaky: Add /flaky-check=Test1,Test2 PR body directive
Allows PR authors to stress-test specific integration tests for
flakiness even when the diff doesn't touch them.

Add a /flaky-check directive on its own line in the PR body:

/flaky-check=TestFoo,TestBar

The GHA integration-flaky job parses the directive from the PR body and
exports the names as FLAKY_EXTRA_TESTS, which the script appends to the
diff-detected set before running the stress loop.

Signed-off-by: Paweł Gronowski <pawel.gronowski@docker.com>
2026-07-03 22:42:12 +02:00
Paweł Gronowski
c6b46e0a3c Revert "gha: Add Docker Agent PR reviewer workflow"
This reverts commit ac82b186fe.

Signed-off-by: Paweł Gronowski <pawel.gronowski@docker.com>
2026-07-03 19:58:01 +02:00
Paweł Gronowski
9fda6bda54 Merge pull request #52727 from vvoland/gha-review
Add Docker Agent PR reviewer workflow
2026-07-03 16:48:24 +02:00
Sebastiaan van Stijn
31bac2b89c gha: remove custom paths and working-directory
Signed-off-by: Sebastiaan van Stijn <github@gone.nl>
2026-07-02 15:38:23 +02:00
Sebastiaan van Stijn
ffd43fa27a use /usr/src/moby instead, and no GOPATH
Signed-off-by: Sebastiaan van Stijn <github@gone.nl>
2026-07-02 15:13:19 +02:00
Sebastiaan van Stijn
4b7a7687a5 update (GOPATH) paths to use github.com/moby/moby/v2
Signed-off-by: Sebastiaan van Stijn <github@gone.nl>
2026-07-02 15:13:16 +02:00
Paweł Gronowski
71fd445284 Merge pull request #52966 from vvoland/work-gha-releases
gha: Add release branch sync workflow
2026-07-01 17:50:52 +02:00
Paweł Gronowski
ac82b186fe gha: Add Docker Agent PR reviewer workflow
Signed-off-by: Paweł Gronowski <pawel.gronowski@docker.com>
2026-07-01 13:37:24 +02:00
Paweł Gronowski
f10a2687af Merge pull request #52962 from vvoland/fix-vm-docker
gha/vm: Reduce Lima guest size for non-Moby org
2026-06-30 16:55:58 +02:00
Paweł Gronowski
178195da33 gha/vm: Utilize full guest RAM
In Docker org, QEMU exits before SSH is available on GitHub-hosted
runners because the VM job asks KVM for 12 GiB of guest memory:

```
qemu-system-x86_64: cannot set up guest memory 'pc.ram': Cannot
allocate memory
```

Set the Lima guest size to the runner's detected memory minus 2 GiB.

This keeps headroom for the host while avoiding fixed assumptions about
which organization or runner size is executing the workflow.

Signed-off-by: Paweł Gronowski <pawel.gronowski@docker.com>
2026-06-30 14:23:20 +02:00
Sebastiaan van Stijn
258ee34b76 Merge pull request #52968 from vvoland/gha-test-validate-cache
gha/test/validate: Build dev image on cache miss
2026-06-26 17:05:36 +02:00
Paweł Gronowski
93360069c4 gha/test/validate: Build dev image on cache miss
Sometimes we get rate limited for cache access.

Drop fail-on-cache-miss and fall back to building the dev image from the
GHA buildkit cache (scope=dev-amd64) when the restore doesn't hit, the
same way the test and test-unit jobs already build it.

The happy path (cache hit + docker load) is unchanged.

Signed-off-by: Paweł Gronowski <pawel.gronowski@docker.com>
2026-06-26 14:48:01 +02:00
Paweł Gronowski
335a604e99 gha: Add release branch sync workflow
Add a manually dispatched workflow for maintainers to sync a docker
release branch to a selected docker release tag.

The job checks out the requested release branch, merges the tag with
`git merge --no-ff`, checks the worktree content out from that tag,
stages the result, and pushes the updated branch.

The actual push is guarded by the `docker-releases` GitHub environment
which requires a manual approval.

Signed-off-by: Paweł Gronowski <pawel.gronowski@docker.com>
2026-06-26 12:50:17 +02:00
Paweł Gronowski
78c2aa4fcc gha/dco: Pass origin branch
The DCO job runs inside an Alpine container and fetches the base
branch from VALIDATE_REPO unless VALIDATE_ORIGIN_BRANCH is set.

In forked runs that repository can point at a private upstream URL, so
the unauthenticated fetch fails with:

```
fatal: could not read Username for 'https://github.com': No such
device or address
```

Pass the pull request base SHA so the validator can resolve the
comparison point locally.

Signed-off-by: Paweł Gronowski <pawel.gronowski@docker.com>
2026-06-25 17:17:19 +02:00
Sebastiaan van Stijn
60966d7495 gha/test: skip pr-gh-references validation
This validation is already run elsewhere, but wasn't excluded in this workflow,
causing failures on non-default branches (because VALIDATE_BRANCH isn't set,
and won't be set when running from a "push" or "workflow_dispatch").

Signed-off-by: Sebastiaan van Stijn <github@gone.nl>
2026-06-24 16:45:35 +02:00
Sebastiaan van Stijn
399066c7a3 gha/validate-milestone: add concurrency check
Signed-off-by: Sebastiaan van Stijn <github@gone.nl>
2026-06-24 10:10:00 +02:00
Sebastiaan van Stijn
98c82b2ba9 gha/validate-check: don't run post-merge
I noticed that this workflow was running both before, and after merging a
pull request. When running after it's merged, it gets confused and the check
will be failing (post merge);

    Modified files: .github/workflows/.dco.yml, .github/workflows/.test-unit.yml, .github/workflows/.test.yml, .github/workflows/.vm.yml, .github/workflows/.windows.yml, .github/workflows/buildkit.yml, .github/workflows/ci.yml, .github/workflows/codeql.yml, .github/workflows/test.yml, .github/workflows/validate-pr.yml
    Touches version: false
    Base ref: master
    Error: PR must have a milestone set (expected: 29.7.0)

Add an extra condition so that the check is only triggered while the PR
is open, not after it's closed or merged. Also add `reopened` as event,
to make sure it's run if someone would close/reopen the PR.

Signed-off-by: Sebastiaan van Stijn <github@gone.nl>
2026-06-24 10:00:32 +02:00
Mend Renovate
7fd309807a Update actions/checkout action to v7
Signed-off-by: Mend Renovate <bot@renovateapp.com>
2026-06-19 09:25:29 +00:00
Sebastiaan van Stijn
9bf5235eb2 Merge pull request #52869 from renovate-bot/renovate/docker-github-builder-1.x
chore(deps): update docker/github-builder action to v1.12.0
2026-06-15 15:44:14 +02:00
Mend Renovate
459ae6439c chore(deps): update crazy-max/.github action to v1.10.1
Signed-off-by: Mend Renovate <bot@renovateapp.com>
2026-06-15 09:58:35 +00:00
Paweł Gronowski
8cd9baf14f gha/labeler: Label libnetwork changes as networking
Signed-off-by: Paweł Gronowski <pawel.gronowski@docker.com>
2026-06-15 10:47:52 +02:00
Mend Renovate
e49df7250f chore(deps): update docker/github-builder action to v1.12.0
Signed-off-by: Mend Renovate <bot@renovateapp.com>
2026-06-14 16:37:11 +00:00
Takumi Akasaka
7e8d2d1c13 gha: Set timeout-minutes on build-dind and labeler jobs
These were the only two jobs running directly via `runs-on:` that did
not have a `timeout-minutes` guardrail. Add 120 minutes as a starting
point to prevent runaway jobs, matching the existing convention used
elsewhere in the workflows. Tuning each job's timeout to its usual
runtime is left as a follow-up, as suggested in the issue.

Signed-off-by: Takumi Akasaka <takumiakasaka1231@gmail.com>
Signed-off-by: Sebastiaan van Stijn <github@gone.nl>
2026-06-12 16:10:56 +02:00
Paweł Gronowski
17ccff865e gha: Avoid cancelling non-PR CI runs
The concurrency groups currently cancel older runs for push, tag,
scheduled, and manually dispatched events.

On maintained refs this canhide a regression when a later run starts
before the earlier validation finishes.

Keep cancellation for stale pull request runs only, while allowing
non-PR validation to complete.

Signed-off-by: Paweł Gronowski <pawel.gronowski@docker.com>
2026-06-12 14:57:03 +02:00
Paweł Gronowski
3407385a86 Revert "gha/ci: Store slim test-results-* artifacts"
This reverts commit 79f5ddc271.

Signed-off-by: Paweł Gronowski <pawel.gronowski@docker.com>
2026-06-12 12:23:55 +02:00