-
Notifications
You must be signed in to change notification settings - Fork 0
Closed
Description
Context
ConfighubScanProvider.ScanCluster() currently delegates entirely to the legacy provider. cub-scan is a static file scanner with no cluster-scanning mode.
What's needed
To use cub-scan for cluster scanning:
- Export live cluster resources to temp files via K8s API
- Invoke
cub-scan --format jsonon each exported manifest - Merge cub-scan findings with runtime scan results (state, timing bombs, dangling, lifecycle hazards) which inherently require live K8s APIs and stay on the legacy provider
Current state
ScanFile()is fully wired tocub-scan(shipped in v1.2, v1.2: Wire ConfighubScanProvider.ScanFile() to cub-scan binary #190)ScanCluster()falls back to legacy — safe, correct, documented- Comment in
internal/scan/confighub_provider.goexplains the limitation
Acceptance criteria
- Cluster resources exported to temp directory (namespace-scoped and cluster-scoped)
-
cub-scaninvoked on exported manifests when binary is available - Runtime scan types (state, timing bombs, dangling, lifecycle hazards) remain on legacy provider
- Graceful fallback to legacy if cub-scan fails or isn't available
- Integration test with fake cub-scan binary (follows existing pattern in
test/integration/scan_provider_test.go)
Spun out from v1.2 (#190) per roadmap known-limitation note.
Reactions are currently unavailable
Metadata
Metadata
Assignees
Labels
No labels