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.

Sign up for GitHub

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

Jump to bottom

Add system / resource usage to status reports. #14975

Merged
merged 2 commits into from
Jan 29, 2024
Merged

Add system / resource usage to status reports. #14975

merged 2 commits into from
Jan 29, 2024

Conversation

Sventimir
Copy link
Contributor

Explain your changes:
Add information on resource usage to node status:

  • uptime
  • load
  • total RAM
  • free RAM
  • total swap
  • free swap
  • number of running processes.

Explain how you tested your changes:

  • Run an ITN node, connecting to a testing status service instance.
  • CI

Checklist:

  • Dependency versions are unchanged
    • Notify Velocity team if dependencies must change in CI
  • Modified the current draft of release notes with details on what is completed or incomplete within this project
  • Document code purpose, how to use it
    • Mention expected invariants, implicit constraints
  • Tests were added for the new behavior
    • Document test purpose, significance of failures
    • Test names should reflect their purpose
  • All tests pass (CI will check this if you didn't)
  • Serialized types are in stable-versioned modules

@Sventimir Sventimir self-assigned this Jan 25, 2024
@Sventimir Sventimir changed the base branch from develop to rampup January 25, 2024 13:09
@Sventimir
Copy link
Contributor Author

!ci-build-me

@Sventimir Sventimir force-pushed the sventimir/status-report-resource-usage branch from 16d0f8e to 8034ed7 Compare January 25, 2024 13:19
@Sventimir
Copy link
Contributor Author

!ci-build-me

@Sventimir
Copy link
Contributor Author

!ci-nightly-me

@Sventimir Sventimir force-pushed the sventimir/status-report-resource-usage branch from 870bf9a to 2ce9695 Compare January 26, 2024 10:31
@Sventimir Sventimir marked this pull request as ready for review January 26, 2024 10:31
@Sventimir Sventimir requested a review from a team as a code owner January 26, 2024 10:31
@Sventimir
Copy link
Contributor Author

!ci-build-me

@Sventimir Sventimir force-pushed the sventimir/status-report-resource-usage branch from 2ce9695 to cd87b31 Compare January 26, 2024 10:56
@Sventimir
Copy link
Contributor Author

!ci-build-me

@Sventimir Sventimir force-pushed the sventimir/status-report-resource-usage branch from cd87b31 to e7243b1 Compare January 26, 2024 13:29
@Sventimir Sventimir changed the base branch from rampup to berkeley January 26, 2024 13:29
@Sventimir
Copy link
Contributor Author

!ci-build-me

@Sventimir Sventimir removed request for a team January 26, 2024 13:30
@Sventimir Sventimir force-pushed the sventimir/status-report-resource-usage branch from e7243b1 to 21b7829 Compare January 29, 2024 08:23
@Sventimir
Copy link
Contributor Author

!ci-build-me

@mrmr1993
Copy link
Member

!approved-for-mainnet

@mrmr1993 mrmr1993 merged commit e7bfee2 into berkeley Jan 29, 2024
39 checks passed
@mrmr1993 mrmr1993 deleted the sventimir/status-report-resource-usage branch January 29, 2024 13:50
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@georgeee georgeee georgeee approved these changes

Assignees

@Sventimir Sventimir

Labels
daemon MinaFoundation node status
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
3 participants
@Sventimir @mrmr1993 @georgeee