Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Warn if the root filesystem is too small #211

Open
bgilbert opened this issue Mar 10, 2021 · 6 comments
Open

Warn if the root filesystem is too small #211

bgilbert opened this issue Mar 10, 2021 · 6 comments
Assignees
Labels
enhancement New feature or request jira validation Issue related to config validation

Comments

@bgilbert
Copy link
Contributor

Add a warning if the root filesystem is too small (coreos/fedora-coreos-tracker#586), either because the underlying storage is explicitly specified, or because the root partition is constrained by an additional partition.

@bgilbert bgilbert added the enhancement New feature or request label Mar 10, 2021
@bgilbert bgilbert self-assigned this Mar 10, 2021
@bgilbert bgilbert added the jira label Mar 10, 2021
@bgilbert bgilbert removed their assignment Jul 27, 2021
@jlebon
Copy link
Member

jlebon commented Aug 24, 2021

We've decided to defer this until we have coreos/fedora-coreos-tracker#759 because otherwise it's impossible in many cases for Butane to know that it's looking at the root disk. There are some cases where it could (e.g. if root reprovisioning/mirroring is requested), but the common case of just adding a "trapping partition" wouldn't be detectable without some dubious heuristics.

@prestist
Copy link
Contributor

prestist commented Jul 29, 2022

@bgilbert This issue might belong in ignition? Similar to coreos/ignition#1397 (comment)
With this change coreos/fedora-coreos-config#1621

@bgilbert
Copy link
Contributor Author

Ignition is distro-independent and doesn't know the partition layout constraints of the OS it runs on, so this needs to go in Butane in the fcos variant.

@prestist
Copy link
Contributor

Darn, ok thank you!

@prestist
Copy link
Contributor

prestist commented Aug 4, 2022

@bgilbert

Add a warning if the root filesystem is too small (coreos/fedora-coreos-tracker#586), either because the underlying storage is explicitly specified, or because the root partition is constrained by an additional partition.

I wanted to ask a clarifying question, do we consider root constrained if there is a partition in front of it aka /var on number 5 ?

or is it only constrained if the underlying storage is explicitly specified and there is a partition in front of it ?

In speaking with @jlebon it is fairly common ask to have a /var partition after root for us, and I was unsure if we wanted a warning in a common configuration case.

@bgilbert
Copy link
Contributor Author

bgilbert commented Aug 5, 2022

It's common to have a /var partition after the root partition, but we should only warn if that would make the root partition too small. In sum, the values we need to check are:

  • The sizeMiB of the root partition, if specified
  • The startMiB of any following partition. If this is unspecified (0) and the root sizeMiB is also unspecified, then the following partition will prevent the root partition from growing at all, and we should warn.

prestist added a commit to prestist/butane that referenced this issue Aug 22, 2022
Fixes coreos#211 , add warning for under allocation of root partition.
additionally add warning when root partition's expansion is constrained
prestist added a commit to prestist/butane that referenced this issue Aug 29, 2022
Fixes coreos#211, the root partition needs a certain amount of space, with
recent changes butane has the opportunity to warn when those expectations
are not met. Add a warn when the root is too small, and additionally
add warn when the root is small and cannot expand.
@bgilbert bgilbert added the validation Issue related to config validation label Jan 10, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request jira validation Issue related to config validation
Projects
None yet
Development

No branches or pull requests

3 participants