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

Crash and remove resources on updating openstack_lb_pool_v1 with new monitor #12495

Closed
carlpett opened this issue Mar 7, 2017 · 5 comments
Closed
Labels

Comments

@carlpett
Copy link
Contributor

carlpett commented Mar 7, 2017

I had accidentally used the wrong monitor for my pool when creating it. I changed the monitor_ids property and ran plan. Plan reports that the value will be changed in-place. Run apply. Terraform crashes, and the pool is lost from the state in the process (still exists, but Terraform does not know about it). Running apply again will re-create all members and a new pool.

Terraform Version

Terraform v0.8.6

Affected Resource(s)

  • openstack_lb_pool_v1

Panic Output

https://gist.github.com/carlpett/e79e0e570a0ee1edcd5e335d5d7eaa5e

Expected Behavior

Monitor is changed.

Actual Behavior

Crash, existing pool is lost.

Steps to Reproduce

Create a pool with a monitor. Apply. Change to another pool. Apply.

@sebastien-prudhomme
Copy link

I don't really know if it's related: got a crash on "terraform plan" with v0.9.0-beta2 when creating a new openstack_lb_monitor_v2 ressource (not a openstack_lb_pool_v1)

resource "openstack_lb_monitor_v2" "test" {
  type           = "HTTP"
  delay          = 20
  timeout        = 10
  max_retries    = 3
  expected_codes = 200
  url_path       = "/v1/status/leader"
  pool_id        = "bf2ab482-42fd-4741-b301-723601169585"
}

crash.log: https://gist.github.com/sebastien-prudhomme/6604df7c75a88b43ba47f0f2eacebd15

@jtopjian
Copy link
Contributor

@carlpett Sorry for the delay on this one. I actually had this fixed soon after you reported it but must have got side tracked and never opened a PR for it. #13069 will resolve this issue.

@sebastien-prudhomme Your crash looks to have been a bug with the new "timeout" feature in the beta2 release. I'm able to reproduce your crash with beta2 and can confirm that any release after beta2 is fine. Let me know if you have any questions though.

Thank you both for reporting the issues!

@carlpett
Copy link
Contributor Author

@jtopjian Nice work!

@stack72
Copy link
Contributor

stack72 commented Mar 25, 2017

Closed via #13069

@stack72 stack72 closed this as completed Mar 25, 2017
@ghost
Copy link

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

No branches or pull requests

5 participants