Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Add a helm golden record #2034

Merged
merged 10 commits into from
Jun 7, 2022
Merged

Add a helm golden record #2034

merged 10 commits into from
Jun 7, 2022

Conversation

Logiraptor
Copy link
Contributor

@Logiraptor Logiraptor commented Jun 6, 2022

What this PR does

Adds a compiled version of the helm chart, useful for validating Helm PRs and later for comparing jsonnet to helm.

The build script re-uses helm's chart-testing tool conventions. In other words, any installation tests we add to the ci directory will have golden records generated.

The diff is mostly from generated files, just the first four files are relevant:

.github/workflows/test-build-deploy.yml
docs/internal/contributing/contributing-to-helm-chart.md
Makefile
operations/helm/tests/build.sh

Which issue(s) this PR fixes or relates to

Fixes #2032

Checklist

  • Automate PR check for changed golden records
  • Tests updated
  • Documentation added
  • CHANGELOG.md updated - the order of entries should be [CHANGE], [FEATURE], [ENHANCEMENT], [BUGFIX]

@Logiraptor Logiraptor force-pushed the logiraptor/helm-golden-record branch from a429c2c to 245cb85 Compare June 6, 2022 17:55
@Logiraptor Logiraptor force-pushed the logiraptor/helm-golden-record branch from e528148 to 1f28881 Compare June 6, 2022 21:04
@Logiraptor Logiraptor marked this pull request as ready for review June 6, 2022 21:10
@Logiraptor Logiraptor requested a review from krajorama June 6, 2022 21:30
Set kubeVersionOverride to generate PodDisruptionBudget API version
consistently.

When I ran the test, I got a diff, because my k8s is newer (1.23).

Signed-off-by: György Krajcsovits <gyorgy.krajcsovits@grafana.com>
Copy link
Contributor

@dimitarvdimitrov dimitarvdimitrov left a comment

Choose a reason for hiding this comment

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

LGTM, this is great!

operations/helm/tests/build.sh Outdated Show resolved Hide resolved
Copy link
Contributor

@krajorama krajorama left a comment

Choose a reason for hiding this comment

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

LGTM, cool stuff, now I can more safely remove some redundant helm stuff and test config management well

Co-authored-by: Dimitar Dimitrov <dimitar.dimitrov@grafana.com>
@krajorama krajorama merged commit 84e4901 into main Jun 7, 2022
@krajorama krajorama deleted the logiraptor/helm-golden-record branch June 7, 2022 15:31
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.

Helm: Commit a "golden-record" of the chart output
3 participants