feat: Transfer charts using Docker Image#217
Open
pranavcracker wants to merge 2 commits intomainfrom
Open
Conversation
Collaborator
|
Why do we need to add the following part:
|
alexarefev
approved these changes
May 6, 2026
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.
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.yamlGitHub 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.