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.

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

don't evaluate destroy instances #25419

Merged
merged 1 commit into from
Jun 29, 2020
Merged

don't evaluate destroy instances #25419

merged 1 commit into from
Jun 29, 2020

Conversation

jbardin
Copy link
Member

@jbardin jbardin commented Jun 26, 2020

Orphaned instances that are create_before_destroy will still be in the
state when their references are evaluated. We need to skip instances
that are planned to be destroyed altogether, as they can't be part of an
evaluation.

Fixes #16473
Fixes #11716

@jbardin jbardin requested a review from a team June 26, 2020 21:57
@codecov
Copy link

codecov bot commented Jun 26, 2020

Codecov Report

Merging #25419 into master will decrease coverage by 0.00%.
The diff coverage is 100.00%.

Impacted Files Coverage Δ
terraform/evaluate.go 53.91% <100.00%> (+0.31%) ⬆️
terraform/node_resource_plan.go 91.80% <0.00%> (-1.64%) ⬇️

Orphaned instances that are create_before_destroy will still be in the
state when their references are evaluated. We need to skip instances
that are planned to be destroyed altogether, as they can't be part of an
evaluation.
@jbardin jbardin force-pushed the jbardin/cbd-scale-in branch from 9a49160 to 6243a63 Compare June 26, 2020 22:06
Copy link
Contributor

@alisdair alisdair left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

This change makes sense to me!

@jbardin jbardin merged commit 8a152f5 into master Jun 29, 2020
@jbardin jbardin deleted the jbardin/cbd-scale-in branch June 29, 2020 16:58
@ghost
Copy link

ghost commented Jul 30, 2020

I'm going to lock this issue because it has been closed for 30 days ⏳. This helps our maintainers find and focus on the active issues.

If you have found a problem that seems similar to this, please open a new issue and complete the issue template so we can capture all the details necessary to investigate further.

@ghost ghost locked and limited conversation to collaborators Jul 30, 2020
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
2 participants