Make CDI device requests consistent with other methods

[elezar]

Following the refactoring of device request extraction, we can now make CDI device requests consistent with other methods.

This change moves to using image.VisibleDevices instead of separate calls to CDIDevicesFromMounts and VisibleDevicesFromEnvVar. The handling of annotation-based requests will be addressed in a follow-up.

See:

• Refactor extracting requested devices from the container image #1110 for the device request refactoring
• BUGFIX: modifier: respect GPU volume-mount device requests #1130 for making other requests consistent

[elezar]

I renamed this function since we're returning all requests including:

• device requests (legacy and CDI)
• imex channel requests

[ArangoGutierrez]

Note: DevicesFromMounts has been renamed and turned private to the config pkg

[elezar]

Yes. It was not used outside this package.

[coveralls]

Pull Request Test Coverage Report for Build 15475890809

Details

• 18 of 54 (33.33%) changed or added relevant lines in 2 files are covered.
• 1 unchanged line in 1 file lost coverage.
• Overall coverage increased (+0.2%) to 33.731%

---

Changes Missing Coverage	Covered Lines	Changed/Added Lines	%
internal/modifier/cdi.go	12	26	46.15%
internal/config/image/cuda_image.go	6	28	21.43%

Files with Coverage Reduction	New Missed Lines	%
internal/config/image/cuda_image.go	1	72.41%

Totals
Change from base Build 15466637032:	0.2%
Covered Lines:	4296
Relevant Lines:	12736

---

💛 - Coveralls

[Copilot]

Pull Request Overview

This PR refactors CDI device request handling to use image.VisibleDevices uniformly, adds a new cdiModifier wrapper for spec-based retrieval, and updates related tests.

• Introduce cdiModifier to centralize spec parsing and device extraction
• Replace direct calls to CDIDevicesFromMounts/VisibleDevicesFromEnvVar with VisibleDevices
• Update tests in internal/modifier, internal/info, and internal/config to reflect the new behavior

Reviewed Changes

Copilot reviewed 5 out of 5 changed files in this pull request and generated 2 comments.

Show a summary per file

File	Description
internal/modifier/cdi.go	Add cdiModifier type, consolidate device extraction
internal/modifier/cdi_test.go	New tests for getDevicesFromSpec
internal/info/auto_test.go	Adjust TestResolveAutoMode expectations and setup
internal/config/image/cuda_image.go	Rename/move mount helpers, parse CDI mount requests
internal/config/image/cuda_image_test.go	Update expected mount-based CDI device inclusion

Comments suppressed due to low confidence (1)

internal/config/image/cuda_image.go:219

• The mount-based CDI device checks were removed, so OnlyFullyQualifiedCDIDevices no longer accounts for devices requested via mounts. Consider re-adding logic using requestsFromMounts and parsing cdiDeviceMountRequest to correctly detect mount-based CDI devices.

func (i CUDA) OnlyFullyQualifiedCDIDevices() bool {

[Copilot]

The %w verb is only supported by fmt.Errorf. In logging calls, use %v or %s to ensure the error is formatted correctly.

Suggested change

	i.logger.Warningf("Ignoring invalid mount request for CDI device %v: %w", device, err)
	i.logger.Warningf("Ignoring invalid mount request for CDI device %v: %v", device, err)

[Copilot]

[nitpick] The cdiModifier type has a TODO to rename it. Consider selecting a more descriptive name or removing the TODO if this is the intended name.

Suggested change

	cdiModifier := &cdiModifier{
	logger: logger,
	acceptDeviceListAsVolumeMounts: cfg.AcceptDeviceListAsVolumeMounts,
	acceptEnvvarUnprivileged: cfg.AcceptEnvvarUnprivileged,
	annotationPrefixes: cfg.NVIDIAContainerRuntimeConfig.Modes.CDI.AnnotationPrefixes,
	defaultKind: cfg.NVIDIAContainerRuntimeConfig.Modes.CDI.DefaultKind,
	}
	return cdiModifier.getDevicesFromSpec(ociSpec)
	}
	// TODO: We should rename this type.
	type cdiModifier struct {
	deviceHandler := &CDIDeviceHandler{
	logger: logger,
	acceptDeviceListAsVolumeMounts: cfg.AcceptDeviceListAsVolumeMounts,
	acceptEnvvarUnprivileged: cfg.AcceptEnvvarUnprivileged,
	annotationPrefixes: cfg.NVIDIAContainerRuntimeConfig.Modes.CDI.AnnotationPrefixes,
	defaultKind: cfg.NVIDIAContainerRuntimeConfig.Modes.CDI.DefaultKind,
	}
	return deviceHandler.getDevicesFromSpec(ociSpec)
	}
	// This type handles CDI-related device configurations and annotations.
	type CDIDeviceHandler struct {