-
Notifications
You must be signed in to change notification settings - Fork 9.4k
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
Add TF_STATE_PERSIST_INTERVAL
environment variable.
#35176
Add TF_STATE_PERSIST_INTERVAL
environment variable.
#35176
Conversation
Signed-off-by: Alex Ott <alexott@gmail.com>
Still missing updating the docs and adding a few more tests, if possible. |
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.
Great, thank you.
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.
thank you - just one comment about boundary check
2993d8b
to
2c03fea
Compare
2c03fea
to
5a625a3
Compare
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.
Left a style suggestion, but feel free to push back if it doesn't feel good.
5a625a3
to
b28c8bf
Compare
Reminder for the merging maintainer: if this is a user-visible change, please update the changelog on the appropriate release branch. |
This PR stems from the initial work done by @alexott (#35116), which fixes #35115. These changes are a simple refactor to delegate handling this env var to
command.Meta
. This allows us to pass the user specified persist interval tobackendrun.Operation
and keep env var handling (with existing vars such asTF_WORKSPACE
,TF_INPUT
) centralized withincommand.Meta
.The default interval remains 20 seconds and serves as a global minimum interval. Remote backends such as
cloud
may override the default using theX-Terraform-Snapshot-Interval
header. If the default interval value is greater than the user-specified value, the default will be used.Target Release
1.9.x
Draft CHANGELOG entry
ENHANCEMENTS