[1.9.3,2.0] cannot create more than 4 partitions when disk is configured with mbr
Bug #1597787 reported by
Jeff Hillman
This bug affects 1 person
| Affects | Status | Importance | Assigned to | Milestone | |
|---|---|---|---|---|---|
| MAAS |
Fix Released
|
High
|
Unassigned | ||
| 1.9 |
Fix Released
|
High
|
Blake Rouse | ||
| 2.0 |
Fix Released
|
High
|
Blake Rouse | ||
| curtin |
Invalid
|
Undecided
|
Unassigned | ||
Bug Description
maas 1.9.3
When trying to create partitions that extend beyond the 4 primary partition (extended partition) when disklabel is set to dos, maas fails.
It appears the issue is related to it not setting the size properly on the extended partition (i.e. us up the remaining space on the disk)
This issue does not happen when disklabel is setup to gpt, likely because no extended partition is required.
Related branches
lp://staging/~blake-rouse/maas/fix-1597787
- Mike Pontillo (community): Approve
-
Diff: 88 lines (+29/-2)2 files modifiedsrc/maasserver/preseed_storage.py (+26/-0)
src/maasserver/tests/test_preseed_storage.py (+3/-2)
lp://staging/~blake-rouse/maas/fix-1597787-2.0
- Blake Rouse (community): Approve
-
Diff: 88 lines (+29/-2)2 files modifiedsrc/maasserver/preseed_storage.py (+26/-0)
src/maasserver/tests/test_preseed_storage.py (+3/-2)
lp://staging/~blake-rouse/maas/fix-1597787-1.9
- Blake Rouse (community): Approve
-
Diff: 88 lines (+29/-2)2 files modifiedsrc/maasserver/preseed_storage.py (+26/-0)
src/maasserver/tests/test_preseed_storage.py (+3/-2)
Revision history for this message
| Jeff Hillman (jhillman) wrote | #1 |
Revision history for this message
| Jeff Hillman (jhillman) wrote | #2 |
Revision history for this message
| Jeff Hillman (jhillman) wrote | #3 |
| Changed in maas: | |
| milestone: | none → 1.9.4 |
| status: | New → Incomplete |
| status: | Incomplete → New |
| importance: | Undecided → High |
Revision history for this message
| Ryan Harper (raharper) wrote | #4 |
| Changed in curtin: | |
| status: | New → Invalid |
| Changed in maas: | |
| status: | New → Triaged |
| milestone: | 1.9.4 → 2.1.0 |
| summary: |
- cannot create more than 4 partitions when disk is configured with dos - DiskLabel + [1.9] cannot create more than 4 partitions when disk is configured with + dos DiskLabel |
| summary: |
- [1.9] cannot create more than 4 partitions when disk is configured with - dos DiskLabel + [1.9.3,2.0] cannot create more than 4 partitions when disk is configured + with dos DiskLabel |
| summary: |
[1.9.3,2.0] cannot create more than 4 partitions when disk is configured - with dos DiskLabel + with mbr |
| Changed in maas: | |
| status: | Triaged → In Progress |
| Changed in maas: | |
| status: | In Progress → Fix Committed |
| Changed in maas: | |
| milestone: | 2.0.1 → none |
| status: | Fix Committed → Fix Released |
To post a comment you must log in.
curtin already supports extended partitions, as the filer suggests, the missing bit is the size of the extended partition in that dictionary:
I don't have the actual disk sizes, but, something like this will work.
- device: sda
flag: extended
id: sda-part4
number: 4
type: partition
size: 9999220736B
- device: sda
flag: logical
id: sda-part5
name: sda-part5
number: 5
size: 9999220736B
curtin already exercises extended/logical partition paths, for example, this 10G disk
defines a 3G primary, and 5G extended, and then 2G logical and 3G logical.
config:
- id: sda
type: disk
ptable: msdos
model: QEMU HARDDISK
path: /dev/vdb
name: main_disk
- id: sda1
type: partition
size: 3GB
device: sda
flag: boot
- id: sda_extended
type: partition
size: 5G
flag: extended
device: sda
- id: sda2
type: partition
size: 2G
flag: logical
device: sda
- id: sda3
type: partition
size: 3G
flag: logical
device: sda