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

(un)deploying prometheus is broken #791

Merged
merged 1 commit into from
Dec 9, 2021

Conversation

jkremser
Copy link
Member

@jkremser jkremser commented Dec 8, 2021

before:

make deploy-prometheus deploy-grafana
/bin/sh: -c: line 0: syntax error near unexpected token `;'
/bin/sh: -c: line 0: `for c in 1 2; do 		echo "\n\033[0;33mLocal cluster \033[1;36mtest-gslb$c\033[0m" ; echo "\n\033[0;33mSet annotations on pods that will be scraped by prometheus\033[0m" ; kubectl annotate pods -l name=k8gb -n k8gb --overwrite prometheus.io/scrape="true" --context=k3d-test-gslb$c ; kubectl annotate pods -l name=k8gb -n k8gb --overwrite prometheus.io/port="8080" --context=k3d-test-gslb$c ; echo "\n\033[0;33minstall prometheus \033[0m" ; helm repo add prometheus-community https://prometheus-community.github.io/helm-charts ; helm repo update ; helm -n k8gb upgrade -i prometheus prometheus-community/prometheus -f deploy/prometheus/values.yaml --version 14.2.0 --wait --timeout=2m0s --kube-context=k3d-test-gslb$c ; ;done'
make: *** [deploy-prometheus] Error 2

after:

make deploy-prometheus deploy-grafana

Local cluster test-gslb1

Set annotations on pods that will be scraped by prometheus
pod/k8gb-fdb8d8894-wthzw annotated
pod/k8gb-fdb8d8894-wthzw annotated
...

(sorry, looks like I broke it)

this PR fixes the makefile, but it still fails to start because of:

k8gb          0s          Normal    FailedBinding             persistentvolumeclaim/prometheus-server              no persistent volumes available for this claim and no storage class is set

^ prolly caused by c36c225#diff-9a834bbeb50f074ae6a033fd21ed1840bb7363625dc52a43d1d45c1925c8803c

Signed-off-by: Jirka Kremser <jiri.kremser@gmail.com>
@jkremser jkremser merged commit c9a0329 into k8gb-io:master Dec 9, 2021
@jkremser jkremser deleted the make-deploy-prometheus branch December 9, 2021 17:19
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.

3 participants