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 .spec.ignoreMissingValuesFiles to HelmChartTemplate API #942

Merged

Conversation

isometry
Copy link
Contributor

This PR introduces the option to ignore missing values files referenced in generated HelmCharts. Full backward compatibility is maintained.

I can't see any meaningful surface to test, as this is simply exposing underlying functionality from the source-controller API.

Depends upon fluxcd/source-controller#1447

@isometry isometry force-pushed the feature/ignore-missing-values-files branch 2 times, most recently from 897e927 to 370be06 Compare April 18, 2024 14:59
@isometry isometry force-pushed the feature/ignore-missing-values-files branch 3 times, most recently from 2d3bb0c to 4d8e07b Compare May 2, 2024 13:12
@isometry isometry changed the title Add .spec.ignoreMissingValuesFiles to HelmChartTemplateSpec Add .spec.ignoreMissingValuesFiles to HelmChartTemplate API May 2, 2024
@isometry isometry force-pushed the feature/ignore-missing-values-files branch from 4d8e07b to 60cda7e Compare May 2, 2024 15:00
@stefanprodan stefanprodan changed the title Add .spec.ignoreMissingValuesFiles to HelmChartTemplate API Add .spec.ignoreMissingValuesFiles to HelmChartTemplate API May 2, 2024
@stefanprodan stefanprodan added area/helm Helm related issues and pull requests area/api API related issues and pull requests labels May 2, 2024
@isometry
Copy link
Contributor Author

isometry commented May 2, 2024

@stefanprodan : will you create a new release tag on github.com/fluxcd/source-controller/api (v1.3.0?) so that I can update go.mod and get the tests passing?

@isometry isometry force-pushed the feature/ignore-missing-values-files branch from 60cda7e to bca7983 Compare May 3, 2024 12:14
Copy link
Member

@stefanprodan stefanprodan left a comment

Choose a reason for hiding this comment

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

LGTM

Thanks @isometry 🎖️

Copy link
Member

@souleb souleb left a comment

Choose a reason for hiding this comment

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

LGTM!

@stefanprodan stefanprodan merged commit 739d0a9 into fluxcd:main May 3, 2024
6 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
area/api API related issues and pull requests area/helm Helm related issues and pull requests
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants