Skip to content
This repository has been archived by the owner on Nov 27, 2018. It is now read-only.

add helm update to draft packs #58

Open
wants to merge 1 commit into
base: master
Choose a base branch
from
Open

add helm update to draft packs #58

wants to merge 1 commit into from

Conversation

i0n
Copy link

@i0n i0n commented Jul 31, 2018

No description provided.

@i0n i0n requested a review from rawlingsj August 6, 2018 10:36
@@ -28,6 +28,8 @@ release: clean
helm init --client-only
helm package .
curl --fail -u $(CHARTMUSEUM_CREDS_USR):$(CHARTMUSEUM_CREDS_PSW) --data-binary "@$(NAME)-$(shell sed -n 's/^version: //p' Chart.yaml).tgz" $(CHART_REPO)/api/charts
helm repo add jenkins-x-api https://chartmuseum.build.cd.jenkins-x.io
helm repo update
rm -rf ${NAME}*.tgz%
Copy link
Member

Choose a reason for hiding this comment

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

I'd have thought a helm repo add and helm repo update would need to happen before the helm package above?

Copy link
Author

Choose a reason for hiding this comment

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

I only need to add the repo to helm in order to run update as the push is done with curl, update is only being used to regenerate the index-cache.

Copy link
Author

Choose a reason for hiding this comment

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

I only need to add the repo to helm in order to run update as the push is done with curl, update is only being used to regenerate the index-cache.

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants