Skip to content

artifactories: pin ecr-token-refresher image to facetscloud/aws-kubectl:1.7.0#562

Merged
sanmesh-kakade merged 1 commit into
masterfrom
ecr-refresher-image-1.7.0-master
Jul 9, 2026
Merged

artifactories: pin ecr-token-refresher image to facetscloud/aws-kubectl:1.7.0#562
sanmesh-kakade merged 1 commit into
masterfrom
ecr-refresher-image-1.7.0-master

Conversation

@sanmesh-kakade

@sanmesh-kakade sanmesh-kakade commented Jul 9, 2026

Copy link
Copy Markdown
Contributor

Promotes to master the ecr-token-refresher image pin merged to develop in #561 (cherry-pick of squash commit b517d9a).

Sets the artifactories ecr-token-refresher cronjob image to facetscloud/aws-kubectl:1.7.0 (modules/artifactories/default/0.1/ecr-token-refresher.tf).

Note: :1.7.0 is amd64-only (cronjob pods pinned to kubernetes.io/arch=amd64); kubectl v1.7.0 is a 2017 binary — validate on a real amd64 node before rollout.

🤖 Generated with Claude Code

Summary by CodeRabbit

  • Chores
    • Updated the container image used by a scheduled background task to a different version for improved compatibility and maintenance.

…tl:1.7.0 (#561)

Set the ecr-token-refresher cronjob image tag to :1.7.0 (matches the original
xynova/aws-kubectl gist, kubectl v1.7.0).

Co-authored-by: Claude Opus 4.8 (1M context) <noreply@anthropic.com>
@coderabbitai

coderabbitai Bot commented Jul 9, 2026

Copy link
Copy Markdown
Contributor

Review Change Stack

Caution

Review failed

The pull request is closed.

ℹ️ Recent review info
⚙️ Run configuration

Configuration used: Organization UI

Review profile: CHILL

Plan: Pro

Run ID: 2095caa6-9325-4146-9d31-a5270dad02bc

📥 Commits

Reviewing files that changed from the base of the PR and between b1a480c and 7e39d1a.

📒 Files selected for processing (1)
  • modules/artifactories/default/0.1/ecr-token-refresher.tf

Walkthrough

This PR changes the container image tag used by the ecr-token-refresher-cron Kubernetes cron job from facetscloud/aws-kubectl:1.31.0 to facetscloud/aws-kubectl:1.7.0 in the Terraform configuration.

Changes

Image Tag Update

Layer / File(s) Summary
Update cron job container image tag
modules/artifactories/default/0.1/ecr-token-refresher.tf
The kubernetes_cron_job_v1.ecr-token-refresher-cron container image tag is changed from facetscloud/aws-kubectl:1.31.0 to facetscloud/aws-kubectl:1.7.0.

Possibly related PRs

Suggested reviewers: rr0hit, vishnukv-facets, unni-facets

✨ Finishing Touches
🧪 Generate unit tests (beta)
  • Create PR with unit tests
  • Commit unit tests in branch ecr-refresher-image-1.7.0-master

Warning

Tools execution failed with the following error:

Failed to run tools: 13 INTERNAL: Received RST_STREAM with code 2 (Internal server error)


Comment @coderabbitai help to get the list of available commands.

@sanmesh-kakade
sanmesh-kakade merged commit 5962c23 into master Jul 9, 2026
1 of 2 checks passed
@sanmesh-kakade
sanmesh-kakade deleted the ecr-refresher-image-1.7.0-master branch July 9, 2026 15:04
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