search

dineshba / tf-summarize

A command-line utility to print the summary of the terraform plan
MIT License
534 stars 38 forks source link

print change summary as json #22

Closed cgroschupp closed 1 month ago

cgroschupp commented 1 year ago

Maybe this is the beginning of this TODO

dineshba commented 1 year ago

Hi @cgroschupp Thanks for the PR

Sorry, I was not very descriptive in the TODO list. What I meant there is We have to show the current terraform state summary. Like what are the resources already created (whatever available in terraform state). And show them in formats like tree/drawable-tree/table

User has to run terraform plan before running tf-summarize. Plan itself will summary count in the last line, which is same as json-sum. Could you please help me in understanding the usecase of the json-sum output.

dineshba commented 1 year ago

@cgroschupp Please share your thoughts

Terraformadillo commented 1 month ago

Is this PR ever planning to get merged?

cgroschupp commented 1 month ago

@dineshba My use case behind the json-sum was to get the change result in structured form (json) and extract information with jq. And my pipeline reacts differently to this result.

@Terraformadillo yes, ready to merge

dineshba commented 1 month ago

Could you help in test fixing ? @cgroschupp

Looks we added new tests which is failing now... we might have to update the error message... I can also look into it if you are packed

cgroschupp commented 1 month ago

Now the tests should be OK.

dineshba commented 1 month ago

Thanks for the PR... Merged and release it now