Skip to content

✨ Update ReportStatusRequestFromAuditConfig to include skipContainerResolution parameter#1481

Merged
slntopp merged 2 commits intomainfrom
mik/fix-air-gapped-status-reporter
Apr 22, 2026
Merged

✨ Update ReportStatusRequestFromAuditConfig to include skipContainerResolution parameter#1481
slntopp merged 2 commits intomainfrom
mik/fix-air-gapped-status-reporter

Conversation

@slntopp
Copy link
Copy Markdown
Member

@slntopp slntopp commented Apr 22, 2026

Resolves #1472

mondoo-code-review[bot]
mondoo-code-review Bot previously approved these changes Apr 22, 2026
Copy link
Copy Markdown

@mondoo-code-review mondoo-code-review Bot left a comment

Choose a reason for hiding this comment

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

Air-gapped environments will now correctly skip container image resolution during status reporting.

Comment thread controllers/status/operator_status_test.go
Copy link
Copy Markdown

@mondoo-code-review mondoo-code-review Bot left a comment

Choose a reason for hiding this comment

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

Previous suggestion addressed: tests now cover skipContainerResolution in both true and false cases.

@github-actions
Copy link
Copy Markdown
Contributor

github-actions Bot commented Apr 22, 2026

Test Results

  7 files  ±0   44 suites  ±0   1h 36m 2s ⏱️ - 3m 9s
465 tests +3  465 ✅ +3  0 💤 ±0  0 ❌ ±0 
531 runs  +3  525 ✅ +3  6 💤 ±0  0 ❌ ±0 

Results for commit e8b6560. ± Comparison against base commit 4f86164.

♻️ This comment has been updated with latest results.

@slntopp slntopp merged commit 2901c34 into main Apr 22, 2026
30 checks passed
@slntopp slntopp deleted the mik/fix-air-gapped-status-reporter branch April 22, 2026 13:46
@github-actions github-actions Bot locked and limited conversation to collaborators Apr 22, 2026
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Status reporter ignores skipContainerResolution

1 participant