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

Add kalray plugin to configure DPU #42

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

Conversation

gthvn1
Copy link
Contributor

@gthvn1 gthvn1 commented Oct 16, 2023

This plugin allows to create raid, logical volume store and logical volumes on the Kalray DPU.

@gthvn1
Copy link
Contributor Author

gthvn1 commented Oct 16, 2023

Still need to add delete lvol, sanity check and unit test....

@stormi
Copy link
Member

stormi commented Oct 16, 2023

Make it a draft PR, then?

README.md Outdated Show resolved Hide resolved
@gthvn1 gthvn1 marked this pull request as draft October 16, 2023 16:14
@gthvn1 gthvn1 force-pushed the gtn-kalray-dpu branch 4 times, most recently from 9e77947 to 32ad725 Compare October 17, 2023 13:58
@gthvn1 gthvn1 marked this pull request as ready for review October 17, 2023 14:05
@gthvn1
Copy link
Contributor Author

gthvn1 commented Oct 17, 2023

any help about how to fix unittest is welcome :) and what to test because most of the jobs is in the RPC that we will need to mock because it is only available on the xcp-ng host when Kalray DPU is deployed...

@gthvn1 gthvn1 marked this pull request as draft October 17, 2023 15:39
@gthvn1
Copy link
Contributor Author

gthvn1 commented Oct 17, 2023

I think I got it for pytest.... So WIP :)

This plugin allows to create raid, logical volume store and logical
volumes on the Kalray DPU. It also allows the deletion of a volume.
Currently restrications are due to the Kalray DPU. Next generation of
the DPU will allow more volumes and won't be restricted on volume name.
See the README for details.

The unittest mocks the RPC server and it checks that required parameters
are well set and that all parameters used are valid.

Signed-off-by: Guillaume <[email protected]>
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