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

Update cloud config and add partitions config #59

Merged
merged 2 commits into from
Aug 2, 2023
Merged

Conversation

Itxaka
Copy link
Member

@Itxaka Itxaka commented Aug 2, 2023

Updates the cloud config page with all the now possible values to override th default configs, now that ythe full config and spec is read from the cloud-config via the agent.

Also adds a page on how to specifically configure the current partitions during install and add extra partitions during install

Fixes: kairos-io/kairos#558

Updates the cloud config page with all the now possible values to
override th default configs, now that ythe full config and spec is read
from the cloud-config via the agent.

Also adds a page on how to specifically configure the current partitions
during install and add extra partitions during install

Signed-off-by: Itxaka <itxaka@kairos.io>
@Itxaka Itxaka requested a review from a team August 2, 2023 11:44
@netlify
Copy link

netlify bot commented Aug 2, 2023

Deploy Preview for kairos-io ready!

Name Link
🔨 Latest commit a27b596
🔍 Latest deploy log https://app.netlify.com/sites/kairos-io/deploys/64ca8b905ab8110008078fa8
😎 Deploy Preview https://deploy-preview-59--kairos-io.netlify.app
📱 Preview on mobile
Toggle QR Code...

QR Code

Use your smartphone camera to open QR code link.

To edit notification comments on pull requests, go to your Netlify site configuration.

Signed-off-by: Itxaka <itxaka@kairos.io>
Copy link
Member

@mauromorales mauromorales left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM

@Itxaka Itxaka merged commit fa336aa into main Aug 2, 2023
5 checks passed
@Itxaka Itxaka deleted the override_mounts branch August 2, 2023 17:11
@@ -22,6 +22,14 @@ users:
ssh_authorized_keys:
# - github:mudler

# enable debug logging
debug: true
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

After the merging of all configs, does the schema need an update? https://github.com/kairos-io/kairos-sdk/tree/main/schema

Copy link
Member Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Good point! I remember updating it already so it would pass the tests. Not sure if the tests are now broken or it's not necessary anymore...

Gonna check

```


Note that there are some caveats in the `extra partitions` setup:
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Shouldn't we also mention that if they try to override the labels of the default partitions, these will be ignored? (as per the PR here: https://github.com/kairos-io/kairos-agent/pull/108/files)

Copy link
Member Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Not really no, the label stuff is for the system partitions and its mentioned above that we only support fs and size overrides.

Copy link
Member Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I dont want to give them ideas that they can indeed override the labels lol, security through obscurity

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

✨ Custom partitioning
3 participants