Skip to content

handle unexpected changes to unknown block#28941

Merged
jbardin merged 1 commit intomainfrom
jbardin/objchange-unknown-blocks
Jun 14, 2021
Merged

handle unexpected changes to unknown block#28941
jbardin merged 1 commit intomainfrom
jbardin/objchange-unknown-blocks

Conversation

@jbardin
Copy link
Copy Markdown
Member

@jbardin jbardin commented Jun 11, 2021

An unknown block represents a dynamic configuration block with an
unknown for_each value. We were not catching the case where a provider
modified this value unexpectedly, which would crash with block of type
NestingList blocks where the config value has no length for comparison.

Fixes #28929

An unknown block represents a dynamic configuration block with an
unknown for_each value. We were not catching the case where a provider
modified this value unexpectedly, which would crash with block of type
NestingList blocks where the config value has no length for comparison.
@jbardin jbardin added the 1.0-backport If you add this label to a PR before merging, backport-assistant will open a new PR once merged label Jun 11, 2021
@jbardin jbardin requested a review from a team June 11, 2021 17:15
@jbardin jbardin merged commit 2ecdf44 into main Jun 14, 2021
@jbardin jbardin deleted the jbardin/objchange-unknown-blocks branch June 14, 2021 14:31
@github-actions
Copy link
Copy Markdown
Contributor

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.
If you have found a problem that seems related to this change, please open a new issue and complete the issue template so we can capture all the details necessary to investigate further.

@github-actions github-actions bot locked as resolved and limited conversation to collaborators Jul 15, 2021
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

1.0-backport If you add this label to a PR before merging, backport-assistant will open a new PR once merged

Projects

None yet

Development

Successfully merging this pull request may close these issues.

state mv on TF 0.15.4 causes later plans to crash with value is not known

2 participants