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

Fix Grafana makefile target (use correct cluster name) #797

Merged
merged 1 commit into from
Dec 13, 2021

Conversation

jkremser
Copy link
Member

@jkremser jkremser commented Dec 9, 2021

Fix #796

my bad, I am pretty sure I was updating this, but it was probably lost during conflict resolutions

❯ make deploy-grafana

Local cluster test-gslb1

install grafana
helm repo add grafana https://grafana.github.io/helm-charts
"grafana" already exists with the same configuration, skipping
helm repo update
Hang tight while we grab the latest from your chart repositories...
...Successfully got an update from the "k8gb" chart repository
...Successfully got an update from the "podinfo" chart repository
...Successfully got an update from the "crossplane-alpha" chart repository
...Successfully got an update from the "nginx-stable" chart repository
...Successfully got an update from the "crossplane-stable" chart repository
...Successfully got an update from the "grafana" chart repository
...Successfully got an update from the "prometheus-community" chart repository
Update Complete. ⎈Happy Helming!⎈
helm -n k8gb upgrade -i grafana grafana/grafana -f deploy/grafana/values.yaml \
	--wait --timeout=2m30s \
	--kube-context=k3d-test-gslb1
W1209 19:51:55.545645   24958 warnings.go:70] policy/v1beta1 PodSecurityPolicy is deprecated in v1.21+, unavailable in v1.25+
W1209 19:51:55.551626   24958 warnings.go:70] policy/v1beta1 PodSecurityPolicy is deprecated in v1.21+, unavailable in v1.25+
W1209 19:51:55.562716   24958 warnings.go:70] policy/v1beta1 PodSecurityPolicy is deprecated in v1.21+, unavailable in v1.25+
W1209 19:51:55.566113   24958 warnings.go:70] policy/v1beta1 PodSecurityPolicy is deprecated in v1.21+, unavailable in v1.25+
W1209 19:51:55.569463   24958 warnings.go:70] policy/v1beta1 PodSecurityPolicy is deprecated in v1.21+, unavailable in v1.25+
W1209 19:51:55.574458   24958 warnings.go:70] policy/v1beta1 PodSecurityPolicy is deprecated in v1.21+, unavailable in v1.25+
Release "grafana" has been upgraded. Happy Helming!
NAME: grafana
LAST DEPLOYED: Thu Dec  9 19:51:55 2021
NAMESPACE: k8gb
STATUS: deployed
REVISION: 2
NOTES:
1. Get your 'admin' user password by running:

   kubectl get secret --namespace k8gb grafana -o jsonpath="{.data.admin-password}" | base64 --decode ; echo

2. The Grafana server can be accessed via port 3000 on the following DNS name from within your cluster:

   grafana.k8gb.svc.cluster.local

   Get the Grafana URL to visit by running these commands in the same shell:
export NODE_PORT=$(kubectl get --namespace k8gb -o jsonpath="{.spec.ports[0].nodePort}" services grafana)
     export NODE_IP=$(kubectl get nodes --namespace k8gb -o jsonpath="{.items[0].status.addresses[0].address}")
     echo http://$NODE_IP:$NODE_PORT


3. Login with the password from step 1 and the username: admin
#################################################################################
######   WARNING: Persistence is disabled!!! You will lose your data when   #####
######            the Grafana pod is terminated.                            #####
#################################################################################
kubectl --context k3d-test-gslb1 apply -f deploy/grafana/dashboard-cm.yaml -n k8gb
configmap/podinfo-dashboard unchanged

Grafana is listening on http://localhost:3000

🖖 credentials are admin:admin

..there are some warnings about PodSecurityPolicy being deprecated, updating the chart can help, I can look into that later on

Signed-off-by: Jirka Kremser [email protected]

@jkremser jkremser requested a review from somaritane December 9, 2021 20:33
Copy link
Contributor

@somaritane somaritane left a comment

Choose a reason for hiding this comment

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

@jkremser looks good, just one nit

Copy link
Contributor

@somaritane somaritane left a comment

Choose a reason for hiding this comment

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

@jkremser LGTM 👍

@jkremser jkremser merged commit 6e4d1db into k8gb-io:master Dec 13, 2021
@jkremser jkremser deleted the grafana-target branch December 13, 2021 11:37
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.

Unable to deploy grafana to the test cluster with deploy-grafana make target
2 participants