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.

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

Move container images to ECR public registry #461

Closed
5 of 6 tasks
mhausenblas opened this issue Nov 4, 2020 · 4 comments
Closed
5 of 6 tasks

Move container images to ECR public registry #461

mhausenblas opened this issue Nov 4, 2020 · 4 comments
Assignees
Labels
kind/enhancement Categorizes issue or PR as related to existing feature enhancements. project-management Infra and communication stuff

Comments

@mhausenblas
Copy link
Contributor

mhausenblas commented Nov 4, 2020

Currently, the service controller images are hosted as public images in Docker Hub. Now that ECR [https://aws.amazon.com/ecr/faqs/) are available, we should move them there, along with the Helm Charts that were being published to the gh-pages branch of the repo.

Things we need to get done here:

  • Create a new AWS account for ACK administration/infra (internal accounts)
  • Create a new AWS sub-account under the top-level AWS account that we use for publishing artifacts (internal accounts)
  • Reserve the ack domain in the ECR public registry list
  • Modify the image publication script to publish to ECR public registry
  • Modify the Helm publication script to publish to an external repo instead of the gh-pages branch of the source repository
  • Update our internal wiki docs about the above changes to our AWS accounts
@mhausenblas mhausenblas added the kind/enhancement Categorizes issue or PR as related to existing feature enhancements. label Nov 4, 2020
@mhausenblas mhausenblas self-assigned this Nov 4, 2020
@mhausenblas mhausenblas added the project-management Infra and communication stuff label Nov 4, 2020
@jaypipes jaypipes changed the title Move container images to AWS public registry Move container images to ECR public registry Dec 7, 2020
@nab-gha
Copy link

nab-gha commented Sep 15, 2021

This issue is closed but it doesn't seem that the helm repo has yet been setup?

@vijtrip2
Copy link
Contributor

This issue is closed but it doesn't seem that the helm repo has yet been setup?

Helm repo is setup and functional. Are you exporting following variable? export HELM_EXPERIMENTAL_OCI=1

Please see this doc which installs ACK controllers using Helm.

@narcosis
Copy link

Hi,
Still having the original issue when trying to add the helm repository for S3.
Has this issue been resolved?
"helm repo add ack https://aws.github.io/aws-controllers-k8s/charts"

Error: looks like "https://aws.github.io/aws-controllers-k8s/charts" is not a valid chart repository or cannot be reached: failed to fetch https://aws.github.io/aws-controllers-k8s/charts/index.yaml : 404 Not Found

@jaypipes
Copy link
Collaborator

Hi, Still having the original issue when trying to add the helm repository for S3. Has this issue been resolved? "helm repo add ack https://aws.github.io/aws-controllers-k8s/charts"

Error: looks like "https://aws.github.io/aws-controllers-k8s/charts" is not a valid chart repository or cannot be reached: failed to fetch https://aws.github.io/aws-controllers-k8s/charts/index.yaml : 404 Not Found

Please see the doc that @vijtrip2 linked. The correct repository URL for you would be public.ecr.aws/aws-controllers-k8s/s3-chart

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
kind/enhancement Categorizes issue or PR as related to existing feature enhancements. project-management Infra and communication stuff
Projects
None yet
Development

No branches or pull requests

5 participants