[dpdk] core mask should not be 0x0

Bug #1562947 reported by Vladimir Eremin on 2016-03-28
6
This bug affects 1 person
Affects Status Importance Assigned to Milestone
Fuel for OpenStack
High
Fedor Zhadaev

Bug Description

ovs_pmd_core_mask and ovs_core_mask should not be 0x0, because it's valid value that means disabling core.
If core is not choosen, value should not be present.

How to reproduce:
Set 1 CPU for DPDK

invalid:
dpdk:
  enabled: true
  ovs_socket_mem: '128'
  ovs_pmd_core_mask: "0x0"
  ovs_core_mask: "0x1"

valid:
dpdk:
  enabled: true
  ovs_socket_mem: '128'
  ovs_core_mask: "0x1"

description: updated

(This check performed automatically)
Please, make sure that bug description contains the following sections filled in with the appropriate data related to the bug you are describing:

actual result

version

expected result

steps to reproduce

For more detailed information on the contents of each of the listed sections see https://wiki.openstack.org/wiki/Fuel/How_to_contribute#Here_is_how_you_file_a_bug

tags: added: need-info
Fedor Zhadaev (fzhadaev) on 2016-03-29
Changed in fuel:
assignee: Fuel Python Team (fuel-python) → Fedor Zhadaev (fzhadaev)

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

Changed in fuel:
status: New → In Progress

Reviewed: https://review.openstack.org/298639
Committed: https://git.openstack.org/cgit/openstack/fuel-web/commit/?id=b31efe6cb128aa4d1c0828ed3e7ab4510f1f2427
Submitter: Jenkins
Branch: master

commit b31efe6cb128aa4d1c0828ed3e7ab4510f1f2427
Author: Fedor Zhadaev <email address hidden>
Date: Tue Mar 29 13:16:13 2016 +0300

    Fix bug with 0x0 core mask

    Exclude ovs_pmd_core_mask from serialization if it contains no CPUs.

    Change-Id: I640aba1f699137e42c05326a281d41d3153c2fb7
    Closes-Bug: #1562947

Changed in fuel:
status: In Progress → Fix Committed
Maksym Strukov (unbelll) wrote :

Verified as fixed in 9.0-mos-364

Changed in fuel:
status: Fix Committed → Fix Released
To post a comment you must log in.
This report contains Public information  Edit
Everyone can see this information.

Other bug subscribers