Aaron Stillwell
Aaron Stillwell
Thanks for your contribution @lipelix! I've never used hashicorp cloud so am unfamiliar with the constraints of operating terraform in their environment. Happy to provide a configuration option to disable...
Yep @pawelad I can investigate a patch update `v0.4.4` including tests against dokku `v0.30.x` and this additional config option
@pawelad please consider using the new config option introduced in `v0.4.4` https://github.com/aaronstillwell/terraform-provider-dokku/releases/tag/v0.4.4 I'm going to leave this issue open for feedback. I do not use terraform cloud and didn't see...
@lipelix any insight you can share with us here?
Came across this after coming back to this project for the first time in a long time, created #16
@jyoost an implementation would be fine without IMO. I already drafted an initial implementation of this feature which ran into a problem or two... I'll look back at what those...
Thanks for your submission @jyoost! LGTM so far (sans acceptance test). This TODO can also be removed 😄 https://github.com/aaronstillwell/terraform-provider-dokku/blob/a7a4f0c465aa7afdfc4f427db33c4ce4266a6d6c/internal/provider/dokku.go#L96
Note no reason for a separate PR - you should be able to just push the test as an additional commit.
@jyoost yes see the `TF_LOG` environment variable. https://www.terraform.io/cli/config/environment-variables The run command automatically logs all dokku commands sent to the server. https://github.com/aaronstillwell/terraform-provider-dokku/blob/48f0f6827a5192e744556011c3bd7e6e4080c0c3/internal/provider/ssh.go#L25-L56
@jyoost a further good starting point for this issue would be to verify the problem exists by updating the following acceptance test to include a config var with a quote....