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 args flag removal to 1.0 proposal #2150

Closed

Conversation

mumoshu
Copy link
Collaborator

@mumoshu mumoshu commented Jun 8, 2022

THIS REPOSITORY HAS BEEN MOVED.
Please see #2148 and submit your pull request to the new repository!

mumoshu and others added 30 commits April 3, 2022 11:18
All the credits goes to @itscaro's work at roboll/helmfile#2084. The only change I made is related to the main branch rename.
Add an issue form for bug report
remove ioutil usage in all project
add unittest for formatters.go
[test] update helm 3.8.0 to 3.8.1
bump go 1.18 & remove .circleci
rodrigodc07 and others added 29 commits May 30, 2022 19:03
Update Helm diff plugin to version 3.5.0 in Dockerfile
Bumps [github.com/imdario/mergo](https://github.com/imdario/mergo) from 0.3.12 to 0.3.13.
- [Release notes](https://github.com/imdario/mergo/releases)
- [Commits](darccio/mergo@0.3.12...v0.3.13)

---
updated-dependencies:
- dependency-name: github.com/imdario/mergo
  dependency-type: direct:production
  update-type: version-update:semver-patch
...

Signed-off-by: dependabot[bot] <[email protected]>
…m/imdario/mergo-0.3.13

Bump github.com/imdario/mergo from 0.3.12 to 0.3.13
Signed-off-by: yxxhero <[email protected]>
Signed-off-by: yxxhero <[email protected]>
* Adds feature to fetch environment values from remote

The releases and environment section allow for values files on the local
disk.
This enhancement allows for referencing remote (go-getter) files to be
fetched, cached and referenced.

In addition when fetching a remote git source with a ssh key the ssh key
will not be part of the caching folder name. This avoids two problems:
1. Don't leak sensitive information in the name of the caching folder
2. Base64 encoded SSH keys are very long. On some file systems the max
lenght of the directory name is hit when using the full base64
information in the path name.

The sshkey informations are reducted. Because of this fixed string
there is a change of colloding cache names. The likelihood of this
collision is very low. The git repo and git reference need to be the
same, but the sshkey can change. This will result in the same source to
be checkout out and referenced.

Signed-off-by: Lüchinger Dominic <[email protected]>

* Update pkg/state/storage.go

Co-authored-by: Yusuke Kuoka <[email protected]>
…tions (#1)

* introduce DISABLE_INSECURE_FEATURES to disable insecure executions

Signed-off-by: Quan TRAN <[email protected]>

* disable remote sources when DISABLE_INSECURE_FEATURES is set to "true"

Signed-off-by: Quan TRAN <[email protected]>

* refactor envvar package

Signed-off-by: Quan TRAN <[email protected]>

* (test) fix test fixtures

Signed-off-by: Quan TRAN <[email protected]>

* use absolute path to avoid unit test failure

Signed-off-by: Quan TRAN <[email protected]>

* Fix conflicts

Co-authored-by: Yusuke Kuoka <[email protected]>
* chore(state): Append Api Versions flags to helm-diff

Signed-off-by: Sergio Morales <[email protected]>

* chore(state): Added unit test to state.flagsForDiff

Signed-off-by: Sergio Morales <[email protected]>

* chore(state): Check Helmstate values if ReleaseSpec values aren't set

Signed-off-by: Sergio Morales <[email protected]>
Signed-off-by: yxxhero <[email protected]>
I've asked a few times by @yxxhero if he deserves a seat as a maintainer of the Helmfile project.
Seeing the amount and the number of his recent contributions, and the nature of his contributions (add tests and fix bugs and CI issues, review PRs), I think he is in a relatively unique position compared to existing maintainers, and we can work better together!

Signed-off-by: Yusuke Kuoka <[email protected]>
It turned out we lack some important information that is necessary to make it a real bug without guesswork.

Signed-off-by: Yusuke Kuoka <[email protected]>
Probably this is the best we can currently offer. Any suggestions are welcomed though.

Ref roboll/helmfile#2147

Signed-off-by: Yusuke Kuoka <[email protected]>
Signed-off-by: Stoned Elipot <[email protected]>
Signed-off-by: Stoned Elipot <[email protected]>
Minor documentation fixes
Add proposals about two breaking changes before we cut Helmfile 1.0
Fix deprecated chart in Getting Started
@mumoshu mumoshu closed this Jun 8, 2022
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.