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

Private data and timeouts were lost in empty plans #21814

Merged
merged 3 commits into from
Jun 20, 2019

Conversation

jbardin
Copy link
Member

@jbardin jbardin commented Jun 20, 2019

Timeouts needs to always exist in the private data with the new protocol, since not all helper/schema operations load the combined config+defaults at the correct time. The private data was lost in PlanResourceChange when there was no diff, which means there was no diff.Meta to encode them into. The read timeouts were also not decoded during ReadResource, so make sure to insert them from the private data.

jbardin added 3 commits June 19, 2019 22:48
simpleDiff is only called from the grpc_provider, and we always need to
make sure the timeouts are encoded in the private data.
Load private data for read, so the resource can get it's configured
timeouts if they exist.

Ensure PlanResourceChange returns the saved private data when there is
an empty diff.

Handle the timeout decoding into Meta in the PlanResourceChange, so that
it's always there for later operations.
@jbardin jbardin requested a review from a team June 20, 2019 02:54
@jbardin jbardin merged commit e281336 into master Jun 20, 2019
@jbardin jbardin deleted the jbardin/private-and-timeout branch June 20, 2019 12:59
@ghost
Copy link

ghost commented Jul 25, 2019

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 25, 2019
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants