prototype: migrate command for terraform stacks#36461
prototype: migrate command for terraform stacks#36461liamcervante wants to merge 1 commit intomainfrom
Conversation
liamcervante
added
the
no-changelog-needed
liamcervante
force-pushed
the
liamcervante/tfstacksmigrate
branch
from
50223a0 to
a6794d7
Compare
liamcervante
force-pushed
the
liamcervante/tfstacksmigrate
branch
from
a6794d7 to
f977c14
Compare
| targetStackAddress := append(current.ConfigAddr(), stackaddrs.StackStep(ref)) | ||
| stack := m.Migration.Config.Stack(targetStackAddress) | ||
|
|
||
| // we have the configurations for the components in this stack, we just | ||
| // need to scope them down to the components that are in the current | ||
| // stack instance. | ||
|
|
||
| for name := range stack.Components { | ||
| configComponentAddress := stackaddrs.ConfigComponent{ |
There was a problem hiding this comment.
It doesn't hurt to leave the code here, but just to validate that my understanding is correct. I don't think this part is actually necessary, especially since we are keeping the tool to just root-level migration for now, i.e no migration into components embedded in child stacks, which is what a StackCall would be
There was a problem hiding this comment.
yes, the mapping doesn't actually have a way of copying data into embedded stacks so this shouldn't ever be triggered
|
I'm going to lock this pull request because it has been closed for 30 days ⏳. This helps our maintainers find and focus on the active contributions. |
2 participants
This PR adds a proof of concept for migrating state between Terraform workspaces and Terraform stacks.