Skip to content

dhi: add fips entropy source#24542

Open
craig-osterhout wants to merge 1 commit intodocker:mainfrom
craig-osterhout:engdocs-3219
Open

dhi: add fips entropy source#24542
craig-osterhout wants to merge 1 commit intodocker:mainfrom
craig-osterhout:engdocs-3219

Conversation

@craig-osterhout
Copy link
Copy Markdown
Contributor

Description

Added FIPs entropy source info

Related issues or tickets

ENGDOCS-3219

Reviews

  • Technical review
  • Editorial review

Signed-off-by: Craig Osterhout <craig.osterhout@docker.com>
@netlify
Copy link
Copy Markdown

netlify bot commented Mar 26, 2026

Deploy Preview for docsdocker ready!

Name Link
🔨 Latest commit c968aa7
🔍 Latest deploy log https://app.netlify.com/projects/docsdocker/deploys/69c5829aec170000089575f9
😎 Deploy Preview https://deploy-preview-24542--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.

@craig-osterhout
Copy link
Copy Markdown
Contributor Author

/review

Copy link
Copy Markdown
Contributor

@docker-agent docker-agent bot left a comment

Choose a reason for hiding this comment

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

Assessment: 🟢 APPROVE

This documentation addition is well-written and follows Docker's style guidelines. The new content about entropy sources for FIPS-compliant images is:

  • Factually accurate: Correctly describes the difference between Debian-based (OpenSSL entropy) and Alpine-based (kernel entropy) images
  • Concise and clear: No hedge words, meta-commentary, or unnecessary verbosity
  • Well-integrated: Maintains parallel structure with existing bullet points
  • Appropriately scoped: Fills a genuine documentation gap without expanding unnecessarily

No issues found. The change is ready to merge.

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

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants