Skip to content

feat(storcli2): physical drive blinker#72

Open
g-carre wants to merge 1 commit into
feature/storcli2-lv-manager-pd-membershipfrom
feature/storcli2-blinker
Open

feat(storcli2): physical drive blinker#72
g-carre wants to merge 1 commit into
feature/storcli2-lv-manager-pd-membershipfrom
feature/storcli2-blinker

Conversation

@g-carre

@g-carre g-carre commented Jun 25, 2026

Copy link
Copy Markdown
Contributor

Summary

Implements ports.Blinker on the storcli2/perccli2 adapter (ARTESCA-17650). Stacked on #70.

  • StartBlink / StopBlink — issue /cx/ex/sx start locate / stop locate (same grammar as storcli), pick the enclosure or no-enclosure selector form from the drive's parsed slot, and surface the in-JSON failure payload via storcli2.Decode.

A single implementation serves both binaries; the runner is injected.

Testing

go build, go vet, gofmt, and the package test suite pass. Tests cover start/stop locate against the blink/storcli2/{start,stop}.json fixtures (enclosure and no-enclosure selectors), the runner-error path, and an invalid-slot rejection.

Note: Targets feature/storcli2-lv-manager-pd-membership (#70), not main, per the stack. Diff narrows as the lower PRs merge.

Issue: ARTESCA-17650

🤖 Generated with Claude Code

Implement ports.Blinker on the storcli2/perccli2 adapter:

- StartBlink / StopBlink issue "/cx/ex/sx start locate" / "stop locate"
  (same grammar as storcli), choosing the enclosure or no-enclosure
  selector form from the drive's parsed slot, and surface the in-JSON
  failure payload via storcli2.Decode.

A single implementation serves both binaries; the runner is injected.

Issue: ARTESCA-17650
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.

1 participant