Skip to content

docs: explain how to fetch values files when installing from OCI#2

Merged
cadolbeau-absyss merged 1 commit into
mainfrom
docs/improve-oci-install-flow
May 26, 2026
Merged

docs: explain how to fetch values files when installing from OCI#2
cadolbeau-absyss merged 1 commit into
mainfrom
docs/improve-oci-install-flow

Conversation

@Cadolbeau

Copy link
Copy Markdown
Contributor

Summary

The current OCI install section jumps from helm install oci://... to -f values-azure.yaml without explaining how to retrieve the values files. Clients pulling through OCI do not have the values-<cloud>.yaml on disk by default, so the instructions are not self-sufficient.

This PR rewrites the "From the public OCI registry" section in both README.md (EN) and README-fr.md (FR) into 3 explicit steps:

  1. Pull the chart locallyhelm pull oci://... --untar (fetches Chart.yaml, values.yaml, all values-<cloud>.yaml, the values-client-template.yaml and the templates/)
  2. Prepare your client values filecp values-client-template.yaml values-mycompany.yaml + fill in the # TODO
  3. Installhelm install visual-tom ./visual-tom -f visual-tom/values-<cloud>.yaml -f values-mycompany.yaml ...

Also adds a short "Upgrades" note describing the upgrade workflow (helm pull --version X.Y.Z + helm upgrade).

Test plan

  • FR section restructured and reviewed
  • EN section restructured to mirror the FR layout
  • CI passes (no Helm files touched)
  • Review pass on wording

The previous OCI install instructions jumped from "helm install
oci://..." to "-f values-azure.yaml" without explaining how to
retrieve the values files. Clients pulling the chart through OCI
do not have the values-<cloud>.yaml on disk by default.

Rework the "From the public OCI registry" section into 3 explicit
steps:

1. helm pull oci://... --untar  -> fetches Chart.yaml, values.yaml,
   all values-<cloud>.yaml and the values-client-template.yaml
2. cp values-client-template.yaml values-mycompany.yaml + edit TODOs
3. helm install ./visual-tom -f visual-tom/values-<cloud>.yaml ...

Add a short "Upgrades" note describing the upgrade workflow with
a later --version.

@cadolbeau-absyss cadolbeau-absyss left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

ok

@cadolbeau-absyss cadolbeau-absyss merged commit 1a790ac into main May 26, 2026
4 checks passed
@cadolbeau-absyss cadolbeau-absyss deleted the docs/improve-oci-install-flow branch May 26, 2026 18:03
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