mirror of
https://github.com/patrickchugh/terravision.git
synced 2026-05-09 08:25:25 +02:00
[GH-ISSUE #66] terravision CLI hangs on last resource #32
Labels
No labels
bug
enhancement
enhancement
good first issue
good first issue
good first issue
pipeline
pull-request
No milestone
No project
No assignees
1 participant
Notifications
Due date
No due date set.
Dependencies
No dependencies set.
Reference
starred/terravision#32
Loading…
Add table
Add a link
Reference in a new issue
No description provided.
Delete branch "%!s()"
Deleting a branch is permanent. Although the deleted branch may continue to exist for a short time before it actually gets removed, it CANNOT be undone in most cases. Continue?
Originally created by @dnk8n on GitHub (Oct 28, 2023).
Original GitHub issue: https://github.com/patrickchugh/terravision/issues/66
CLI output hung, in case template is useful for debugging:
terraform.tfvars
@dnk8n commented on GitHub (Oct 28, 2023):
Command I ran:
poetry run terravision draw --source ~/src/dtrack --varfile ~/src/dtrack/terraform.tfvarsMy setup/install procedure:
@maker2413 commented on GitHub (Oct 30, 2023):
I'm also encountering this. Did you find a way to get around this issue?
@patrickchugh commented on GitHub (Nov 21, 2023):
@dnk8n @maker2413 Hello, thank you for your feedback and thanks for testing out Terravision and helping to make this a better product. I have recently completed a major revamp to the code which uses the terraform binary to download source files and generate initial relationships, before augmenting it with source code parsing as before. This is a significant departure from the previous approach and whilst slower, ensures a more accurate output and will handle all terraform in built functions now when encountered in the source. Please do a git pull and re-test again with the latest source code. If your problem is resolved, please close the the thread in github issues and mark it as completed. If not, please give as much detail as you can about the errors and include any source files if possible so I can reproduce your issue. Once again, thanks for helping and look forward to hearing from you soon.
P.
@patrickchugh commented on GitHub (Nov 29, 2023):
can this issue be closed now?
@dnk8n commented on GitHub (Nov 29, 2023):
I plan to be able to give feedback this week. It’s on my task list. Please
understand that terravision is not right at the top. Anyone else able to
validate and report their experience?
On Wed, 29 Nov 2023 at 11:31, Patrick Chugh @.***>
wrote:
@dnk8n commented on GitHub (Nov 30, 2023):
I had a go with the new code, thanks for your patience... I understand how exciting it is to roll out a new feature and how frustrating it is waiting for feedback!
First, my project used a backend conf for terraform init, so I needed to hardcode that for this exercise - logged an issue here: https://github.com/patrickchugh/terravision/issues/85
Then when reinstalling from scratch, I needed to manually install debugpy before I could continue.
Now that I am properly installed, I could run:
It did all the right things until the end:
Even changing current directory to where my main.tf lives didn't produce a different result.
@patrickchugh commented on GitHub (Dec 2, 2023):
@dnk8n Thanks for the feedback and patience. I have just made some bugfixes to this. Can you please try to run with latest code?
Working on the terraform init part separately
@dnk8n commented on GitHub (Dec 3, 2023):
It gets this far, and then hangs:
Not sure how it found
Parsing /home/dean/src/dtrack/terraform.tfvars.tpl@dnk8n commented on GitHub (Dec 3, 2023):
Deleting the .tpl file and rerunning doesn't change the result.
@dnk8n commented on GitHub (Dec 3, 2023):
How long is the process expected to take. Unfortunately in South Africa we have electricity load shedding, and my laptop battery is about to die. Will try again when back online.
@dnk8n commented on GitHub (Dec 3, 2023):
Seems to just hang. 37min and still going (using 100% of one thread) ...

@patrickchugh commented on GitHub (Dec 20, 2023):
@dnk8n Would you have time to git pull the latest code and try again?
@dnk8n commented on GitHub (Dec 21, 2023):
Looks much better! I think enough to close this issue.
Maybe you know better than me @patrickchugh if it is showing all the available AWS resources (compare the dictionaries to the image). Seems to be missing detail?
FYI:
@patrickchugh commented on GitHub (Dec 28, 2023):
@dnk8n I found there are a few glitches in the diagram you got. If you can open a new issue regarding this bug handling security groups and share your code I can make it output the correct image similar to this one attached:
