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
[O11y][Docker] Migrate Docker Overview dashboard to lens #7942
[O11y][Docker] Migrate Docker Overview dashboard to lens #7942
Conversation
🌐 Coverage report
|
/test |
} | ||
} | ||
}, | ||
"title": "Docker containers [Metrics Docker] (converted)", |
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.
"title": "Docker containers [Metrics Docker] (converted)", | |
"title": "Docker containers [Metrics Docker]", |
"shape": "donut" | ||
} | ||
}, | ||
"title": "Docker containers per host [Metrics Docker] (converted)", |
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.
"title": "Docker containers per host [Metrics Docker] (converted)", | |
"title": "Docker containers per host [Metrics Docker]", |
"y": 0 | ||
}, | ||
"panelIndex": "6cb38196-b994-40f8-b871-09d8e532a843", | ||
"title": "Number of Running Containers [Metrics Docker]", |
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.
"title": "Number of Running Containers [Metrics Docker]", | |
"title": "Running Containers [Metrics Docker]", |
"y": 0 | ||
}, | ||
"panelIndex": "e5f670cb-36d3-4481-be82-733775edcb83", | ||
"title": "Number of Paused Containers [Metrics Docker]", |
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.
"title": "Number of Paused Containers [Metrics Docker]", | |
"title": "Paused Containers [Metrics Docker]", |
"y": 0 | ||
}, | ||
"panelIndex": "58e96448-b689-45bd-b0b2-a0ee57fc0235", | ||
"title": "Number of Stopped Containers [Metrics Docker]", |
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.
"title": "Number of Stopped Containers [Metrics Docker]", | |
"title": "Stopped Containers [Metrics Docker]", |
"shape": "donut" | ||
} | ||
}, | ||
"title": "Docker images and names [Metrics Docker] (converted)", |
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.
"title": "Docker images and names [Metrics Docker] (converted)", | |
"title": "Docker images and names [Metrics Docker] ", |
"yTitle": "Count" | ||
} | ||
}, | ||
"title": "CPU usage [Metrics Docker] (converted)", |
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.
"title": "CPU usage [Metrics Docker] (converted)", | |
"title": "CPU usage [Metrics Docker]", |
"yTitle": "Count" | ||
} | ||
}, | ||
"title": "Memory usage [Metrics Docker] (converted)", |
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.
"title": "Memory usage [Metrics Docker] (converted)", | |
"title": "Memory usage [Metrics Docker]", |
"yTitle": "Count" | ||
} | ||
}, | ||
"title": "Network IO [Metrics Docker] (converted)", |
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.
"title": "Network IO [Metrics Docker] (converted)", | |
"title": "Network IO [Metrics Docker] ", |
"panelIndex": "2", | ||
"panelRefName": "panel_1", | ||
"version": "7.3.0" | ||
"title": "Docker containers [Metrics Docker]", |
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.
Do we need [Metrics Docker] to be visible to all titles? Shall we remove it?
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.
@gizas, According to Elastic documentation for integration development it is recommended to add suffix <NAME> [<Metrics | Logs> <PACKAGE NAME>]
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.
I think it is best to remove it, according to the dashboard guidelines.
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.
Here is the issue for removing suffix from docker visualization.
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 docker - 2.8.3 containing this change is available at https://epr.elastic.co/search?package=docker |
Urgency
Activity Type
What does this PR do?
Docker
visualizations to the lens in the current Kibana version8.8.0
itself.Migration stats
Checklist
changelog.yml
file.Author's Checklist
Related issues
Notes
Docker containers
panel [O11y][Docker] Update format in table visualization #7915