l23network is not able to save use_carrier=1 bond property

Bug #1658981 reported by Aleksandr Didenko
6
This bug affects 1 person
Affects Status Importance Assigned to Milestone
Fuel for OpenStack
Fix Committed
High
Sergey Vasilenko
Nominated for Ocata by Aleksandr Didenko
Mitaka
Fix Committed
High
Sergey Vasilenko
Newton
Fix Committed
High
Sergey Vasilenko

Bug Description

Steps to reproduce:
1) Install 9.0
2) Upgrade to 9.2 from http://mirror.fuel-infra.org/mos-repos/centos/mos9.0-centos7/snapshots/proposed-2017-01-13-184421/x86_64
3) Deploy env with LACP bonds
4) Run re-deploy without changing bond properties (or re-run netconfig task only)

Expected result:
No changes in bond properties

Actual result:
Puppet changes bond properties on every run, it tries to set use_carrier=1 property:
http://paste.openstack.org/show/596222/

Tags: l23network
Changed in fuel:
milestone: none → 9.3
Changed in fuel:
milestone: 9.3 → 11.0
Revision history for this message
OpenStack Infra (hudson-openstack) wrote : Fix proposed to fuel-library (master)

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

Changed in fuel:
status: New → In Progress
Revision history for this message
OpenStack Infra (hudson-openstack) wrote : Change abandoned on fuel-library (master)

Change abandoned by Sergey Vasilenko (<email address hidden>) on branch: master
Review: https://review.openstack.org/426791
Reason: due https://review.openstack.org/#/c/427123/

Revision history for this message
OpenStack Infra (hudson-openstack) wrote : Fix merged to fuel-library (master)

Reviewed: https://review.openstack.org/427123
Committed: https://git.openstack.org/cgit/openstack/fuel-library/commit/?id=6fde41c44af5ce785053fdf40eddad65a1a1bff4
Submitter: Jenkins
Branch: master

commit 6fde41c44af5ce785053fdf40eddad65a1a1bff4
Author: Sergey Vasilenko <email address hidden>
Date: Mon Jan 30 17:53:13 2017 +0300

    Prevent a bond for unrequired re-assembles

    * Assemble bond members under bond in UP state
    * FIX lost bridge parameter
    * FIX lost 'use_carrier' bond property
    * waiting for bond UP after slaves added
    * additional flush IP addresses from slave while bond assemble

    Change-Id: I568d852e65dc5d5c246e11deb0740f4a608f5ecc
    Closes-bug: #1658981
    Related-bug: #1657750

Changed in fuel:
status: In Progress → Fix Committed
Revision history for this message
OpenStack Infra (hudson-openstack) wrote : Fix proposed to fuel-library (stable/newton)

Fix proposed to branch: stable/newton
Review: https://review.openstack.org/430173

Revision history for this message
OpenStack Infra (hudson-openstack) wrote : Fix proposed to fuel-library (stable/mitaka)

Fix proposed to branch: stable/mitaka
Review: https://review.openstack.org/430174

Revision history for this message
OpenStack Infra (hudson-openstack) wrote : Fix included in openstack/fuel-library 11.0.0.0rc1

This issue was fixed in the openstack/fuel-library 11.0.0.0rc1 release candidate.

Revision history for this message
OpenStack Infra (hudson-openstack) wrote : Fix merged to fuel-library (stable/mitaka)

Reviewed: https://review.openstack.org/430174
Committed: https://git.openstack.org/cgit/openstack/fuel-library/commit/?id=a848384706cdf69e7241ef4f51c9e2534805f586
Submitter: Jenkins
Branch: stable/mitaka

commit a848384706cdf69e7241ef4f51c9e2534805f586
Author: Sergey Vasilenko <email address hidden>
Date: Mon Jan 30 17:53:13 2017 +0300

    Prevent a bond for unrequired re-assembles

    * Assemble bond members under bond in UP state
    * FIX lost bridge parameter
    * FIX lost 'use_carrier' bond property
    * waiting for bond UP after slaves added
    * additional flush IP addresses from slave while bond assemble

    Change-Id: I568d852e65dc5d5c246e11deb0740f4a608f5ecc
    Closes-bug: #1658981
    Related-bug: #1657750
    (cherry picked from commit 6fde41c44af5ce785053fdf40eddad65a1a1bff4)

Revision history for this message
OpenStack Infra (hudson-openstack) wrote : Fix merged to fuel-library (stable/newton)

Reviewed: https://review.openstack.org/430173
Committed: https://git.openstack.org/cgit/openstack/fuel-library/commit/?id=dedeaa789c78f8f4a9282e87210468a90eb09b37
Submitter: Jenkins
Branch: stable/newton

commit dedeaa789c78f8f4a9282e87210468a90eb09b37
Author: Sergey Vasilenko <email address hidden>
Date: Mon Jan 30 17:53:13 2017 +0300

    Prevent a bond for unrequired re-assembles

    * Assemble bond members under bond in UP state
    * FIX lost bridge parameter
    * FIX lost 'use_carrier' bond property
    * waiting for bond UP after slaves added
    * additional flush IP addresses from slave while bond assemble

    Change-Id: I568d852e65dc5d5c246e11deb0740f4a608f5ecc
    Closes-bug: #1658981
    Related-bug: #1657750
    (cherry picked from commit 6fde41c44af5ce785053fdf40eddad65a1a1bff4)

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.