Skip to content

Operator v1.6.0 content#969

Open
rb-kurrent wants to merge 3 commits into
masterfrom
operator-v1-6-0-content
Open

Operator v1.6.0 content#969
rb-kurrent wants to merge 3 commits into
masterfrom
operator-v1-6-0-content

Conversation

@rb-kurrent
Copy link
Copy Markdown
Contributor

This includes 1.6.0 features and also substantial typo detection + correction.

@rb-kurrent rb-kurrent requested a review from a team as a code owner June 5, 2026 20:02
@github-actions
Copy link
Copy Markdown

github-actions Bot commented Jun 5, 2026

@qodo-code-review
Copy link
Copy Markdown
Contributor

Review Summary by Qodo

Add Kubernetes Operator v1.6.0 documentation and content

✨ Enhancement 📝 Documentation

Grey Divider

Walkthroughs

Description
• Add comprehensive v1.6.0 documentation for Kubernetes Operator
• Document new features including PodDisruptionBudget, improved health checks, online license checks
• Provide installation guides for Helm, OpenShift, and OLM deployment methods
• Include detailed API reference, networking, backup, and certificate management guides
• Fix certificate authority secret documentation for clarity on empty string handling
Diagram
flowchart LR
  A["v1.6.0 Release"] --> B["Getting Started Guide"]
  A --> C["Installation Methods"]
  A --> D["API Reference"]
  A --> E["Operations Guides"]
  B --> F["README with Features"]
  C --> G["Helm/OpenShift/OLM"]
  D --> H["Resource Types & Config"]
  E --> I["Deployment Examples"]
  E --> J["Backup & Certificates"]
  E --> K["Advanced Networking"]

Loading

Grey Divider

File Changes

1. docs/server/kubernetes-operator/v1.5.0/getting-started/resource-types.md 📝 Documentation +5/-4

Clarify certificate authority secret handling

docs/server/kubernetes-operator/v1.5.0/getting-started/resource-types.md


2. docs/server/kubernetes-operator/v1.6.0/README.md 📝 Documentation +5/-0

Add v1.6.0 root documentation page

docs/server/kubernetes-operator/v1.6.0/README.md


3. docs/server/kubernetes-operator/v1.6.0/getting-started/README.md 📝 Documentation +80/-0

Add comprehensive getting started guide

docs/server/kubernetes-operator/v1.6.0/getting-started/README.md


View more (9)
4. docs/server/kubernetes-operator/v1.6.0/getting-started/installation.md 📝 Documentation +300/-0

Add detailed installation instructions for multiple methods

docs/server/kubernetes-operator/v1.6.0/getting-started/installation.md


5. docs/server/kubernetes-operator/v1.6.0/getting-started/resource-types.md 📝 Documentation +463/-0

Add complete API reference and configuration documentation

docs/server/kubernetes-operator/v1.6.0/getting-started/resource-types.md


6. docs/server/kubernetes-operator/v1.6.0/operations/README.md 📝 Documentation +11/-0

Add operations section overview page

docs/server/kubernetes-operator/v1.6.0/operations/README.md


7. docs/server/kubernetes-operator/v1.6.0/operations/advanced-networking.md 📝 Documentation +236/-0

Add advanced networking configuration guide

docs/server/kubernetes-operator/v1.6.0/operations/advanced-networking.md


8. docs/server/kubernetes-operator/v1.6.0/operations/database-backup.md 📝 Documentation +126/-0

Add backup and restore procedures documentation

docs/server/kubernetes-operator/v1.6.0/operations/database-backup.md


9. docs/server/kubernetes-operator/v1.6.0/operations/database-deployment.md 📝 Documentation +666/-0

Add comprehensive deployment examples and patterns

docs/server/kubernetes-operator/v1.6.0/operations/database-deployment.md


10. docs/server/kubernetes-operator/v1.6.0/operations/managing-certificates.md 📝 Documentation +231/-0

Add certificate management and migration guide

docs/server/kubernetes-operator/v1.6.0/operations/managing-certificates.md


11. docs/server/kubernetes-operator/v1.6.0/operations/modify-deployments.md 📝 Documentation +144/-0

Add deployment modification and update procedures

docs/server/kubernetes-operator/v1.6.0/operations/modify-deployments.md


12. docs/server/kubernetes-operator/versions.json ⚙️ Configuration changes +5/-0

Register v1.6.0 in documentation versions list

docs/server/kubernetes-operator/versions.json


Grey Divider

Qodo Logo

@qodo-code-review
Copy link
Copy Markdown
Contributor

qodo-code-review Bot commented Jun 5, 2026

Code Review by Qodo

🐞 Bugs (2) 📘 Rule violations (1) 📎 Requirement gaps (0) 🎨 UX issues (0)

Grey Divider


Action required

1. ReadOnly-Replicas term inconsistent ✓ Resolved 📘 Rule violation ⚙ Maintainability
Description
The new v1.6.0 docs use ReadOnly-Replicas/ReadOnlyReplicas instead of the consistent generic
term read-only replicas. This violates the requirement to normalize terminology consistently
across the documentation.
Code

docs/server/kubernetes-operator/v1.6.0/getting-started/resource-types.md[21]

+| `replicas` _integer_                                     | Yes      | Number of nodes in a database cluster.  May be 1, 3, 5, or, for [standalone ReadOnly-Replicas][ror], it may be 0.                        |
Evidence
Rule 3 requires consistent casing/terminology. The added KurrentDBSpec table introduces
standalone ReadOnly-Replicas and later references ReadOnlyReplicas, which conflicts with the
otherwise-used read-only replicas terminology.

docs/server/kubernetes-operator/v1.6.0/getting-started/resource-types.md[21-21]
docs/server/kubernetes-operator/v1.6.0/getting-started/resource-types.md[36-36]
Best Practice: Repository guidelines

Agent prompt
The issue below was found during a code review. Follow the provided context and guidance below and implement a solution

## Issue description
The documentation uses inconsistent terminology for read-only replicas (e.g., `ReadOnly-Replicas`, `ReadOnlyReplicas`) instead of consistent reader-facing wording.

## Issue Context
Rule requires consistent capitalization/terminology across headings and prose.

## Fix Focus Areas
- docs/server/kubernetes-operator/v1.6.0/getting-started/resource-types.md[21-21]
- docs/server/kubernetes-operator/v1.6.0/getting-started/resource-types.md[36-36]

ⓘ Copy this prompt and use it to remediate the issue with your preferred AI generation tools


2. Kurrent DB naming inconsistent ✓ Resolved 📘 Rule violation ⚙ Maintainability
Description
The new v1.6.0 docs mix KurrentDB with Kurrent DB/kurrentdb in prose and descriptions. This
violates the requirement to keep terminology consistently normalized throughout the docs.
Code

docs/server/kubernetes-operator/v1.6.0/getting-started/resource-types.md[R31-33]

+| `constraints` _[KurrentDBConstraints][d6]_               | No       | Scheduling constraints for the Kurrent DB pod.                                                                                           |
+| `readOnlyReplicas` _[KurrentDBReadOnlyReplicasSpec][d7]_ | No       | Read-only replica configuration for the Kurrent DB Cluster.                                                                              |
+| `archiver` _[KurrentDBArchiverSpec][d8]_                 | No       | Archiver replica configuration for the Kurrent DB Cluster.                                                                               |
Evidence
Rule 3 requires consistent terminology/casing. The added content uses KurrentDB but also
introduces spaced/lowercased variants (Kurrent DB, kurrentdb) in the same document, creating
inconsistent terminology.

docs/server/kubernetes-operator/v1.6.0/getting-started/resource-types.md[31-33]
docs/server/kubernetes-operator/v1.6.0/getting-started/resource-types.md[309-311]
docs/server/kubernetes-operator/v1.6.0/getting-started/resource-types.md[417-419]
Best Practice: Repository guidelines

Agent prompt
The issue below was found during a code review. Follow the provided context and guidance below and implement a solution

## Issue description
Inconsistent product/term formatting appears in the documentation (`KurrentDB` vs `Kurrent DB` vs `kurrentdb`).

## Issue Context
Rule requires consistent terminology in prose and descriptions.

## Fix Focus Areas
- docs/server/kubernetes-operator/v1.6.0/getting-started/resource-types.md[31-33]
- docs/server/kubernetes-operator/v1.6.0/getting-started/resource-types.md[309-311]
- docs/server/kubernetes-operator/v1.6.0/getting-started/resource-types.md[417-419]

ⓘ Copy this prompt and use it to remediate the issue with your preferred AI generation tools



Remediation recommended

3. Kind's` grammar typo 📘 Rule violation ≡ Correctness
Description
The phrase Kind's is grammatically incorrect and should be Kinds (or kinds`). This reduces
clarity and violates the documentation grammar requirement.
Code

docs/server/kubernetes-operator/v1.6.0/getting-started/resource-types.md[6]

+The Operator supports the following resource types (known as `Kind`'s):
Evidence
Rule 4 requires correct grammar/tense. The added line uses an incorrect plural form (Kind's`)
which is a grammatical error.

docs/server/kubernetes-operator/v1.6.0/getting-started/resource-types.md[6-6]
Best Practice: Repository guidelines

Agent prompt
The issue below was found during a code review. Follow the provided context and guidance below and implement a solution

## Issue description
A grammatical error (`Kind`'s`) appears in the introduction to the resource types list.

## Issue Context
This is reader-facing documentation and should use correct grammar.

## Fix Focus Areas
- docs/server/kubernetes-operator/v1.6.0/getting-started/resource-types.md[6-6]

ⓘ Copy this prompt and use it to remediate the issue with your preferred AI generation tools


4. CA keyName rules unclear 🐞 Bug ≡ Correctness
Description
The updated v1.5.0 note for certificateAuthoritySecret.keyName only describes non-empty vs
empty-string behavior and no longer mentions the valid “missing/unset” case. This creates
inconsistency with the v1.6.0 reference, which documents that missing *or* empty trusts all keys,
and can mislead users configuring CA rotation.
Code

docs/server/kubernetes-operator/v1.5.0/getting-started/resource-types.md[R226-230]

+Note that in `certificateAuthoritySecret`, only `.name` needs to be non-empty.  If `.keyName` is
+non-empty, only that Secret key will be mounted into the pod as a CA.  If it is set to the empty
+string (`""`), all Secret keys will be mounted as CAs, which allows for rotating CAs without
+downtime, by trusting both old and new CAs for a period of time.  `.privateKeyName` is deprecated
+and ignored.
Evidence
The reworded v1.5.0 note now only talks about non-empty vs empty-string keyName, while v1.6.0
explicitly documents that keyName being missing OR empty results in trusting all keys—highlighting
the v1.5.0 ambiguity/inconsistency.

docs/server/kubernetes-operator/v1.5.0/getting-started/resource-types.md[219-230]
docs/server/kubernetes-operator/v1.6.0/getting-started/resource-types.md[253-259]

Agent prompt
The issue below was found during a code review. Follow the provided context and guidance below and implement a solution

## Issue description
The v1.5.0 documentation note for `certificateAuthoritySecret.keyName` became ambiguous after rewording: it discusses `keyName` being non-empty vs explicitly set to `""`, but doesn’t explicitly say what happens if `keyName` is omitted.

## Issue Context
v1.6.0’s `CertificateAuthoritySecret` table explicitly states that if `keyName` is missing or empty, all keys are trusted. v1.5.0 should match that wording to avoid confusing readers and introducing version-to-version inconsistency.

## Fix Focus Areas
- docs/server/kubernetes-operator/v1.5.0/getting-started/resource-types.md[226-230]

ⓘ Copy this prompt and use it to remediate the issue with your preferred AI generation tools


5. scheulde typo in table ✓ Resolved 📘 Rule violation ≡ Correctness
Description
The v1.6.0 configuration reference misspells “schedule” as scheulde in the
KurrentDBBackupScheduleSpec table. This violates the requirement for correct grammar and will be
published verbatim, reducing documentation clarity and searchability.
Code

docs/server/kubernetes-operator/v1.6.0/getting-started/resource-types.md[378]

+| `suspend` _boolean_                | No       | While true, pauses the creation of new backups for this scheulde.                                                                    |
Evidence
Rule 4 requires correct grammar/spelling in documentation, and the cited section of the generated
v1.6.0 resource-types reference table for backup schedules contains the misspelling scheulde in
the KurrentDBBackupScheduleSpec table, demonstrating a clear spelling error in user-facing
content.

docs/server/kubernetes-operator/v1.6.0/getting-started/resource-types.md[378-378]
docs/server/kubernetes-operator/v1.6.0/getting-started/resource-types.md[370-379]
Best Practice: Repository guidelines

Agent prompt
The issue below was found during a code review. Follow the provided context and guidance below and implement a solution

## Issue description
The v1.6.0 documentation contains a typo in the `KurrentDBBackupScheduleSpec` table where “schedule” is misspelled as `scheulde`; this should be corrected to meet grammar/spelling requirements and maintain documentation quality.

## Issue Context
This is user-facing, generated reference documentation that will be rendered verbatim to end users; typos reduce clarity, documentation quality, and searchability.

## Fix Focus Areas
- docs/server/kubernetes-operator/v1.6.0/getting-started/resource-types.md[372-379]

ⓘ Copy this prompt and use it to remediate the issue with your preferred AI generation tools



Advisory comments

6. No newline at EOF 🐞 Bug ⚙ Maintainability
Description
The newly added v1.6.0 operations README is missing a trailing newline at end-of-file. This can
break markdown/text linting and some POSIX-oriented tooling in CI pipelines.
Code

docs/server/kubernetes-operator/v1.6.0/operations/README.md[R9-11]

+A number of operations can be performed with the Operator which are catalogued below:
+
+<Catalog/>
Evidence
The file is introduced in this PR and the PR diff indicates it lacks a final newline; the cited
region is the file end.

docs/server/kubernetes-operator/v1.6.0/operations/README.md[1-11]

Agent prompt
The issue below was found during a code review. Follow the provided context and guidance below and implement a solution

## Issue description
`docs/server/kubernetes-operator/v1.6.0/operations/README.md` is missing a final newline, which some linters and tooling treat as an error.

## Issue Context
The PR diff marks this file with `No newline at end of file`.

## Fix Focus Areas
- docs/server/kubernetes-operator/v1.6.0/operations/README.md[9-11]

ⓘ Copy this prompt and use it to remediate the issue with your preferred AI generation tools


Grey Divider

Qodo Logo

Comment thread docs/server/kubernetes-operator/v1.6.0/getting-started/resource-types.md Outdated
Comment thread docs/server/kubernetes-operator/v1.6.0/getting-started/resource-types.md Outdated
@github-actions
Copy link
Copy Markdown

github-actions Bot commented Jun 5, 2026

@cloudflare-workers-and-pages
Copy link
Copy Markdown

cloudflare-workers-and-pages Bot commented Jun 5, 2026

Deploying documentation with  Cloudflare Pages  Cloudflare Pages

Latest commit: 170760e
Status: ✅  Deploy successful!
Preview URL: https://4416081a.documentation-21k.pages.dev
Branch Preview URL: https://operator-v1-6-0-content.documentation-21k.pages.dev

View logs

@rb-kurrent rb-kurrent force-pushed the operator-v1-6-0-content branch from 64fd462 to cd47e12 Compare June 5, 2026 20:17
@rb-kurrent rb-kurrent force-pushed the operator-v1-6-0-content branch from cd47e12 to 170760e Compare June 5, 2026 20:18
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants