No ansible checks to ensure loaded ebtables module

Bug #1546818 reported by Miguel Alejandro Cantu
6
This bug affects 1 person
Affects Status Importance Assigned to Milestone
OpenStack-Ansible
Fix Released
High
Ala Raddaoui

Bug Description

There are no checks within ansible that check to see if the ebtables module is loaded on the networking hosts.
This will cause the neutron-linuxbridge-agents to fail if the user hasn't already loaded the module on the networking host.

I ran into this problem on Friday when my linuxbridge-agents were logging that ebtables wasn't available on my kernel. I attempted to load the module using modprobe on the neutron agents container, but it failed. The module must be loaded on the networking host, not the container.

description: updated
Changed in openstack-ansible:
assignee: nobody → Ala Raddaoui (raddaoui-ala)
Revision history for this message
Matthew Thode (prometheanfire) wrote :

what if the module is staticly built into the kernel?

Revision history for this message
Jesse Pretorius (jesse-pretorius) wrote :

A fairly simple solution here would be to add ebtables to https://github.com/openstack/openstack-ansible-openstack_hosts/blob/master/defaults/main.yml#L35 - but there are other ways of being more discerning.

Changed in openstack-ansible:
importance: Undecided → Medium
status: New → Confirmed
milestone: none → mitaka-3
Revision history for this message
Matthew Thode (prometheanfire) wrote :

Probably easiest to do where jesse said, though that method will eventually need to be fixed to allow for static kernels.

better check would be:

1. zgrep KCONFIG_PARAM /proc/config.gz
2. register output
3.
  a. if output ends in '=y' be happy
  b. if output ends in '=m' do as is being done now
  c. if output ends in 'is not set' fail

Revision history for this message
Matthew Thode (prometheanfire) wrote :

might need to be key value based, kconfig and module name

Changed in openstack-ansible:
milestone: mitaka-3 → 13.0.0
importance: Medium → High
Revision history for this message
OpenStack Infra (hudson-openstack) wrote : Fix proposed to openstack-ansible (master)

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

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

Reviewed: https://review.openstack.org/290902
Committed: https://git.openstack.org/cgit/openstack/openstack-ansible-openstack_hosts/commit/?id=d9185f72b803daba0c866e63fb21d41792bc4b5f
Submitter: Jenkins
Branch: master

commit d9185f72b803daba0c866e63fb21d41792bc4b5f
Author: Kevin Carter <email address hidden>
Date: Wed Mar 9 17:15:14 2016 -0600

    Add ebtables to openstack_hosts role

    This change adds the ebtables package and kernel module to the
    openstack_host role. This is being added because without it
    the neutron linuxbridge agent will not start in a container
    sighting that the ebtables filter is missing. While we do not
    see this issue in the gate, because its being masked by the os_nova
    role, it is a problem on a multi-node setup when using network nodes
    that are not also compute nodes. With this simple change neutron had
    no issues running in a true multi-node setup.

    Change-Id: Ib178bbeed14c3e6c17613ad72545bcadf66b98cb
    Closes-Bug: 1546818
    Signed-off-by: Kevin Carter <email address hidden>

Changed in openstack-ansible:
status: In Progress → Fix Released
Revision history for this message
OpenStack Infra (hudson-openstack) wrote : Fix proposed to openstack-ansible (liberty)

Fix proposed to branch: liberty
Review: https://review.openstack.org/291281

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

Reviewed: https://review.openstack.org/291281
Committed: https://git.openstack.org/cgit/openstack/openstack-ansible/commit/?id=005e064da52d81518d30a7ef019ba1e93e47e500
Submitter: Jenkins
Branch: liberty

commit 005e064da52d81518d30a7ef019ba1e93e47e500
Author: Kevin Carter <email address hidden>
Date: Thu Mar 10 10:25:28 2016 -0600

    Add ebtables to openstack_hosts role

    This change adds the ebtables package and kernel module to the
    openstack_host role. This is being added because without it
    the neutron linuxbridge agent will not start in a container
    sighting that the ebtables filter is missing. While we do not
    see this issue in the gate, because its being masked by the os_nova
    role, it is a problem on a multi-node setup when using network nodes
    that are not also compute nodes. With this simple change neutron had
    no issues running in a true multi-node setup.

    Change-Id: Ie0cad4efa2d65697a49bbb5da70a59fca85d5fa4
    Backport-of: Ib178bbeed14c3e6c17613ad72545bcadf66b98cb
    Closes-Bug: 1546818
    Signed-off-by: Kevin Carter <email address hidden>

tags: added: in-liberty
Revision history for this message
OpenStack Infra (hudson-openstack) wrote : Fix proposed to openstack-ansible-openstack_hosts (master)

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

Revision history for this message
Doug Hellmann (doug-hellmann) wrote : Fix included in openstack/openstack-ansible 12.0.8

This issue was fixed in the openstack/openstack-ansible 12.0.8 release.

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

Reviewed: https://review.openstack.org/289622
Committed: https://git.openstack.org/cgit/openstack/openstack-ansible/commit/?id=0849367b62aa4f37243bb3a90e3b200c0075e572
Submitter: Jenkins
Branch: master

commit 0849367b62aa4f37243bb3a90e3b200c0075e572
Author: Ala Raddaoui <email address hidden>
Date: Mon Mar 7 23:15:27 2016 +0000

    Add ebtables check for network hosts

    Check how ebtables module is configured in
    kernel for network hosts and make sure it is
    loaded.

    Change-Id: I5f0410dfc6cd347920658b74ae0aadf8e4af78b1
    Closes-Bug: 1546818

Revision history for this message
Doug Hellmann (doug-hellmann) wrote : Fix included in openstack/openstack-ansible 12.0.9

This issue was fixed in the openstack/openstack-ansible 12.0.9 release.

Revision history for this message
Doug Hellmann (doug-hellmann) wrote : Fix included in openstack/openstack-ansible 13.0.0

This issue was fixed in the openstack/openstack-ansible 13.0.0 release.

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

Reviewed: https://review.openstack.org/291797
Committed: https://git.openstack.org/cgit/openstack/openstack-ansible-openstack_hosts/commit/?id=1b6bc3efd9bb37150980c08e68e7fa045f10039d
Submitter: Jenkins
Branch: master

commit 1b6bc3efd9bb37150980c08e68e7fa045f10039d
Author: Ala Raddaoui <email address hidden>
Date: Fri Mar 11 16:03:10 2016 +0000

    Add a check for hosts particular kernel modules

    Some hosts needs to have more specific kernel
    modules loaded in memory. Based on a list of
    those modules and their target group hosts, this
    patch checks how these modules are implemented
    in the kernel by searching for a specified pattern
    for each module in /boot/config-$kernel_version,
    loads them if they are dynamic and fails in case
    they are not set in the kernel configuration.

    Change-Id: I728f36146bd99193b346ebf4158d3716f0fbbed8
    Closes-Bug: 1546818

Revision history for this message
Davanum Srinivas (DIMS) (dims-v) wrote : Fix included in openstack/openstack-ansible 13.0.0

This issue was fixed in the openstack/openstack-ansible 13.0.0 release.

Revision history for this message
Davanum Srinivas (DIMS) (dims-v) wrote : Fix included in openstack/openstack-ansible 12.0.11

This issue was fixed in the openstack/openstack-ansible 12.0.11 release.

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

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

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

Reviewed: https://review.openstack.org/321819
Committed: https://git.openstack.org/cgit/openstack/openstack-ansible-openstack_hosts/commit/?id=eb1b3afc7414cd25effd29335be7d6cc73c2a0ba
Submitter: Jenkins
Branch: stable/mitaka

commit eb1b3afc7414cd25effd29335be7d6cc73c2a0ba
Author: Ala Raddaoui <email address hidden>
Date: Fri Mar 11 16:03:10 2016 +0000

    Add a check for hosts particular kernel modules

    Some hosts needs to have more specific kernel
    modules loaded in memory. Based on a list of
    those modules and their target group hosts, this
    patch checks how these modules are implemented
    in the kernel by searching for a specified pattern
    for each module in /boot/config-$kernel_version,
    loads them if they are dynamic and fails in case
    they are not set in the kernel configuration.

    Note: This is a manual cherry-pick of
    I728f36146bd99193b346ebf4158d3716f0fbbed8 from master

    Change-Id: I728f36146bd99193b346ebf4158d3716f0fbbed8
    Closes-Bug: 1546818

tags: added: in-stable-mitaka
Revision history for this message
Thierry Carrez (ttx) wrote : Fix included in openstack/openstack-ansible-openstack_hosts 14.0.0.0b1

This issue was fixed in the openstack/openstack-ansible-openstack_hosts 14.0.0.0b1 development milestone.

Revision history for this message
Davanum Srinivas (DIMS) (dims-v) wrote : Fix included in openstack/openstack-ansible-openstack_hosts 13.1.3

This issue was fixed in the openstack/openstack-ansible-openstack_hosts 13.1.3 release.

Revision history for this message
OpenStack Infra (hudson-openstack) wrote : Fix included in openstack/openstack-ansible 12.0.8

This issue was fixed in the openstack/openstack-ansible 12.0.8 release.

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.