Skip to content

docs: promote HelmRelease tutorial to weight 10 (and fix typo)#3233

Open
pipo02mix wants to merge 6 commits into
migrate-to-hrfrom
promote-helmrelease-tutorial-minimal
Open

docs: promote HelmRelease tutorial to weight 10 (and fix typo)#3233
pipo02mix wants to merge 6 commits into
migrate-to-hrfrom
promote-helmrelease-tutorial-minimal

Conversation

@pipo02mix

@pipo02mix pipo02mix commented Jun 26, 2026

Copy link
Copy Markdown
Contributor

Towards https://github.com/giantswarm/giantswarm/issues/36892

Minimal-surgery retry of the previously-reverted tutorial-promotion PR.

pipo02mix and others added 6 commits June 17, 2026 22:33
Creates a new continuous-deployment subsection (tutorials/continuous-
deployment/helm-releases/) and its first guide, the HelmRelease
equivalent of apps/add-appcr.md.

- _index.md scaffolds the section at weight 80 (ahead of the legacy
  apps section at weight 90).
- add-helmrelease.md mirrors the App CR GitOps guide's shape:
  environment variables, gitops-template directory structure, then
  rendering OCIRepository + HelmRelease YAML with flux create
  --export, wiring up valuesFrom (ConfigMap + SOPS-encrypted Secret),
  and including the new files in kustomization.yaml.
- Links to the conceptual overview and to the imperative
  deploy-app-helmrelease tutorial; references the App CR equivalent
  for context. Avoids forward-links to the three upcoming companion
  guides (update / automatic updates / multiple releases) so the
  Hugo build doesn't break on missing relrefs until those pages land.

Co-Authored-By: Claude Opus 4.7 (1M context) <noreply@anthropic.com>
Mirrors apps/update-appcr.md for the HelmRelease world. Covers the four
edit surfaces that map to "update an existing deployment":

- ocirepository.yaml for chart version bumps (the version moves out of
  the release CR and into the source, vs the App CR model).
- helmrelease.yaml for install/upgrade behavior, dependsOn, drift
  detection, and so on.
- configmap.yaml for non-secret values.
- secret.enc.yaml for credentials, via the same sops decrypt-edit-
  re-encrypt cycle used in the App CR guide.

Also adds a "Force an immediate reconciliation" section showing
flux reconcile source oci / helmrelease for when waiting for the
interval isn't acceptable.

Page is branched off add-helmrelease-gitops-guide so the section
_index.md exists when this PR builds; PR base is migrate-to-hr and
should be merged after page 1.

Co-Authored-By: Claude Opus 4.7 (1M context) <noreply@anthropic.com>
add-helmrelease.md:
- "repo" -> "repository" (singular not in vocab).
- "SOPS-encrypted" -> "encrypted with sops" so the bare uppercase
  acronym no longer triggers Microsoft.Acronyms.

update-helmrelease.md:
- Splits the SemVer-range note into two sentences to drop a semicolon
  Microsoft.Semicolon flagged as too complex.
- "afterwards" -> "afterward" per Microsoft.Terms.

Co-Authored-By: Claude Opus 4.7 (1M context) <noreply@anthropic.com>
… page 3/4)

Mirrors apps/automatic-updates-appcr.md but reframes around the two
update mechanisms available for HelmRelease:

1. SemVer range on the OCIRepository (recommended). One-line change to
   ocirepository.yaml; no Git commits, no extra Flux resources. This
   is the native Flux way to auto-roll chart versions and didn't
   exist for App CRs.

2. Image automation with Git commits (optional). The ImageRepository
   + ImagePolicy + ImageUpdateAutomation triad from the App CR guide,
   adapted to write the new tag into ocirepository.yaml's
   .spec.ref.tag via a setter comment. For users who want an audit
   trail of every version bump in Git.

Private-registry section is included for both approaches via the
ImageRepository secretRef.

Stacked on add-update-helmrelease-guide so the section _index.md
plus pages 1+2 are present at build time. PR base is migrate-to-hr;
merge in order 1 -> 2 -> 3.

Co-Authored-By: Claude Opus 4.7 (1M context) <noreply@anthropic.com>
Replacement for apps/app-sets.md and the conceptual content of
app-platform/app-bundle/. Covers three patterns for grouping
HelmReleases:

1. Helm umbrella chart: one parent chart with Chart.yaml
   dependencies, deployed as a single HelmRelease. Strongest coupling,
   shared values, one Helm history.

2. Kustomize over multiple HelmReleases: the closest 1:1 translation
   of App Sets. Per-component HelmRelease + OCIRepository, grouped by
   a Kustomize base with commonLabels, common namespace, and inline
   patches pinning each OCIRepository's tag.

3. dependsOn: native Flux ordering with no App CR equivalent. For
   release-A-before-release-B sequencing across namespaces.

Calls out that the three patterns compose. Links the App Sets page
for users coming from the App CR model.

Stacked on add-auto-updates-helmrelease-guide. PR base is
migrate-to-hr; merge in order 1 -> 2 -> 3 -> 4 to complete the
helm-releases section.

Co-Authored-By: Claude Opus 4.7 (1M context) <noreply@anthropic.com>
Minimal-surgery retry of the previously-reverted tutorial-promotion
PR. Two tiny changes only:

- weight: 21 -> 10 so the HelmRelease tutorial sorts above the App
  CR tutorial (which is at weight 20) under App management.
- Fix a pre-existing typo "OCIRespository" -> "OCIRepository" in the
  Background paragraph.

NOT in this PR (compared to the earlier reverted attempt):
- No edits to deploy-app/ (the App CR tutorial stays at its current
  weight, no banner added).
- No comparison table between App CR and HelmRelease.
- No new "GitOps workflow" section.

Those can be follow-up PRs if you want them, but kept out of this one
since the earlier combined attempt was reverted.

Co-Authored-By: Claude Opus 4.7 (1M context) <noreply@anthropic.com>
@pipo02mix pipo02mix requested a review from a team as a code owner June 26, 2026 13:01
@pipo02mix pipo02mix self-assigned this Jun 26, 2026
@marians

marians commented Jun 26, 2026

Copy link
Copy Markdown
Member

PR title and the actual change are not in sync here. The title suggests a small frontmatter change, but the PR actually adds a lot of content.

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