-
Notifications
You must be signed in to change notification settings - Fork 415
download terraform.py using local_exec #1582
base: master
Are you sure you want to change the base?
Conversation
Is there anything else we need to do for this, or just testing? |
@sehqlr You tell me! This has passed it's CI builds, so I'm not sure it even needs testing, just merging. |
@ryane: What do you think? Can we include this in 1.3? |
I'm just wondering if we need a little documentation around upgrades. For example, it looks a user would have to run terraform apply or the equivalent curl command to download the inventory script. And, when we bring this in, we'll definitely want to make sure the updated sample tf files are mentioned in the release notes (#1733) |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
thanks!
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Let's get the base branch stabilized before approaching this.
Resolved conflicts |
fixes #1120