pytest to reduce an already running storage pool / SR with specified device (disk & host) #309
Conversation
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
…hecks. Added docstring explaining what this new method does vs `disk_is_available`. Added type hints. Signed-off-by: Rushikesh Jadhav <[email protected]>
…specifying or auto-detecting Linstor SR expansion disks. Signed-off-by: Rushikesh Jadhav <[email protected]>
Signed-off-by: Rushikesh Jadhav <[email protected]>
…tor_sr_expand_host` Signed-off-by: Rushikesh Jadhav <[email protected]>
Added `test_linstor_sr_expand_disk` to detect & assert change in SR size. The test: - Ensures `--expansion-sr-disk` disk(s) are available on the hosts. - Configures LVM onto it and integrates into Linstor SR pool. - Expands SR capacity and validates the increase. Signed-off-by: Rushikesh Jadhav <[email protected]>
…tor SR by integrating a new host into the storage pool. The test: - Ensures the new host has available disks and installs required Linstor packages. - Configures LVM and integrates it into Linstor SR pool. - Expands SR capacity and validates the increase. - Handles cleanup by removing the node. Signed-off-by: Rushikesh Jadhav <[email protected]>
…ving a disk. The test: - Does not apply to "thin" type SR due to thin LVM limitations. - Identifies disks used for LVM, chooses a disk and empty it. - Once the disk is removed from LVM, linstor service is restarted to recognise the SR size change. Signed-off-by: Rushikesh Jadhav <[email protected]>
Signed-off-by: Rushikesh Jadhav <[email protected]>
…ving a host. The test: - Selects a non master host to evacuate and eject from pool. - Ensures that the host selected is not linstor-controller, if it is then moves the controller services to another host. - Evacuates host selected, stops linstor services and unplugs PBD. - Deletes selected node from linstor pool. - Restarts services to recognise the SR size change. Signed-off-by: Rushikesh Jadhav <[email protected]>
rushikeshjadhav
requested review from
ydirson,
Nambrok and
stormi
and removed request for
ydirson
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
The test by reducing disk:
- Does not apply to "thin" type SR due to thin LVM limitations.
- Identifies disks used for LVM, chooses a disk and empty it.
- Once the disk is removed from LVM, linstor service is restarted to recognise the SR size change.
The test by reducing host:
- Selects a non master host to evacuate and eject from pool.
- Ensures that the host selected is not linstor-controller, if it is then moves the controller services to another host.
- Evacuates host selected, stops linstor services and unplugs PBD.
- Deletes selected node from linstor pool.
- Restarts services to recognise the SR size change.