Commit Graph

1318 Commits

Author SHA1 Message Date
Yusuke Kuoka f8f50c8bcd
Merge pull request #45 from yxxhero/remove_gotest
remove gotest deps
2022-04-19 09:24:46 +09:00
yxxhero a0d4047cc7 remove gotest deps
Signed-off-by: yxxhero <aiopsclub@163.com>
2022-04-19 08:02:21 +08:00
Yusuke Kuoka 9e9567ed7d
Merge pull request #44 from yxxhero/add_unittest_for_constants
add unittest for constants.go
2022-04-19 08:51:41 +09:00
yxxhero b895f75198 add unittest for constants.go
Signed-off-by: yxxhero <aiopsclub@163.com>
2022-04-18 17:15:04 +08:00
Yusuke Kuoka 21ab18b906
Merge pull request #39 from itscaro/qtran/helm382
(test) update helm to 3.8.2
2022-04-15 07:25:09 +09:00
Yusuke Kuoka 42ad7beb6e
Merge pull request #24 from helmfile/auto-detect-term-for-color-diff
feat: Auto-detect term for coloring helm-diff output
2022-04-15 07:24:42 +09:00
Yusuke Kuoka a4542265ca
Merge pull request #23 from helmfile/fix-helmfile-deps-chartify
Fix helmfile deps not to remove entries for charts that are being chartified
2022-04-15 07:24:20 +09:00
Quan TRAN dad58533b5 (test) update helm to 3.8.2
Signed-off-by: Quan TRAN <account@itscaro.me>
2022-04-14 10:13:24 +02:00
Quan TRAN 86c9a1ae3d
Fix canary image publication 2022-04-14 10:04:26 +02:00
Yusuke Kuoka f84ab66ba0 Make --color and --no-color exclusive
Signed-off-by: Yusuke Kuoka <ykuoka@gmail.com>
2022-04-13 23:43:41 +00:00
Yusuke Kuoka 47e13fcdad
Merge pull request #34 from dol/feature/redacted-url-in-chart-name
Redact sensitive informations like password in chart url
2022-04-14 08:28:25 +09:00
Yusuke Kuoka 7af57add9d
Merge pull request #37 from itscaro/qtran/fix-temp-dir
Fix non-existent custom temp dir
2022-04-14 08:27:41 +09:00
Yusuke Kuoka d0841c72aa
Merge pull request #22 from itscaro/qtran/issue-2118
(fix) chartify process when chart path is absolute
2022-04-14 08:25:13 +09:00
Yusuke Kuoka b5480123a2
Merge pull request #31 from yxxhero/fix_outout_dir
fix output-dir issue
2022-04-14 08:20:14 +09:00
Lüchinger Dominic f89234e5dc
Redact sensitive informations like password in chart url
Signed-off-by: Lüchinger Dominic <dev@snowgarden.ch>
2022-04-13 00:47:32 +02:00
Quan TRAN 12e103d510 (fix) non-existent custom temp dir
Signed-off-by: Quan TRAN <account@itscaro.me>
2022-04-12 22:27:59 +02:00
yxxhero ec648c4011 use os.PathSeparator replace /
Signed-off-by: yxxhero <aiopsclub@163.com>
2022-04-12 13:13:53 +08:00
Quan TRAN c7b23a67cb
Merge pull request #32 from yxxhero/fix_typo_state
(fix) typo in state.go
2022-04-11 23:47:59 +02:00
Quan TRAN 3cae3f4f89 add integration test
Signed-off-by: Quan TRAN <account@itscaro.me>
2022-04-11 23:41:28 +02:00
Quan TRAN 88127d641b [fix] chartify process when chart path is absolute
Signed-off-by: Quan TRAN <account@itscaro.me>
2022-04-11 23:34:52 +02:00
yxxhero 9c0c021dc9 fix typo in state.go
Signed-off-by: yxxhero <aiopsclub@163.com>
2022-04-11 23:33:25 +02:00
yxxhero 7743f317d1 fix output-dir issue
Signed-off-by: yxxhero <aiopsclub@163.com>
2022-04-11 12:57:09 +08:00
Yusuke Kuoka f1a250cab3
Merge pull request #28 from itscaro/qtran/image
[Dockerfile] only use image alpine for compilation, add sops & age to pre-built image, remove helm v2 image
2022-04-08 11:20:05 +09:00
Yusuke Kuoka 89a00ca3e3
Merge pull request #25 from helmfile/bump-go-getter
Bump go-getter to 1.5.11
2022-04-08 11:18:20 +09:00
Yusuke Kuoka b672af6c06 Bump go-getter to 1.5.11
Ref https://github.com/roboll/helmfile/issues/2106

Signed-off-by: Yusuke Kuoka <ykuoka@gmail.com>
2022-04-08 02:03:40 +00:00
Quan TRAN a4555f34ff test build image without publication
Signed-off-by: Quan TRAN <account@itscaro.me>
2022-04-06 11:30:10 +02:00
Quan TRAN 570a2c22e9 [Dockerfile] only use image alpine for compilation, add sops & age to pre-built image, remove helm v2 image
Signed-off-by: Quan TRAN <account@itscaro.me>
2022-04-06 10:59:24 +02:00
Yusuke Kuoka 97e0ca73ca feat: Auto-detect term for coloring helm-diff output
Since helm-diff has added an ability to auto-detect the term to decide if it should output with color or not, helmfile had been defaulted to no-color.
This resoloves that, by adding a term-detection logic that is same as helm-diff.

As a part of this work, I have also implemented a new global flag `--color`, which is used for forcing color without relying on the term-detection logic implemented in helmfile or explicitly setting the HELM_DIFF_COLOR envvar. I hope it is useful for folks.

Ref https://github.com/roboll/helmfile/issues/2043

Signed-off-by: Yusuke Kuoka <ykuoka@gmail.com>
2022-04-06 02:38:36 +00:00
Yusuke Kuoka 8e50dbc46d Fix helmfile deps not to remove entries for charts that are being chartified
When chartify is involved due to the use of `forceNamespace`, `strategicMergePatches`, `jsonPatches`, and so on, We had been internally mutating the Release.Chart with the path to the local temporary directory that contains the modified version of the chart.

This resulted in us unintentionally making `helmfile deps` to remove entries for the chart being modified out of helmfile.lock file, which resulted in issues like #2110.

To be clear, although the original issue is reported to occur for `strategicMergePatches`, I believe that it occurered also for any remote charts using `jsonPatches` and `forceNamespace` too.

I also believe this has been the issue since our introduction of chartify (maybe a year or so ago??), and I guess why it took so much time to be found and reported is that not so many people with chartify in combination with `helmfile deps` 🤔

Lastly, this changes chart names surfaced in the various log output from Helmfile, from temporary chart paths to the chart name/path declared in the helmfile.yaml. I think this is generally a good change, no fear of being a breaking change. But if anyone has any concern about that, please feel free to comment/report/etc.

Ref https://github.com/roboll/helmfile/issues/2110

Signed-off-by: Yusuke Kuoka <ykuoka@gmail.com>
2022-04-06 02:12:08 +00:00
Yusuke Kuoka 8fb418e3c9
Merge pull request #21 from itscaro/qtran.fix-golang1.18
[fix] bump go 1.18 & optional TAG definition in makefile
2022-04-06 08:39:14 +09:00
Yusuke Kuoka f49d181b41
Merge pull request #20 from itscaro/qtran/fix-decrypted-file
[fix] unclosed file stream when creating temporary decrypted files
2022-04-06 08:37:13 +09:00
Yusuke Kuoka f7eb4a9e3e
Merge pull request #19 from itscaro/qtran/badge
Update badge to reflect Github Action status
2022-04-06 08:36:23 +09:00
Yusuke Kuoka 3f1d01a688
Merge pull request #18 from yxxhero/add_unittest_RequiredEnv
add unittest for RequiredEnv func
2022-04-06 08:35:26 +09:00
Yusuke Kuoka e8212276d5
Merge pull request #17 from yxxhero/update_args_unitest
update args_test.go unittest
2022-04-06 08:32:35 +09:00
Quan TRAN 03cbf5f9f4 [fix] bump go 1.18 & optional TAG definition in makefile
Signed-off-by: Quan TRAN <account@itscaro.me>
2022-04-06 00:35:23 +02:00
Quan TRAN a793797404 [fix] unclosed file stream when creating temporary decrypted files
Signed-off-by: Quan TRAN <account@itscaro.me>
2022-04-05 22:52:17 +02:00
Quan TRAN eed489efc1 Update badge to reflect Github Action status
Signed-off-by: Quan TRAN <account@itscaro.me>
2022-04-05 21:16:52 +02:00
yxxhero 5cb82ec3d1 add unittest for RequiredEnv func
Signed-off-by: yxxhero <aiopsclub@163.com>
2022-04-05 22:53:07 +08:00
Yusuke Kuoka 4247b97eb1
Merge pull request #2 from itscaro/qtran/go1.18
bump go 1.18 & remove .circleci
2022-04-05 08:51:15 +09:00
Yusuke Kuoka 4480ec2cdc
Merge pull request #11 from yxxhero/add_unittest_for_apptest
add unittest for load_opts.go
2022-04-05 08:50:11 +09:00
Yusuke Kuoka 7cc9778a13
Merge pull request #16 from itscaro/qtran/helm-3.8.1
[test] update helm 3.8.0 to 3.8.1
2022-04-05 08:49:38 +09:00
yxxhero 8e5c779626 update args_test.go unittest
Signed-off-by: yxxhero <aiopsclub@163.com>
2022-04-05 06:26:42 +08:00
Quan TRAN b2798315b1 [test] update helm 3.8.0 to 3.8.1
Signed-off-by: Quan TRAN <account@itscaro.me>
2022-04-04 16:39:06 +02:00
Quan TRAN a1dfcb3480 bump go 1.18 & remove .circleci
Signed-off-by: Quan TRAN <account@itscaro.me>
2022-04-04 16:33:48 +02:00
yxxhero 02058cc1f5 add unittest for load_opts.go
Signed-off-by: yxxhero <aiopsclub@163.com>
2022-04-04 17:20:54 +08:00
Yusuke Kuoka 4a52ce0ac2
Merge pull request #13 from yxxhero/mv_captureStdout_as_public_func
mv captureStdout as public func
2022-04-04 10:06:03 +09:00
yxxhero e0244bc341 mv captureStdout as public func
Signed-off-by: yxxhero <aiopsclub@163.com>
2022-04-03 22:24:38 +08:00
Yusuke Kuoka 01b5b79a01
Merge pull request #9 from yxxhero/unit_formatters
add unittest for formatters.go
2022-04-03 19:22:32 +09:00
yxxhero 7f70138b5f add unit for formatters.go
Signed-off-by: yxxhero <aiopsclub@163.com>
2022-04-03 17:31:59 +08:00
Yusuke Kuoka ed27147cac Add more links to the new-issue page 2022-04-03 08:35:22 +00:00