Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom

Image Repositories #601

Closed
TreverW opened this issue Oct 27, 2021 · 4 comments
Closed

Image Repositories #601

TreverW opened this issue Oct 27, 2021 · 4 comments

Comments

@TreverW
Copy link

TreverW commented Oct 27, 2021

The controller must be run with the flag --aws-autologin-for-ecr set for this to work.

Which controller? When I tried adding this to the Image Reflector Controller, it failed to run.

unknown flag: --aws-autologin-for-ecr
@stefanprodan
Copy link
Member

You need to upgrade to the latest Flux release v0.19 that contains the image-reflector-controller v0.13.0.

@dholbach
Copy link
Member

Is this still an issue now?

@kingdonb kingdonb self-assigned this Oct 31, 2021
@kingdonb kingdonb mentioned this issue Oct 31, 2021
Closed
@kingdonb
Copy link
Member

Yes, it should be fixed by fluxcd/image-reflector-controller#193.

@kingdonb kingdonb mentioned this issue Oct 31, 2021
Merged
lloydchang added a commit to lloydchang/image-reflector-controller that referenced this issue Oct 31, 2021
@lloydchang
docs(image-reflector-controller/docs/spec/v1beta1/imagerepositories.m…
44baa4c 
…d): elaborate on how to enable autologin for ECR

fluxcd/flux#1619
fluxcd/flux2#107
fluxcd#11
fluxcd#139
fluxcd#147
fluxcd#174
fluxcd#179
fluxcd#180
fluxcd/website#601
fluxcd#193
fluxcd/website#611
https://fluxcd.io/docs/guides/image-update/#aws-elastic-container-registry

Signed-off-by: lloydchang <lloydchang@gmail.com>
@lloydchang lloydchang mentioned this issue Oct 31, 2021
Merged
kingdonb pushed a commit to lloydchang/image-reflector-controller that referenced this issue Oct 31, 2021
@lloydchang
docs(image-reflector-controller/docs/spec/v1beta1/imagerepositories.m…
e34d8b7 
…d): elaborate on how to enable autologin for ECR

fluxcd/flux#1619
fluxcd/flux2#107
fluxcd#11
fluxcd#139
fluxcd#147
fluxcd#174
fluxcd#179
fluxcd#180
fluxcd/website#601
fluxcd#193
fluxcd/website#611
https://fluxcd.io/docs/guides/image-update/#aws-elastic-container-registry

Signed-off-by: lloydchang <lloydchang@gmail.com>
kingdonb pushed a commit to kingdonb/image-reflector-controller that referenced this issue Oct 31, 2021
@lloydchang
docs(image-reflector-controller/docs/spec/v1beta1/imagerepositories.m…
0e2f925 
…d): elaborate on how to enable autologin for ECR

fluxcd/flux#1619
fluxcd/flux2#107
fluxcd#11
fluxcd#139
fluxcd#147
fluxcd#174
fluxcd#179
fluxcd#180
fluxcd/website#601
fluxcd#193
fluxcd/website#611
https://fluxcd.io/docs/guides/image-update/#aws-elastic-container-registry

Signed-off-by: lloydchang <lloydchang@gmail.com>
Signed-off-by: Kingdon Barrett <kingdon@weave.works>
kingdonb pushed a commit to kingdonb/image-reflector-controller that referenced this issue Dec 16, 2021
@lloydchang
docs(image-reflector-controller/docs/spec/v1beta1/imagerepositories.m…
9c125b5 
…d): elaborate on how to enable autologin for ECR

fluxcd/flux#1619
fluxcd/flux2#107
fluxcd#11
fluxcd#139
fluxcd#147
fluxcd#174
fluxcd#179
fluxcd#180
fluxcd/website#601
fluxcd#193
fluxcd/website#611
https://fluxcd.io/docs/guides/image-update/#aws-elastic-container-registry

Signed-off-by: lloydchang <lloydchang@gmail.com>
Signed-off-by: Kingdon Barrett <kingdon@weave.works>
@chedim chedim mentioned this issue Dec 21, 2021
Closed
@dholbach
Copy link
Member

dholbach commented Nov 7, 2022

Is this still an issue?

@kingdonb kingdonb removed their assignment Nov 7, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging a pull request may close this issue.

Elaborate on how to enable autologin for ECR kingdonb/image-reflector-controller
4 participants
@TreverW @dholbach @kingdonb @stefanprodan