Skip to content

Conversation

@crazy-max
Copy link
Member

Related issues or tickets

Reviews

  • Technical review
  • Editorial review
  • Product review

@netlify
Copy link

netlify bot commented Jul 10, 2025

Deploy Preview for docsdocker ready!

Name Link
🔨 Latest commit c2a12e2
🔍 Latest deploy log https://app.netlify.com/projects/docsdocker/deploys/686fb78b0a91970008202104
😎 Deploy Preview https://deploy-preview-23059--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 project configuration.

@github-actions github-actions bot added area/cli Relates to the CLI client dependencies Vendoring, packaging updates labels Jul 10, 2025
@crazy-max
Copy link
Member Author

@thaJeztah @vvoland Seems cli reference docs is broken: https://github.com/docker/docs/actions/runs/16195638531/job/45721313558?pr=23059#step:3:481

#26 [test 5/5] RUN htmltest
#26 0.057 htmltest started at 12:53:30 on public
#26 0.057 ========================================================================
#26 0.788 reference/cli/docker/container/run/index.html
#26 0.788   hash does not exist --- reference/cli/docker/container/run/index.html --> /reference/cli/dockerd/#enable-cdi-devices
#26 0.788 ========================================================================
#26 5.164 ✘✘✘ failed in 5.102860805s
#26 5.164 1 errors in 3177 documents
#26 5.164 
#26 ERROR: process "/bin/sh -c htmltest" did not complete successfully: exit code: 1

@thaJeztah
Copy link
Member

Yes, that was why we temporarily dropped the CLI update; an anchor changed; see #23042 (comment)

Copy link
Member

@thaJeztah thaJeztah left a comment

Choose a reason for hiding this comment

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

leaving a "request changes" review because we need to fix up the link in docker/cli (or add some temporary exception)

@crazy-max
Copy link
Member Author

Yes, that was why we temporarily dropped the CLI update; an anchor changed; see #23042 (comment)

Ah my bad closing this one then

@crazy-max crazy-max closed this Jul 11, 2025
@crazy-max crazy-max deleted the fix-docker-vendoring branch July 11, 2025 08:09
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

area/cli Relates to the CLI client dependencies Vendoring, packaging updates

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants