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

fix-chart #7

Merged
merged 3 commits into from Dec 15, 2023
Merged

fix-chart #7

merged 3 commits into from Dec 15, 2023

Conversation

mathieu-benoit
Copy link
Contributor

@mathieu-benoit mathieu-benoit commented Dec 15, 2023

Issue with #4, my bad with my local tests.

helm template test workload --repo https://score-spec.github.io/score-helm-charts or helm template charts/workload returns this error:

Error: template: workload/templates/deployment.yaml:6:22: executing "workload/templates/deployment.yaml" at <.Values.workload.annotations>: nil pointer evaluating interface {}.annotations

Fix by safer tests in templating + proposing a smoke test in CI in order to catch such a trivial errors in the future.

Tested:

  • helm template charts/workload/
  • helm template test charts/workload/ --set workload.annotations.test=test
  • helm template test charts/workload/ --set service.type=LoadBalancer
  • helm template test charts/workload/ --set service.type=LoadBalancer --set service.annotations.test=test

I also re-packaged the chart with the same version, in order to fix that version out there.

Copy link
Member

@astromechza astromechza left a comment

Choose a reason for hiding this comment

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

Good test to have! (also this is why the values.yaml file is useful - it provides safe defaults)

I'm good with the change but I think it could be cleaner?

Signed-off-by: Mathieu Benoit <mathieu-benoit@hotmail.fr>
Signed-off-by: Mathieu Benoit <mathieu-benoit@hotmail.fr>
Signed-off-by: Mathieu Benoit <mathieu-benoit@hotmail.fr>
@astromechza astromechza merged commit a493c77 into score-spec:main Dec 15, 2023
1 check passed
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.

None yet

2 participants