Skip to content

Commit

Permalink
test/suites: Ignore errors when removing ceph pool
Browse files Browse the repository at this point in the history
Seems this always returns an error because the pool was removed with the
LXD storage pool. However, because it was the last line, the error was
not propagating.

To ensure there's no leftover state, we still run the command but do not
expect it to pass.

Signed-off-by: Max Asnaashari <[email protected]>
  • Loading branch information
masnax committed Oct 22, 2024
1 parent b8905a4 commit 0c39aed
Showing 1 changed file with 1 addition and 1 deletion.
2 changes: 1 addition & 1 deletion test/suites/storage_driver_ceph.sh
Original file line number Diff line number Diff line change
Expand Up @@ -129,7 +129,7 @@ test_storage_driver_ceph() {
lxc profile device remove default root
lxc storage delete "lxdtest-$(basename "${LXD_DIR}")-pool1"
lxc storage delete "lxdtest-$(basename "${LXD_DIR}")-pool2"
ceph --cluster "${LXD_CEPH_CLUSTER}" osd pool rm "lxdtest-$(basename "${LXD_DIR}")-existing-osd-pool" "lxdtest-$(basename "${LXD_DIR}")-existing-osd-pool" --yes-i-really-really-mean-it
ceph --cluster "${LXD_CEPH_CLUSTER}" osd pool rm "lxdtest-$(basename "${LXD_DIR}")-existing-osd-pool" "lxdtest-$(basename "${LXD_DIR}")-existing-osd-pool" --yes-i-really-really-mean-it || true

# Test that the pool size setting only applies to the created pool, and not any other pools.
pool1="lxdtest-$(basename "${LXD_DIR}")-pool1"
Expand Down

0 comments on commit 0c39aed

Please sign in to comment.