fix(sm): use env script reference instead of hardcoded Helm version#8439
fix(sm): use env script reference instead of hardcoded Helm version#8439leiicamundi merged 5 commits intomainfrom
Conversation
There was a problem hiding this comment.
Pull request overview
Updates the operator-based infrastructure deployment docs to avoid hardcoding a Helm chart version and instead point readers to the shared environment setup script from camunda-deployment-references.
Changes:
- Replaces the inline
export HELM_CHART_VERSION=...snippet with a0-set-environment.shreference block. - Removes
camunda-values-identity-secrets.ymlfrom the examplehelm install ... -f ...commands (in both “with domain” and “local development” variants).
Reviewed changes
Copilot reviewed 2 out of 2 changed files in this pull request and generated 6 comments.
| File | Description |
|---|---|
| docs/self-managed/deployment/helm/configure/operator-based-infrastructure.md | Switches chart version guidance to a referenced env script; adjusts Helm install examples. |
| versioned_docs/version-8.8/self-managed/deployment/helm/configure/operator-based-infrastructure.md | Same changes as above, pinned to stable/8.8 references for the 8.8 docs set. |
💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.
docs/self-managed/deployment/helm/configure/operator-based-infrastructure.md
Outdated
Show resolved
Hide resolved
docs/self-managed/deployment/helm/configure/operator-based-infrastructure.md
Show resolved
Hide resolved
...ned_docs/version-8.8/self-managed/deployment/helm/configure/operator-based-infrastructure.md
Outdated
Show resolved
Hide resolved
...ned_docs/version-8.8/self-managed/deployment/helm/configure/operator-based-infrastructure.md
Show resolved
Hide resolved
docs/self-managed/deployment/helm/configure/operator-based-infrastructure.md
Show resolved
Hide resolved
...ned_docs/version-8.8/self-managed/deployment/helm/configure/operator-based-infrastructure.md
Show resolved
Hide resolved
Replace hardcoded HELM_CHART_VERSION export with a reference snippet to 0-set-environment.sh in the operator-based infrastructure guide.
e024a76 to
50aec0e
Compare
🔍 InfraEx Content DetectedThis PR modifies documentation pages owned or referenced by the Infrastructure Experience team. A review from @camunda/infraex-medic has been requested to ensure technical consistency. 📎 Files referencing monitored repos
❌ Broken links
ℹ️ What to check
|
a973296
docs/self-managed/deployment/helm/configure/operator-based-infrastructure.md
Outdated
Show resolved
Hide resolved
…rastructure.md Co-authored-by: Christina Ausley <84338309+christinaausley@users.noreply.github.com>
|
👋 🤖 🤔 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. |
versioned_docs/version-8.7/self-managed/setup/deploy/openshift/redhat-openshift.md
Show resolved
Hide resolved
|
@christinaausley thanks to the review, I edited the docs to include the replcaement of |
|
🧹 Preview environment for this PR has been torn down. |
Description
Replace hardcoded HELM_CHART_VERSION export with a reference snippet to 0-set-environment.sh in the operator-based infrastructure guide.
Related to https://camunda.slack.com/archives/C07KBQY7H0C/p1774964251761389
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.