fixed_ips list out of order

Bug #1630410 reported by Armando Migliaccio
This bug affects 1 person
Affects Status Importance Assigned to Milestone
Kevin Benton

Bug Description

Change [1], led to failures like [2], in that the order of fixed_ips is no longer preserved between POST and GET requests. This was taken care for some other attributes of the Port resource like allowed address pairs, but not all.

Even though the API is lax about the order of specific attributes, we should attempt to restore the old behavior to avoid more damaging side effects in clients that are assuming the list be returned in the order in which fixed IPs are created.


Changed in neutron:
importance: Undecided → Medium
status: New → Confirmed
assignee: nobody → Kevin Benton (kevinbenton)
milestone: none → ocata-1
tags: added: newton-backport-potential
description: updated
description: updated
description: updated
description: updated
Revision history for this message
OpenStack Infra (hudson-openstack) wrote : Fix proposed to neutron (master)

Fix proposed to branch: master

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

Submitter: Jenkins
Branch: master

commit bd1c454c4f6de69eec7b3814b90faeb4db371ba6
Author: Kevin Benton <email address hidden>
Date: Tue Oct 4 18:38:07 2016 -0600

    Deterministic ordering of fixed_ips

    This adds an order_by clause to the fixed_ips relationship
    on the port object to ensure that the fixed_ip ordering is
    consistent between a create, an update, and a get request
    for a port. Without it we were at the mercy of the sql backend
    to determine how it felt like ordering them on the join condition.

    Closes-Bug: #1630410
    Change-Id: I523e0ab6e376f5ff6205b1cc1748aa6d546919cb

Changed in neutron:
status: In Progress → Fix Released
Revision history for this message
OpenStack Infra (hudson-openstack) wrote : Fix included in openstack/neutron

This issue was fixed in the openstack/neutron development milestone.

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

Other bug subscribers