Bump aws-kubectl to latest (based on Ubuntu 24.04)#4262
Bump aws-kubectl to latest (based on Ubuntu 24.04)#4262imnasnainaec wants to merge 2 commits intomasterfrom
Conversation
📝 WalkthroughWalkthroughUpdates the aws-kubectl container image version from 0.4.2 to 0.5.0 across Helm chart values files and the maintenance Dockerfile. No other configurations, schedules, or references are modified. Changes
Estimated code review effort🎯 1 (Trivial) | ⏱️ ~2 minutes Possibly related PRs
Suggested labels
Suggested reviewers
Poem
🚥 Pre-merge checks | ✅ 3✅ 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)
Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out. Comment |
There was a problem hiding this comment.
🧹 Nitpick comments (1)
maintenance/Dockerfile (1)
19-19: Tag bump is correct; consider pinning by digest if available.Line 19 correctly updates the base image tag. As an optional hardening step, pinning to an immutable digest would improve reproducibility and prevent tag mutation. If you have AWS credentials, retrieve the digest using:
aws ecr-public describe-images --repository-name thecombine/aws-kubectl --image-ids imageTag=0.5.0 --region us-east-1Then apply the manifest digest syntax (for multi-arch support):
FROM public.ecr.aws/thecombine/aws-kubectl:0.5.0@sha256:<manifest-digest>🤖 Prompt for AI Agents
Verify each finding against the current code and only fix it if needed. In `@maintenance/Dockerfile` at line 19, The FROM line currently uses a mutable tag "public.ecr.aws/thecombine/aws-kubectl:0.5.0-$TARGETARCH"; to pin immutably, fetch the repository's manifest digest for tag 0.5.0 (e.g. via AWS ECR Public describe-images) and replace the base image reference with the multi-arch manifest form that appends `@sha256`:<manifest-digest> while keeping the $TARGETARCH qualifier as needed (i.e., change the FROM to include :0.5.0@sha256:<manifest-digest> so the image is pinned to the specific digest).
🤖 Prompt for all review comments with AI agents
Verify each finding against the current code and only fix it if needed.
Nitpick comments:
In `@maintenance/Dockerfile`:
- Line 19: The FROM line currently uses a mutable tag
"public.ecr.aws/thecombine/aws-kubectl:0.5.0-$TARGETARCH"; to pin immutably,
fetch the repository's manifest digest for tag 0.5.0 (e.g. via AWS ECR Public
describe-images) and replace the base image reference with the multi-arch
manifest form that appends `@sha256`:<manifest-digest> while keeping the
$TARGETARCH qualifier as needed (i.e., change the FROM to include
:0.5.0@sha256:<manifest-digest> so the image is pinned to the specific digest).
ℹ️ Review info
⚙️ Run configuration
Configuration used: defaults
Review profile: CHILL
Plan: Pro
Run ID: 0f2e1c3e-3e06-480e-b68b-2c0951efe084
📒 Files selected for processing (3)
deploy/helm/aws-login/values.yamldeploy/helm/thecombine/charts/maintenance/values.yamlmaintenance/Dockerfile
Codecov Report✅ All modified and coverable lines are covered by tests. Additional details and impacted files@@ Coverage Diff @@
## master #4262 +/- ##
=======================================
Coverage 75.94% 75.94%
=======================================
Files 303 303
Lines 11352 11352
Branches 1403 1403
=======================================
Hits 8621 8621
Misses 2330 2330
Partials 401 401
Flags with carried forward coverage won't be shown. Click here to find out more. ☔ View full report in Codecov by Sentry. 🚀 New features to boost your workflow:
|
Pairs with sillsdev/aws-kubectl#9
Devin review: https://app.devin.ai/review/sillsdev/TheCombine/pull/4262
This change is
Summary by CodeRabbit