dns_remove_record() fails with KeyError exception
Bug #1793132 reported by
Lucas Alvares Gomes
This bug affects 1 person
Affects | Status | Importance | Assigned to | Milestone | |
---|---|---|---|---|---|
ovsdbapp |
Fix Released
|
Undecided
|
Unassigned |
Bug Description
Original report: https:/
The method dns_remove_record() from ovsdbapp does not support the parameter "if_exist" therefore, if the record being removed do not exist in OVNDB it will raise an KeyError exception which causing other methods in networking ovn such as add_txns_
Changed in networking-ovn: | |
status: | New → Confirmed |
Changed in ovsdbapp: | |
status: | New → Confirmed |
Changed in networking-ovn: | |
importance: | Undecided → High |
no longer affects: | networking-ovn |
To post a comment you must log in.
Reviewed: https:/ /review. openstack. org/603333 /git.openstack. org/cgit/ openstack/ ovsdbapp/ commit/ ?id=31a47643537 3a8c6581ad66d96 fbe314711e7f36
Committed: https:/
Submitter: Zuul
Branch: master
commit 31a476435373a8c 6581ad66d96fbe3 14711e7f36
Author: Lucas Alvares Gomes <email address hidden>
Date: Tue Sep 18 10:07:05 2018 +0100
NBDB API: Add param if_exists for methods using db_remove()
This patch is adding the parameter "if_exists" to the methods in the
Northbound DB API that uses the db_remove() method underneath.
The following methods were affected:
* dns_remove_record() dns_record( )
* ls_remove_
The intention of this patch is to make the API more flexible for
projects relying on those methods so they can choose whether they want
to raise an exception if the item being removed exist or not.
Closes-Bug: #1793132 91a9db29e6f5ba0 c8b7a6cc622
Change-Id: I1e6749a9f3827f