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

New module for LVM #58915

Open
wants to merge 1 commit into
base: devel
from

Conversation

Projects
None yet
3 participants
@japokorn
Copy link
Contributor

commented Jul 10, 2019

added new module for handling LVM:

  • high level module for LVM handling
  • added unit tests for LVM module
SUMMARY

New module for high level handling of Logical Volume Management (LVM):

  • lvm - handles the whole LVM
  • includes tests for LVM module
ISSUE TYPE
  • New Module Pull Request
COMPONENT NAME

lvm

ADDITIONAL INFORMATION

The goal of the lvm module is to provide high level interface which automates some tasks
related to creating, removing and maintaining LVM. Its design also allows adding new
functionality into it without affecting the interface.
Since the module logic is non-trivial, the code is separated into multiple files.
The files in module_utils (pv.py, vg.py, lv.py) provide functionality for the individual LVM layers (Physical Volume, Volume Group, Logical Volume).
LVM commands are executed only in these files using third party library (*libblockdev).
These files are used by lvm module which contains argument processing and most of the logic.
The design also allows creation of simple modules: lvm_pv, lvm_vg and lvm_lv which provide control over their respective LVM layers by utilizing module_utils files. These modules are not part of this PR and will be creatred later.

  • Even though there are cli tools for LVM handling, using third party library has large advantages. The library interface is stable, its output does not need to be parsed and is already well tested.
New module for LVM
added new module for handling LVM:
 - high level module for LVM handling
 - added unit tests for LVM module
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
You can’t perform that action at this time.