Add TODO re attached.uuid use for Pool member attribution rockstor#2824

Minor test comment updates.

src/rockstor/storageadmin/views/disk.py

@@ -211,6 +211,9 @@ def _update_disk_state():
                                     disk_roles_identified["redirect"] = part_byid_name
                             break
                 pool_info = dev_pool_info[dev_name]
+                # TODO: pool_info.uuid should be canonical: not pool_info.label
+                #  As we move to this posture the following will need attention.
+                #  And we already have btrfs uuid from attached.uuid.
                 pool_name = pool_info.label
                 # TODO: First call we reset none pool label member-count times!
                 # Corner case but room for efficiency improvement.

src/rockstor/system/tests/test_osi.py

@@ -1784,13 +1784,12 @@ def dyn_disk_serial_return(*args, **kwargs):
 
     def test_scan_disks_btrfs_in_partition(self):
         """
-        Test btrfs in partition on otherwise generic install. System disk sda
-        data disk (for btrfs in partition) virtio with serial "serial-1"
-        prepared as follows with regard to partition / formatting:
+        Test btrfs in partition on an otherwise generic install. System disk sda
+        data disk (for btrfs in partition) virtio with serial "serial-1" 10 GB,
+        prepared as follows with regard to partitioning / formatting:
 
         First data set:
 
-        yum install dosfstools
         parted -a optimal /dev/disk/by-id/virtio-serial-1
         mklabel msdos
         mkpart primary fat32 1 50%