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.

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

Change series name to "statusok.request" and store URL as a tag instead #51

Open
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

jinnatar
Copy link

@jinnatar jinnatar commented Jan 4, 2020

The way data is pushed into influxdb right now is somewhat unorthodox and difficult to work with due to the URL being used as the series name. This means that adding any new URL to the config adds new series instead of extending the existing corpus of data.

TL;DR; a much more traditional way of structuring data in InfluxDB is to have a single series that I chose to call "statusok.request" and store the URL as a tag of each datapoint. This way the data is much easier to work with, for example in Grafana.

I can't actually build and test if this works since the project is a mess, all the dependencies have changed and nothing was pinned to versions that still worked. BUT, the change is simple enough that it ought to work if the project ever becomes buildable again.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

1 participant