Logical volume size is required
Affects | Status | Importance | Assigned to | Milestone | |
---|---|---|---|---|---|
MAAS |
Fix Released
|
Low
|
Jack Lloyd-Walters |
Bug Description
Hi,
I'm defining the config of a machine through fce using the following
bucketsconfig.yaml for disks section:
- id: vda
disk: 0
type: disk
- id: vda-vg
name: vgroot
type: lvm_volgroup
- vda-part2
- id: vda-part1
name: efi
size: 100M
type: partition
- id: vda-part1-format
type: format
- id: vda-part1-mount
path: /boot/efi
type: mount
- id: vda-part2
name: lvm
type: partition
- id: vda-part2-lvm1
name: lvroot
type: lvm_partition
- id: vda-part2-
type: format
- id: vda-part2-
path: /
type: mount
So in summary I just create an efi partition, a vg, and I want only
a root lv of the size I can afford. Generating the partition would
work, generating the lv will output the following:
2020-06-09-04:54:32 root ERROR [localhost] Command failed: maas admin volume-group create-
2020-06-09-04:54:32 root ERROR [localhost] STDOUT follows:
{"size": ["This field is required."]}
This is based in maas 2.7.1 (8262-g.
Thanks!
Jose.
Related branches
- Adam Collard (community): Approve
- Anton Troyanov: Approve
- MAAS Lander: Approve
-
Diff: 90 lines (+30/-5)5 files modifiedsrc/maasserver/api/tests/test_volume_groups.py (+25/-0)
src/maasserver/api/volume_groups.py (+2/-2)
src/maasserver/forms/__init__.py (+1/-1)
src/maasserver/forms/tests/test_volume_group.py (+1/-1)
src/maasserver/models/filesystemgroup.py (+1/-1)
- MAAS Lander: Needs Fixing
- Adam Collard: Pending requested
-
Diff: 103 lines (+43/-5)5 files modifiedsrc/maasserver/api/tests/test_volume_groups.py (+38/-0)
src/maasserver/api/volume_groups.py (+2/-2)
src/maasserver/forms/__init__.py (+1/-1)
src/maasserver/forms/tests/test_volume_group.py (+1/-1)
src/maasserver/models/filesystemgroup.py (+1/-1)
Changed in maas: | |
assignee: | nobody → Jack Lloyd-Walters (lloydwaltersj) |
status: | Triaged → In Progress |
Changed in maas: | |
milestone: | none → 3.3.0 |
Changed in maas: | |
status: | In Progress → Fix Committed |
Changed in maas: | |
milestone: | 3.3.0 → 3.3.0-beta1 |
Changed in maas: | |
status: | Fix Committed → Fix Released |
Confirmed that the size is currently required in the form. group.get_ lvm_free_ space() if not provided
We could make it optional and just use self.volume_