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

Added iSCSI multipath status to the API #934

Open
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

BrooklynDewolf
Copy link
Contributor

@BrooklynDewolf BrooklynDewolf commented Apr 11, 2024

Fixes issue #366

Changes introduced with this PR

  • A new field has been added <available_paths> to the /ovirt-engine/api/hosts/xxxx/storage endpoint that represents the multipaths that are NOT faulty as an integer value. This is useful for monitoring (you can compare 'available_paths' to 'paths' and trigger an alert when 'available_paths' is lower than 'paths'.

This commit needs oVirt/ovirt-engine-api-model#106 before merge.

Are you the owner of the code you are sending in, or do you have permission of the owner?

Yes

BrooklynDewolf pushed a commit to BrooklynDewolf/ovirt-engine-api-model that referenced this pull request Apr 11, 2024
This is necessary for the iSCI multipath status oVirt/ovirt-engine#934

Signed-off-by: Brooklyn Dewolf <[email protected]>
@BrooklynDewolf BrooklynDewolf marked this pull request as ready for review April 11, 2024 11:39
BrooklynDewolf pushed a commit to BrooklynDewolf/ovirt-engine-api-model that referenced this pull request Apr 11, 2024
This is necessary for the iSCI multipath status oVirt/ovirt-engine#934

Signed-off-by: Brooklyn Dewolf <[email protected]>
@sandrobonazzola sandrobonazzola requested review from dupondje and removed request for oliel May 30, 2024 08:54
In this commit the multipath status is available in the following
endpoint:
/ovirt-engine/api/hosts/xxxx/storage

A new field has been added <available_paths> that represents the
multipaths that are NOT faulty as an integer value. This is useful for monitoring (you can
compare 'available_paths' to 'paths' and trigger an alert when
'available_paths' is lower than 'paths'.

Signed-off-by: Brooklyn Dewolf <[email protected]>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants