Nish Krishnan

Results 18 comments of Nish Krishnan

Yeah i'd be in favor of this, been thinking about this for a while, but didn't have the bandwidth to look into it and see whether it's something the community...

Commented on #932. But I'm not inclined to support this. Autoplan should be a default and likely will be in a future release (when we tackle some of the cleanup...

Hmmm, this doesn't happen for us. It happens almost immediately. We have Atlantis deployed as a statefulset. Can you pin point where exactly it's getting stuck in the code?

We have metrics support in our fork, however it uses statsd in the form of github.com/lyft/gostats. Here is the commit: https://github.com/lyft/atlantis/commit/37c200fee9f3dcd30471a99045a87e9ec902b275 If theres enough likes on this, seeing as it's...

yeah will do, sorry about that i must have missed all this stuff.

https://github.com/runatlantis/atlantis/pull/2147

if you use [extra-args](https://www.runatlantis.io/docs/repo-level-atlantis-yaml.html#adding-extra-arguments-to-terraform-commands) it'll use the version you specify in your atlantis.yaml. referencing the binary directly in a custom run command doesn't work if you're using the non-default. We...

Since it's a custom run step it doesn't get formatted since we don't know whether it's a plan or not. https://github.com/runatlantis/atlantis/blob/master/server/events/runtime/plan_step_runner.go#L226 ^the above happens for a plan step specifically.

So the workaround for this is to send a passing status for the apply check? Have we considered running the following logic as a fallback to checking PR mergeability: 1....

This was our work around since we have an org wide status check we wanted atlantis to ignore in addition to atlantis/apply: https://github.com/lyft/atlantis/blob/release-v0.17.0-beta-lyft.1/server/events/vcs/github_client.go#L296 If this gets enough likes I can...