weaveworks / service

☁️ Images for Weave Cloud (R) (TM) (C) ☁️
https://cloud.weave.works
2 stars 2 forks source link

billing-ingester: Add internal_instance_id as label #2752

Closed ozamosi closed 2 years ago

ozamosi commented 2 years ago

The billing system tries to solve one specific problem:

It does this using a 20TB+ bigquery table.

With this tiny change that increases entropy merely a little, we could replace that with sum by (job, event_type, internal_instance_id) (rate(billing_ingester_events_total[1h])) as long as we take care to preserve hour alignment.

This only adds the label to dev, because I'm still trying to see that it works.

This could also replace the awful tables in https://cloud.weave.works/admin/billing/organizations with something like a grafana dashboard.