Description of problem:

As per bug 1163890, the LVs are deactivated when SD is put into maintenance mode.

===
2021-04-13 06:15:26,360+0000 INFO  (monitor/d2831c8) [storage.BlockSD] Tearing down domain d2831c8a-2e31-4d13-9fc3-9116debab149 (blockSD:992)
===

However, When the user detaches the SD, vdsm starts a "detachStorageDomain" in the SPM host which activates the LVs again.

===
2021-04-13 06:15:56,371+0000 INFO  (jsonrpc/7) [vdsm.api] START detachStorageDomain(sdUUID='d2831c8a-2e31-4d13-9fc3-9116debab149', spUUID='fd4eb21e-72d8-11eb-80ee-00163e75cb19', msdUUID='00000000-0000-0000-0000-000000000000', masterVersion=1, options=None) from=::ffff:10.74.252.109,59212, flow_id=3500b73f-e735-41c9-b866-5d8e0b94cee2, task_id=1b2e0d8d-2905-4c9f-a6dc-8ad7c9d9bcff (api:48)

2021-04-13 06:15:56,988+0000 INFO  (jsonrpc/7) [storage.LVM] Activating lvs: vg=d2831c8a-2e31-4d13-9fc3-9116debab149 lvs=['ids'] (lvm:1738)
2021-04-13 06:15:57,678+0000 INFO  (jsonrpc/7) [storage.LVM] Activating lvs: vg=d2831c8a-2e31-4d13-9fc3-9116debab149 lvs=['leases'] (lvm:1738)
2021-04-13 06:15:58,030+0000 INFO  (jsonrpc/7) [storage.LVM] Activating lvs: vg=d2831c8a-2e31-4d13-9fc3-9116debab149 lvs=['metadata', 'inbox', 'outbox', 'xleases', 'master'] (lvm:1738)
===

This leaves all the default SD LVs active after the detach operation.

===
lvs --config 'devices { filter = [ "a|/dev/mapper/*|", "r|.*|" ] }' 
  LV                                   VG                                   Attr       LSize   Pool Origin Data%  Meta%  Move Log Cpy%Sync Convert
  c9f66cdc-e6ee-4900-92d6-e13d98f6660a d2831c8a-2e31-4d13-9fc3-9116debab149 -wi------- 128.00m                                                    
  fe74fe69-12f6-49c9-a253-04d54fc8fbcd d2831c8a-2e31-4d13-9fc3-9116debab149 -wi------- 128.00m                                                    
  ids                                  d2831c8a-2e31-4d13-9fc3-9116debab149 -wi-a----- 128.00m                                                    
  inbox                                d2831c8a-2e31-4d13-9fc3-9116debab149 -wi-a----- 128.00m                                                    
  leases                               d2831c8a-2e31-4d13-9fc3-9116debab149 -wi-a-----   2.00g                                                    
  master                               d2831c8a-2e31-4d13-9fc3-9116debab149 -wi-a-----   1.00g                                                    
  metadata                             d2831c8a-2e31-4d13-9fc3-9116debab149 -wi-a----- 128.00m                                                    
  outbox                               d2831c8a-2e31-4d13-9fc3-9116debab149 -wi-a----- 128.00m                                                    
  xleases                              d2831c8a-2e31-4d13-9fc3-9116debab149 -wi-a-----   1.00g                                        
===


Version-Release number of selected component (if applicable):


vdsm-4.40.40-1.el8ev.x86_64

How reproducible:

100%

Steps to Reproduce:

1. Put an FC SD in maintenance mode.  
2. Check the lvs output in SPM. All the LVs will be inactive for this SD.
3. Deatch the storage domain.
4. Check the lvs output in SPM. All the default SD LVs active will be in "active" status.

Actual results:

LVs are left open in SPM host after the storage domain detach.

Expected results:

LVs should be inactive to allow easier unmapping of LUN from the host.

Additional info: