]> review.fuel-infra Code Review - openstack-build/neutron-build.git/commit
Make security_groups_provider_updated work with Kilo agents
authorIhar Hrachyshka <ihrachys@redhat.com>
Wed, 13 Jan 2016 11:37:21 +0000 (12:37 +0100)
committerIhar Hrachyshka <ihrachys@redhat.com>
Fri, 15 Jan 2016 12:40:06 +0000 (13:40 +0100)
commitf8f366024052a191eb0fc74af1643be15c541aef
treeede81434c2e0d04af9ebbe4b3eff9ca06819754a
parentfeced76488ea99355c605e0bc719723931621693
Make security_groups_provider_updated work with Kilo agents

Initially, we bumped the required version for the agent endpoint from
1.1 (the initial version that implemented security groups) to 1.3
without considering that the code should work with old agents that do
not yet know about the new devices_to_update argument.

Actually, there was no need to bump the version: old agent side code
already captures all unknown arguments that could be passed from the
server, ignoring them:

https://github.com/openstack/neutron/blob/608b54137fb67512c07099089ea7e074176e12df/neutron/agent/securitygroups_rpc.py#L155

(^ the link shows the latest Kilo code as of writing)

Note: some people may argue that the approach that is taken in Neutron
to support backwards compatibility for server notifications is wrong,
and we instead should adopt some stricter mechanism like nova version
pinning. While that is a noble thing to do, it's out of scope for the
patch that is designed to be easily backportable to stable/liberty.

Note: some people may also argue that the patch should go straight into
stable/liberty because we don't claim support for rolling upgrade
scenarios that span multiple releases. That's indeed true, though my
take on it is that if we have a way to handle more unofficial scenarios
without more coding effort, it's worth doing it.

Change-Id: I741e6e5c460658ac17095551040e67e8d1990812
Closes-Bug: #1531772
neutron/api/rpc/handlers/securitygroups_rpc.py