Yurii Matsiuk

Results 12 comments of Yurii Matsiuk

I'm using this PR as a daily driver for a month and it works perfectly, however I don't want to add pressure, just saying - it works 😉 Thanks to...

> Just a small suggestion. I think: 1. serve CLI command should start the connection in the background 2. And there should be another stop CLI command to disconnect from...

AFAIK the [update](https://github.com/DataDog/datadog-operator/blob/c32af591bc2b96dd956a2e56a55ae81b4715fe14/controllers/datadogmonitor/controller.go#L157) is only triggered if hashed spec of the monitor [differs](https://github.com/DataDog/datadog-operator/blob/c32af591bc2b96dd956a2e56a55ae81b4715fe14/controllers/datadogmonitor/controller.go#L126) from the hash in status. Nowhere during the reconciliation loop I see the monitors from DD are...

alternative option might be `--disable-absolute-path` argument that changes behavior of the template

Another way (might be more efficient one) that came to my mind recently is to extend [getExecutable](https://github.com/synfinatic/aws-sso-cli/blob/5cad7e52e06f3f0a94b87a6829d2d49ee490a7ea/sso/settings.go#L414) function to handle nix store paths, so we can leave the rest of...

@synfinatic thanks a lot, this is exactly what I was asking for!

Is that ok that the issue is closed before the change is merged?

Result of `nixpkgs-review pr 192706` run on x86_64-linux [1](https://github.com/Mic92/nixpkgs-review) 1 package built: appgate-sdp

Reproducible on: ``` Server: Engine: Version: 17.12.0-ce API version: 1.35 (minimum version 1.12) Go version: go1.9.2 Git commit: c97c6d6 Built: Wed Dec 27 20:09:53 2017 OS/Arch: linux/amd64 Experimental: false ```...

The same issue with `argo-cd` chart: ``` Terraform will perform the following actions: # module..helm_release.argo-cd[0] will be updated in-place ~ resource "helm_release" "argo-cd" { id = "argo-cd" ~ manifest =...