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
[AWS] Add Firewall metrics dashboard input control groups #5650
[AWS] Add Firewall metrics dashboard input control groups #5650
Conversation
🌐 Coverage report
|
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.
Is there a reason you chose to use the Lens legacy metric visualization? In the future, this will likely be removed in favor of the metric visualization.
Hi! We just realized that we haven't looked into this PR in a while. We're sorry! We're labeling this issue as |
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.
@gpop63 sorry about this—let's take these back to the legacy metric for now. The way this dashboard is set up, especially after removing the legacy dashboard controls from the layout, just leaves too much vertical room for the new metric to make much sense.
Approving so as not to hold you up.
From meeting
|
…s-input-control-groups
@drewdaemon what do you think about the new layout? |
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 aws - 1.40.7 containing this change is available at https://epr.elastic.co/search?package=aws |
* add input control groups * bump package version * replace legacy with metric charts * fix data stream filter * fix layout
What does this PR do?
Adds input control groups.
I'm not sure if the markdown visualization (first one from images) can be converted to a lens. Should it be left like that or removed?
Before:
After:
Checklist
changelog.yml
file.Author's Checklist
How to test this PR locally
Related issues
Screenshots