Project

General

Profile

Actions
Copy link

Bug #18209

closed

Cannot destroy host if DHCP record is found but is not a lease

Added by Daniel Lobato Garcia about 7 years ago. Updated about 5 years ago.

Status:
Closed
Priority:
Normal
Assignee:
Daniel Lobato Garcia
Category:
DHCP
Target version:
-
Difficulty:
Triaged:
Yes
Bugzilla link:
Pull request:
Fixed in Releases:
Found in Releases:
Red Hat JIRA:

Description

When you try to destroy a DHCP record in Libvirt, it's possible that the record is found but it's not in the list of leased_ips.
In such case an error like this shows up:

#=> #<Libvirt::Error: Call to virNetworkUpdate failed: Requested operation is not valid: couldn't locate a matching dhcp host entry in network 'vagrant-libvirt'>

To avoid it, let's return a 404 in case the record is not in the list of leased ips.

Actions
Copy link
 #1

Updated by The Foreman Bot about 7 years ago

  • Status changed from New to Ready For Testing
  • Assignee set to Daniel Lobato Garcia
  • Pull request https://github.com/theforeman/smart-proxy/pull/499 added
Actions
Copy link
 #2

Updated by Daniel Lobato Garcia about 7 years ago

  • Target version set to 169
Actions
Copy link
 #3

Updated by Brad Buckingham about 7 years ago

  • Target version deleted (169)
Actions
Copy link
 #4

Updated by Daniel Lobato Garcia about 7 years ago

  • Target version set to 1.11.0
Actions
Copy link
 #5

Updated by Tomer Brisker over 5 years ago

  • Target version deleted (1.11.0)
Actions
Copy link
 #6

Updated by Lukas Zapletal about 5 years ago

  • Status changed from Ready For Testing to Closed
  • Triaged set to Yes
  • Pull request deleted (https://github.com/theforeman/smart-proxy/pull/499)

Thanks for the patch, however libvirt provider was vastly refactored. Please rebase on top of develop and reopen if you still want this to get it through.

Actions
Copy link

Also available in: Atom PDF