-
Notifications
You must be signed in to change notification settings - Fork 157
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
Set variable id to null if workspace id doesnt exist #227
Conversation
Also, the managing workspace gets stuck in the "Error" state since it can't find the manually deleted workspaces anymore. Any workaround suggestions? |
@engd which version of the TFE provider are you using? This should be handled in the most recent version of the TFE provider - https://github.com/terraform-providers/terraform-provider-tfe/blob/master/tfe/resource_tfe_workspace.go#L211-L219. |
@im2nguyen thanks for your answer. I updated to 0.21.0 but it didn't fix it. |
Can you include the logs attached with the run?
|
We are using TFE to create an organization and multiple workspaces in it. I am getting the same error for each workspace.
Also the following 404s for each of them as well:
|
I was able to work around it by removing workspaces manually with |
Hi @engd , can you show which resource is refreshing its state? The line should look similar to the following and come slightly before the
|
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM
Description
Addresses #195
Configuration containing a TFE variable will fail if the workspace is destroyed in the UI. This change destroys the variable resource in state if the workspace it belongs to no longer exists.
BREAKING CHANGE
Delete this section if your change is not introducing a breaking change to the provider or a breaking change from the TFE API
Testing plan
External links
Include any links here that might be helpful for people reviewing your PR. If there are none, feel free to delete this section.
Output from acceptance tests
Please run the full suite of acceptance tests locally and include the output here.