-
Notifications
You must be signed in to change notification settings - Fork 113
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Support Terraform 0.12.x #101
Comments
What benefit do you see in landscape for 012?
The plan is so much more readable that I no longer see a need for it
…--
Fernando Miguel
On Tue, 11 Jun 2019, 08:18 Kohei Ota, ***@***.***> wrote:
The latest released version doesn't seem to work well with Terraform 0.12.x
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub
<#101?email_source=notifications&email_token=AABJDLVNIGYZAPLVDPQ6KGDPZ5GTBA5CNFSM4HW3HYW2YY3PNVWWK3TUL52HS4DFUVEXG43VMWVGG33NNVSW45C7NFSM4GYXNGHA>,
or mute the thread
<https://github.com/notifications/unsubscribe-auth/AABJDLQD4FTJ7TKAZOH5Y7DPZ5GTBANCNFSM4HW3HYWQ>
.
|
It's true the 012 plan is much more readable but way too verbose. However Landscape is doing a great job of just output what is actually needed in most case. I have in mind the infinite "refreshing state". |
I've been using landscape as a "simpler-terraform-plan-result" tool. |
@inductor you can easily "hide" the "refreshing state" with |
Completely agree with @remipichon . Terraform 0.12 is way verbose showing things that will remain the same with apply. |
Is this gonna happen or not? :P |
We would welcome a well-tested pull request that preserves functionality for users who are still on Terraform 0.11.x. Thanks! |
The v0.12 plan is definitely more readable, but I would agree with @remipichon that landscape does a better job of showing the actual diffs. It would be great if it could support v0.12! |
Landscape is still useful for diffing large blocks of text. For example when using the helm provider, you can pass in a yaml file. Landscape will pretty format it and show the relevant diff, while terraform 0.12 does not do that and that makes it difficult to see what changed. |
an improvement would be to add color to the entire element being created, destroy, or modified, instead of just the -/+/~ |
Terraform landscape output was really great and definitively more readable with colors and diff-like output. I would love to be able to use it for Terraform 0.12.x 👍 |
Same |
For things like ECS task definitions where you have many tasks in a single file ? That would be a good one :) |
for jsonencoded maps for ECS task defs in 0.12 is not showing the changes, but the replacement of the whole json :( it'd be nice to use the tool for things like that |
Would like this feature also. |
I, too, would love to see this. |
+1 |
I've locked this issue to prevent further "+1" comments that aren't helpful. Would welcome a pull request adding support for the 0.12 output syntax! |
The latest released version doesn't seem to work well with Terraform 0.12.x
The text was updated successfully, but these errors were encountered: