Support for Btrfs and XFS in parted and mkfs

openSUSE · Mar 23, 2020 · fa34f4b · fa34f4b
1 parent f7b4c38
commit fa34f4b
Show file tree

Hide file tree

Showing 2 changed files with 18 additions and 2 deletions.
diff --git a/salt/modules/parted_partition.py b/salt/modules/parted_partition.py
@@ -390,8 +390,8 @@ def _is_fstype(fs_type):
     :param fs_type: file system type
     :return: True if fs_type is supported in this module, False otherwise
     '''
-    return fs_type in set(['ext2', 'ext3', 'ext4', 'fat32', 'fat16', 'linux-swap', 'reiserfs',
-                           'hfs', 'hfs+', 'hfsx', 'NTFS', 'ntfs', 'ufs'])
+    return fs_type in set(['btrfs', 'ext2', 'ext3', 'ext4', 'fat32', 'fat16', 'linux-swap', 'reiserfs',
+                           'hfs', 'hfs+', 'hfsx', 'NTFS', 'ntfs', 'ufs', 'xfs'])
 
 
 def mkfs(device, fs_type):

diff --git a/tests/unit/modules/test_parted_partition.py b/tests/unit/modules/test_parted_partition.py
@@ -376,6 +376,22 @@ def test_list__valid_unit_valid_legacy_cmd_output(self):
             }
             self.assertEqual(output, expected)
 
+    def test_btrfs_fstypes(self):
+        '''Tests if we see btrfs as valid fs type'''
+        with patch('salt.modules.parted_partition._validate_device', MagicMock()):
+            try:
+                parted.mkfs('/dev/foo', 'btrfs')
+            except CommandExecutionError:
+                self.fail("Btrfs is not in the supported fstypes")
+
+    def test_xfs_fstypes(self):
+        '''Tests if we see xfs as valid fs type'''
+        with patch('salt.modules.parted_partition._validate_device', MagicMock()):
+            try:
+                parted.mkfs('/dev/foo', 'xfs')
+            except CommandExecutionError:
+                self.fail("XFS is not in the supported fstypes")
+
     def test_disk_set(self):
         with patch('salt.modules.parted_partition._validate_device', MagicMock()):
             self.cmdrun.return_value = ''