[pve-devel] applied: [PATCH v2 storage 3/3] rbd: add integration test for namespace handling

[Thomas Lamprecht]

On 07.04.21 16:22, Aaron Lauterer wrote:
> This test is intended to be run on a hyperconverged PVE cluster to test
> the most common operations of VMs using a namespaced Ceph RBD pool.
> 
> Signed-off-by: Aaron Lauterer <a.lauterer at proxmox.com>
> ---
> v1 -> v2:
> reworked the test from the feedback I got [0].
> 
> * tests are now defined in deeper hashes/arrays and can contain testing
>   steps as well as preparation and cleanup steps where needed.
> * command calls don't use fixed paths
> * command calls use arrays
> * a new Ceph pool will be created and removed at the end by default. Can
>   be overriden with the '--use_existing' parameter. Most likely in
>   combination with the '--pool' parameter.
> * debug flag has been introduced to be used if some debug printing via
>   the `jp` sub is needed. Currenctly not used but might be useful when
>   tinkering with the test in the future. Thx @Thomas for the hint
> 
> 
> [0] https://lists.proxmox.com/pipermail/pve-devel/2021-April/047472.html
> 
>  test/rbd_namespace.pl | 370 ++++++++++++++++++++++++++++++++++++++++++
>  1 file changed, 370 insertions(+)
>  create mode 100755 test/rbd_namespace.pl
> 
>

applied, thanks!

I made three small follow-ups though:

* switched --use_existing to --use-existing
* exit cleanly on --help, its not an error after all!
* allow also the short-opt version for help, -h