docs(DOC-1834, DOC-1841): FIPS Docker image, 140-3 update, and style fixes#1630
docs(DOC-1834, DOC-1841): FIPS Docker image, 140-3 update, and style fixes#1630micheleRP merged 3 commits intov-WIP/26.1from
Conversation
✅ Deploy Preview for redpanda-docs-preview ready!
To edit notification comments on pull requests, go to your Netlify project configuration. |
📝 WalkthroughWalkthroughThe documentation page for FIPS compliance was updated to standardize terminology, expand deployment guidance, and restructure instructional content. The page metadata was updated with learning objectives, and terminology was refined to consistently use "FIPS-compliant" language. Docker deployment prerequisites were added, including guidance on using versioned FIPS images and Linux host requirements. The limitations section was adjusted to reflect host-level requirements for FIPS mode. A new "Configure FIPS mode with Docker" section replaced the "Suggested reading" section with concrete configuration examples, while existing reference links in the "Next steps" section were retained. Estimated code review effort🎯 2 (Simple) | ⏱️ ~10 minutes Suggested reviewers
🚥 Pre-merge checks | ✅ 3✅ Passed checks (3 passed)
✏️ Tip: You can configure your own custom pre-merge checks in the settings. ✨ Finishing Touches📝 Generate docstrings
🧪 Generate unit tests (beta)
Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out. Comment |
There was a problem hiding this comment.
🧹 Nitpick comments (1)
modules/manage/pages/security/fips-compliance.adoc (1)
32-32: Use auto-title xrefs instead of hard-coded link text.Prefer
xref:...[]here for consistency with repo AsciiDoc linking conventions.Suggested doc-only refactor
-Before configuring brokers to run in FIPS mode on Linux, install the `redpanda-rpk-fips` and `redpanda-fips` xref:deploy:deployment-option/self-hosted/manual/production/production-deployment.adoc#install-redpanda-for-fips-compliance[packages]. +Before configuring brokers to run in FIPS mode on Linux, install the `redpanda-rpk-fips` and `redpanda-fips` xref:deploy:deployment-option/self-hosted/manual/production/production-deployment.adoc#install-redpanda-for-fips-compliance[]. -- Redpanda does not support PKCS#12 keys for xref:manage:security/encryption.adoc[TLS encryption] when FIPS mode is enabled. The PKCS12KDF algorithm used in PKCS#12 is not FIPS-compliant. To use Redpanda in FIPS mode with TLS enabled, configure your certificates and keys in PEM format instead. +- Redpanda does not support PKCS#12 keys for xref:manage:security/encryption.adoc[] when FIPS mode is enabled. The PKCS12KDF algorithm used in PKCS#12 is not FIPS-compliant. To use Redpanda in FIPS mode with TLS enabled, configure your certificates and keys in PEM format instead.Based on learnings: AsciiDoc linking should prefer
xref:...[]so link titles are pulled from target docs automatically.Also applies to: 40-40
🤖 Prompt for AI Agents
Verify each finding against the current code and only fix it if needed. In `@modules/manage/pages/security/fips-compliance.adoc` at line 32, Replace the hard-coded link text in the sentence beginning "Before configuring brokers to run in FIPS mode on Linux, install the `redpanda-rpk-fips` and `redpanda-fips` ..." with an auto-title AsciiDoc cross-reference by using xref:deploy:deployment-option/self-hosted/manual/production/production-deployment.adoc#install-redpanda-for-fips-compliance[] instead of the explicit link text; apply the same change to the other occurrence mentioned (lines 40-40) so both links use xref:...[] and pull titles automatically.
🤖 Prompt for all review comments with AI agents
Verify each finding against the current code and only fix it if needed.
Nitpick comments:
In `@modules/manage/pages/security/fips-compliance.adoc`:
- Line 32: Replace the hard-coded link text in the sentence beginning "Before
configuring brokers to run in FIPS mode on Linux, install the
`redpanda-rpk-fips` and `redpanda-fips` ..." with an auto-title AsciiDoc
cross-reference by using
xref:deploy:deployment-option/self-hosted/manual/production/production-deployment.adoc#install-redpanda-for-fips-compliance[]
instead of the explicit link text; apply the same change to the other occurrence
mentioned (lines 40-40) so both links use xref:...[] and pull titles
automatically.
ℹ️ Review info
⚙️ Run configuration
Configuration used: Organization UI
Review profile: CHILL
Plan: Pro
Run ID: b3e187ec-6311-465d-b3a1-c26bd1b75e0c
📒 Files selected for processing (1)
modules/manage/pages/security/fips-compliance.adoc
Co-Authored-By: Claude Opus 4.6 (1M context) <[email protected]>
a3729c6 to
0723738
Compare
| * xref:deploy:deployment-option/self-hosted/manual/production/production-deployment.adoc#install-redpanda-for-fips-compliance[Install Redpanda for FIPS Compliance] | ||
| * xref:deploy:redpanda/manual/production/production-deployment.adoc#install-redpanda-for-fips-compliance[Install Redpanda for FIPS Compliance] | ||
| // TODO: Confirm OpenSSL version and FIPS 140-3 certificate number with engineering (ENG-307). Update links below accordingly. | ||
| * https://github.com/openssl/openssl/blob/openssl-3.0.9/README-FIPS.md[OpenSSL FIPS Readme^] |
There was a problem hiding this comment.
this could be https://github.com/openssl/openssl/blob/master/README-FIPS.md so that it is version-agnostic
ivotron
left a comment
There was a problem hiding this comment.
lgtm. minor comment on updating a URL
Summary
Consolidates DOC-1834 (Docker image for FIPS binary) and DOC-1841 (FIPS 140-3 support) into a single PR, since both modify the same page.
DOC-1834: Docker image documentation
== Configure FIPS mode with Dockersection with two deployment methods (config file mount / flags)docker.redpanda.com/redpandadata/redpanda:<version>-fipsDOC-1841: FIPS 140-3 support
Style guide fixes
should,ensure,verify)Preview page
Configure Redpanda for FIPS
What's New
🤖 Generated with Claude Code