fix(docs): align quickstart PQC keygen and config with nightly platform image#353
Conversation
…rm image The quickstart docker-compose pinned the keygen source and config download to service/v0.16.0 while using the nightly platform image. After opentdf/platform#3563 changed the hybrid key PEM format, the nightly image can no longer load keys generated by the v0.16.0 keygen, causing a 502 on /healthz. Switch both refs to main so they stay in sync with the nightly image. Closes #352 Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com> Signed-off-by: Mary Dickson <mary.dickson@virtru.com>
📝 WalkthroughWalkthroughThe quickstart docker-compose setup now downloads ChangesQuickstart source updates
Estimated code review effort🎯 1 (Trivial) | ⏱️ ~2 minutes Possibly related issues
Possibly related PRs
Suggested reviewers
Poem
🚥 Pre-merge checks | ✅ 5✅ Passed checks (5 passed)
✨ 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.
Code Review
This pull request updates the docs/getting-started/docker-compose.yaml file to fetch the example configuration and pull source code from the main branch instead of the service/v0.16.0 release branch. There are no review comments, and I have no feedback to provide.
Important
The consumer version of Gemini Code Assist on GitHub is being sunset. Starting June 18, 2026, new organization installations will be blocked, and all code review activity will officially cease on July 17, 2026.
For more details on the timeline and next steps, please review the Help Documentation.
|
Confirmed that main fails on the script job, here: https://github.com/opentdf/docs/actions/runs/28251987695/job/83706206391 If it passes in the PR with this fix that is our validation that short term fix works. |
|
📄 Preview deployed to https://opentdf-docs-pr-353.surge.sh |
Summary
service/v0.16.0while using thenightlyplatform image/healthzmainso they stay in sync with the nightly imageReported in opentdf/platform#3068 (comment)
Longer-term fix (adding
keygenas a platform subcommand) tracked in opentdf/platform#3673.Test plan
test-stackCI job passes (currently failing on main)mainCloses #352
🤖 Generated with Claude Code
Summary by CodeRabbit