ctl: require ctladm in addition to sg_opcodes
ClosedPublic
Actions

Authored by olivier on Apr 22 2026, 7:48 AM.

Tags

None

Referenced Files

	F168195526: D56568.diff
	Wed, Aug 26, 8:03 PM

	F168086820: D56568.id.diff
	Wed, Aug 26, 7:50 AM

	Unknown Object (File)
	Fri, Aug 21, 2:34 PM

	Unknown Object (File)
	Tue, Aug 18, 8:37 PM

	Unknown Object (File)
	Tue, Aug 18, 4:16 AM

	Unknown Object (File)
	Sat, Aug 15, 12:47 AM

	Unknown Object (File)
	Thu, Aug 13, 8:39 AM

	Unknown Object (File)
	Sun, Aug 9, 8:27 PM

View All 47 Files

Subscribers

imp

Details

Reviewers

asomers

Group Reviewers

cam

Commits

rG19213b8eb33f: ctl: require ctladm in addition to sg_opcodes

Summary

Each test case in opcodes.sh uses ctladm to create and remove the LUN it exercises, but only sg_opcodes was listed in require.progs. On systems where ctladm is not builded the tests would fail at setup instead of being skipped cleanly.

Diff Detail

Repository

rG FreeBSD src repository

Lint

Lint Not Applicable

Unit

Tests Not Applicable

Event Timeline

olivier created this revision.Apr 22 2026, 7:48 AM

Herald added a reviewer: cam. · View Herald TranscriptApr 22 2026, 7:48 AM

olivier requested review of this revision.Apr 22 2026, 7:48 AM

olivier added a reviewer: asomers.

Good catch.

This revision is now accepted and ready to land.Apr 22 2026, 12:47 PM

Closed by commit rG19213b8eb33f: ctl: require ctladm in addition to sg_opcodes (authored by olivier). · Explain WhyApr 22 2026, 2:17 PM

This revision was automatically updated to reflect the committed changes.

olivier added a commit: rG19213b8eb33f: ctl: require ctladm in addition to sg_opcodes.

Herald added a subscriber: imp. · View Herald TranscriptApr 22 2026, 2:17 PM

Revision Contents
Changeset List

Path

Size

tests/

sys/

cam/

ctl/

opcodes.sh

18 lines

Diff 176112

View Options

tests/sys/cam/ctl/opcodes.sh

ctl: require ctladm in addition to sg_opcodesClosedPublicActions