docs(certimate): document writable temporary storage - #502
Conversation
|
No actionable comments were generated in the recent review. 🎉 ℹ️ Recent review info⚙️ Run configurationConfiguration used: Organization UI Review profile: CHILL Plan: Pro Plus Run ID: 📒 Files selected for processing (2)
📝 WalkthroughWalkthroughCertimate now includes an optional temporary storage size field. Its chart documentation describes the dedicated ChangesCertimate temporary storage
Estimated code review effort: 1 (Trivial) | ~3 minutes Possibly related PRs
🚥 Pre-merge checks | ✅ 5✅ Passed checks (5 passed)
✨ Finishing Touches📝 Generate docstrings
🧪 Generate unit tests (beta)
Warning There were issues while running some tools. Please review the errors and either fix the tool's configuration or disable the tool if it's a critical failure. 🔧 ESLint
ESLint install failed: dependency version conflict. Check your lock file or package.json. 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 |
## Summary - mount an ephemeral writable `emptyDir` at `/tmp` for Certimate certificate workflows - keep the container root filesystem read-only - cover the storage and security contract in unit tests and chart documentation ## Validation - `make validate-chart CHART=certimate` — PASS, 17 layers including all 8 k3d scenarios - `make standards-check CHART=certimate` — PASS - `make preflight` — PASS Site PR: helmforgedev/site#502 Resolves #970 <!-- This is an auto-generated comment: release notes by coderabbit.ai --> ## Summary by CodeRabbit * **New Features** * Added writable temporary storage at `/tmp` for certificate workflows. * Preserved read-only container root filesystems for improved runtime security. * **Documentation** * Documented the temporary storage configuration and production defaults. * **Tests** * Added coverage verifying read-only root filesystems and writable `/tmp` storage. <!-- end of auto-generated comment: release notes by coderabbit.ai -->
Summary
/tmpstorageCompanion documentation for helmforgedev/charts#973 and helmforgedev/charts#970.
Validation
npm run lintnpm run format:checknpm run buildmake release-check REPO=sitemake attribution-check REPO=siteSummary by CodeRabbit
New Features
/tmpstorage size.Documentation