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

Filtering InfluxDB series by parts of thier name #613

Closed
sontags opened this issue Jul 22, 2014 · 12 comments
Closed

Filtering InfluxDB series by parts of thier name #613

sontags opened this issue Jul 22, 2014 · 12 comments

Comments

@sontags
Copy link

sontags commented Jul 22, 2014

I am feeding nmon data from multiple servers into an InfluxDB and use Grafana for basic analytics/monitoring. The series feed into InfluxDB are named in simple patterns where elements are separated by a dot, for example:

[prefix].[servername].[nmon-section].[metric]

For example raw.raspi.CPU_ALL.SysPercent contains system cpu usage over all CPU's on the node raspi.

Now, for example, I would like to have a filter at the place of the [servername]. The query would look somehow like that:

from /raw.*'[[servername]]'*.CPU_ALL.*/ where  time > 1406045460s and time < 1406047512s group by time(1s)  order asc

A input box (in this case better than a dropdown) would allow me to filter.

Currently I do this via scripted dashboards which works fine, but thats somehow lacking of flexibility...

Cheers Daniel

@tmonk42
Copy link
Contributor

tmonk42 commented Aug 5, 2014

+1

This was one of my favorite features when I was using graphite/cyanite as the backend.

@torkelo
Copy link
Member

torkelo commented Aug 6, 2014

An overhaul of the filter feature (with big InfluxDB improvements) is planed for v1.8

@torkelo torkelo added this to the v1.8.0 milestone Aug 10, 2014
@dzrw
Copy link

dzrw commented Aug 13, 2014

👍

@pauldix
Copy link

pauldix commented Aug 14, 2014

The list series query now has support for matching on regexes influxdata/influxdb#376. It's in master now and will be in the 0.8 release. We'll push an RC by tomorrow afternoon with this for testing.

That should make this kind of filtering fast when you're building the query and you want to do quick lookups to see which series will match

@torkelo
Copy link
Member

torkelo commented Aug 14, 2014

@pauldix that is great. Will start working on the filter feature overhaul next week I hope.

@sevastos
Copy link

sevastos commented Sep 1, 2014

👍

@torkelo
Copy link
Member

torkelo commented Sep 1, 2014

Progress on InfluxDB and filter/templating support!

image

Only need to solve how to handle an "All" option that is when a templated variable has an "all" option that should show all series. The metric queries will need all need to be using a regex syntax and the "all" value would need to be a regex pattern.

@torkelo
Copy link
Member

torkelo commented Sep 5, 2014

Ready to test in master! There are some breaking changes for InfluxDB users in master (read changelog for more details)

image

@sevastos
Copy link

Looking good! However, I'm temporarily stuck on Influx 0.7 and list series /regex/ seems like a 0.8 feature, so can't test atm 😦

@sevastos
Copy link

sevastos commented Oct 4, 2014

Amazing job @torkelo, works great and the new series specific overrides is really awesome!

@blufor
Copy link

blufor commented Oct 6, 2014

that is great work! goodbye, last remnants of graphite :)

@joelittlejohn
Copy link

A note for anyone that gets here from Google, since InfluxDB 0.9 you'll need SHOW series instead of list series.

sunker added a commit that referenced this issue Nov 28, 2022
* refactor resource response type

* remove not used file.

* go lint

* fix tests

* remove commented code
sunker added a commit that referenced this issue Nov 28, 2022
* refactor resource response type

* remove not used file.

* go lint

* fix tests

* remove commented code
sunker added a commit that referenced this issue Nov 28, 2022
* Lattice: Point to private prerelease of aws-sdk-go (#515)

* point to private prerelease of aws-sdk-go

* fix build issue

* Lattice: Adding a feature toggle (#549)

* Adding a feature toggle for lattice

* Change name of feature toggle

* Lattice: List accounts (#543)

* Separate layers

* Introduce testify/mock library

Co-authored-by: Shirley Leu <4163034+fridgepoet@users.noreply.github.com>

* point to version that includes metric api changes (#574)

* add accounts component (#575)

* Test refactor: remove unneeded clientFactoryMock (#581)

* Lattice: Add monitoring badge (#576)

* add monitoring badge

* fix tests

* solve conflict

* Lattice: Add dynamic label for account display name (#579)

* Build: Automatically sync lattice-main with OSS

* Lattice: Point to private prerelease of aws-sdk-go (#515)

* point to private prerelease of aws-sdk-go

* fix build issue

* Lattice: Adding a feature toggle (#549)

* Adding a feature toggle for lattice

* Change name of feature toggle

* Lattice: List accounts (#543)

* Separate layers

* Introduce testify/mock library

Co-authored-by: Shirley Leu <4163034+fridgepoet@users.noreply.github.com>

* point to version that includes metric api changes (#574)

* add accounts component (#575)

* Test refactor: remove unneeded clientFactoryMock (#581)

* Lattice: Add monitoring badge (#576)

* add monitoring badge

* fix tests

* solve conflict

* add account label

Co-authored-by: Shirley Leu <4163034+fridgepoet@users.noreply.github.com>
Co-authored-by: Sarah Zinger <sarah.zinger@grafana.com>

* fix import

* solve merge related problem

* add account info (#608)

* add back namespaces handler

* Lattice: Parse account id and return it to frontend (#609)

* parse account id and return to frontend

* fix route test

* only show badge when feature toggle is enabled (#615)

* Lattice: Refactor resource response type and return account (#613)

* refactor resource response type

* remove not used file.

* go lint

* fix tests

* remove commented code

* Lattice: Use account as input when listing metric names and dimensions (#611)

* use account in resource requests

* add account to response

* revert accountInfo to accountId

* PR feedback

* unit test account in list metrics response

* remove not used asserts

* don't assert on response that is not relevant to the test

* removed dupe test

* pr feedback

* rename request package (#626)

* Lattice: Move account component and add tooltip (#630)

* move accounts component to the top of metric stat editor

* add tooltip

* CloudWatch: add account to GetMetricData queries (#627)

* Add AccountId to metric stat query

* Lattice: Account variable support  (#625)

* add variable support in accounts component

* add account variable query type

* update variables

* interpolate variable before its sent to backend

* handle variable change in hooks

* remove not used import

* Update public/app/plugins/datasource/cloudwatch/components/Account.tsx

Co-authored-by: Sarah Zinger <sarah.zinger@grafana.com>

* Update public/app/plugins/datasource/cloudwatch/hooks.ts

Co-authored-by: Sarah Zinger <sarah.zinger@grafana.com>

* add one more unit test

Co-authored-by: Sarah Zinger <sarah.zinger@grafana.com>

* cleanup (#629)

* Set account Id according to crossAccountQuerying feature flag in backend (#632)

* CloudWatch: Change spelling of feature-toggle (#634)

* Lattice Logs (#631)

* Lattice Logs

* Fixes after CR

* Lattice: Bug: fix dimension keys request (#644)

* fix dimension keys

* fix lint

* more lint

* CloudWatch: Add tests for QueryData with AccountId (#637)

* Update from breaking change (#645)

* Update from breaking change

* Remove extra interface and methods

Co-authored-by: Shirley Leu <4163034+fridgepoet@users.noreply.github.com>

* CloudWatch: Add business logic layer for getting log groups (#642)



Co-authored-by: Sarah Zinger <sarah.zinger@grafana.com>

* Lattice: Fix - unset account id in region change handler (#646)

* move reset of account to region change handler

* fix broken test

* Lattice: Add account id to metric stat query deep link (#656)

add account id to metric stat link

* CloudWatch: Add new log groups handler for cross-account querying (#643)

* Lattice: Add feature tracking (#660)

* add tracking for account id prescense in metrics query

* also check feature toggle

* fix broken test

* CloudWatch: Add route for DescribeLogGroups for cross-account querying (#647)

Co-authored-by: Erik Sundell <erik.sundell87@gmail.com>

* Lattice: Handle account id default value (#662)

* make sure right type is returned

* set right default values

* Suggestions to lattice changes (#663)

* Change ListMetricsWithPageLimit response to slice of non-pointers

* Change GetAccountsForCurrentUserOrRole response to be not pointer

* Clean test Cleanup calls in test

* Remove CloudWatchAPI as part of mock

* Resolve conflicts

* Add Latest SDK (#672)

* add tooltip (#674)

* Docs: Add documentation for CloudWatch cross account querying (#676)

* wip docs

* change wordings

* add sections about metrics and logs

* change from monitoring to observability

* Update docs/sources/datasources/aws-cloudwatch/_index.md

Co-authored-by: Sarah Zinger <sarah.zinger@grafana.com>

* Update docs/sources/datasources/aws-cloudwatch/query-editor/index.md

Co-authored-by: Fiona Artiaga <89225282+GrafanaWriter@users.noreply.github.com>

* Update docs/sources/datasources/aws-cloudwatch/query-editor/index.md

Co-authored-by: Fiona Artiaga <89225282+GrafanaWriter@users.noreply.github.com>

* Update docs/sources/datasources/aws-cloudwatch/query-editor/index.md

Co-authored-by: Sarah Zinger <sarah.zinger@grafana.com>

* Update docs/sources/datasources/aws-cloudwatch/query-editor/index.md

Co-authored-by: Fiona Artiaga <89225282+GrafanaWriter@users.noreply.github.com>

* apply pr feedback

* fix file name

* more pr feedback

* pr feedback

Co-authored-by: Sarah Zinger <sarah.zinger@grafana.com>
Co-authored-by: Fiona Artiaga <89225282+GrafanaWriter@users.noreply.github.com>

* use latest version of the aws-sdk-go

* Fix tests' mock response type

* Remove change in Azure Monitor

Co-authored-by: Sarah Zinger <sarah.zinger@grafana.com>
Co-authored-by: Shirley Leu <4163034+fridgepoet@users.noreply.github.com>
Co-authored-by: Fiona Artiaga <89225282+GrafanaWriter@users.noreply.github.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

9 participants