chore(docker): update docker#11601
Conversation
|
✅ Conflict Markers Resolved All conflict markers have been successfully resolved in this pull request. |
66ff82d to
1657cc9
Compare
|
Note Reviews pausedIt looks like this branch is under active development. To avoid overwhelming you with review comments due to an influx of new commits, CodeRabbit has automatically paused this review. You can configure this behavior by changing the Use the following commands to manage reviews:
Use the checkboxes below for quick actions:
📝 WalkthroughWalkthroughThe default value of the ChangesDocker Image Digest Bump
Estimated code review effort🎯 1 (Trivial) | ⏱️ ~2 minutes 🚥 Pre-merge checks | ✅ 3 | ❌ 2❌ Failed checks (1 warning, 1 inconclusive)
✅ Passed checks (3 passed)
✏️ Tip: You can configure your own custom pre-merge checks in the settings. ✨ Finishing Touches📝 Generate docstrings
🧪 Generate unit tests (beta)
Comment |
There was a problem hiding this comment.
Actionable comments posted: 1
🤖 Prompt for all review comments with AI agents
Verify each finding against current code. Fix only still-valid issues, skip the
rest with a brief reason, keep changes minimal, and validate.
Inline comments:
In `@contrib/aws/multi-account-securityhub/Dockerfile`:
- Line 4: The PROWLER_VERSION argument in the Dockerfile uses an invalid Docker
image digest that does not exist in the official toniblyx/prowler Docker Hub
repository, which will cause build failures. Replace the digest value in the ARG
PROWLER_VERSION line with a valid tag such as `latest` or `5.30.1` from the
official Docker Hub repository.
🪄 Autofix (Beta)
Fix all unresolved CodeRabbit comments on this PR:
- Push a commit to this branch (recommended)
- Create a new PR with the fixes
ℹ️ Review info
⚙️ Run configuration
Configuration used: Organization UI
Review profile: ASSERTIVE
Plan: Pro Plus
Run ID: b0472b3d-8afd-401b-b861-b91b558f9d7f
📒 Files selected for processing (1)
contrib/aws/multi-account-securityhub/Dockerfile
1657cc9 to
15eefaa
Compare
There was a problem hiding this comment.
Actionable comments posted: 1
🤖 Prompt for all review comments with AI agents
Verify each finding against current code. Fix only still-valid issues, skip the
rest with a brief reason, keep changes minimal, and validate.
Inline comments:
In `@contrib/aws/multi-account-securityhub/Dockerfile`:
- Line 4: The PROWLER_VERSION ARG in the Dockerfile contains an invalid image
digest that does not exist on Docker Hub. Replace the current digest in the ARG
PROWLER_VERSION line with either a valid known version tag such as 5.30.1 or use
just the latest tag without a custom digest appended. Verify the correct digest
on Docker Hub before applying the change to ensure the image exists and is
available for pulling during the build process.
🪄 Autofix (Beta)
Fix all unresolved CodeRabbit comments on this PR:
- Push a commit to this branch (recommended)
- Create a new PR with the fixes
ℹ️ Review info
⚙️ Run configuration
Configuration used: Organization UI
Review profile: ASSERTIVE
Plan: Pro Plus
Run ID: 4e2b8296-52a5-442c-a139-2a3b1e6783fb
📒 Files selected for processing (1)
contrib/aws/multi-account-securityhub/Dockerfile
6999f9a to
f78e420
Compare
There was a problem hiding this comment.
♻️ Duplicate comments (1)
contrib/aws/multi-account-securityhub/Dockerfile (1)
4-4:⚠️ Potential issue | 🔴 CriticalRe-pin this digest to a published image.
sha256:f61e0dfb4b07eebdef884392955ba799b9c77d321ced1578bfd403814f4aec8ddoes not appear on the currenttoniblyx/prowlertags page, whilelatestand5.28.1currently resolve to different digests (c1513c235682…andb5ad69727547…), so thisFROMreference will still fail to pull. Please re-verify the exact digest before merging. (hub.docker.com)🤖 Prompt for AI Agents
Verify each finding against current code. Fix only still-valid issues, skip the rest with a brief reason, keep changes minimal, and validate. In `@contrib/aws/multi-account-securityhub/Dockerfile` at line 4, The ARG PROWLER_VERSION digest in the Dockerfile does not correspond to a published image on Docker Hub for toniblyx/prowler. Verify the correct digest by checking the current tags page on Docker Hub (hub.docker.com/r/toniblyx/prowler/tags) and update the ARG PROWLER_VERSION line to use an actual published digest that exists for a current version like latest or 5.28.1, ensuring the image can be successfully pulled during the build process.
🤖 Prompt for all review comments with AI agents
Verify each finding against current code. Fix only still-valid issues, skip the
rest with a brief reason, keep changes minimal, and validate.
Duplicate comments:
In `@contrib/aws/multi-account-securityhub/Dockerfile`:
- Line 4: The ARG PROWLER_VERSION digest in the Dockerfile does not correspond
to a published image on Docker Hub for toniblyx/prowler. Verify the correct
digest by checking the current tags page on Docker Hub
(hub.docker.com/r/toniblyx/prowler/tags) and update the ARG PROWLER_VERSION line
to use an actual published digest that exists for a current version like latest
or 5.28.1, ensuring the image can be successfully pulled during the build
process.
ℹ️ Review info
⚙️ Run configuration
Configuration used: Organization UI
Review profile: ASSERTIVE
Plan: Pro Plus
Run ID: 723bc930-e2e3-4b05-b53c-ffb95420360f
📒 Files selected for processing (1)
contrib/aws/multi-account-securityhub/Dockerfile
0d094e7 to
0f4948d
Compare
ebc99d4 to
fc7e73e
Compare
fc7e73e to
5a0a488
Compare
This PR contains the following updates:
8b1e787→20316564b796c6→e6b9100Configuration
📅 Schedule: (in timezone Europe/Madrid)
🚦 Automerge: Disabled by config. Please merge this manually once you are satisfied.
♻ Rebasing: Whenever PR becomes conflicted, or you tick the rebase/retry checkbox.
👻 Immortal: This PR will be recreated if closed unmerged. Get config help if that's undesired.
This PR was generated by Mend Renovate. View the repository job log.