Improve logic for nvp plugin nat rules ordering

Bug #1176947 reported by Salvatore Orlando
6
This bug affects 1 person
Affects Status Importance Assigned to Milestone
neutron
Fix Released
Low
Salvatore Orlando

Bug Description

NAT rule ordering for the nicira plugin is currently trivial. A higher order is assigned to floating IP rules whereas a lower order is assigned to default SNAT rules.

A better logic would leverage the cidr prefix, establishing the rule order as MAX_ORDER - PREFIX_LEN.
In this way, floating IP rules (/32) will still have highest priority.

This will also avoid problems with deployments upgraded from 2.x to 3.x where NAT rules order are rearranged according to this logic.

Tags: nicira
Revision history for this message
OpenStack Infra (hudson-openstack) wrote : Fix proposed to quantum (master)

Fix proposed to branch: master
Review: https://review.openstack.org/28314

Changed in quantum:
status: New → In Progress
Revision history for this message
OpenStack Infra (hudson-openstack) wrote : Fix merged to quantum (master)

Reviewed: https://review.openstack.org/28314
Committed: http://github.com/openstack/quantum/commit/6acbc0520422ab08e87caf7e7ac0f1ebe7683823
Submitter: Jenkins
Branch: master

commit 6acbc0520422ab08e87caf7e7ac0f1ebe7683823
Author: Salvatore Orlando <email address hidden>
Date: Mon May 6 17:13:15 2013 +0200

    Calculate nicira plugin NAT rules order according to CIDR prefix

    Bug 1176947

    Change-Id: Ibe81b9905cacea51d954416588f5c50d6337bbb0

Changed in quantum:
status: In Progress → Fix Committed
Thierry Carrez (ttx)
Changed in quantum:
status: Fix Committed → Fix Released
Thierry Carrez (ttx)
Changed in neutron:
milestone: havana-1 → 2013.2
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.