netaddr 0.7.16 causes gate havoc

Bug #1490380 reported by Armando Migliaccio
18
This bug affects 3 people
Affects Status Importance Assigned to Milestone
neutron
Fix Released
Critical
Kevin Benton
Kilo
Fix Released
Undecided
Unassigned
Changed in neutron:
assignee: nobody → Kevin Benton (kevinbenton)
importance: Undecided → Critical
status: New → Confirmed
milestone: none → liberty-3
Revision history for this message
Armando Migliaccio (armando-migliaccio) wrote :

Looks like unit tests, functional, grenade and dvr are affected.

description: updated
summary: - netaddr 0.7.16 causes UT havoc
+ netaddr 0.7.16 causes gate havoc
Revision history for this message
OpenStack Infra (hudson-openstack) wrote : Fix proposed to neutron (master)

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

Changed in neutron:
status: Confirmed → In Progress
Revision history for this message
Miguel Angel Ajo (mangelajo) wrote :

Did they revert for 0.7.17 ?

It works locally for such version.

Revision history for this message
Miguel Angel Ajo (mangelajo) wrote :

yes, fails for 0.7.16 and not for 0.7.17 (locally).

Revision history for this message
Miguel Angel Ajo (mangelajo) wrote :

neiljerram comments that's not true in his local environment when running all tests. (I was running an offending subset)

Changed in neutron:
assignee: Kevin Benton (kevinbenton) → Miguel Angel Ajo (mangelajo)
Revision history for this message
OpenStack Infra (hudson-openstack) wrote : Fix proposed to neutron (stable/kilo)

Fix proposed to branch: stable/kilo
Review: https://review.openstack.org/218852

Changed in neutron:
assignee: Miguel Angel Ajo (mangelajo) → Kevin Benton (kevinbenton)
Revision history for this message
OpenStack Infra (hudson-openstack) wrote : Fix merged to neutron (stable/kilo)

Reviewed: https://review.openstack.org/218852
Committed: https://git.openstack.org/cgit/openstack/neutron/commit/?id=cc791b04f08518f895b8885e40aed358a4a9cfac
Submitter: Jenkins
Branch: stable/kilo

commit cc791b04f08518f895b8885e40aed358a4a9cfac
Author: Kevin Benton <email address hidden>
Date: Sun Aug 30 19:15:27 2015 -0700

    Fix usage of netaddr '.broadcast'

    netaddr 0.7.16 changed the behavior of ipnetworks with /31 and /32
    prefixes to make their 'broadcast' attribute return none. this patch
    replaces the use of the attribute with a -1 index lookup to get the
    last address instead.

    for stable kilo we also fix neutron/db/db_base_plugin_v2.py which had
    the checks in neutron/ipam/utils.py for Liberty.

    closes-bug: #1490380
    (cherry picked from commit 6324f7f23d5debecf7201a371050e20527550a8c)

    conflicts:
     neutron/ipam/utils.py
     neutron/plugins/linuxbridge/agent/linuxbridge_neutron_agent.py

    Change-Id: I97d71c4051882ddd9e496c78cfbce840ad7a2b67

tags: added: in-stable-kilo
Revision history for this message
OpenStack Infra (hudson-openstack) wrote : Fix merged to neutron (master)

Reviewed: https://review.openstack.org/218723
Committed: https://git.openstack.org/cgit/openstack/neutron/commit/?id=6324f7f23d5debecf7201a371050e20527550a8c
Submitter: Jenkins
Branch: master

commit 6324f7f23d5debecf7201a371050e20527550a8c
Author: Kevin Benton <email address hidden>
Date: Sun Aug 30 19:15:27 2015 -0700

    Fix usage of netaddr '.broadcast'

    netaddr 0.7.16 changed the behavior of IPNetworks with /31 and /32
    prefixes to make their 'broadcast' attribute return None. This patch
    replaces the use of the attribute with a -1 index lookup to get the
    last address instead.

    Closes-Bug: #1490380
    Change-Id: I97d71c4051882ddd9e496c78cfbce840ad7a2b67

Changed in neutron:
status: In Progress → Fix Committed
Revision history for this message
Matt Riedemann (mriedem) wrote :

g-r block for netaddr 0.7.16:

https://review.openstack.org/#/c/218720/

Revision history for this message
OpenStack Infra (hudson-openstack) wrote : Fix proposed to neutron (feature/pecan)

Fix proposed to branch: feature/pecan
Review: https://review.openstack.org/219146

Revision history for this message
OpenStack Infra (hudson-openstack) wrote :

Fix proposed to branch: feature/pecan
Review: https://review.openstack.org/219156

Revision history for this message
OpenStack Infra (hudson-openstack) wrote : Change abandoned on neutron (feature/pecan)

Change abandoned by Kevin Benton (<email address hidden>) on branch: feature/pecan
Review: https://review.openstack.org/219146
Reason: merge commit here is better

I9df72f59c7e65414ddbc8431676312a0d4fd36c8

Revision history for this message
OpenStack Infra (hudson-openstack) wrote : Fix merged to neutron (feature/pecan)
Download full text (3.8 KiB)

Reviewed: https://review.openstack.org/219156
Committed: https://git.openstack.org/cgit/openstack/neutron/commit/?id=22188bb0d96b05e9e2a8eb6ed73b18b6d36fc2ac
Submitter: Jenkins
Branch: feature/pecan

commit 1ec34b7d0d38324053996394be90d6fdecfed3ef
Author: Nick <email address hidden>
Date: Wed Aug 26 18:16:14 2015 +0800

    Linuxbridge-agent: fix bridge deletion

    Make sure that linuxbridge agent doesn't delete
    physical interfaces if they are specified in the
    physical_interface_mappings.

    Change-Id: I95e3c46b03ba2ae0872f5def47f26d4e8ea13094
    Closes-Bug: #1334634

commit 6324f7f23d5debecf7201a371050e20527550a8c
Author: Kevin Benton <email address hidden>
Date: Sun Aug 30 19:15:27 2015 -0700

    Fix usage of netaddr '.broadcast'

    netaddr 0.7.16 changed the behavior of IPNetworks with /31 and /32
    prefixes to make their 'broadcast' attribute return None. This patch
    replaces the use of the attribute with a -1 index lookup to get the
    last address instead.

    Closes-Bug: #1490380
    Change-Id: I97d71c4051882ddd9e496c78cfbce840ad7a2b67

commit 8a588dc95ef6486f3d28706052ca8873210d32f7
Author: Neil Jerram <email address hidden>
Date: Mon Aug 31 11:16:24 2015 +0100

    Add lieutenants contact for networking-calico

    Change-Id: Idc688a380e82ea6a385efc7b459b46e2daed0b4a

commit 81c2a7787fc6991db3fd3550f803e60994a7689a
Author: Neil Jerram <email address hidden>
Date: Mon Aug 31 11:10:53 2015 +0100

    Adding networking-calico to sub_projects document

    Change-Id: I537936dcc1723c76c12e816ff04645abaa7a8431

commit bac73541f420d1802cdf48f4d35b3a6ad1f5b7ee
Author: Ihar Hrachyshka <email address hidden>
Date: Mon Aug 24 17:58:12 2015 +0200

    [neutron-db-manage] Introduce contract and expand commands

    Those are just wrappers that pass proper @head revision into upgrade
    alembic API.

    Change-Id: I4d7e1bd992c3ad75fca6d72defa9f96565ad6ba9
    Partially-Implements: blueprint online-schema-migrations

commit 298d225e2962d41ae08f2777facc938a84bf4ef9
Author: Neil Jerram <email address hidden>
Date: Mon Jul 27 14:26:59 2015 +0100

    DHCP agent: add 'bridged' property to interface driver

    This allows a custom interface driver to use the reference DHCP agent
    to provide DHCP service to unbridged TAP interfaces, by overriding the
    definition of the property as follows.

        @property
        def bridged(self):
            return False

    Partial-Bug: #1486649
    Change-Id: I2c6bbd9df6833a65088173b021790eb23e64616b

commit cf003af20906f3ef556a39d4f4f7067193dbfb68
Author: Moshe Levi <email address hidden>
Date: Tue Aug 18 23:10:05 2015 +0300

    SR-IOV: deprecate agent_required option

    When SR-IOV introduce in Juno Agent supported only link state change
    Some Intel cards don't support setting link state, so to
    resolve it the SR-IOV mech driver supports agent and agent less mode.
    From Liberty the SR-IOV agent brings more functionality like
    qos and port security so we want to make the agent mandatory.
    (of course I already talked with Intel Guys to get their approval)

    This patch deprec...

Read more...

tags: added: in-feature-pecan
Thierry Carrez (ttx)
Changed in neutron:
status: Fix Committed → Fix Released
Thierry Carrez (ttx)
Changed in neutron:
milestone: liberty-3 → 7.0.0
To post a comment you must log in.
This report contains Public information  
Everyone can see this information.

Duplicates of this bug

Other bug subscribers

Remote bug watches

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