Skip to content

Conversation

@adrianchiris
Copy link
Collaborator

@adrianchiris adrianchiris commented Aug 18, 2025

use devlink to get VF and SF representors.
fallback to sysfs in case of failure.

@adrianchiris
Copy link
Collaborator Author

depends on: vishvananda/netlink#1115

@adrianchiris
Copy link
Collaborator Author

vishvananda/netlink#1115 was merged.

work can be resumed on this MR

@adrianchiris adrianchiris force-pushed the get-representor-devlink branch from c93214e to ce55a2d Compare November 23, 2025 18:26
use devlink to get VF and SF representors.
fallback to sysfs in case of failure.

Signed-off-by: adrianc <[email protected]>
@adrianchiris adrianchiris force-pushed the get-representor-devlink branch from ce55a2d to 9a6a167 Compare November 24, 2025 09:07
@coveralls
Copy link

Pull Request Test Coverage Report for Build 19628897106

Details

  • 89 of 550 (16.18%) changed or added relevant lines in 4 files are covered.
  • 3 unchanged lines in 2 files lost coverage.
  • Overall coverage decreased (-4.7%) to 31.53%

Changes Missing Coverage Covered Lines Changed/Added Lines %
sriovnet_helper.go 6 11 54.55%
pkg/utils/netlinkops/netlinkops.go 0 11 0.0%
sriovnet_switchdev.go 43 56 76.79%
pkg/utils/netlinkops/mocks/mocks.go 40 472 8.47%
Files with Coverage Reduction New Missed Lines %
sriovnet_helper.go 1 7.5%
sriovnet_switchdev.go 2 83.83%
Totals Coverage Status
Change from base Build 19472276574: -4.7%
Covered Lines: 548
Relevant Lines: 1738

💛 - Coveralls

@adrianchiris adrianchiris changed the title WIP: Get representor devlink use devlink to get representor name Nov 24, 2025
@adrianchiris
Copy link
Collaborator Author

@SchSeba @zeeke PTAL

Copy link
Collaborator

@SchSeba SchSeba left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

lgtm just a small comment from my side

@SchSeba SchSeba merged commit 65c5d34 into k8snetworkplumbingwg:master Dec 2, 2025
5 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants