stratis-storage / stratis-cli

CLI for the Stratis project
https://stratis-storage.github.io
Apache License 2.0
106 stars 20 forks source link

add support to destroy pool directly #957

Open yizhanglinux opened 1 year ago

yizhanglinux commented 1 year ago

I'm trying to destroy pool directly, but I've to destroy filesystem first, could we add support for such option? Like add --force option?

[root@storageqe-62 ~]# stratis pool list
Name                           Total / Used / Free    Properties                                   UUID   Alerts
olympic         186.31 GiB / 1.13 GiB / 185.18 GiB   ~Ca,~Cr, Op   9f7343ef-6cce-452a-bedb-8079e5d95046         
stratis_howto       1.46 TiB / 1.78 GiB / 1.45 TiB   ~Ca,~Cr, Op   0a2ff1a5-dcb8-4d1e-8f3d-1c59fe92db35  
[root@storageqe-62 ~]# stratis pool destroy olympic
Execution failed:
stratisd failed to perform the operation that you requested. It returned the following information via the D-Bus: ERROR: filesystems remaining on pool. 

[root@storageqe-62 ~]# stratis daemon version
3.4.4
mulkieran commented 1 year ago

Part of our design is to not add a --force option anywhere. If we override any safety mechanism, the specific mechanism being overridden must be specified. An example of a specific kind of force option is the option, --trust-url.

For the filesystems, we have always taken the position that the user must umount the filesystem themselves.