Note: This bug is displayed in read-only format because the product is no longer active in Red Hat Bugzilla.

Bug 1750612

Summary: [BACKPORT] backport of upstream patch to Replace coremask with cpu_list in vrouter dpdk schema
Product: Red Hat OpenStack Reporter: PURANDHAR SAIRAM MANNIDI <pmannidi>
Component: os-net-configAssignee: RHOS Maint <rhos-maint>
Status: CLOSED ERRATA QA Contact: nlevinki <nlevinki>
Severity: medium Docs Contact:
Priority: medium    
Version: 14.0 (Rocky)CC: bfournie, bshephar, dsedgmen, hbrock, jslagle, mburns, nlevinki, rhos-maint, skramaja, tkajinam
Target Milestone: z4Keywords: Triaged, ZStream
Target Release: 14.0 (Rocky)   
Hardware: Unspecified   
OS: Unspecified   
Whiteboard:
Fixed In Version: os-net-config-9.3.1-0.20190513134526.9ed8760.el7ost Doc Type: If docs needed, set a value
Doc Text:
Story Points: ---
Clone Of: 1750259 Environment:
Last Closed: 2019-11-06 16:48:54 UTC Type: ---
Regression: --- Mount Type: ---
Documentation: --- CRM:
Verified Versions: Category: ---
oVirt Team: --- RHEL 7.3 requirements from Atomic Host:
Cloudforms Team: --- Target Upstream Version:
Embargoed:
Bug Depends On: 1750259    
Bug Blocks:    

Description PURANDHAR SAIRAM MANNIDI 2019-09-10 04:50:41 UTC
+++ This bug was initially created as a clone of Bug #1750259 +++

Description of problem:
Os-net-config validation throws error when cpu_list parameter is used for vrouter dpdk nic type.

Sep  4 09:00:07 compute1 os-collect-config: [2019/09/04 08:57:03 AM] [WARNING] Config file failed schema validation at network_config/4:
Sep  4 09:00:07 compute1 os-collect-config: {'persist_mapping': False, 'addresses': [{'ip_netmask': '172.16.0.12/23'}], 'cpu_list': '0x1554', 'nic_mapping': None, 'driver': 'vfio-pci', 'mtu': 9000, 'bond_
mode': 4, 'members': [{'type': 'interface', 'name': 'p1p1', 'mtu': 9096}, {'type': 'interface', 'name': 'p1p2', 'mtu': 9096}], 'use_dhcp': False, 'bond_policy': 'layer3+4', 'type': 'contrail_vrouter_dpdk', 'name': 'vhost0'} is not valid under any of the given schemas
Sep  4 09:00:07 compute1 os-collect-config: Sub-schemas tested and not matching:
Sep  4 09:00:07 compute1 os-collect-config: - items/oneOf/contrail_vrouter_dpdk/additionalProperties: Additional properties are not allowed ('cpu_list' was unexpected)

RH OpenStack Platform 13

How reproducible:
Always

Actual results:
os-net-config throws validation error

Expected results:
os-net-config validation should succeed

Additional info:
This should be backport of https://review.opendev.org/#/c/601615/1

--- Additional comment from PURANDHAR SAIRAM MANNIDI on 2019-09-09 07:45:47 UTC ---

There was a typo with MTU.

Sep  4 09:00:07 compute1 os-collect-config: [2019/09/04 08:57:03 AM] [WARNING] Config file failed schema validation at network_config/4:
Sep  4 09:00:07 compute1 os-collect-config: {'persist_mapping': False, 'addresses': [{'ip_netmask': '172.16.0.12/23'}], 'cpu_list': '0x1554', 'nic_mapping': None, 'driver': 'vfio-pci', 'mtu': 9000, 'bond_
mode': 4, 'members': [{'type': 'interface', 'name': 'p1p1', 'mtu': 9000}, {'type': 'interface', 'name': 'p1p2', 'mtu': 9000}], 'use_dhcp': False, 'bond_policy': 'layer3+4', 'type': 'contrail_vrouter_dpdk', 'name': 'vhost0'} is not valid under any of the given schemas
Sep  4 09:00:07 compute1 os-collect-config: Sub-schemas tested and not matching:
Sep  4 09:00:07 compute1 os-collect-config: - items/oneOf/contrail_vrouter_dpdk/additionalProperties: Additional properties are not allowed ('cpu_list' was unexpected)

Comment 2 Bob Fournier 2019-10-30 16:48:55 UTC
With puddle 2019-10-18.1, verified that schema is updated in /usr/lib/python2.7/site-packages/os_net_config/schema.yaml

   contrail_vrouter_dpdk:
        type: object
        properties:
            type:
                enum: ["contrail_vrouter_dpdk"]
            name:
                $ref: "#/definitions/string_or_param"
            members:
                type: array
                items:
                  - $ref: "#/definitions/interface"
                minItems: 1
            bond_mode:
                oneOf:
                  - $ref: "#/definitions/string_or_param"
                  - $ref: "#/definitions/int_or_param"
            bond_policy:
                oneOf:
                  - $ref: "#/definitions/string_or_param"
                  - $ref: "#/definitions/int_or_param"
            driver:
                $ref: "#/definitions/string_or_param"
            cpu_list:
                $ref: "#/definitions/string_or_param"

Comment 4 errata-xmlrpc 2019-11-06 16:48:54 UTC
Since the problem described in this bug report should be
resolved in a recent advisory, it has been closed with a
resolution of ERRATA.

For information on the advisory, and where to find the updated
files, follow the link below.

If the solution does not work for you, open a new bug report.

https://access.redhat.com/errata/RHBA-2019:3745