helm-secrets icon indicating copy to clipboard operation
helm-secrets copied to clipboard

Fix template command to not print list of deleted files

Open jimeh opened this issue 4 years ago • 9 comments

For all wrapped Helm commands the list of decrypted files is printed as they get removed at the end of plugin execution. For the template command this is undesired, as it means you cannot simply run helm secrets template and get valid YAML output.

This change suppresses the output of deleted files for the template command only, meaning it does not alter the behavior of other commands, but just fixes the template command to work as I imagine most people would expect it to work.

Fixes issues #126 and #132.

jimeh avatar Sep 23 '19 12:09 jimeh

Can deletion output go to stderr instead? In either case this should be merged

dev-kpyc avatar Oct 08 '19 17:10 dev-kpyc

@dev-kpyc I'd say changes to the delete command's output should probably be a separate PR :)

jimeh avatar Oct 10 '19 11:10 jimeh

@szibis

dev-kpyc avatar Oct 25 '19 22:10 dev-kpyc

Bump. Why has this not been approved yet? 👀

lodotek avatar Feb 17 '20 17:02 lodotek

Bump because it's been 5 months since this was pulled.

@mateuszzawisza can you help review this 🙏

rkt2spc avatar Mar 23 '20 16:03 rkt2spc

Any news? I've used the forked plugin over @jimeh repo and it seems to work fine. What are the blockers?

rkt2spc avatar Aug 17 '20 21:08 rkt2spc

When you get tired of an issue and go to fix it yourself, only to find that a former teammate of yours has already raised a PR for this 14 months ago that still hasn't been reviewed. 😂

jimbali avatar Nov 24 '20 10:11 jimbali

Hi @kaarolch, you were the last person to merge a PR. Are you available to review this please?

jimbali avatar Dec 02 '20 10:12 jimbali

@jimbali Use this fork instead https://github.com/jkroepke/helm-secrets (I'm using this on our production CD for 3 months already, no problem so far) this repo is dead

rkt2spc avatar Jan 05 '21 02:01 rkt2spc