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

Unable to add helm charts repo for ACK #550

Closed
kumargauravsharma opened this issue Dec 3, 2020 · 1 comment
Closed

Unable to add helm charts repo for ACK #550

kumargauravsharma opened this issue Dec 3, 2020 · 1 comment
Assignees
Labels
kind/bug Categorizes issue or PR as related to a bug.

Comments

@kumargauravsharma
Copy link
Contributor

Describe the bug
404 Not Found error observed during helm repo add.

Reference documentation: https://github.com/aws/aws-controllers-k8s/blob/main/docs/contents/user-docs/install.md

Steps to reproduce
Command:

$ helm repo add ack https://aws.github.io/aws-controllers-k8s/charts

produces error:

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

Expected outcome
Repo to get added successfully.

Environment

  • Kubernetes version
  • Using EKS (yes/no), if so version?
  • AWS service targeted (S3, RDS, etc.)
@kumargauravsharma kumargauravsharma added the kind/bug Categorizes issue or PR as related to a bug. label Dec 3, 2020
@kumargauravsharma kumargauravsharma added this to the Common service Beta preparation milestone Dec 3, 2020
@jaypipes jaypipes self-assigned this Dec 3, 2020
@jaypipes
Copy link
Collaborator

jaypipes commented Dec 7, 2020

OK, we identified the source of this problem. Turns out that the Github Action we use for auto-generating our docs site is overwriting the Git commits in the gh-pages branch.

We were storing our Helm Charts in the charts/ directory of the gh-pages branch and unfortunately the last commit that modified the docs/ content overwrote our Charts. :(

So, I need to regenerate the charts and push them. However, we have decided that since we still have not published our updated install docs that refer to the gh-pages Helm Charts repository, we are going to instead publish to a new ECR Public repository and use ECR instead of Dockerhub for storing the binary ACK service controller images.

Please see #461 for information on what we need to do here. I'm going to close this bug out in favor of #461

@jaypipes jaypipes closed this as completed Dec 7, 2020
@jaypipes jaypipes removed this from the Common service Beta preparation milestone Dec 7, 2020
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
kind/bug Categorizes issue or PR as related to a bug.
Projects
None yet
Development

No branches or pull requests

2 participants