Skip to content

Commit

Permalink
storage: do not add /boot among required partitions
Browse files Browse the repository at this point in the history
In general it is recommended to have a separate /boot, but not strictly
required.

In some cases it is required but we need to add API for that, because it
depends mainly on the filesystem of the root partition.

So now the requirement regards mainly biosboot and efi partitions, which
is dependent on the platform.
  • Loading branch information
rvykydal committed Dec 15, 2023
1 parent 931c5a8 commit 0e28a7d
Showing 1 changed file with 7 additions and 1 deletion.
8 changes: 7 additions & 1 deletion pyanaconda/modules/storage/devicetree/viewer.py
Original file line number Diff line number Diff line change
Expand Up @@ -484,6 +484,9 @@ def get_mount_point_constraints(self):
This includes mount points required to boot (e.g. /boot/efi, /boot)
and the / partition which is always considered to be required.
/boot is not required in general but can be required in some cases,
depending on the filesystem on the root partition (ie crypted root).
:return: a list of mount points with its constraints
"""

Expand All @@ -499,7 +502,10 @@ def get_mount_point_constraints(self):
for p in platform.partitions:
if p.mountpoint:
constraint = self._get_mount_point_constraints_data(p)
constraint.required = True
if p.mountpoint == "/boot":
constraint.recommended = True
else:
constraint.required = True
constraints.append(constraint)

return constraints
Expand Down

0 comments on commit 0e28a7d

Please sign in to comment.