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

Update cassandra and ALB chart repo to use latest migrated repository #222

Merged
merged 12 commits into from
Jan 7, 2021
Merged

Update cassandra and ALB chart repo to use latest migrated repository #222

merged 12 commits into from
Jan 7, 2021

Conversation

MadhuriArugula
Copy link
Collaborator

@MadhuriArugula MadhuriArugula commented Dec 23, 2020

Travis builds are failing with https://travis-ci.org/github/pegasystems/pega-helm-charts/builds/751186081 chart not found in corresponding repo issues.

Downloading traefik from repo https://charts.helm.sh/stable
Save error occurred: could not find : chart aws-alb-ingress-controller not found in https://kubernetes-charts-incubator.storage.googleapis.com/

Deleting newly downloaded charts, restoring pre-update state

Error: could not find : chart aws-alb-ingress-controller not found in https://kubernetes-charts-incubator.storage.googleapis.com/

Makefile:2: recipe for target 'dependencies' failed

make: *** [dependencies] Error 1

The command "make dependencies" exited with 2.

To Reproduce:
Perform helm dep update and check if dependencies are downloaded properly using "helm dep list" under addons

Expected behavior
All chart dependencies should be downloaded properly without any issues.
The issue is because stable, incubator chart repo used in our charts are deprecated. It should be moved to another repo as per https://helm.sh/blog/new-location-stable-incubator-charts/

References - https://kubernetes-sigs.github.io/aws-load-balancer-controller/guide/upgrade/migrate_v1_v2/

Updated ALB chart and Cassandra chart repositories.

@MadhuriArugula MadhuriArugula requested review from dcasavant and a team as code owners December 23, 2020 12:16
@MadhuriArugula MadhuriArugula changed the title Update cassandra chart repo to use latest migrated repository Update cassandra and ALB chart repo to use latest migrated repository Jan 5, 2021
Copy link
Contributor

@taz-pega-work taz-pega-work left a comment

Choose a reason for hiding this comment

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

excellent!

@Saurabh-16 Saurabh-16 merged commit 8325204 into pegasystems:master Jan 7, 2021
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.

6 participants