SchSeba / sriov-network-operator-1

63%

master: 63%

LAST BUILD ON BRANCH support_vfio_ib_in_webhook

branch: SELECT

CHANGE BRANCH

x

Sync Branches

• No branch selected
• add_device_injector_validation
• add_hugepages_func_test
• add_logs
• add_mtu_func_test
• add_predicate_for_nodes
• add_rbac_to_nri
• add_rdma_documentation
• add_resync
• add_retry_on_delete
• allow_to_remove_default_pool
• block_resource_injector
• bug_remove_nic_config_from_host
• bump-dependencies-pr1027
• bump_cni_version
• bump_controller_runtime_0.20.2
• bump_ginkgo_version
• bump_k8s_1.30
• bump_k8s_version_ci
• bump_mock_package
• bump_ocp_version_4.18
• bumps_1.24
• change_vlan_max
• check_MCP_after_pod_restart
• check_lock_down_on_mlx_plugin
• ci_improvements
• clean_ci_images
• conditions/github-action-update
• conditions/network-crds
• conditions/nodestate
• conditions/operator-config
• conditions/policy-poolconfig
• conditions/redeploy-script-fix
• create_desing_proposal_template
• create_platform_and_orchestrator_packages
• daemon_redesign
• design_conditions
• design_conditions_implementation
• design_externally_manage
• device_plugin_redesign
• documentation-re-org
• documentation_update
• drain_controller
• drain_daemon
• drain_improve
• enable_delve
• enable_mtu_test
• extend_time
• externally_manage
• externally_manage_tests
• fast-leader-election
• fix-webhook-image-user
• fix_device_plugin_label
• fix_drain_handler
• fix_empty_nodeselector_in_policy
• fix_githubaction_artifact
• fix_ip_link_command_tests
• fix_k8s_ci
• fix_k8s_ci_multus
• fix_k8s_ci_virtual_cluster
• fix_k8s_ci_virtual_cluster_1
• fix_lock_race
• fix_log_test
• fix_matchcondition
• fix_mlx_plugin
• fix_mlx_plugin_1
• fix_mtu_disconnected_test
• fix_mtu_test_1
• fix_ocp_ci
• fix_ocp_ci_login
• fix_policy_order
• fix_race_condition
• fix_render
• fix_return_error
• fix_root_device_overlap
• fix_sriov_test_for_sno
• fix_status_patch
• fix_test
• fix_unittest
• fix_virtual_cluster_script
• fix_waiting_for_dp_when_no_policy
• fix_webhook
• func_e2e_fixes
• improve_drain_check
• improve_hypershift_check
• improve_logs
• improve_nic_selection
• interface_redesign
• master
• move_systemd_to_host_package
• multiple_bumps
• parallel_drain
• parallel_drain_design
• platform_and_orchestrator_packages_design
• rdma-functest
• rdma-subsytem-mode
• reduce_rbac_resource_injector
• refs/pull/3/merge
• remove_DP_from_nodestate_spec
• remove_client
• remove_drain_on_none_vfs
• remove_leader_election
• remove_leader_election_test
• remove_timeout
• run_make_bundle
• security_improvements
• skip_drain_on_reboot
• skip_drain_on_reboot_single_node
• skip_virtual_nic_udev
• support_BF2_connectx
• support_PSA
• support_aws
• support_ib_netlink
• support_k8s_cluster_version
• support_vfio_ib_in_webhook
• support_virtual_cluster
• sync-2025-02-03
• systemd
• temp_fix_ocp_ci
• unit_test_no_mock
• unit_tests
• unit_tests_1
• update_bundle_to_stable
• update_ci_system
• update_github_actions
• update_ocp_ci_version
• use_context_to_annotate
• virtual_webhook

Build # 22179240298

Build Type

push

github

Committed by SchSeba

Commit Message

fix: skip isRdma mutation for vfio-pci with InfiniBand linkType

The mutating webhook unconditionally set isRdma to true for policies
with InfiniBand link type, even when deviceType was vfio-pci. This
caused the validating webhook to reject the resource since deviceType
vfio-pci conflicts with isRdma: true.

Check the deviceType before applying the InfiniBand isRdma patch and
skip it when deviceType is vfio-pci.

Add unit tests for mutateSriovNetworkNodePolicy covering default
values, InfiniBand with various device types, Ethernet, and
preservation of user-specified fields.

Signed-off-by: Sebastian Sch <sebassch@gmail.com>

Run Details

12 of 12 new or added lines in 1 file covered. (100.0%)

5 existing lines in 1 file now uncovered.

9342 of 14799 relevant lines covered (63.13%)

0.7 hits per line