Marton Greber has posted comments on this change. ( 
http://gerrit.cloudera.org:8080/20814 )

Change subject: Add blog post about Prometheus/Grafana
......................................................................


Patch Set 1:

(8 comments)

Removed self hosted website url, adding it here in the comments for easier 
review:
https://kudu-site-659861.gitlab.io/2026/07/03/kudu-prometheus-grafana.html

http://gerrit.cloudera.org:8080/#/c/20814/1/_posts/2023-12-18-kudu-prometheus-grafana.md
File _posts/2023-12-18-kudu-prometheus-grafana.md:

http://gerrit.cloudera.org:8080/#/c/20814/1/_posts/2023-12-18-kudu-prometheus-grafana.md@24
PS1, Line 24:
> I have one question I'd like to ask. Now that tablet-level metrics are full
Added a small capability section where `prometheus_use_entity_labels` is 
mentioned.
Regarding metric cardinality and load: I dont think we've any 
reports/measurements for that so far. But its a valid point that we could keep 
in mind for testing etc.


http://gerrit.cloudera.org:8080/#/c/20814/2/_posts/2026-07-03-kudu-prometheus-grafana.md
File _posts/2026-07-03-kudu-prometheus-grafana.md:

http://gerrit.cloudera.org:8080/#/c/20814/2/_posts/2026-07-03-kudu-prometheus-grafana.md@8
PS2, Line 8:
> nit: the outer parentheses around the link read like a typo. I think withou
Done


http://gerrit.cloudera.org:8080/#/c/20814/2/_posts/2026-07-03-kudu-prometheus-grafana.md@20
PS2, Line 20:
> web server
Done


http://gerrit.cloudera.org:8080/#/c/20814/2/_posts/2026-07-03-kudu-prometheus-grafana.md@21
PS2, Line 21:
> Does it make sense to mention tweaking --prometheus_use_entity_labels flag
Yes, added a section at the very end "Capability Notes" where i outlined this 
flag.


http://gerrit.cloudera.org:8080/#/c/20814/2/_posts/2026-07-03-kudu-prometheus-grafana.md@42
PS2, Line 42:
            :
            :
            :
            :
            :
            :
            :
            :
            :
            :
            :
            :
            :
            :
            :
> I think this YAML isn't valid: scrape_interval/evaluation_interval are at t
Done


http://gerrit.cloudera.org:8080/#/c/20814/2/_posts/2026-07-03-kudu-prometheus-grafana.md@80
PS2, Line 80:
> nit: trailing whitespace.
Yes the numbering was not working as expected here, fixed.


http://gerrit.cloudera.org:8080/#/c/20814/2/_posts/2026-07-03-kudu-prometheus-grafana.md@107
PS2, Line 107:
             :
             :
             :
             :
             :
             :
             :
             :
             :
             :
             :
             :
> same as the other yaml.
Done


http://gerrit.cloudera.org:8080/#/c/20814/2/_posts/2026-07-03-kudu-prometheus-grafana.md@154
PS2, Line 154:
> nit: missing trailing newline at the end of the file.
Done



--
To view, visit http://gerrit.cloudera.org:8080/20814
To unsubscribe, visit http://gerrit.cloudera.org:8080/settings

Gerrit-Project: kudu
Gerrit-Branch: gh-pages
Gerrit-MessageType: comment
Gerrit-Change-Id: I85878378f883b2cc1f8136bde2c9f0db7ceccd8f
Gerrit-Change-Number: 20814
Gerrit-PatchSet: 1
Gerrit-Owner: Marton Greber <[email protected]>
Gerrit-Reviewer: Alexey Serbin <[email protected]>
Gerrit-Reviewer: Attila Bukor <[email protected]>
Gerrit-Reviewer: Gabriella Lotz <[email protected]>
Gerrit-Reviewer: Marton Greber <[email protected]>
Gerrit-Reviewer: Yan-Daojiang <[email protected]>
Gerrit-Reviewer: Zoltan Chovan <[email protected]>
Gerrit-Comment-Date: Wed, 08 Jul 2026 09:38:16 +0000
Gerrit-HasComments: Yes

Reply via email to