Skip to content

Fix Podman compatibility for Prowler image pull#19

Merged
carlosinfantes merged 3 commits into
devfrom
fix/podman-prowler-image-prefix
Mar 6, 2026
Merged

Fix Podman compatibility for Prowler image pull#19
carlosinfantes merged 3 commits into
devfrom
fix/podman-prowler-image-prefix

Conversation

@jorgemarti
Copy link
Copy Markdown
Collaborator

Add docker.io/ registry prefix to PROWLER_IMAGE so Podman can resolve the image name without unqualified-search registries configured.

Checklist

  • Pre-commit hooks pass (pre-commit run --all-files)
  • No real AWS account IDs or credentials
  • Tests pass (pytest / npm test)
  • CDK synthesizes (cd infrastructure && npm run build && npx cdk synth)

Add docker.io/ registry prefix to PROWLER_IMAGE so Podman can resolve
the image name without unqualified-search registries configured.
@carlosinfantes
Copy link
Copy Markdown
Owner

Hey Jorge! CI is failing on ruff lint — 11 errors in the test files (unused imports and unused variables).

Quick fix from lambdas/:

ruff check . --fix

This auto-fixes the 7 fixable ones (unused imports). For the 4 F841 (unused result variables), either prefix with _ or remove the assignment:

# Before:
result = analyzer.analyze()

# After (if you don't need the return value):
analyzer.analyze()

I'll push a fix commit to your branch to unblock this. 🤝

Carlos Infantes and others added 2 commits March 6, 2026 16:04
Remove unused imports and unused variable assignments flagged by ruff.

Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
@carlosinfantes carlosinfantes merged commit 67ce14f into dev Mar 6, 2026
4 checks passed
@carlosinfantes carlosinfantes deleted the fix/podman-prowler-image-prefix branch March 6, 2026 15:59
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.

2 participants