-
Notifications
You must be signed in to change notification settings - Fork 10.1k
Actions in plan/changes #37320
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Actions in plan/changes #37320
Conversation
1dcb773 to
4169c85
Compare
4e218f9 to
6c35cd9
Compare
6c35cd9 to
fb0ab7a
Compare
| ) | ||
|
|
||
| type ActionInvocation struct { | ||
| // Address is the absolute action address |
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.
I think we will need the address of the triggering resource, the trigger block index and the index for the event here to clearly link the action invocation back to its place in the config and to be able to base apply on it. I think we could get if it's run before or after from this info as well by looking at the config so we don't necessarily need it in the plan.
That being said, we can either add these now or in a follow up PR that integrates this with the plan action changes. The heavy lifting is done so I'm also ok with folding this into the upcoming work 👍
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.
Yep, agreed - I didn't do it here because I'd have to rework the hooks too, and also I didn't have a good picture of when or where we would build that info
|
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. |
Adding ActionInvocations to the planfile
Part of TF-27252 (internal): adding ActionInvocations to planfile and stopping just short of the rendering (which got moved to a different ticket).
I was making up the types as I go, and while most areas support LinkedResources I ignored them in the jsonplan package, so some random notes on TBD future work -
Still gated behind the alpha gate, so this should not have any user-facing impact.
Fixes #
Target Release
1.14
1.14.x
Rollback Plan
Changes to Security Controls
Are there any changes to security controls (access controls, encryption, logging) in this pull request? If so, explain.
CHANGELOG entry