feat: workload ApplicationSet + charts - #14
Merged
Merged
Conversation
…ineered, mealie, changedetection, api)
Helm template diffOnly in /tmp/head: changedetection.yaml
Only in /tmp/head: mealie.yaml
Only in /tmp/head: plausible.yaml |
8gears.container-registry.com/chartrepo/library now serves a Harbor web UI instead of a valid Helm index. imio.github.io/helm-charts mirrors the same plausible-analytics chart at the identical version (0.4.2), so values.yaml needs no changes.
None of the three have a source repo or built image yet. Keeping mealie, plausible, and changedetection, which use real upstream images.
jonathandieu
force-pushed
the
workload-charts
branch
from
July 4, 2026 02:03
a4c337e to
176550f
Compare
jonathandieu
marked this pull request as ready for review
July 4, 2026 03:07
Contributor
There was a problem hiding this comment.
Pull request overview
Adds GitOps-managed “workloads” applications (Helm charts + Argo CD ApplicationSet) for Mealie, changedetection.io, and Plausible, intended to deploy using real upstream images and cluster-specific overrides.
Changes:
- Introduces new workload Helm charts for
mealieandchangedetection(Deployment/Service/PVC + Gateway API HTTPRoute). - Adds a wrapper Helm chart for
plausiblethat depends onplausible-analytics. - Adds an Argo CD
ApplicationSetto deploy these workload charts to selected clusters.
Reviewed changes
Copilot reviewed 15 out of 15 changed files in this pull request and generated 5 comments.
Show a summary per file
| File | Description |
|---|---|
| charts/workloads/plausible/values.yaml | Configures the plausible dependency chart (image tag, ingress toggle, app settings). |
| charts/workloads/plausible/Chart.yaml | Declares wrapper chart + dependency on plausible-analytics. |
| charts/workloads/mealie/values.yaml | Defines Mealie image/env/route/persistence/resources defaults. |
| charts/workloads/mealie/templates/service.yaml | Exposes Mealie on port 9000 via a ClusterIP Service. |
| charts/workloads/mealie/templates/pvc.yaml | Adds optional PVC for Mealie data. |
| charts/workloads/mealie/templates/httproute.yaml | Routes external traffic to Mealie via Gateway API HTTPRoute. |
| charts/workloads/mealie/templates/deployment.yaml | Deploys Mealie pod + mounts persistence volume. |
| charts/workloads/mealie/Chart.yaml | Defines the custom Mealie Helm chart. |
| charts/workloads/changedetection/values.yaml | Defines changedetection image/route/persistence/resources defaults. |
| charts/workloads/changedetection/templates/service.yaml | Exposes changedetection on port 5000 via a ClusterIP Service. |
| charts/workloads/changedetection/templates/pvc.yaml | Adds optional PVC for changedetection datastore. |
| charts/workloads/changedetection/templates/httproute.yaml | Routes external traffic to changedetection via Gateway API HTTPRoute. |
| charts/workloads/changedetection/templates/deployment.yaml | Deploys changedetection pod + mounts persistence volume (Recreate strategy). |
| charts/workloads/changedetection/Chart.yaml | Defines the custom changedetection Helm chart. |
| argocd/apps/workloads/appset.yaml | Adds the Argo CD ApplicationSet that instantiates these workload apps per cluster. |
💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.
Comment on lines
+16
to
+18
| - name: mealie | ||
| image: "{{ .Values.image.repository }}:{{ .Values.image.tag }}" | ||
| ports: |
Owner
Author
There was a problem hiding this comment.
Fixed, falls back to :latest when tag is empty.
Comment on lines
+18
to
+20
| - name: changedetection | ||
| image: "{{ .Values.image.repository }}:{{ .Values.image.tag }}" | ||
| ports: |
Comment on lines
+5
to
+8
| ingress: | ||
| enabled: false # routing via Envoy Gateway HTTPRoute | ||
|
|
||
| baseURL: "https://analytics.dieu.dev" |
Fall back to :latest when image.tag is empty, so mealie and changedetection can deploy before Kargo's first promotion sets a real tag. Add Recreate strategy to mealie, matching changedetection, to avoid multi-attach errors on a ReadWriteOnce PVC. Pin releaseName on the workloads appset for stable resource naming. Add plausible's missing HTTPRoute, it had no route into the cluster at all.
11 tasks
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Workload charts for mealie, plausible, and changedetection all use real, already-
pullable upstream images. Re-add portfolio ones once their app repo + CI + image exist.