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.

Sign up for GitHub

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

Jump to bottom

Use Ubuntu-based grafana image instead of the alpine image #3480

Closed
sayf-eddine-scality opened this issue Aug 4, 2021 · 1 comment
Closed

Use Ubuntu-based grafana image instead of the alpine image #3480

sayf-eddine-scality opened this issue Aug 4, 2021 · 1 comment
Assignees
@sayf-eddine-scality

Comments

@sayf-eddine-scality
Copy link
Contributor

Component:
monitoring,grafana

What happened:
Grafna image is based on alpine image, which handles DNS SERVFAIL very poorly and result in non working datasources
What was expected:

Steps to reproduce
configure a DNS server that returns SERVFAIL in the host machine, grafana wont be able to reach loki
Resolution proposal (optional):
Use ubuntu-based image for grafana
@sayf-eddine-scality sayf-eddine-scality self-assigned this Aug 4, 2021
sayf-eddine-scality added a commit that referenced this issue Aug 4, 2021
@sayf-eddine-scality
Bump Grafana image to 8.0.6-ubuntu
28f8963 
We moved to Ubuntu-based image because the Alpine-based image
handle DNS SERVFAIL errors poorly.

Grafana image tag set to 8.0.6-ubuntu then
```
./charts/render.py prometheus-operator \
      charts/kube-prometheus-stack.yaml \
      charts/kube-prometheus-stack/ \
      --namespace metalk8s-monitoring \
      --service-config grafana \
      metalk8s-grafana-config \
      metalk8s/addons/prometheus-operator/config/grafana.yaml \
      metalk8s-monitoring \
      --service-config prometheus \
      metalk8s-prometheus-config \
      metalk8s/addons/prometheus-operator/config/prometheus.yaml \
      metalk8s-monitoring \
      --service-config alertmanager \
      metalk8s-alertmanager-config \
      metalk8s/addons/prometheus-operator/config/alertmanager.yaml \
      metalk8s-monitoring \
      --service-config dex \
      metalk8s-dex-config \
      metalk8s/addons/dex/config/dex.yaml.j2 metalk8s-auth \
      --drop-prometheus-rules charts/drop-prometheus-rules.yaml \
      > salt/metalk8s/addons/prometheus-operator/deployed/chart.sls
```

Refs: #3480
@sayf-eddine-scality sayf-eddine-scality mentioned this issue Aug 4, 2021
Merged
sayf-eddine-scality added a commit that referenced this issue Aug 4, 2021
@sayf-eddine-scality
Bump Grafana image to 8.0.6-ubuntu
99b0e93 
We moved to Ubuntu-based image because the Alpine-based image
handle DNS SERVFAIL errors poorly.

Grafana image tag set to 8.0.6-ubuntu then
```
./charts/render.py prometheus-operator \
      charts/kube-prometheus-stack.yaml \
      charts/kube-prometheus-stack/ \
      --namespace metalk8s-monitoring \
      --service-config grafana \
      metalk8s-grafana-config \
      metalk8s/addons/prometheus-operator/config/grafana.yaml \
      metalk8s-monitoring \
      --service-config prometheus \
      metalk8s-prometheus-config \
      metalk8s/addons/prometheus-operator/config/prometheus.yaml \
      metalk8s-monitoring \
      --service-config alertmanager \
      metalk8s-alertmanager-config \
      metalk8s/addons/prometheus-operator/config/alertmanager.yaml \
      metalk8s-monitoring \
      --service-config dex \
      metalk8s-dex-config \
      metalk8s/addons/dex/config/dex.yaml.j2 metalk8s-auth \
      --drop-prometheus-rules charts/drop-prometheus-rules.yaml \
      > salt/metalk8s/addons/prometheus-operator/deployed/chart.sls
```

Refs: #3480
@gdemonet
Copy link
Contributor

gdemonet commented Aug 5, 2021

Fixed by #3481

@gdemonet gdemonet closed this as completed Aug 5, 2021
TeddyAndrieux added a commit that referenced this issue Jan 4, 2022
@TeddyAndrieux
build,salt,charts: Switch back to grafana ubuntu image
6b056c5 
Re-render the chart using:
```
./charts/render.py prometheus-operator \
  charts/kube-prometheus-stack.yaml \
  charts/kube-prometheus-stack/ \
  --namespace metalk8s-monitoring \
  --service-config grafana \
  metalk8s-grafana-config \
  metalk8s/addons/prometheus-operator/config/grafana.yaml \
  metalk8s-monitoring \
  --service-config prometheus \
  metalk8s-prometheus-config \
  metalk8s/addons/prometheus-operator/config/prometheus.yaml \
  metalk8s-monitoring \
  --service-config alertmanager \
  metalk8s-alertmanager-config \
  metalk8s/addons/prometheus-operator/config/alertmanager.yaml \
  metalk8s-monitoring \
  --service-config dex \
  metalk8s-dex-config \
  metalk8s/addons/dex/config/dex.yaml.j2 metalk8s-auth \
  --drop-prometheus-rules charts/drop-prometheus-rules.yaml \
  --patch 'PrometheusRule,metalk8s-monitoring,prometheus-operator-kubernetes-system-kubelet,spec:groups:0:rules:1:for,"5m"' \
  > salt/metalk8s/addons/prometheus-operator/deployed/chart.sls
```

See: #3480
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@sayf-eddine-scality sayf-eddine-scality

Labels
None yet
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
2 participants
@sayf-eddine-scality @gdemonet