Skip to content

add argocd-image-updater component#316

Merged
svghadi merged 6 commits into
rh-gitops-midstream:mainfrom
chengfang:image.updater.in.1.19
Oct 12, 2025
Merged

add argocd-image-updater component#316
svghadi merged 6 commits into
rh-gitops-midstream:mainfrom
chengfang:image.updater.in.1.19

Conversation

@chengfang
Copy link
Copy Markdown
Collaborator

@chengfang chengfang commented Oct 8, 2025

Note: argocd-image-updater upstream Dockerfile uses alpines and installs awscli along with other alpine packages. In the downstream Dockerfile in this PR, awscli is not included because it is not available as a package in ubi. The other option would be to install python3 package, and use pip install awscli (v1 only, current version is 2), but even that would require adding a lot of dependencies to prefetch, if it is possible at all.

@chengfang chengfang force-pushed the image.updater.in.1.19 branch from 5f88161 to 043a715 Compare October 8, 2025 12:48
@svghadi
Copy link
Copy Markdown
Collaborator

svghadi commented Oct 8, 2025

/retest

2 similar comments
@svghadi
Copy link
Copy Markdown
Collaborator

svghadi commented Oct 8, 2025

/retest

@chengfang
Copy link
Copy Markdown
Collaborator Author

/retest

@chengfang chengfang marked this pull request as draft October 8, 2025 19:29
@chengfang chengfang force-pushed the image.updater.in.1.19 branch from 043a715 to 79ec266 Compare October 8, 2025 19:41
Signed-off-by: Cheng Fang <cfang@redhat.com>
@chengfang chengfang force-pushed the image.updater.in.1.19 branch from 79ec266 to 243ab57 Compare October 8, 2025 20:21
…vailable in ubi-minimal

Signed-off-by: Cheng Fang <cfang@redhat.com>
…1.34.34

Signed-off-by: Cheng Fang <cfang@redhat.com>
@chengfang chengfang force-pushed the image.updater.in.1.19 branch from 136723b to 5ae17e5 Compare October 8, 2025 21:57
…n building argocd-image-updater

Signed-off-by: Cheng Fang <cfang@redhat.com>
@chengfang chengfang force-pushed the image.updater.in.1.19 branch from 5ae17e5 to f9957ee Compare October 8, 2025 22:07
Signed-off-by: Cheng Fang <cfang@redhat.com>
Signed-off-by: Cheng Fang <cfang@redhat.com>
@chengfang chengfang marked this pull request as ready for review October 9, 2025 01:25
@svghadi
Copy link
Copy Markdown
Collaborator

svghadi commented Oct 9, 2025

Is aws-cli a hard requirement for image updater?

Copy link
Copy Markdown
Collaborator

@svghadi svghadi left a comment

Choose a reason for hiding this comment

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

LGTM

@chengfang
Copy link
Copy Markdown
Collaborator Author

chengfang commented Oct 9, 2025

Is aws-cli a hard requirement for image updater?

No, it's nice to have for those ECR users calling awscli to get a token.

@svghadi svghadi merged commit 2c1e110 into rh-gitops-midstream:main Oct 12, 2025
3 of 7 checks passed
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