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

Fixes time out when applying updates to Triton machine metadata. #6149

Merged
merged 2 commits into from
Apr 20, 2016
Merged

Fixes time out when applying updates to Triton machine metadata. #6149

merged 2 commits into from
Apr 20, 2016

Conversation

sodre
Copy link
Contributor

@sodre sodre commented Apr 13, 2016

This is the fix for GH-6148.

The test starts the basic machine and then adds the metadata field
user_data.
Test fails if the user_data field does not match what we expect
OR it times out.

Related to #6148
The code waiting for the entire Machine Metadata to "deep equal" the Terraform
metadata modifications. These two sets will only be the same if the user
changes all metadata fields of the resource before calling `apply`.

Closes #6148
@stack72
Copy link
Contributor

stack72 commented Apr 20, 2016

Hi @sodre

this looks good to me. I will be providing an update to the tests as the instances in the test are not in the default Terraform test region :)

Paul

@stack72 stack72 merged commit 33d4c44 into hashicorp:master Apr 20, 2016
@sodre sodre deleted the b-triton-machine-metadata-convergence branch April 26, 2016 02:41
@ghost
Copy link

ghost commented Apr 26, 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 Apr 26, 2020
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants