Incomplete bond information in interfaces template

Bug #1402028 reported by Nolan Brubaker
6
This bug affects 1 person
Affects Status Importance Assigned to Milestone
OpenStack-Ansible
Fix Released
Medium
Nolan Brubaker
Juno
Fix Released
Medium
Jesse Pretorius
Trunk
Fix Released
Medium
Nolan Brubaker

Bug Description

When creating a network interfaces file with the new host_interfaces role, bond interfaces are being created without an address information, even if an address is provided in the variable.

In addition, vlan tagged interfaces for a bond are not being reported at all by `ip a` if a bond interface doesn't have an IP address.

Changed in openstack-ansible:
status: New → In Progress
Revision history for this message
Nolan Brubaker (nolan-brubaker) wrote :

Not sure why gerrit didn't link the review, but it is here: https://review.openstack.org/#/c/141204/2

Revision history for this message
Dolph Mathews (dolph) wrote :

I think that's because the original commit message did not include a bug reference, so when one was added, the bot didn't comment - only updated the status of the bug.

Revision history for this message
OpenStack Infra (hudson-openstack) wrote : Fix merged to os-ansible-deployment (master)

Reviewed: https://review.openstack.org/141204
Committed: https://git.openstack.org/cgit/stackforge/os-ansible-deployment/commit/?id=067119ba89f454c576042ab3413d6b3a16c82464
Submitter: Jenkins
Branch: master

commit 067119ba89f454c576042ab3413d6b3a16c82464
Author: Nolan Brubaker <email address hidden>
Date: Thu Dec 11 18:05:57 2014 -0500

    Correct interface information for bonds

    When generating bonded interfaces, we still want to have an IP available
    if the user has defined it. This change will make sure that the address,
    netmask, and DNS information gets set on bonded interfaces.

    Also, tagged interfaces that are based off a bond need to be set to
    `manual`, so that they are not forced to have IPs.

    Closes-Bug: #1402028
    Change-Id: I6ad29f875be6ae8b770f0608874f6a650204f681

Changed in openstack-ansible:
status: In Progress → Fix Committed
Changed in openstack-ansible:
milestone: none → 10.1.0
Changed in openstack-ansible:
milestone: 10.1.0 → 10.1.2
Revision history for this message
OpenStack Infra (hudson-openstack) wrote : Fix proposed to os-ansible-deployment (juno)

Fix proposed to branch: juno
Review: https://review.openstack.org/151021

Revision history for this message
OpenStack Infra (hudson-openstack) wrote : Fix merged to os-ansible-deployment (juno)

Reviewed: https://review.openstack.org/151021
Committed: https://git.openstack.org/cgit/stackforge/os-ansible-deployment/commit/?id=f508f23a43c816077bf103a32a511aed09500350
Submitter: Jenkins
Branch: juno

commit f508f23a43c816077bf103a32a511aed09500350
Author: Nolan Brubaker <email address hidden>
Date: Thu Dec 11 18:05:57 2014 -0500

    Correct interface information for bonds

    When generating bonded interfaces, we still want to have an IP available
    if the user has defined it. This change will make sure that the address,
    netmask, and DNS information gets set on bonded interfaces.

    Also, tagged interfaces that are based off a bond need to be set to
    `manual`, so that they are not forced to have IPs.

    Closes-Bug: #1402028
    Change-Id: I6ad29f875be6ae8b770f0608874f6a650204f681
    (cherry picked from commit 067119ba89f454c576042ab3413d6b3a16c82464)

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.