-
Notifications
You must be signed in to change notification settings - Fork 375
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
Add global level data_stream.dataset dashboard filter #9869
Add global level data_stream.dataset dashboard filter #9869
Conversation
🚀 Benchmarks reportTo see the full report comment with |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM! 🚀
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM!
Dismissing the review as more changes may be needed here.
@harnish-elastic , can you verify - if entries in |
Okay, let me take a look at this with @milan-elastic and update if needed. |
Done! |
…-dataset-filters-dashboad-level-set2
💚 Build Succeeded
History
|
Quality Gate passedIssues Measures |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM!
Package coredns - 0.7.0 containing this change is available at https://epr.elastic.co/search?package=coredns |
Package couchbase - 1.6.0 containing this change is available at https://epr.elastic.co/search?package=couchbase |
Package couchdb - 1.2.0 containing this change is available at https://epr.elastic.co/search?package=couchdb |
Package golang - 1.5.0 containing this change is available at https://epr.elastic.co/search?package=golang |
Package iis - 1.18.0 containing this change is available at https://epr.elastic.co/search?package=iis |
Package influxdb - 0.8.0 containing this change is available at https://epr.elastic.co/search?package=influxdb |
Package memcached - 1.4.0 containing this change is available at https://epr.elastic.co/search?package=memcached |
Package nats - 1.6.0 containing this change is available at https://epr.elastic.co/search?package=nats |
Package stan - 1.6.0 containing this change is available at https://epr.elastic.co/search?package=stan |
Package zookeeper - 1.11.0 containing this change is available at https://epr.elastic.co/search?package=zookeeper |
Proposed commit message
Add a global filter to the dashboard and remove the subfilters on the dataset and a global filter on the dataset is introduced. Filters on data_stream.dataset are applied with a phrase type filter on the dataset that is being used in that particular dashboard.
Expect the above mentioned changes done in following selected packages to ease the review
Checklist
changelog.yml
file.Related issues
Screenshot
Stan
Nats
Golang
Couchbase
Couchdb
IIS
Memcached
Zookeeper
InfluxDB
Coredns