-
Notifications
You must be signed in to change notification settings - Fork 0
docs: publish latest Control Plane and Helm releases, switch to minor version scheme #27
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Conversation
… version only scheme
Deploying pgedge-docs with
|
| Latest commit: |
2e0b456
|
| Status: | ✅ Deploy successful! |
| Preview URL: | https://c25861e7.pgedge-docs.pages.dev |
| Branch Preview URL: | https://docs-plat-359-helm-control-p.pgedge-docs.pages.dev |
WalkthroughUpdated documentation redirects and configuration files to reflect version changes. Control Plane moved from v0.6.0 to v0.6.1, pgEdge Containers updated from v0.0.4 to v0.1, and CloudNative-PG manifest bumped to 1.28.0. Navigation entries and redirect targets adjusted accordingly. Changes
Estimated code review effort🎯 2 (Simple) | ⏱️ ~10 minutes
Poem
Pre-merge checks and finishing touches✅ Passed checks (3 passed)
✨ Finishing touches🧪 Generate unit tests (beta)
📜 Recent review detailsConfiguration used: defaults Review profile: CHILL Plan: Pro 📒 Files selected for processing (3)
⏰ Context from checks skipped due to timeout of 90000ms. You can increase the timeout in your CodeRabbit configuration to a maximum of 15 minutes (900000ms). (1)
🔇 Additional comments (6)
Comment |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Actionable comments posted: 0
Caution
Some comments are outside the diff and can’t be posted inline due to platform limitations.
⚠️ Outside diff range comments (1)
mkdocs.yml (1)
10-10: Update the JavaScript path to match the new control plane version.The comment on line 9 explicitly states "Must update version when control-plane latest version changes", but the path still references
v0-6-0while the control plane version is being updated to v0.6.1 (line 106) and redirects are being changed to v0-6/ (see docs/control-plane/index.md).🔎 Proposed fix
- - /control-plane/v0-6-0/scripts/generate-stack.js + - /control-plane/v0-6/scripts/generate-stack.js
📜 Review details
Configuration used: defaults
Review profile: CHILL
Plan: Pro
📒 Files selected for processing (3)
docs/control-plane/index.md(1 hunks)docs/pgedge-containers/index.md(1 hunks)mkdocs.yml(3 hunks)
⏰ Context from checks skipped due to timeout of 90000ms. You can increase the timeout in your CodeRabbit configuration to a maximum of 15 minutes (900000ms). (1)
- GitHub Check: Cloudflare Pages
🔇 Additional comments (6)
docs/control-plane/index.md (1)
2-6: LGTM! Redirect targets are consistent.The redirect URLs and link text have been correctly updated to use the minor version scheme (v0-6/), which aligns with the PR objective.
docs/pgedge-containers/index.md (1)
2-6: LGTM! Redirect targets are consistent.The redirect URLs and link text have been correctly updated to use the minor version scheme (v0-1/), which aligns with the PR objective.
mkdocs.yml (4)
381-382: LGTM! Navigation structure correctly implements minor version scheme.The pgEdge Containers navigation correctly displays "v0.1" (minor version) while referencing the full semantic version "v0.1.0" in the branch. This aligns with the PR's objective to switch to a minor version only display scheme.
403-404: LGTM! Navigation structure correctly implements minor version scheme.The Control Plane navigation correctly displays "v0.6" (minor version) while referencing the full semantic version "v0.6.1" in the branch. This aligns with the PR's objective to switch to a minor version only display scheme and is consistent with the pgEdge Containers implementation.
106-106: Check Control Plane v0.6.1 ReleaseThe
control_plane_versionin mkdocs.yml has been updated to v0.6.1. While the pgEdge Control Plane project maintains a CHANGELOG with full release notes for each version, specific details about v0.6.1 are not publicly visible. Review the project's GitHub releases page or CHANGELOG directly to confirm this release exists and assess any relevant changes before merging.
104-104: CNPG 1.28.0 manifest URL is valid, but note breaking changes for existing deployments.CloudNativePG 1.28.0 is released and the manifest URL https://raw.githubusercontent.com/cloudnative-pg/cloudnative-pg/release-1.28/releases/cnpg-1.28.0.yaml is documented. Users upgrading from 1.27.x should review two significant changes: Native Barman Cloud support has been removed, and the monitoring.enablePodMonitor field is deprecated and will be removed in a future release. Additionally, quorum-based failover is now configured via spec.postgresql.synchronous.failoverQuorum instead of the previous alpha.cnpg.io/failoverQuorum annotation.
Uh oh!
There was an error while loading. Please reload this page.