build: Buildx OCI artifact opt-out - #25713
Conversation
Document BUILDX_NO_DEFAULT_OCI_ARTIFACT as a compatibility escape hatch for registries that reject OCI artifact attestations. Add the corresponding Buildx version requirement to the summary bar metadata. Signed-off-by: CrazyMax <1951866+crazy-max@users.noreply.github.com>
✅ Deploy Preview for docsdocker ready!
To edit notification comments on pull requests, go to your Netlify project configuration. |
github-actions
Bot
added
the
area/build
docker-agent
left a comment
docker-agent
left a comment
There was a problem hiding this comment.
Assessment: 🟢 APPROVE
The PR adds documentation for the new BUILDX_NO_DEFAULT_OCI_ARTIFACT environment variable and a corresponding summary bar entry. The writing is direct and factual, there are no style violations (no hedge words, no AI-isms, no passive voice), markdown formatting is correct, and the Hugo shortcode is used properly. No issues found.
docker-agent
left a comment
docker-agent
left a comment
There was a problem hiding this comment.
Assessment: 🟢 APPROVE
No issues found. The documentation is clear, accurate, and follows the Docker style guide. The new variable is well-documented as a compatibility escape hatch, and the corresponding update adds the correct Buildx version requirement.
docker-agent
left a comment
docker-agent
left a comment
There was a problem hiding this comment.
Assessment: 🟢 APPROVE
No issues found. The new BUILDX_NO_DEFAULT_OCI_ARTIFACT documentation is well-scoped and follows the established patterns for similar variables in this file. The data/summary.yaml entry correctly matches the feature_name used in the shortcode. No style guide violations detected in the added lines.
docker-agent
left a comment
docker-agent
left a comment
There was a problem hiding this comment.
Assessment: 🟢 APPROVE
The changes are concise, accurate, and follow the documentation style guide. The new BUILDX_NO_DEFAULT_OCI_ARTIFACT section is well-structured, uses correct code formatting, avoids hedge words, and matches the existing section style. The data/summary.yaml entry is correctly added.
docker-agent
left a comment
docker-agent
left a comment
There was a problem hiding this comment.
Assessment: 🟡 NEEDS ATTENTION
3 participants
Description
Document BUILDX_NO_DEFAULT_OCI_ARTIFACT as a compatibility escape hatch for registries that reject OCI artifact attestations.
Add the corresponding Buildx version requirement to the summary bar metadata.
Related issues or tickets
Reviews