[stable/train] member_batch_update breaks contract with octavia-api interface

Bug #1979661 reported by Fernando Royo
6
This bug affects 1 person
Affects Status Importance Assigned to Milestone
neutron
Fix Released
High
Fernando Royo

Bug Description

The prototype of the member_batch_update function in Octavia provider API in Train (16.x) [1]
is member_batch_update(self, members). A change [2] was later done in Ussuri, to move to member_batch_update(self, pool_id, members) but it was not backported to train to keep backwards compatibility.

So looks like the change on ovn-octavia-provider [3] was backported by error to stable/train and now the call from octavia to the provider is triggering:

2022-06-22 14:05:54.634 14 ERROR octavia.api.drivers.utils [req-8ad8a6ce-59cc-4531-94fa-d7918406d19f - 25b1efe043b54816ab6bd8e2b6b0d9c8 - default default] Provider 'ovn' raised an unknown error: member_batch_update() missing 1 required positional argument: 'members': TypeError: member_batch_update() missing 1 required positional argument: 'members'
2022-06-22 14:05:54.634 14 ERROR octavia.api.drivers.utils Traceback (most recent call last):
2022-06-22 14:05:54.634 14 ERROR octavia.api.drivers.utils File "/usr/lib/python3.6/site-packages/octavia/api/drivers/utils.py", line 55, in call_provider
2022-06-22 14:05:54.634 14 ERROR octavia.api.drivers.utils return driver_method(*args, **kwargs)
2022-06-22 14:05:54.634 14 ERROR octavia.api.drivers.utils TypeError: member_batch_update() missing 1 required positional argument: 'members'

[1] https://opendev.org/openstack/octavia/src/branch/stable/train/octavia/api/drivers/amphora_driver/v2/driver.py#L217
[2] https://review.opendev.org/c/openstack/octavia/+/688548/
[3] https://review.opendev.org/c/openstack/networking-ovn/+/746134

This is an issue just only over stable/train branch.

Changed in neutron:
assignee: nobody → Fernando Royo (froyoredhat)
summary: - member_batch_update breaks contract with octavia-api interface
+ [stable/train] member_batch_update breaks contract with octavia-api
+ interface
description: updated
Changed in neutron:
status: New → Triaged
importance: Undecided → High
tags: added: ovn-octavia-provider
Changed in neutron:
status: Triaged → Fix Committed
Changed in neutron:
status: Fix Committed → Fix Released
Revision history for this message
OpenStack Infra (hudson-openstack) wrote : Fix included in openstack/networking-ovn train-eol

This issue was fixed in the openstack/networking-ovn train-eol release.

To post a comment you must log in.
This report contains Public information  
Everyone can see this information.

Other bug subscribers

Remote bug watches

Bug watches keep track of this bug in other bug trackers.