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

Feature: System resources were not exposed #224

Merged
merged 1 commit into from
Sep 13, 2022
Merged

Conversation

hoh
Copy link
Member

@hoh hoh commented Sep 13, 2022

The scheduling of persistent VMs requires external services to fetch the available system resources of the host.

Solution: Add a new HTTP endpoint on /about/usage/system that exposes system resources and system properties of the host machine.

@hoh hoh requested a review from odesenfans September 13, 2022 06:26
Copy link
Collaborator

@odesenfans odesenfans left a comment

Choose a reason for hiding this comment

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

Mostly cosmetic changes.

vm_supervisor/resources.py Outdated Show resolved Hide resolved
vm_supervisor/resources.py Outdated Show resolved Hide resolved
vm_supervisor/resources.py Outdated Show resolved Hide resolved
vm_supervisor/resources.py Outdated Show resolved Hide resolved
vm_supervisor/supervisor.py Outdated Show resolved Hide resolved
vm_supervisor/resources.py Outdated Show resolved Hide resolved
@odesenfans
Copy link
Collaborator

Btw, isn't this the same PR as #194 ?

@hoh
Copy link
Member Author

hoh commented Sep 13, 2022

Btw, isn't this the same PR as #194 ?

That branch contained a lot of other changes and is now closed.

The scheduling of persistent VMs requires external services to fetch
the available system resources of the host.

Solution: Add a new HTTP endpoint on `/about/usage/system`
that exposes system resources and system properties of
the host machine.
@hoh hoh merged commit 75425cb into main Sep 13, 2022
@hoh hoh deleted the hoh-expose-system-usage branch September 13, 2022 15:58
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.

2 participants