Skip to content

Conversation

@rb-kurrent
Copy link
Contributor

@rb-kurrent rb-kurrent commented Dec 22, 2025

User description

Description

This PR is a direct copy of the v1.4.3 files. The more reviewable content pr is at #941.

Page previews


PR Type

Documentation


Description

  • Create v1.5.0 documentation directory structure

  • Copy v1.4.3 documentation content as baseline

  • Include getting started guides and installation instructions

  • Add comprehensive operations and deployment examples


Diagram Walkthrough

flowchart LR
  A["v1.4.3 Docs"] -- "copy content" --> B["v1.5.0 Docs"]
  B --> C["Getting Started"]
  B --> D["Operations"]
  C --> E["README.md"]
  C --> F["Installation.md"]
  C --> G["Resource Types.md"]
  D --> H["Database Deployment"]
  D --> I["Database Backup"]
  D --> J["Advanced Networking"]
  D --> K["Managing Certificates"]
Loading

File Walkthrough

Relevant files
Documentation
10 files
README.md
Root documentation page for v1.5.0                                             
+5/-0     
README.md
Getting started guide with features and overview                 
+93/-0   
installation.md
Installation instructions for Helm and OLM                             
+280/-0 
resource-types.md
API reference for KurrentDB resource types                             
+290/-0 
README.md
Operations section overview and catalog                                   
+11/-0   
advanced-networking.md
Advanced networking configuration and strategies                 
+236/-0 
database-backup.md
Database backup and restore procedures                                     
+115/-0 
database-deployment.md
Example deployments and configuration patterns                     
+559/-0 
managing-certificates.md
TLS certificate management and configuration                         
+182/-0 
modify-deployments.md
Deployment modification and update procedures                       
+118/-0 

@rb-kurrent rb-kurrent requested a review from a team as a code owner December 22, 2025 23:21
@github-actions
Copy link

github-actions bot commented Dec 22, 2025

@qodo-code-review
Copy link
Contributor

qodo-code-review bot commented Dec 22, 2025

PR Compliance Guide 🔍

Below is a summary of compliance checks for this PR:

Security Compliance
🟢
No security concerns identified No security vulnerabilities detected by AI analysis. Human verification advised for critical code.
Ticket Compliance
🎫 No ticket provided
  • Create ticket/issue
Codebase Duplication Compliance
Codebase context is not defined

Follow the guide to enable codebase context checks.

Custom Compliance
🟢
Generic: Comprehensive Audit Trails

Objective: To create a detailed and reliable record of critical system actions for security analysis
and compliance.

Status: Passed

Learn more about managing compliance generic rules or creating your own custom rules

Generic: Robust Error Handling and Edge Case Management

Objective: Ensure comprehensive error handling that provides meaningful context and graceful
degradation

Status: Passed

Learn more about managing compliance generic rules or creating your own custom rules

Generic: Secure Error Handling

Objective: To prevent the leakage of sensitive system information through error messages while
providing sufficient detail for internal debugging.

Status: Passed

Learn more about managing compliance generic rules or creating your own custom rules

Generic: Secure Logging Practices

Objective: To ensure logs are useful for debugging and auditing without exposing sensitive
information like PII, PHI, or cardholder data.

Status: Passed

Learn more about managing compliance generic rules or creating your own custom rules

Generic: Security-First Input Validation and Data Handling

Objective: Ensure all data inputs are validated, sanitized, and handled securely to prevent
vulnerabilities

Status: Passed

Learn more about managing compliance generic rules or creating your own custom rules

🔴
Generic: Meaningful Naming and Self-Documenting Code

Objective: Ensure all identifiers clearly express their purpose and intent, making code
self-documenting

Status:
Misnamed identifiers: The PR introduces multiple misspelled or incorrect field/identifier names (e.g.,
readOnlyReplias, KurretnDB, KurrnetDBNetwork, AdveritseHostToClientAs, newtork) which
makes the documentation non-self-documenting and potentially misleading.

Referred Code
| `readOnlyReplias` _[KurrentDBReadOnlyReplicasSpec][d7]_ | No       | Read-only replica configuration the Kurrent DB Cluster.                                                                                  |
| `extraMetadata` _[KurrentDBExtraMetadataSpec][d8]_      | No       | Additional annotations and labels for child resources.                                                                                   |
| `quorumNodes` _string array_                            | No       | A list of endpoints (in host:port notation) to reach the quorum nodes when .Replicas is zero, see [standalone ReadOnlyReplicas][ror]     |

[d1]: https://kubernetes.io/docs/reference/generated/kubernetes-api/v1.26/#resourcerequirements-v1-core
[d2]: https://kubernetes.io/docs/reference/generated/kubernetes-api/v1.26/#persistentvolumeclaimspec-v1-core
[d3]: #kurrentdbnetwork
[d4]: #kurrentdbsecurity
[d5]: https://kubernetes.io/docs/reference/generated/kubernetes-api/v1.26/#secretkeyselector-v1-core
[d6]: #kurrentdbconstraints
[d7]: #kurrentdbreadonlyreplicasspec
[d8]: #kurrentdbextrametadataspec
[ror]: ../operations/database-deployment.md#deploying-standalone-read-only-replicas

#### KurrentDBReadOnlyReplicasSpec

Other than `replicas`, each of the fields in `KurrentDBReadOnlyReplicasSpec` default to the corresponding values from the main KurrentDBSpec.

| Field                                        | Required | Description                                                      |
|----------------------------------------------|----------|------------------------------------------------------------------|
| `replicas` _integer_                         | No       | Number of read-only replicas in the cluster.  Defaults to zero.  |


 ... (clipped 238 lines)

Learn more about managing compliance generic rules or creating your own custom rules

  • Update
Compliance status legend 🟢 - Fully Compliant
🟡 - Partial Compliant
🔴 - Not Compliant
⚪ - Requires Further Human Verification
🏷️ - Compliance label

@qodo-code-review
Copy link
Contributor

qodo-code-review bot commented Dec 22, 2025

PR Code Suggestions ✨

Explore these optional code suggestions:

CategorySuggestion                                                                                                                                    Impact
Possible issue
Update Helm install command version
Suggestion Impact:The documentation was updated to replace 1.4.3 with 1.5.0 in the Helm install command, and the same version bump was also applied to related commands/examples (helm pull, upgrade example version, and startingCSV).

code diff:

@@ -46,7 +46,7 @@
 
 ```bash
 helm install kurrentdb-operator kurrent-latest/kurrentdb-operator \
-  --version 1.4.3 \
+  --version 1.5.0 \
   --create-namespace \
   --namespace kurrent-system \
   --set-file operator.license.key=/path/to/license.key \
@@ -97,7 +97,7 @@
 
 ```bash
 # Download the kurrentdb-operator Helm chart
-helm pull kurrent-latest/kurrentdb-operator --version 1.4.3 --untar
+helm pull kurrent-latest/kurrentdb-operator --version 1.5.0 --untar
 
 # Install the CRDs
 kubectl apply -f kurrentdb-operator/templates/crds
@@ -139,7 +139,7 @@
 Here's what these commands do:
 - Refresh the local Helm repository index
 - Locate an existing operator installation in namespace `kurrent`
-- Select the target upgrade version `{version}` e.g. `1.4.3`
+- Select the target upgrade version `{version}` e.g. `1.5.0`
 - Perform the upgrade, preserving values that were set during installation
 
 ## Install Using OLM
@@ -237,7 +237,7 @@
   name: kurrentdb-operator
   source: kurrentdb-operator
   sourceNamespace: default
-  startingCSV: kurrentdb-operator.v1.4.3
+  startingCSV: kurrentdb-operator.v1.5.0
 EOF

Update the Helm installation command in the documentation to use --version 1.5.0
instead of 1.4.3 to ensure users install the correct operator version.

docs/server/kubernetes-operator/v1.5.0/getting-started/installation.md [48-53]

 helm install kurrentdb-operator kurrent-latest/kurrentdb-operator \
-  --version 1.4.3 \
+  --version 1.5.0 \
   --create-namespace \
   --namespace kurrent-system \
   --set-file operator.license.key=/path/to/license.key \
   --set-file operator.license.file=/path/to/license.lic

[Suggestion processed]

Suggestion importance[1-10]: 7

__

Why: This is an important correction, as the incorrect version in the helm install command would cause users to install an outdated operator, leading to confusion and potential issues.

Medium
Update Helm pull command version
Suggestion Impact:The documentation was updated to change the `helm pull` command from `--version 1.4.3` to `--version 1.5.0`. The commit also updated other related version references (helm install version, upgrade example, and OLM startingCSV) to 1.5.0.

code diff:

@@ -97,7 +97,7 @@
 
 ```bash
 # Download the kurrentdb-operator Helm chart
-helm pull kurrent-latest/kurrentdb-operator --version 1.4.3 --untar
+helm pull kurrent-latest/kurrentdb-operator --version 1.5.0 --untar
 

Update the helm pull command in the manual installation guide to specify
--version 1.5.0 instead of 1.4.3.

docs/server/kubernetes-operator/v1.5.0/getting-started/installation.md [99-100]

 # Download the kurrentdb-operator Helm chart
-helm pull kurrent-latest/kurrentdb-operator --version 1.4.3 --untar
+helm pull kurrent-latest/kurrentdb-operator --version 1.5.0 --untar

[Suggestion processed]

Suggestion importance[1-10]: 7

__

Why: This correction is important because the helm pull command in the documentation points to an old version, which would cause users following the manual installation guide to download the wrong chart.

Medium
Update OLM startingCSV to correct version
Suggestion Impact:The commit updates the OLM `startingCSV` field to v1.5.0 as suggested (and also updates related Helm version references to 1.5.0).

code diff:

@@ -237,7 +237,7 @@
   name: kurrentdb-operator
   source: kurrentdb-operator
   sourceNamespace: default
-  startingCSV: kurrentdb-operator.v1.4.3
+  startingCSV: kurrentdb-operator.v1.5.0
 EOF

Update the startingCSV field in the OLM Subscription example from
kurrentdb-operator.v1.4.3 to kurrentdb-operator.v1.5.0.

docs/server/kubernetes-operator/v1.5.0/getting-started/installation.md [240]

-startingCSV: kurrentdb-operator.v1.4.3
+startingCSV: kurrentdb-operator.v1.5.0

[Suggestion processed]

Suggestion importance[1-10]: 7

__

Why: This is an important fix, as the incorrect startingCSV version in the OLM installation guide would lead users to install an outdated operator version.

Medium
General
Fix typo in API field name
Suggestion Impact:The documentation table entry was updated to use the corrected field name `readOnlyReplicas` (replacing the misspelled `readOnlyReplias`) as part of a broader rewrite/expansion of the configuration reference.

code diff:

-| Field                                                   | Required | Description                                                                                                                              |
-|---------------------------------------------------------|----------|------------------------------------------------------------------------------------------------------------------------------------------|
-| `replicas` _integer_                                    | Yes      | Number of nodes in a database cluster.  May be 1, 3, or, for [standalone ReadOnly-Replicas][ror], it may be 0.                            |
-| `image` _string_                                        | Yes      | KurrentDB container image URL                                                                                                            |
-| `resources` _[ResourceRequirements][d1]_                | No       | Database container resource limits and requests                                                                                          |
-| `storage` _[PersistentVolumeClaim][d2]_                 | Yes      | Persistent volume claim settings for the underlying data volume                                                                          |
-| `network` _[KurrentDBNetwork][d3]_                      | Yes      | Defines the network configuration to use with the database                                                                               |
-| `configuration` _yaml_                                  | No       | Additional configuration to use with the database, see [below](#configuring-kurrent-db)                                                  |
-| `sourceBackup` _string_                                 | No       | Backup name to restore a cluster from                                                                                                    |
-| `security` _[KurrentDBSecurity][d4]_                    | No       | Security configuration to use for the database. This is optional, if not specified the cluster will be created without security enabled. |
-| `licenseSecret` _[SecretKeySelector][d5]_               | No       | A secret that contains the Enterprise license for the database                                                                           |
-| `constraints` _[KurrentDBConstraints][d6]_              | No       | Scheduling constraints for the Kurrent DB pod.                                                                                           |
-| `readOnlyReplias` _[KurrentDBReadOnlyReplicasSpec][d7]_ | No       | Read-only replica configuration the Kurrent DB Cluster.                                                                                  |
-| `extraMetadata` _[KurrentDBExtraMetadataSpec][d8]_      | No       | Additional annotations and labels for child resources.                                                                                   |
-| `quorumNodes` _string array_                            | No       | A list of endpoints (in host:port notation) to reach the quorum nodes when .Replicas is zero, see [standalone ReadOnlyReplicas][ror]     |
-
+| Field                                                    | Required | Description                                                                                                                              |
+|----------------------------------------------------------|----------|------------------------------------------------------------------------------------------------------------------------------------------|
+| `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.                        |
+| `image` _string_                                         | Yes      | KurrentDB container image URL.  See [Selecting An Image][img], below.                                                                    |
+| `resources` _[ResourceRequirements][d1]_                 | No       | Database container resource limits and requests                                                                                          |
+| `storage` _[PersistentVolumeClaim][d2]_                  | Yes      | Persistent volume claim settings for the underlying data volume                                                                          |
+| `network` _[KurrentDBNetwork][d3]_                       | Yes      | Defines the network configuration to use with the database                                                                               |
+| `configuration` _yaml_                                   | No       | Additional configuration to use with the database, see [below](#configuring-kurrent-db)                                                  |
+| `environmentSecret` _string_                             | No       | The name of a Secret to populate environment variables.  If the secret changes a rolling restart occurs.                                 |
+| `sourceBackup` _string_                                  | No       | Backup name to restore a cluster from                                                                                                    |
+| `security` _[KurrentDBSecurity][d4]_                     | No       | Security configuration to use for the database. This is optional, if not specified the cluster will be created without security enabled. |
+| `licenseSecret` _[SecretKeySelector][d5]_                | No       | A secret that contains the Enterprise license for the database                                                                           |
+| `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.                                                                               |

Correct the typo in the API documentation from readOnlyReplias to
readOnlyReplicas in the KurrentDBSpec table.

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

-| `readOnlyReplias` _[KurrentDBReadOnlyReplicasSpec][d7]_ | No       | Read-only replica configuration the Kurrent DB Cluster.                                                                                  |
+| `readOnlyReplicas` _[KurrentDBReadOnlyReplicasSpec][d7]_ | No       | Read-only replica configuration the Kurrent DB Cluster.                                                                                  |

[Suggestion processed]

Suggestion importance[1-10]: 6

__

Why: This suggestion correctly identifies and fixes a typo in an API field name (readOnlyReplias) within the documentation, which improves clarity and prevents user confusion.

Low
Update documentation title to correct version
Suggestion Impact:The README frontmatter title was changed from "Kubernetes Operator v1.4.3" to "Kubernetes Operator v1.5.0", correcting the version in the documentation title.

code diff:

 ---
 # title is for breadcrumb and sidebar nav
-title: Kubernetes Operator v1.4.3
+title: Kubernetes Operator v1.5.0
 order: 1

Update the documentation title in
docs/server/kubernetes-operator/v1.5.0/README.md from v1.4.3 to v1.5.0 to match
the correct version.

docs/server/kubernetes-operator/v1.5.0/README.md [3]

-title: Kubernetes Operator v1.4.3
+title: Kubernetes Operator v1.5.0

[Suggestion processed]

Suggestion importance[1-10]: 5

__

Why: This is a valid correction for a copy-paste error in the documentation, ensuring the title accurately reflects the v1.5.0 version of the document.

Low
  • Update

@github-actions
Copy link

@cloudflare-workers-and-pages
Copy link

cloudflare-workers-and-pages bot commented Dec 22, 2025

Deploying documentation with  Cloudflare Pages  Cloudflare Pages

Latest commit: 86ba9c3
Status: ✅  Deploy successful!
Preview URL: https://db751371.documentation-21k.pages.dev
Branch Preview URL: https://operator-v1-5-0.documentation-21k.pages.dev

View logs

@stktung stktung merged commit 1ecfba9 into master Dec 23, 2025
3 checks passed
@stktung stktung deleted the operator-v1-5-0 branch December 23, 2025 03:33
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants