-
Notifications
You must be signed in to change notification settings - Fork 94
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
Custom partitioning #209
Comments
When the config is mounted as a datasource iso (as suggested here), it partly works:
|
I seems to be failing:
|
The error seems to originate here: https://github.com/kairos-io/elemental-cli/blob/main/pkg/partitioner/parted.go#L155 I created this PR to improve the error message: https://github.com/kairos-io/elemental-cli/pull/45/files
If I run this first:
the error goes away. |
updated the config.yaml like this:
and now it works:
|
Currently working on a refactoring of the kairos config here: https://github.com/kairos-io/kairos/tree/209-custom-partitioning-refactor-config We are extracting the code that collects configuration from 3 different sources:
When this is done, we will use this package to parse configuration in kairos-agent, elemental-cli, kcrypt and whatever consumer needs access to kairos-configuration. The consumers will unmarshal the merged Yamls into whatever Config object they mainain and they can do their own validations on that too. This should fix the issue where elemental-cli doesn't read the |
Currently blocked by #1225 |
We have planned a refactoring that will make elemental-cli a kairos-agent subcommand. When that happens, #1225 will be easier to implement (config will probably be available in memory). We came up with an alternative fix for this issue for now, which is to write down the kairos config to |
I'm trying the config above with a kairos-agent I've built from this: kairos-io/kairos-agent#32 I had to change the stage from I also had to add |
to let the user set this in the cloud-config file instead of /etc/elemental/config.yaml See also here: kairos-io/kairos#209 (comment) Signed-off-by: Dimitris Karakasilis <dimitris@karakasilis.me>
This is the config that works with the PR branch:
I had to make some fixes so that all partitions are created and "expand" happens after creation. |
to let the user set this in the cloud-config file instead of /etc/elemental/config.yaml See also here: kairos-io/kairos#209 (comment) Signed-off-by: Dimitris Karakasilis <dimitris@karakasilis.me>
to let the user set this in the cloud-config file instead of /etc/elemental/config.yaml See also here: kairos-io/kairos#209 (comment) Signed-off-by: Dimitris Karakasilis <dimitris@karakasilis.me>
Let's write some docs for this an then it's done |
Looking into this I see that it is possible to create a system despite the system not having enough space to handle the indicated partitions. The errors show up in the installation logs but it doesn't fail. Is this the expected UX? it means the user is required to check that there were no errors during installation
|
I don't know. But it already tell you to use |
Is your feature request related to a problem? Please describe.
A way to configure partitions via config file
Describe the solution you'd like
A simple and detailed way in the config to declare the partitions after install. Ideally should just enlarge the elemental config so it just exposes the underlying configuration
Describe alternatives you've considered
Dropping the config file manually, but the solution is more a workaround. It can be achieved by adding an initramfs stage which overwrites the elemental-config
Additional context
The text was updated successfully, but these errors were encountered: