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 support for LVM LocalPV #33

Closed
6 tasks done
vharsh opened this issue Jun 18, 2021 · 0 comments · Fixed by #45, #74 or #75
Closed
6 tasks done

Add support for LVM LocalPV #33

vharsh opened this issue Jun 18, 2021 · 0 comments · Fixed by #45, #74 or #75
Assignees

Comments

@vharsh
Copy link
Member

vharsh commented Jun 18, 2021

Potential scope

More details will be added here, as more info drops by

Some more fields are getting added to the LVM resources -> @ openebs/lvm-localpv -> 115

  • Add a sub-command for getting volume-group(vg), as squishing this into the pool sub-command doesn't make good UX sense.
  • pool sub-command will be re-named to something generic like storage, so that it can mean a pool of resources(& not a pool as in a cstor pool or an LVM volume-group) as mentioned in this recent talks
  • CSI -> Get CR structures from https://github.com/openebs/lvm-localpv
  • Listing LVM volumes
  • Listing pools/LVM-nodes
  • Describing LVM volumes
  • Describing pools/LVM-nodes, etc
@vharsh vharsh assigned vharsh and unassigned vharsh Jul 5, 2021
@vharsh vharsh changed the title Add support for LocalPV Add support for LVM LocalPV Jul 5, 2021
@vharsh vharsh self-assigned this Jul 5, 2021
@prateekpandey14 prateekpandey14 linked a pull request Jul 8, 2021 that will close this issue
@vharsh vharsh closed this as completed in #45 Jul 8, 2021
@vharsh vharsh reopened this Jul 8, 2021
@vharsh vharsh added the pr/hold label Jul 26, 2021
@vharsh vharsh reopened this Aug 12, 2021
@vharsh vharsh linked a pull request Aug 12, 2021 that will close this issue
@vharsh vharsh removed the pr/hold label Aug 12, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment