Restore built-in OpenVSwitch kernel module for Centos

Bug #1400355 reported by Miroslav Anashkin
14
This bug affects 2 people
Affects Status Importance Assigned to Milestone
Fuel for OpenStack
Fix Released
High
Aleksander Mogylchenko
6.0.x
Won't Fix
High
Fuel Library (Deprecated)

Bug Description

OpenVSwitch datapath module, shipped with CentOS 6.6 kernel has very important fix to OVS flow hashing algorithm.
Without this fix everyone with hping may hang-up OVS.
Additionally, OVS may hang up itself inder high enough load (1 Gbit and higher) and we already observing such issues at highly loaded customer installations.

Related issue and fix to revert:
https://bugs.launchpad.net/bugs/1397290

Changed in fuel:
importance: Undecided → Critical
milestone: none → 6.0
tags: added: customer-found
Changed in fuel:
assignee: nobody → Fuel Library Team (fuel-library)
Revision history for this message
Vladimir Kuklin (vkuklin) wrote :

As far as I see, this bug has been always present (if at all) in 1.10.2 as I do not see any info on it. We can try to to apply the suggested fix, but we have no guarantee that it will not break anything along with integration with other hardware and network appliances.

Considering that there is no clear bug description or link to the upstream bug tracker and the implications of such a change are unclear, I suggest to set this bug to high priority and move to 6.1 milestone for further investigation.

Revision history for this message
Dmitry Borodaenko (angdraug) wrote :

Looks like a duplicate of bug #1386110

tags: added: release-notes
Revision history for this message
Miroslav Anashkin (manashkin) wrote :

Related info
http://openvswitch.org/pipermail/dev/2013-April/026424.html

Workaround:
Either update kernel module openvswitch-kmod back to built-in, shipped with CentOS 6.6 kernel, or compile the latest OpenVSwitch 2.3 kernel module and use it.

Changed in fuel:
milestone: 6.0 → 6.1
status: New → Confirmed
no longer affects: fuel/6.1.x
Mike Scherbakov (mihgen)
no longer affects: fuel/6.0.x
Revision history for this message
Mike Scherbakov (mihgen) wrote :

We have updated kernel version in CentOS: https://review.fuel-infra.org/gitweb?p=packages/centos6/kernel.git;a=commit;h=b5eea5381bf6a2e0e5aa738395614bbac4db5114

In 5.1 we used openvswitch as separate module (not built-in one) due to some issue we had before. We could switch to built-in one now, however I consider this as too much of a change for 6.0 release, which is about to call for HCF and release in a couple of weeks from now.
We simply won't be able to get reliable answer on question whether it works in all possible combinations during the timeframe we have.
This actually seems to me more of a feature request - as it changes the way how things work. We are obviously way behind of the schedule for it. So, this has to be considered for 6.1, and I'm marking this bug as Won't fix for 6.0.

Changed in fuel:
milestone: 6.0 → 6.1
no longer affects: fuel/6.1.x
Revision history for this message
Bogdan Dobrelya (bogdando) wrote :

Please update the status of this issue do we have a plans to address it as a blueprint of resolve it as a bug for 6.1?

Changed in fuel:
status: Confirmed → Triaged
Revision history for this message
Bogdan Dobrelya (bogdando) wrote :

As Miroslav provided a w/a, the status of this issue looks triaged

Revision history for this message
Miroslav Anashkin (manashkin) wrote :

If we decide to switch to OVS 2.3 in 6.1 - this bug may be marked as invalid.
But please pay attention to VMWare NSX integration first, they depend on OVS kernel module version.

Revision history for this message
Andrey Danin (gcon-monolake) wrote :

It's up to NSX bits package to provide a proper version to OVS. I have no objections about upgrading OVS to 2.3.

Changed in fuel:
status: Triaged → Won't Fix
status: Won't Fix → Triaged
Revision history for this message
Bogdan Dobrelya (bogdando) wrote :

Thanks for the feedback, @Andrey. Let's update the status of this bug once we have a final decision.

Changed in fuel:
assignee: Fuel Library Team (fuel-library) → Aleksander Mogylchenko (amogylchenko)
status: Triaged → In Progress
Revision history for this message
OSCI Robot (oscirobot) wrote :

package openvswitch-kmod has been built for project packages/centos6/openvswitch-kmod
Package version == , package release ==

Changeset: https://review.fuel-infra.org/2783
project: packages/centos6/openvswitch-kmod
branch: 6.1
author: Aleksandr Mogylchenko
committer: Aleksandr Mogylchenko
subject: OpenVSwtich 2.3.1 LTS kernel module for CentOS
status: patchset-created

Files placed on repository:

NOTE: Changeset is not merged, created temporary package repository.
 repository URL: /centos-fuel-6.1-stable-2783/

Revision history for this message
OSCI Robot (oscirobot) wrote :

package openvswitch-kmod has been built for project packages/centos6/openvswitch-kmod
Package version == , package release ==

Changeset: https://review.fuel-infra.org/2783
project: packages/centos6/openvswitch-kmod
branch: 6.1
author: Aleksandr Mogylchenko
committer: Aleksandr Mogylchenko
subject: OpenVSwtich 2.3.1 LTS kernel module for CentOS
status: change-merged

Files placed on repository:

Changeset merged. Package placed on primary repository
 repository URL: /centos-fuel-6.1-stable/

Changed in fuel:
status: In Progress → Fix Committed
Revision history for this message
Fuel Devops McRobotson (fuel-devops-robot) wrote : Fix proposed to packages/centos6/openvswitch-kmod (6.1)

Fix proposed to branch: 6.1
Change author: Aleksandr Mogylchenko <email address hidden>
Review: https://review.fuel-infra.org/6474

Revision history for this message
Fuel Devops McRobotson (fuel-devops-robot) wrote : Fix proposed to packages/centos6/openvswitch-kmod (6.0.1)

Fix proposed to branch: 6.0.1
Change author: Aleksandr Mogylchenko <email address hidden>
Review: https://review.fuel-infra.org/6478

tags: added: release-notes-done
Stanislav Makar (smakar)
Changed in fuel:
status: Fix Committed → Fix Released
tags: removed: release-notes
Revision history for this message
OpenStack Infra (hudson-openstack) wrote : Related fix proposed to fuel-docs (stable/6.1)

Related fix proposed to branch: stable/6.1
Review: https://review.openstack.org/194961

Revision history for this message
OpenStack Infra (hudson-openstack) wrote : Related fix merged to fuel-docs (stable/6.1)
Download full text (45.4 KiB)

Reviewed: https://review.openstack.org/194961
Committed: https://git.openstack.org/cgit/stackforge/fuel-docs/commit/?id=0e26e7d7cc153d179ec34985645dd23cdd239ddb
Submitter: Jenkins
Branch: stable/6.1

commit 5cc5f0c643aebecaf3bf4580535a3ea7c3334a6c
Author: Mike Scherbakov <email address hidden>
Date: Tue Jun 23 13:43:35 2015 -0700

    Removed streamlined patching backend pieces

    Change-Id: I955e76ccdbd12a9145f4e9b689f80bdf9fcaf929

commit 563c4b5c78ebfcb1f4f91047c2919f6270f9a1d4
Author: Mike Scherbakov <email address hidden>
Date: Tue Jun 23 13:30:30 2015 -0700

    Removed outdated patching guide

    Change-Id: I76180c277789ade9c5ebedd19fe2092847c0b7d9

commit 8d120c14bec1ab41d448683ad146a3053a57c4ee
Author: Irina Povolotskaya <email address hidden>
Date: Tue Jun 23 19:59:11 2015 +0300

    Add dual hypervisor ref arch into 6.1 docs

    Change-Id: I900c24c9de878eafadbfc995aa879b7f55737fac

commit feebd1592d3305b64bbdfd0bc5fe108190aef120
Author: OlgaGusarenko <email address hidden>
Date: Tue Jun 23 18:38:17 2015 +0300

    [OPs guide] Running Ceilometer section edits

    1. conf file extract is updated
    2. note is updated

    Closes-bug: 1467817
    Change-Id: I0217e164108e0ba6c1397045a5e57d13ff429223

commit 44a93f9dead7511a3461ec35248dbb689c81eafd
Author: OlgaGusarenko <email address hidden>
Date: Tue Jun 23 18:04:40 2015 +0300

    [RN6_1] Final changes

    1. capitalization
    2. 2014.2 to 2014.2.2
    3. general improvements

    Change-Id: I45057e90c90550559f66bc67ccdf97a559fd9000

commit bb41389cae58084285688853281516b659686422
Author: evkonstantinov <email address hidden>
Date: Tue Jun 23 16:45:35 2015 +0300

    Update patching decription

    Update patching description with
    the standard Linux commands.

    Change-Id: Ia1a8346639c468fdfce15a11d2430bf3a4731244

commit bf3018fae3f2e564413d33aba6cdebf8868f0b4e
Author: OlgaGusarenko <email address hidden>
Date: Tue Jun 23 15:55:49 2015 +0300

    [RN6_1] Clean up

    1. Rearranges sections
    2. Improves RST
    3. Changes titles order

    Change-Id: I6110bf515667d3d6ba08ad35ff5d593dbc96641e

commit 1c7e4457808e8f2d6c56fdf31252170972e444b9
Author: Maria Zlatkova <email address hidden>
Date: Tue Jun 23 15:26:28 2015 +0300

    Replaces VBOX screenshots

    This patch:
    - replaces VBOX screenshots
    - changes the link for Download Mirantis VirtualBox scripts
     to https://docs.mirantis.com/openstack/fuel/fuel-master/#downloads

    Change-Id: I58dede960c5c3355d39b07ff44b757403f6af02c
    Closes-Bug: #1467872

commit 0a568bf53fc0e25d1d692d5d74b4a7b4d983bbcc
Author: evkonstantinov <email address hidden>
Date: Tue Jun 23 14:01:55 2015 +0300

    6.1 --separate repos

    change wording and add links to the
    separate repos feature.

    Change-Id: Ib5d0778a0d8f1534f79ed2f553574cb69a3150b0

commit 95a188b21cbdd064d92696b7920e6a0105fe0c56
Author: Maria Zlatkova <email address hidden>
Date: Tue Jun 23 12:07:28 2015 +0300

    Corrects the output 'pcs status'

    Changes the example outputs to appropriate ones.

    Change-Id: Ib6d83...

Revision history for this message
Fuel Devops McRobotson (fuel-devops-robot) wrote : Change abandoned on packages/centos6/openvswitch-kmod (6.1)

Change abandoned by Albert Syriy <email address hidden> on branch: 6.1
Review: https://review.fuel-infra.org/6474
Reason: Not need any more

Revision history for this message
Fuel Devops McRobotson (fuel-devops-robot) wrote : Change abandoned on packages/centos6/openvswitch-kmod (6.0.1)

Change abandoned by Albert Syriy <email address hidden> on branch: 6.0.1
Review: https://review.fuel-infra.org/6478
Reason: Not need any more

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.