Skip to content

Conversation

@Xeratec
Copy link
Member

@Xeratec Xeratec commented Jul 9, 2025

This PR improves the Docker image publishing workflow by switching from per-architecture tags to digest-based references when assembling multi-architecture manifests. This simplifies the tagging strategy and avoids excessive tag proliferation in the container registry. It also includes minor CI fixes and version bumps.

Added

  • Output Docker image digests per platform (amd64, arm64) after build, which is used to construct the multi-arch Docker manifest. This prevents registry clutter caused by unnecessary per-architecture Docker tags.

Changed

  • Replaced platform-specific tags (*-amd64, *-arm64) with direct digest references in Noelware/docker-manifest-action.

Fixed

  • Resolved issue with missing id in the Build Cache for Docker step, used in the Inject build-cache step.

PR Merge Checklist

  1. The PR is rebased on the latest devel commit and points to devel.
  2. Your PR was reviewed and approved.
  3. All checks are passing.
  4. The CHANGELOG.md file has been updated.
  5. If the docker was modified, change back its link after review.

@Xeratec Xeratec added this to the Release 0.2.1 milestone Jul 9, 2025
@Xeratec Xeratec self-assigned this Jul 9, 2025
@Xeratec Xeratec added the Feature Addition of new features label Jul 9, 2025
@Xeratec Xeratec added this to Deeploy Jul 9, 2025
Xeratec added 3 commits July 9, 2025 17:41
Replaces per-architecture tags (e.g., *-amd64, *-arm64) with direct digest references
when creating the multi-arch Docker manifest. This reduces tag clutter in the registry.
@Xeratec Xeratec force-pushed the pr/improve_docker_build branch from 124b064 to a84f6e5 Compare July 9, 2025 15:42
Copy link
Contributor

@lukamac lukamac left a comment

Choose a reason for hiding this comment

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

Merge approved.

This is not part of this PR, but just for my curiosity, why are we using the ubuntu-22.04-arm runner and not the ubuntu-24.04-arm one?

@Xeratec
Copy link
Member Author

Xeratec commented Jul 28, 2025

Merge approved.

This is not part of this PR, but just for my curiosity, why are we using the ubuntu-22.04-arm runner and not the ubuntu-24.04-arm one?

No specific reason, it is just the one I used from the beginning, and I did not test the latest one. It is expected to reach the end of life Linux after April 2027, which should be fine for now. I suggest we update if we face an issue, but it is worth even the 10 min to test this now.

@Xeratec Xeratec merged commit 9ef3217 into pulp-platform:devel Jul 30, 2025
118 checks passed
@Xeratec Xeratec deleted the pr/improve_docker_build branch July 30, 2025 13:22
@github-project-automation github-project-automation bot moved this to Done in Deeploy Jul 30, 2025
diaconuccalin pushed a commit to Aldrago98/Deeploy that referenced this pull request Oct 17, 2025
* Update action versions and fix missing id

* Avoid platform-specific image tags by using digests in Docker manifest

* Replaces per-architecture tags (e.g., *-amd64, *-arm64) with direct digest references
when creating the multi-arch Docker manifest. This reduces tag clutter in the registry.

* Update Changelog
diaconuccalin pushed a commit to diaconuccalin/Deeploy that referenced this pull request Oct 27, 2025
* Update action versions and fix missing id

* Avoid platform-specific image tags by using digests in Docker manifest

* Replaces per-architecture tags (e.g., *-amd64, *-arm64) with direct digest references
when creating the multi-arch Docker manifest. This reduces tag clutter in the registry.

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

Labels

Feature Addition of new features

Projects

Status: Done

Development

Successfully merging this pull request may close these issues.

2 participants