Skip to content
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.

Sign up for GitHub

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

Jump to bottom

Cloud: State snapshots are always enabled, despite missing x-terraform-snapshot-interval header #687

Closed
RLRabinowitz opened this issue Oct 8, 2023 · 0 comments · Fixed by #696
Closed

Cloud: State snapshots are always enabled, despite missing x-terraform-snapshot-interval header #687

RLRabinowitz opened this issue Oct 8, 2023 · 0 comments · Fixed by #696
Assignees
@RLRabinowitz
Labels
accepted This issue has been accepted for implementation. bug Something isn't working
Milestone
First Stable Rele...

Comments

@RLRabinowitz
Copy link
Contributor

Summary:

The cloud APIs for creating/listing versions could return a x-terraform-snapshot-interval header, which tells tofu how often is should save intermediate snapshots
If the x-terraform-snapshot-interval header is missing, then tofu should not save intermediate snapshots. However, currently that is not the case, and if x-terraform-snapshot-interval is missing - tofu still saves intermediate snapshots
@RLRabinowitz RLRabinowitz self-assigned this Oct 8, 2023
@RLRabinowitz RLRabinowitz added bug Something isn't working pending-decision This issue has not been accepted for implementation nor rejected. It's still open to discussion. labels Oct 8, 2023
@RLRabinowitz RLRabinowitz added this to the First Stable Release milestone Oct 8, 2023
@eranelbaz eranelbaz added accepted This issue has been accepted for implementation. and removed pending-decision This issue has not been accepted for implementation nor rejected. It's still open to discussion. labels Oct 9, 2023
@eranelbaz eranelbaz assigned eranelbaz and unassigned RLRabinowitz Oct 9, 2023
@eranelbaz eranelbaz mentioned this issue Oct 9, 2023
Merged
@cube2222 cube2222 assigned RLRabinowitz and unassigned eranelbaz Oct 11, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@RLRabinowitz RLRabinowitz

Labels
accepted This issue has been accepted for implementation. bug Something isn't working
Projects
None yet
Milestone
First Stable Release (1.6.0)
Development

Successfully merging a pull request may close this issue.

Consider missing x-terraform-snapshot-interval for State Snapshot opentofu/opentofu
2 participants
@eranelbaz @RLRabinowitz