Description of problem: By default, mellanox plugin sets total VFs to 0 for mellanox interfaces with no spec (user specified). however, this is only applied to unsupported NICs instead of supported NICs due to a wrong conditional check. Version-Release number of selected component (if applicable): How reproducible: 100% Steps to Reproduce: 1. deploy sriov operator in a cluster which contains unsupported Mellanox NICs. for the list of supported NICs, refers to openshift official document: https://docs.openshift.com/container-platform/4.5/networking/hardware_networks/about-sriov.html#supported-devices_about-sriov 2. 3. Actual results: mellanox plugin tries to set total VFs to 0 for unsupported NICs even it's not configured. Expected results: mellanox plugin doesn't touch unsupported NICs, if not explicitly configured. Additional info:
Verified this bug on 4.6.0-202008240540.p0 1. Create one policy and delete it. 2. ssh into config daemon pod 3. Check the num_of_vfs changed to 0. sh-4.2# mstconfig -d 5e:00.0 q | grep -i num_of_vfs NUM_OF_VFS 0
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 (OpenShift Container Platform 4.6 GA Images), 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-2020:4196