mirror of
https://github.com/patrickchugh/terravision.git
synced 2026-05-09 08:25:25 +02:00
[GH-ISSUE #162] GraphMaker: AttributeError: 'int' object has no attribute 'replace' #92
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#92
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 @2k-joker on GitHub (Jan 24, 2026).
Original GitHub issue: https://github.com/patrickchugh/terravision/issues/162
Problem
These two lines of code are making an incorrect assumption of always having to operate on a string literal
This assumption results in an unhandled exception when a non-string is encountered. Sample stacktrace;
Potential Fix
A type check or a cast to string somewhere up the call chain is needed
@patrickchugh commented on GitHub (Jan 24, 2026):
Hi @2k-joker Thanks for bringing this to my attention. I have implemented a new fix for this and refactored the function in question. Can you try again and let me know if I can close this issue now?
@2k-joker commented on GitHub (Jan 24, 2026):
This particular error is now fix. Thanks @patrickchugh
However, I am running into a different unrelated issue where my regular
terraform planruns successfully, but when I run terravision, I get this:It seems to be opinionated about certain things?
Here's my regular terraform plan output:
@patrickchugh commented on GitHub (Jan 25, 2026):
I have not seen this type of error reported before. But it is definitely a Terraform code issue not a TerraVision bug. Your current terraform plan from the command line is most likely not evaluating the line in question since the resource doesn't need to be refreshed. If you force a local backend, using override.tf like TerraVision does, or try a terraform plan on an account with no existing resources (blank state file) then you will most likely get that error as well. Give it a try and let me know. Closing this for now.