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

Allow helmfile statefile to passed by stdin #520

Merged
merged 5 commits into from
Nov 22, 2022
Merged

Conversation

xiaomudk
Copy link
Contributor

fix: #505

example:

cat examples/helmfile.yaml | helmfile -f - template

Signed-off-by: xiaomudk xiaomudk@gmail.com

@yxxhero
Copy link
Member

yxxhero commented Nov 13, 2022

@xiaomudk ci issue.

pkg/filesystem/fs_test.go Outdated Show resolved Hide resolved
@yxxhero
Copy link
Member

yxxhero commented Nov 14, 2022

@xiaomudk add a e2e test will be better.

@mumoshu
Copy link
Contributor

mumoshu commented Nov 14, 2022

Yeah, this is worth being covered in an E2E test!

@xiaomudk xiaomudk force-pushed the patch-7 branch 2 times, most recently from a0527d3 to cd449aa Compare November 14, 2022 07:42
@xiaomudk xiaomudk force-pushed the patch-7 branch 7 times, most recently from c19e3f8 to bfc1e49 Compare November 14, 2022 11:04
@xiaomudk
Copy link
Contributor Author

@yxxhero @mumoshu done.

pkg/filesystem/fs_test.go Outdated Show resolved Hide resolved
pkg/filesystem/fs_test.go Outdated Show resolved Hide resolved
Signed-off-by: xiaomudk <xiaomudk@gmail.com>
@xiaomudk
Copy link
Contributor Author

@yxxhero @mumoshu Any more questions?

@yxxhero
Copy link
Member

yxxhero commented Nov 16, 2022

@xiaomudk I have no questions.

@yxxhero
Copy link
Member

yxxhero commented Nov 16, 2022

@mumoshu @itscaro ping

@xiaomudk
Copy link
Contributor Author

@mumoshu ping

docs/paths.md Outdated Show resolved Hide resolved
Signed-off-by: Yusuke Kuoka <ykuoka@gmail.com>
docs/index.md Outdated Show resolved Hide resolved
Signed-off-by: Yusuke Kuoka <ykuoka@gmail.com>
cmd/root.go Outdated Show resolved Hide resolved
docs/index.md Outdated Show resolved Hide resolved
Signed-off-by: Yusuke Kuoka <ykuoka@gmail.com>
Signed-off-by: Yusuke Kuoka <ykuoka@gmail.com>
Copy link
Contributor

@mumoshu mumoshu left a comment

Choose a reason for hiding this comment

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

LGTM overall! Let me add minor updates on the doc though. Thanks for your contribution ☺️

@xiaomudk
Copy link
Contributor Author

@mumoshu Thank you so much, you are such a helpful person!

@yxxhero yxxhero merged commit 7844145 into helmfile:main Nov 22, 2022
@github-actions github-actions bot locked as resolved and limited conversation to collaborators Nov 24, 2022
@xiaomudk xiaomudk deleted the patch-7 branch January 1, 2023 09:06
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Allow helmfile statefile to passed by stdin
3 participants