I was running <a href="https://github.com/prometheus/prometheus/tree/master/cmd/promto

I'm on the same jsonnet version. <div class="snippet-clipboard-content notranslate

Oh. I know the issue. These rules are vendored into <a href="https:/

I'm going to see what the prometheus folks say. <a class="issue-link js-issue-link" da

"mapping values are not allowed in this context" error from promtool,about kubernetes-monitoring/kubernetes-mixin

Comments (5)

tomwilkie commented on June 26, 2024

Hi Adam - what version of jsonnet are you using? Does make lint also fail?

We use jsonnet 0.10 in the CI and it works...

from kubernetes-mixin.

adamdecaf commented on June 26, 2024

I'm on the same jsonnet version.

$ jsonnet -version
Jsonnet commandline interpreter v0.10.0

$ make lint
jsonnet -S lib/alerts.jsonnet > prometheus_alerts.yaml
find . -name 'vendor' -prune -o -name '*.libsonnet' -print -o -name '*.jsonnet' -print | \
		while read f; do \
			jsonnet fmt -n 2 --max-blank-lines 2 --string-style s --comment-style s "$f" | diff -u "$f" -; \
		done
promtool check rules prometheus_rules.yaml
Checking prometheus_rules.yaml
  SUCCESS: 29 rules found

promtool check rules prometheus_alerts.yaml
Checking prometheus_alerts.yaml
  SUCCESS: 15 rules found

The generated yaml has quotes around it, so maybe this was my fault.

- "expr": |
      sum(irate(node_network_receive_drop{job="default/node-exporter",device="eth0"}[1m])) +
      sum(irate(node_network_transmit_drop{job="default/node-exporter",device="eth0"}[1m]))
    "record": ":node_net_saturation:sum_irate"

from kubernetes-mixin.

tomwilkie commented on June 26, 2024

Weird, would be good to get to the bottom of what happened. Perhaps the original set was generated with a different version of jsonnet? Feel free to close if you can't reproduce.

from kubernetes-mixin.

adamdecaf commented on June 26, 2024

Oh. I know the issue.

These rules are vendored into coreos/prometheus-operator, which is what I'm using. To make some edits locally I rewrote that ConfigMap from the /rules html, which doesn't quote the recording rules.