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

feat: impl patch with alpha plugin flag. #59

Merged

Conversation

Peefy
Copy link
Contributor

@Peefy Peefy commented Aug 1, 2023

Purpose

feat: impl patch with alpha plugin flag. More information is here: helmfile/helmfile#827

I found that the helmfile HelmState structure passed chartifyOpts.EnableKustomizeAlphaPlugins=true to chartify, but it did not take effect in chartify.

Tests

For transformer and alpha plugin, I have added a integration test using the AnnotationsTransformer kustomize plugin

  • integration_test.go

@Peefy
Copy link
Contributor Author

Peefy commented Aug 1, 2023

Thank you @yxxhero PTAL.

@Peefy Peefy force-pushed the feat-chartify-kustomize-patch-plugin-flag branch from e0841b0 to 4547d55 Compare August 1, 2023 13:06
kustomize.go Outdated Show resolved Hide resolved
@Peefy Peefy force-pushed the feat-chartify-kustomize-patch-plugin-flag branch from 4547d55 to 2cef52c Compare August 1, 2023 14:22
@yxxhero
Copy link
Member

yxxhero commented Aug 1, 2023

@Peefy please fix the ci error. then I will merge this PR.

@yxxhero yxxhero added the enhancement New feature or request label Aug 1, 2023
@yxxhero yxxhero self-assigned this Aug 1, 2023
@Peefy Peefy force-pushed the feat-chartify-kustomize-patch-plugin-flag branch from 2cef52c to feef492 Compare August 2, 2023 02:31
@Peefy
Copy link
Contributor Author

Peefy commented Aug 2, 2023

@Peefy please fix the ci error. then I will merge this PR.

@yxxhero Sorry, I forgot to submit the want file and I've added it.

@Peefy
Copy link
Contributor Author

Peefy commented Aug 2, 2023

@yxxhero yxxhero merged commit e649b1d into helmfile:master Aug 2, 2023
3 checks passed
@Peefy Peefy deleted the feat-chartify-kustomize-patch-plugin-flag branch August 2, 2023 05:29
yxxhero pushed a commit that referenced this pull request Sep 19, 2023
* feat: impl patch with alpha plugin flag

Signed-off-by: peefy <[email protected]>

* refactor: kustomize build arguments and remove duplicate code

Signed-off-by: peefy <[email protected]>

---------

Signed-off-by: peefy <[email protected]>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants