Skip to content

docs: clarify provenance attestation defaults#16764

Merged
usha-mandya merged 4 commits into
docker:mainfrom
sozercan:docs-clarify-prov
Mar 7, 2023
Merged

docs: clarify provenance attestation defaults#16764
usha-mandya merged 4 commits into
docker:mainfrom
sozercan:docs-clarify-prov

Conversation

@sozercan
Copy link
Copy Markdown
Contributor

Proposed changes

Clarify provenance defaults
BUILDX_NO_DEFAULT_ATTESTATIONS is not cherry picked yet. docker/buildx#1612

Related issues (optional)

@netlify
Copy link
Copy Markdown

netlify Bot commented Feb 22, 2023

Deploy Preview for docsdocker ready!

Name Link
🔨 Latest commit fdf4d46
🔍 Latest deploy log https://app.netlify.com/sites/docsdocker/deploys/640257ec8fe1bf0008aac130
😎 Deploy Preview https://deploy-preview-16764--docsdocker.netlify.app
📱 Preview on mobile
Toggle QR Code...

QR Code

Use your smartphone camera to open QR code link.

To edit notification comments on pull requests, go to your Netlify site settings.

@sozercan sozercan changed the title docs: clarify prov defaults docs: clarify provenance attestation defaults Feb 22, 2023
Signed-off-by: Sertac Ozercan <sozercan@gmail.com>
@usha-mandya usha-mandya added the area/build Relates to Dockerfiles or docker build command label Feb 23, 2023
Comment thread build/attestations/index.md Outdated
Co-authored-by: David Karlsson <35727626+dvdksn@users.noreply.github.com>
dvdksn
dvdksn previously approved these changes Feb 25, 2023
@crazy-max
Copy link
Copy Markdown
Member

Before merging we should wait for next Buildx 0.10.4 release to avoid confusion.

@crazy-max
Copy link
Copy Markdown
Member

Hum now that we have a new page for env vars with build, I wonder if we should just add a link to it: https://docs.docker.com/build/building/env-vars/#buildx_no_default_attestations

@dvdksn WDYT?

sozercan added 2 commits March 3, 2023 20:23
Signed-off-by: Sertac Ozercan <sozercan@gmail.com>
@sozercan
Copy link
Copy Markdown
Contributor Author

sozercan commented Mar 3, 2023

@crazy-max @dvdksn updated to link to env vars page, lmk if this works

Copy link
Copy Markdown
Contributor

@dvdksn dvdksn left a comment

Choose a reason for hiding this comment

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

LGTM thanks @sozercan !

@usha-mandya usha-mandya enabled auto-merge March 7, 2023 14:20
@usha-mandya usha-mandya merged commit 1f1a8de into docker:main Mar 7, 2023
@sozercan sozercan deleted the docs-clarify-prov branch March 15, 2023 17:08
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

area/build Relates to Dockerfiles or docker build command

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants