profile
viewpoint
If you are wondering where the data of this site comes from, please visit https://api.github.com/users/tomwilkie/events. GitMemory does not store any data, but only uses NGINX to cache data for a period of time. The idea behind GitMemory is simply to give users a better reading experience.
Tom Wilkie tomwilkie @grafana London https://grafana.com @grafana VP Product, @prometheus & @cortexproject maintainer. Previously @kausalco, @weaveworks, @google, @acunu

grafana/loki 13806

Like Prometheus, but for logs.

cortexproject/cortex 4300

A horizontally scalable, highly available, multi-tenant, long term Prometheus.

grafana/tanka 1445

Flexible, reusable and concise configuration for Kubernetes

pywemo/pywemo 118

Python module to discover and control WeMo devices.

prometheus/compliance 69

A set of tests to check compliance with various Prometheus interfaces

tomwilkie/awesomation 26

Home Awesomation; python home automation system.

grafana/cortex 25

A multitenant, horizontally scalable Prometheus as a Service

tomwilkie/cubienetes 11

Cubienetes: A Kubernetes Cluster on Cubieboard2s

tomwilkie/aws-self-serve 4

A self service portal for ec2 instances

pull request commentgrafana/jsonnet-libs

alertmanager: restart pods on config change

The watch container will try to reload the config, but if its invalid that won't disrupt the working Alertmanager.

We have an alert to detect that.

If you would redeploy the StatefulSet with a new config that is invalid, it might prevent it from starting.

Indeed; its deemed more important that the alerting work than it uses the latest config. So we'd rather a config reload fails than we restart the service and it fails to start (causing an outage).

asymmetric

comment created time in 40 minutes

push eventgrafana/grafonnet-lib

Russell Howe

commit sha f7fae27965036dd6f931c7cec169e5fb9acceb57

Fix links to datasource docs These links went to pages for Grafana 7.5. Update them to the latest URLs.

view details

Tom Wilkie

commit sha 05fb200ee1a1816fc1b4c522071d5606d8dd71c1

Merge pull request #344 from rhowe/fix-cw-docs-url Fix links to datasource docs

view details

push time in 10 days

PR merged grafana/grafonnet-lib

Fix links to datasource docs

These links went to pages for Grafana 7.5. Update them to the latest URLs.

This is probably a bug in the docs website tbh.

+7 -7

1 comment

7 changed files

rhowe

pr closed time in 10 days

pull request commentgrafana/grafonnet-lib

Fix links to datasource docs

Thank you!

rhowe

comment created time in 10 days

push eventgrafana/grafonnet-lib

Tom Wilkie

commit sha f68483101ffd3aa8a7193a96eda801bc1c4dee5d

Typo in usage example. Signed-off-by: Tom Wilkie <tom@grafana.com>

view details

push time in 11 days

push eventgrafana/grafonnet-lib

Tom Wilkie

commit sha 3521d13cf6fd669f5392ab97c49a88a388c8b43b

jsonnetfmt Signed-off-by: Tom Wilkie <tom@grafana.com>

view details

push time in 12 days

pull request commentgrafana/grafonnet-lib

Add a little helper library to automatically manage the layout of panels

Let me know if you think this belongs here; if so I'll tidy it up a little and get it ready for commit.

tomwilkie

comment created time in 12 days

create barnchgrafana/grafonnet-lib

branch : layout

created branch time in 12 days

PullRequestReviewEvent

push eventmonitoring-mixins/mixtool

Ryan Geyer

commit sha bca30663e83cc9574fa9e841e2990bd0f6963d01

Update the grafana/grafana dependency to latest (#39) * Update the grafana/grafana dependency to latest, which resolves the Unknwon/unknwon conflict * Update golang version to latest (1.16)

view details

push time in a month

PR merged monitoring-mixins/mixtool

Update the grafana/grafana dependency to latest

This actually resolves what I was trying to resolve in #37, but didn't fully test.

This PR resolves the inability to install using go get due to issues resolving the dependency tree.

Coincidentally, it also sets us up to check against the latest Grafana models (warn of deprecated visualizations, perhaps?).

+1782 -18

1 comment

3 changed files

rgeyer

pr closed time in a month

push eventtomwilkie/prom-run

Annanay Agarwal

commit sha cfd0fe97e6874a56d7828cf9e6d3c6a84ceca7c0

Print output even on success Signed-off-by: Annanay Agarwal <annanay.agarwal@grafana.com>

view details

Tom Wilkie

commit sha 27fabd294517887a703e22a0b5f7aaf3950bdf0d

Merge pull request #10 from annanay25/print-output-on-success Print output even on success

view details

push time in a month

PR merged tomwilkie/prom-run

Print output even on success

The output of a cronjob is often useful to track over time as well.

Signed-off-by: Annanay Agarwal annanay.agarwal@grafana.com

+2 -1

0 comment

1 changed file

annanay25

pr closed time in a month

pull request commentprometheus/node_exporter

[WIP] Update the node mixin to add some more info and make the dashboards more attractive.

Oh sorry I completely forgot! No I think this was a failed experiment. Thanks!

tomwilkie

comment created time in a month

delete branch cortexproject/cortex

delete branch : community-call-times

delete time in a month

push eventcortexproject/cortex

Tom Wilkie

commit sha e658571295c09c4426c25511597c8e2ee0e1a524

Update community call times. (#4405) Signed-off-by: Tom Wilkie <tom@grafana.com>

view details

push time in a month

PR merged cortexproject/cortex

Reviewers
Update community call times. size/XS
+1 -1

1 comment

1 changed file

tomwilkie

pr closed time in a month

push eventcortexproject/cortex

Tom Wilkie

commit sha 3cf97f54cec7955071f838630fa8bf7880c92d83

Update community call times. Signed-off-by: Tom Wilkie <tom@grafana.com>

view details

push time in a month

PR opened cortexproject/cortex

Reviewers
Update community call times.
+1 -1

0 comment

1 changed file

pr created time in a month

create barnchcortexproject/cortex

branch : community-call-times

created branch time in a month

push eventgrafana/tns

Connor Lindsey

commit sha e24910f27392035ed19b0319d056513431821bfd

Update data source instruction wording

view details

Tom Wilkie

commit sha 7809f56397c87f31b3af8f66056450993ea3ca3d

Merge pull request #10 from grafana/connor/docs Update data source instruction wording

view details

push time in 2 months

delete branch grafana/tns

delete branch : connor/docs

delete time in 2 months

PR merged grafana/tns

Reviewers
Update data source instruction wording

Exemplars are now included in the Prometheus data source so rewording docs to match.

+2 -2

0 comment

1 changed file

connorlindsey

pr closed time in 2 months

PullRequestReviewEvent
PullRequestReviewEvent

Pull request review commentgrafana/cortex-jsonnet

Fix scaling dashboard to work on multi-zone ingesters

 local utils = import 'mixin-utils/utils.libsonnet';         rules: [           {             // Convenience rule to get the number of replicas for both a deployment and a statefulset.+            // Multi-zone deployments are grouped together removing the "zone-X" suffix.             record: 'cluster_namespace_deployment:actual_replicas:count',             expr: |||-              sum by (cluster, namespace, deployment) (kube_deployment_spec_replicas)-                or               sum by (cluster, namespace, deployment) (-                label_replace(kube_statefulset_replicas, "deployment", "$1", "statefulset", "(.*)")+                label_replace(+                  kube_deployment_spec_replicas,+                  "deployment", "$1", "deployment", "(.*?)(?:-zone-[a-z])?"

TIL! Can you add a comment to this effect please?

pracucci

comment created time in 2 months

PullRequestReviewEvent

Pull request review commentgrafana/cortex-jsonnet

Fix scaling dashboard to work on multi-zone ingesters

 local utils = import 'mixin-utils/utils.libsonnet';         rules: [           {             // Convenience rule to get the number of replicas for both a deployment and a statefulset.+            // Multi-zone deployments are grouped together removing the "zone-X" suffix.             record: 'cluster_namespace_deployment:actual_replicas:count',             expr: |||-              sum by (cluster, namespace, deployment) (kube_deployment_spec_replicas)-                or               sum by (cluster, namespace, deployment) (-                label_replace(kube_statefulset_replicas, "deployment", "$1", "statefulset", "(.*)")+                label_replace(+                  kube_deployment_spec_replicas,+                  "deployment", "$1", "deployment", "(.*?)(?:-zone-[a-z])?"+                )+              )+              or+              sum by (cluster, namespace, deployment) (+                label_replace(+                  label_replace(kube_statefulset_replicas, "deployment", "$1", "statefulset", "(.*)"),+                  "deployment", "$1", "deployment", "(.*?)(?:-zone-[a-z])?"+                )

The inner label replace is just moving the statefuleset label to the deployment label, so could be done with this I believe:

                label_replace(kube_statefulset_replicas, "deployment", "$1", "statefulset", "(.*?)(?:-zone-[a-z])?"),
pracucci

comment created time in 2 months