fix(helm): quickstart deployment command nb of brokers#8090
fix(helm): quickstart deployment command nb of brokers#8090leiicamundi merged 6 commits intomainfrom
Conversation
|
👋 🤖 🤔 Hello, @leiicamundi! Did you make your changes in all the right places? These files were changed only in docs/. You might want to duplicate these changes in versioned_docs/version-8.8/.
You may have done this intentionally, but we wanted to point it out in case you didn't. You can read more about the versioning within our docs in our documentation guidelines. |
There was a problem hiding this comment.
Pull request overview
Updates the Helm quick-install documentation to avoid misconfigured multi-broker Orchestration Cluster setups when using embedded H2 as RDBMS secondary storage.
Changes:
- Updates Helm install command examples to explicitly set
orchestration.clusterSize=1,orchestration.partitionCount=1, andorchestration.replicationFactor=1. - Expands the explanatory text to describe why embedded H2 requires a single-broker/single-partition setup.
💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.
giorgionaps
left a comment
There was a problem hiding this comment.
Looks like some of the updates should be slightly reworded/reconciled with other docs. I'm requesting changes for now
Co-authored-by: Copilot <175728472+Copilot@users.noreply.github.com>
|
Thanks for the review @giorgionaps - I also edited the outdated pages, and fixed the directions around embedded elasticsearch that is now deprecated in 8.9 Happy to get your counter review! |
There was a problem hiding this comment.
Pull request overview
Copilot reviewed 3 out of 3 changed files in this pull request and generated no new comments.
💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.
ingorichtsmeier
left a comment
There was a problem hiding this comment.
Looks good to me.
...elf-managed/deployment/helm/configure/database/elasticsearch/using-external-elasticsearch.md
Outdated
Show resolved
Hide resolved
…arch/using-external-elasticsearch.md Co-authored-by: Christina Ausley <84338309+christinaausley@users.noreply.github.com>
296c0e0
|
Thanks @christinaausley, can you re-approve it so it can be automerge? ty! |
Pull request was closed
|
The preview environment relating to the commit 5ac86e2 has successfully been deployed. You can access it at https://preview.docs.camunda.cloud/pr-8090/ |
Description
Fix quickstart instructions, related to https://camunda.slack.com/archives/C03UR0V2R2M/p1772126738381749
This pull request updates the quick installation instructions for Camunda 8 using Helm, clarifying the recommended cluster configuration when using the embedded H2 database as secondary storage. The changes ensure users configure a single-broker, single-partition cluster to avoid issues with the local nature of H2, and update the example Helm commands accordingly.
Documentation improvements for Helm installation:
quick-install.mdto explicitly setorchestration.clusterSize=1,orchestration.partitionCount=1, andorchestration.replicationFactor=1, ensuring a single-broker, single-partition setup when using embedded H2. [1] [2]When should this change go live?
bugorsupportlabel)available & undocumentedlabel)holdlabel)low priolabel)PR Checklist
{type}(scope): {description}commit message(s)/docsdirectory (version 8.9)./versioned_docsdirectory.@camunda/tech-writersunless working with an embedded writer.