Merge lp:~paelzer/curtin/minor-mdadm-fs-fixes into lp:~curtin-dev/curtin/trunk

Proposed by Christian Ehrhardt 
Status: Merged
Merged at revision: 283
Proposed branch: lp:~paelzer/curtin/minor-mdadm-fs-fixes
Merge into: lp:~curtin-dev/curtin/trunk
Diff against target: 26 lines (+6/-2)
1 file modified
curtin/commands/block_meta.py (+6/-2)
To merge this branch: bzr merge lp:~paelzer/curtin/minor-mdadm-fs-fixes
Reviewer Review Type Date Requested Status
curtin developers Pending
Review via email: mp+274309@code.launchpad.net

Description of the change

Minor fixes that came up when testing various mdadm options and filesystems.

To post a comment you must log in.
Revision history for this message
Scott Moser (smoser) wrote :

i'm gonna pull this.

Revision history for this message
Scott Moser (smoser) wrote :

actually, this was part of reno 283 pull.

Preview Diff

[H/L] Next/Prev Comment, [J/K] Next/Prev File, [N/P] Next/Prev Hunk
=== modified file 'curtin/commands/block_meta.py'
--- curtin/commands/block_meta.py 2015-10-02 18:30:42 +0000
+++ curtin/commands/block_meta.py 2015-10-13 19:17:09 +0000
@@ -36,7 +36,7 @@
36SIMPLE_BOOT = 'simple-boot'36SIMPLE_BOOT = 'simple-boot'
37CUSTOM = 'custom'37CUSTOM = 'custom'
3838
39CUSTOM_REQUIRED_PACKAGES = ['mdadm', 'lvm2', 'bcache-tools']39CUSTOM_REQUIRED_PACKAGES = ['mdadm', 'lvm2', 'bcache-tools', 'xfsprogs', 'btrfs-tools']
4040
41CMD_ARGUMENTS = (41CMD_ARGUMENTS = (
42 ((('-D', '--devices'),42 ((('-D', '--devices'),
@@ -812,8 +812,12 @@
812 spare_devices = info.get('spare_devices')812 spare_devices = info.get('spare_devices')
813 if not devices:813 if not devices:
814 raise ValueError("devices for raid must be specified")814 raise ValueError("devices for raid must be specified")
815 if raidlevel not in [0, 1, 5]:815 if raidlevel not in ['linear', 'raid0', 0, 'stripe', 'raid1', 1, 'mirror',
816 'raid4', 4, 'raid5', 5, 'raid6', 6, 'raid10', 10]:
816 raise ValueError("invalid raidlevel '%s'" % raidlevel)817 raise ValueError("invalid raidlevel '%s'" % raidlevel)
818 if raidlevel in ['linear', 'raid0', 0, 'stripe']:
819 if spare_devices:
820 raise ValueError("spares unsupported in raidlevel '%s'" % raidlevel)
817821
818 device_paths = list(get_path_to_storage_volume(dev, storage_config) for822 device_paths = list(get_path_to_storage_volume(dev, storage_config) for
819 dev in devices)823 dev in devices)

Subscribers

People subscribed via source and target branches