John Eikenberry

Consider using https://pkg.go.dev/golang.org/x/sync/errgroup along with Run(ctx) to wait for all of them to shutdown. The Group.Go method will generally call the Run(ctx) .

Hey @mkam, I assigned myself as a reviewer of this as I have additional context (was working with @angrycub with a similar PR for consul-template). So feel free to skip...

Didn't we decide to remove the PaloAlto/PanOS support in favor of the more general post update triggers (or whatever we called them)? Unless that changed this probably should be removed,...

Consul-template re-uses it's child process running sub-module for this. It is most likely overkill for this case and I'd probably not recommend it (if you were even thinking of this)....

Thanks for the suggestion @aphorise, but getting it to work is not the problem (ie. I can work around it). The main problems is that GOPATH is a colon separated...

If you have GOPATH set, you can set GOBIN based on GOPATH with any POSIX compatible shell with: `GOBIN=${GOPATH%%:*}` (that strips off everything after the first path delimiter `:`). Once...

Hey @dizeee, thanks for reporting this. The k8ts auth support was added in a PR and I'm hoping the people who contributed might have an idea. @Kryvchun, @ygworldr... any thoughts?

Re-reading this I'm wondering if this could be related to #310? That bug would impact this type of auth as well. I'm going to merge the fix for that today....

Thanks for the followup @dizeee. > The only difference is that now envconsul doesn't reload static secrets when they are changed [..] Sorry, being a bit dense... Would you mind...

Thanks. Secrets stored using the KV values should always be monitored and kept up to date. The default is to check every 5 minutes.