diff --git a/.github/workflows/ci.yml b/.github/workflows/ci.yml index db67084..64c2d39 100644 --- a/.github/workflows/ci.yml +++ b/.github/workflows/ci.yml @@ -1464,35 +1464,6 @@ jobs: env: DOCKER_BUILD_RECORD_RETENTION_DAYS: ${{ matrix.days }} - export-legacy: - runs-on: ubuntu-latest - strategy: - fail-fast: false - matrix: - legacy: - - false - - true - steps: - - - name: Checkout - uses: actions/checkout@v6 - with: - path: action - - - name: Set up Docker Buildx - uses: docker/setup-buildx-action@v3 - with: - version: ${{ inputs.buildx-version || env.BUILDX_VERSION }} - driver-opts: | - image=${{ inputs.buildkit-image || env.BUILDKIT_IMAGE }} - - - name: Build - uses: ./action - with: - file: ./test/Dockerfile - env: - DOCKER_BUILD_EXPORT_LEGACY: ${{ matrix.legacy }} - checks: runs-on: ubuntu-latest strategy: diff --git a/README.md b/README.md index 364eb58..b50e801 100644 --- a/README.md +++ b/README.md @@ -277,7 +277,6 @@ The following outputs are available: | `DOCKER_BUILD_SUMMARY` | Bool | `true` | If `false`, [build summary](https://docs.docker.com/build/ci/github-actions/build-summary/) generation is disabled | | `DOCKER_BUILD_RECORD_UPLOAD` | Bool | `true` | If `false`, build record upload as [GitHub artifact](https://docs.github.com/en/actions/using-workflows/storing-workflow-data-as-artifacts) is disabled | | `DOCKER_BUILD_RECORD_RETENTION_DAYS` | Number | | Duration after which build record artifact will expire in days. Defaults to repository/org [retention settings](https://docs.github.com/en/actions/learn-github-actions/usage-limits-billing-and-administration#artifact-and-log-retention-policy) if unset or `0` | -| `DOCKER_BUILD_EXPORT_LEGACY` | Bool | `false` | If `true`, exports build using legacy export-build tool instead of [`buildx history export` command](https://docs.docker.com/reference/cli/docker/buildx/history/export/) | ## Troubleshooting diff --git a/src/main.ts b/src/main.ts index 9946ca0..2e3610c 100644 --- a/src/main.ts +++ b/src/main.ts @@ -175,8 +175,8 @@ actionsToolkit.run( core.info(`Build summary skipped for ${inputs.call} subrequest`); } else if (GitHub.isGHES) { core.info('Build summary is not yet supported on GHES'); - } else if (!(await toolkit.buildx.versionSatisfies('>=0.13.0'))) { - core.info('Build summary requires Buildx >= 0.13.0'); + } else if (!(await toolkit.buildx.versionSatisfies('>=0.23.0'))) { + core.info('Build summary requires Buildx >= 0.23.0'); } else if (!ref) { core.info('Build summary requires a build reference'); } else { @@ -202,8 +202,7 @@ actionsToolkit.run( const buildxHistory = new BuildxHistory(); const exportRes = await buildxHistory.export({ - refs: stateHelper.buildRef ? [stateHelper.buildRef] : [], - useContainer: buildExportLegacy() + refs: stateHelper.buildRef ? [stateHelper.buildRef] : [] }); core.info(`Build record written to ${exportRes.dockerbuildFilename} (${Util.formatFileSize(exportRes.dockerbuildSize)})`); @@ -298,10 +297,3 @@ function buildRecordRetentionDays(): number | undefined { return res; } } - -function buildExportLegacy(): boolean { - if (process.env.DOCKER_BUILD_EXPORT_LEGACY) { - return Util.parseBool(process.env.DOCKER_BUILD_EXPORT_LEGACY); - } - return false; -}