]> review.fuel-infra Code Review - openstack-build/cinder-build.git/commitdiff
HDS HNAS Driver fails when FS unhealthy
authorAdriano Rosso <adriano.rosso@fit-tecnologia.org.br>
Tue, 5 May 2015 18:58:38 +0000 (15:58 -0300)
committerAdriano Rosso <adriano.rosso@fit-tecnologia.org.br>
Wed, 6 May 2015 20:09:43 +0000 (17:09 -0300)
This patch fixes the error when there is a file system in an
unhealthy HNAS storage pool.

Change-Id: Ic17c540039996f4a1060829d5f1f12a5f3155271
Closes-bug: #1452002
(cherry picked from commit 418cf4b989c8df983c1109c8c8851d9d4c3ffbfe)

cinder/tests/test_hds_hnas_backend.py
cinder/volume/drivers/hds/hnas_backend.py

index 8f2070a7564ccd5e40faf61ff90d934d0affe452..6af18bb929b43ada81b02073bc8afab476af8fa9 100644 (file)
@@ -102,6 +102,10 @@ HNAS_RESULT5 = "\n\
 1032         test    2  3.97 GB 2.12 GB (53%) 0 B (0%)   NA      \
    1.85 GB (47%)  No                                        4 KB,\
    WFS-2,128 DSBs\n\
+1058         huge_FS    7  1.50 TB  Not determined\n\
+1053              fs-unmounted    4   108 GB     Not mounted \
+   NA  943 MB (18%)  39.2 GB (36%)    No                    4 KB,\
+   WFS-2,128 DSBs,dedupe enabled\n\
 \n"
 
 HNAS_RESULT6 = "\n\
@@ -173,6 +177,10 @@ Deduped          Avail  Thin  ThinSize  ThinAvail              FS Type\n\
   No                       4 KB,WFS-2,128 DSBs\n\
 1047  manage_test02  19.9 GB  9.29 GB (47%)   0 B (0%)       NA  10.6 GB (53%)\
   No                       4 KB,WFS-2,128 DSBs\n\
+1058         huge_FS    7  1.50 TB  Not determined\n\
+1053              fs-unmounted    4   108 GB     Not mounted \
+   NA  943 MB (18%)  39.2 GB (36%)    No                 4 KB,\
+   WFS-2,128 DSBs,dedupe enabled\n\
 \n"
 
 HNAS_RESULT20 = "\n\
index 64087d1459b8e1db135a09dde2e08491afa9c728..c449cd468d3d16177a28957b3e8711bb44626a8d 100644 (file)
@@ -182,7 +182,7 @@ class HnasBackend(object):
 
         newout = ""
         for line in lines:
-            if 'Not mounted' in line:
+            if 'Not mounted' in line or 'Not determined' in line:
                 continue
             if 'not' not in line and 'EVS' in line:
                 single_evs = False