terraform: when pruning destroy, only match exact nodes, or exact counts #1527
Conversation
| if k == prefix+".0" { | ||
| continue | ||
| } | ||
|
|
||
| if v.Primary != nil { | ||
| println("FUCK: " + prefix + " ; " + k) |
There was a problem hiding this comment.
I prefer the lowercase version here, I think it's less "yelling" 👅
There was a problem hiding this comment.
Nothing to see here, just @mitchellh's Fast Understanding, Confusion Killing ™️ debugging system.
There was a problem hiding this comment.
I do this on the plane a lot and one time a lady looked over and was somewhat confused as to why "FUCK" was just scrolling matrix-style down my terminal.
Anyways, NSFW, will be more careful.
|
Ah.... b-sad-destroy indeed. Yeah I squinted at the prefix matching the first time around - shoulda asked. Welp it's all good now! 👍 |
terraform: when pruning destroy, only match exact nodes, or exact counts
|
I'm going to lock this issue because it has been closed for 30 days ⏳. This helps our maintainers find and focus on the active issues. If you have found a problem that seems similar to this, please open a new issue and complete the issue template so we can capture all the details necessary to investigate further. |
ghost
locked and limited conversation to collaborators
This fixes an issue where the Terraform prune destroy transform was invalidly keeping the destroy node around for things that didn't need it. Example: if you had a resource "foo" and "foo-bar" and "foo-bar" needed to be kept around, it'd also keep "foo" because of the way prefix checking was before.