particle-os/debian-forge-composer
1
0
Fork 0
Code Issues Pull requests Projects Releases Packages Wiki Activity Actions

cloudapi: make fs_type required for filesystems and logical volumes

Browse source 
While technically this is a backwards incompatible API change, the
core blueprint schema didn't allow for empty or undefined fs_type [1,2],
so the general behaviour hasn't changed, we're just catching it early
now.

[1] de18902e3f/pkg/blueprint/disk_customizations.go (L624-L627)
[2] de18902e3f/pkg/blueprint/disk_customizations.go (L677-L680)
This commit is contained in:
Achilleas Koutsou 2025-06-03 17:12:26 +02:00 • committed by Tomáš Hozza
parent a03d01c92c
commit ea0b5e8661
4 changed files with 146 additions and 141 deletions
Download patch file Download diff file Expand all files Collapse all files

4
internal/cloudapi/v2/compose.go
View file

@ -1367,7 +1367,7 @@ func convertDiskCustomizations(disk *Disk) (*blueprint.DiskCustomization, error)
FilesystemTypedCustomization: blueprint.FilesystemTypedCustomization{
Mountpoint: common.DerefOrDefault(fs.Mountpoint),
Label: common.DerefOrDefault(fs.Label),
FSType: string(common.DerefOrDefault(fs.FsType)),
FSType: string(fs.FsType),
},
}
case string(Btrfs):
@ -1422,7 +1422,7 @@ func convertDiskCustomizations(disk *Disk) (*blueprint.DiskCustomization, error)
FilesystemTypedCustomization: blueprint.FilesystemTypedCustomization{
Mountpoint: common.DerefOrDefault(lv.Mountpoint),
Label: common.DerefOrDefault(lv.Label),
FSType: string(common.DerefOrDefault(lv.FsType)),
FSType: string(lv.FsType),
},
}
bpPartition.LogicalVolumes = append(bpPartition.LogicalVolumes, bpLV)