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

feat: cAdvisor Docker System Metrics and Dashboard for Dev Docker Stack #258

Merged
merged 1 commit into from
Oct 15, 2024

Conversation

AlfredoG87
Copy link
Contributor

@AlfredoG87 AlfredoG87 commented Oct 15, 2024

Description:

  • Improving Grafana configuration to remove login and need to change admin user
  • Added cAdvisor to Docker dev Stack to collect system metrics on host and containers
  • Added cAdvisor to list of targets to scrap from prometheus instance of the stack
  • Added a Dashboard for System Host and Container metrics visualization on grafana

Related issue(s):

Fixes #257

Notes for reviewer:
image
image
image
image

Checklist

  • Documented (Code comments, README, etc.)
  • Tested (unit, integration, etc.)

…admin user

- Added cAdvisor to Docker dev Stack to collect system metrics on host and containers
- Added cAdvisor to list of targets to scrap from prometheus instance of the stack
- Added a Dashboard for System Host and Container metrics visualization on grafana

Signed-off-by: Alfredo Gutierrez <[email protected]>
@AlfredoG87 AlfredoG87 self-assigned this Oct 15, 2024
@AlfredoG87 AlfredoG87 added Improvement Code changes driven by non business requirements P2 Required to be completed in the assigned milestone, but may or may not impact release schedule. labels Oct 15, 2024
@AlfredoG87 AlfredoG87 added this to the 0.2.0 milestone Oct 15, 2024
@AlfredoG87 AlfredoG87 marked this pull request as ready for review October 15, 2024 16:45
@AlfredoG87 AlfredoG87 requested a review from a team as a code owner October 15, 2024 16:45
Copy link
Contributor

@mattp-swirldslabs mattp-swirldslabs left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM... were you able to get the Network dashboards to work?

@AlfredoG87 AlfredoG87 merged commit c2a360a into main Oct 15, 2024
16 of 20 checks passed
@AlfredoG87 AlfredoG87 deleted the cadvisor-system-dashboard branch October 15, 2024 20:22
Copy link

codecov bot commented Oct 15, 2024

Codecov Report

All modified and coverable lines are covered by tests ✅

Project coverage is 99.66%. Comparing base (e89eef4) to head (75ab5a6).
Report is 1 commits behind head on main.

Additional details and impacted files
@@            Coverage Diff            @@
##               main     #258   +/-   ##
=========================================
  Coverage     99.66%   99.66%           
  Complexity      248      248           
=========================================
  Files            51       51           
  Lines           907      907           
  Branches         61       61           
=========================================
  Hits            904      904           
  Misses            3        3           

@AlfredoG87
Copy link
Contributor Author

LGTM... were you able to get the Network dashboards to work?

No I was not, but maybe we can work it out on a follow up PR, since is not as straight forwards as I originally thought, I think there is something missing or wrong on the docker-compose.yaml for all containers in order to get them.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Improvement Code changes driven by non business requirements P2 Required to be completed in the assigned milestone, but may or may not impact release schedule.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Add System Stat Metrics to the docker dev stack
2 participants