Partitioning should align for performance
Affects | Status | Importance | Assigned to | Milestone | |
---|---|---|---|---|---|
MAAS |
Fix Released
|
High
|
Jeffrey C Jones | ||
curtin |
Fix Released
|
Medium
|
Unassigned |
Bug Description
Watching a recent MAAS install I noticed this in the console output:
mdadm: No arrays found in config file or automatically
Error: /dev/sdb: unrecognised disk label
mdadm: No arrays found in config file or automatically
Error: /dev/sdb: unrecognised disk label
Error: /dev/sdc: unrecognised disk label
mdadm: No arrays found in config file or automatically
Error: /dev/sdc: unrecognised disk label
Error: /dev/sdd: unrecognised disk label
mdadm: No arrays found in config file or automatically
Error: /dev/sdd: unrecognised disk label
Warning: The resulting partition is not properly aligned for best performance.
Error: /dev/sdd: unrecognised disk label
UUID: ac6fd136-
Set UUID: 98ff872c-
version: 0
nbuckets: 488396
block_size: 1
bucket_size: 1024
nr_in_set: 1
nr_this_dev: 0
first_bucket: 1
UUID: 14e85839-
Set UUID: 98ff872c-
version: 1
block_size: 1
data_offset: 16
Error: /dev/bcache0: unrecognised disk label
Error: /dev/bcache0: unrecognised disk label
Warning: The resulting partition is not properly aligned for best performance.
Error: /dev/bcache0: unrecognised disk label
--2015-11-04 14:19:59-- http://
Connecting to 192.168.9.2:5248... connected.
HTTP request sent, awaiting response... 200 OK
Length: 322584502 (308M) [text/html]
In particular I noticed the "Warning: The resulting partition is not properly aligned for best performance." Using parted, I've found that it's best to specify the first partitioning as starting at "1" using the compact units, which avoids this error, though TBH I have no idea why that works and look forward to having that all explained to me ;)
Related branches
- curtin developers: Pending requested
-
Diff: 99 lines (+52/-8)2 files modifiedcurtin/commands/block_meta.py (+51/-7)
examples/tests/lvm.yaml (+1/-1)
- Blake Rouse (community): Approve
-
Diff: 1347 lines (+256/-192)21 files modifiedsrc/maasserver/api/tests/test_raid.py (+6/-12)
src/maasserver/api/tests/test_volume_groups.py (+13/-4)
src/maasserver/forms.py (+2/-1)
src/maasserver/models/blockdevice.py (+1/-1)
src/maasserver/models/partition.py (+7/-7)
src/maasserver/models/partitiontable.py (+21/-17)
src/maasserver/models/tests/test_blockdevice.py (+4/-2)
src/maasserver/models/tests/test_filesystemgroup.py (+27/-33)
src/maasserver/models/tests/test_partition.py (+9/-8)
src/maasserver/models/tests/test_partitiontable.py (+27/-18)
src/maasserver/models/virtualblockdevice.py (+21/-11)
src/maasserver/testing/factory.py (+1/-3)
src/maasserver/tests/test_forms_blockdevice.py (+5/-1)
src/maasserver/tests/test_forms_partition.py (+5/-5)
src/maasserver/tests/test_forms_raid.py (+5/-9)
src/maasserver/tests/test_forms_volume_group.py (+7/-2)
src/maasserver/tests/test_preseed_storage.py (+14/-14)
src/maasserver/tests/test_storage_layouts.py (+43/-32)
src/maasserver/utils/converters.py (+5/-3)
src/maasserver/utils/tests/test_converters.py (+22/-6)
src/maasserver/websockets/handlers/tests/test_node.py (+11/-3)
- Mike Pontillo (community): Approve
-
Diff: 1347 lines (+256/-192)21 files modifiedsrc/maasserver/api/tests/test_raid.py (+6/-12)
src/maasserver/api/tests/test_volume_groups.py (+13/-4)
src/maasserver/forms.py (+2/-1)
src/maasserver/models/blockdevice.py (+1/-1)
src/maasserver/models/partition.py (+7/-7)
src/maasserver/models/partitiontable.py (+21/-17)
src/maasserver/models/tests/test_blockdevice.py (+4/-2)
src/maasserver/models/tests/test_filesystemgroup.py (+27/-33)
src/maasserver/models/tests/test_partition.py (+9/-8)
src/maasserver/models/tests/test_partitiontable.py (+27/-18)
src/maasserver/models/virtualblockdevice.py (+21/-11)
src/maasserver/testing/factory.py (+1/-3)
src/maasserver/tests/test_forms_blockdevice.py (+5/-1)
src/maasserver/tests/test_forms_partition.py (+5/-5)
src/maasserver/tests/test_forms_raid.py (+5/-9)
src/maasserver/tests/test_forms_volume_group.py (+7/-2)
src/maasserver/tests/test_preseed_storage.py (+14/-14)
src/maasserver/tests/test_storage_layouts.py (+43/-32)
src/maasserver/utils/converters.py (+5/-3)
src/maasserver/utils/tests/test_converters.py (+22/-6)
src/maasserver/websockets/handlers/tests/test_node.py (+11/-3)
Changed in maas: | |
status: | New → Triaged |
importance: | Undecided → Critical |
milestone: | none → 1.9.0 |
importance: | Critical → High |
Changed in curtin: | |
status: | New → Confirmed |
Changed in curtin: | |
status: | In Progress → Fix Committed |
importance: | Undecided → Medium |
Changed in maas: | |
assignee: | nobody → Jeffrey C Jones (trapnine) |
status: | Triaged → In Progress |
Changed in maas: | |
status: | In Progress → Fix Committed |
Changed in maas: | |
status: | Fix Committed → Fix Released |
It might be best for MAAS to make sure that all partition sizes are a multiple of 4MiB. That should align the partitions correctly on disk and fix an issue where the LVM PV extents need to be a multiple of 4MiB.