Skip to content

diagnostics: include resource usage - #530

Merged
petrutlucian94 merged 2 commits into
cloudbase:mainfrom
petrutlucian94:diagnostics
Sep 25, 2026
Merged

petrutlucian94 merged 2 commits into
cloudbase:mainfrom
petrutlucian94:diagnostics

Conversation

@petrutlucian94

Copy link
Copy Markdown
Member

We're updating the diagnostic report to include resource usage (disk, memory and cpu).

See the updated API samples for examples.

We're updating the diagnostic report to include resource usage
(disk, memory and cpu).

See the updated API samples for examples.

@fabi200123 fabi200123 left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

LGTM! Resource usage can come handy for discovering different issues.

},
"cpu_usage": [
{"core": 0, "percent": 3.2},
{"core": 1, "percent": 1.0}

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

Is this an average, or a momentary snapshot of the cpu usage?

Copy link
Copy Markdown
Member Author

Choose a reason for hiding this comment

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

It's sampling over 0.1s

Comment thread coriolis/api-refs/api_samples/diagnostics/diagnostics-get-resp.json Outdated
Comment thread coriolis/tests/test_utils.py
Comment thread coriolis/utils.py Outdated
…eaning

"capacity" is misleading, we'll report it as "used_percentage" instead.

Also, we'll ensure that "used" also includes reserved space so that
used + free == total.
@petrutlucian94
petrutlucian94 merged commit 778e001 into cloudbase:main Sep 25, 2026
5 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants