bagpipe: IPAllocation DB query failing, engine facade mismatch
Affects | Status | Importance | Assigned to | Milestone | |
---|---|---|---|---|---|
BaGPipe |
Fix Released
|
Undecided
|
Unassigned | ||
networking-bgpvpn |
Fix Released
|
Undecided
|
Unassigned | ||
neutron |
Fix Released
|
High
|
Unassigned |
Bug Description
networking_
Traceback (most recent call last):
File "/home/
context, router_id, network_id)
File "/home/
return method(*args, **kwargs)
File "/home/
network_
File "neutron/
return [cls._load_
File "neutron/
obj.
File "/home/
self.
File "/home/
subnets_info = _get_subnets_
File "/home/
for subnet in subnets
File "/home/
port = Port.get_
File "neutron/
return cls._load_
File "neutron/
obj.
File "neutron/
super(Port, self).from_
File "neutron/
self.
File "neutron/
for obj in synth_db_objs]
File "neutron/
context.
File "/home/
state_
InvalidRequestE
"git bisect" show that neutron commit 906eda44d2c230b
Changed in neutron: | |
milestone: | none → queens-rc1 |
Changed in bgpvpn: | |
status: | New → Confirmed |
Changed in neutron: | |
status: | New → Confirmed |
importance: | Undecided → High |
Changed in neutron: | |
status: | Confirmed → Fix Released |
Changed in bgpvpn: | |
status: | Confirmed → Fix Released |
tags: | added: neutron-proactive-backport-potential |
tags: | removed: neutron-proactive-backport-potential |
The issue we have seems to be related to the fact that this unit test mixes:
- old engine facade code from networking-bagpipe OVO code
- new engine facade code for IPAllocations
This issue is strongly related to bug 1744829 .
My current understanding is that:
- [2] resulted in the use of OVO for IPAllocations
- networking-bgpvpn then started to make use of OVO, resulting in IPAllocation in OVO objects construction (both old engine facade, so all good)
- when [1] merged to revert [2] because of bug 174829, IPAllocation went back to non-OVO, hence
using the new engine facade, while we still use OVO to query IPAllocations in networking-bagpipe OVO objects
[1] https:/ /review. openstack. org/#/c/ 536913 /review. openstack. org/#/c/ 407868
[2] https:/