Skip to content

feat: Transfer charts using Docker Image#217

Open
pranavcracker wants to merge 2 commits intomainfrom
transfer-charts
Open

feat: Transfer charts using Docker Image#217
pranavcracker wants to merge 2 commits intomainfrom
transfer-charts

Conversation

@pranavcracker
Copy link
Copy Markdown
Collaborator

Problem

The Helm charts for site-manager live directly in this repository, which means every chart change requires a manual copy into downstream distribution repositories. There is no automated way for consumers to pull a specific version of the charts without cloning the full repository or relying on a manual sync step.

Solution

Introduce a dedicated sm-charts-transfer image built from a minimal FROM scratch Dockerfile. This image contains only the Helm chart directory and is published automatically via the existing publish.yaml GitHub Actions workflow on every push to main and on every release tag — no new workflows or manual steps are required. Downstream consumers can pull a specific chart version by referencing the corresponding image tag.

Changes Introduced

  • ci/docker-charts/Dockerfile — New FROM scratch image that packages charts/ into /charts inside the image, keeping the artifact size minimal.
  • .github/workflows/publish.yaml — Added sm-charts-transfer to the existing build matrix so the image is built and pushed automatically alongside all other component images.
  • charts/site-manager/values.yaml — Added missing default values: profile, MONITORING_ENABLED, DBAAS_ENABLED, and paasGeoMonitor.monitoring.serviceCIDRRegex.
  • charts/site-manager/templates/ — Added ServiceMonitor, grafana-dashboard, and paas-geo-monitor-service-monitor templates for Prometheus/Grafana monitoring integration.
  • charts/site-manager/resource-profiles/ — Added dev, small, medium, large, and prod resource profile definitions.

@alexarefev
Copy link
Copy Markdown
Collaborator

Why do we need to add the following part:

  • charts/site-manager/templates/ — Added ServiceMonitor, grafana-dashboard, and paas-geo-monitor-service-monitor templates for Prometheus/Grafana monitoring integration.

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.

3 participants