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

Kargs integration test #1

Closed
wants to merge 15 commits into from
Closed

Kargs integration test #1

wants to merge 15 commits into from

Conversation

lukewarmtemp
Copy link
Owner

No description provided.

jeckersb and others added 14 commits June 5, 2024 13:19
tests/integration: install libzstd-devel
Fixes the authfile brown paper bag bug.

Signed-off-by: Colin Walters <[email protected]>
Signed-off-by: Platform Engineering Bot <[email protected]>
This ensures that when we rerun the container build without
changing the source, we get the same thing.  Which then
further ensures we don't invalidate any cached disk images,
etc.

Signed-off-by: Colin Walters <[email protected]>
Let's give some recommendations here.

Signed-off-by: Colin Walters <[email protected]>
Signed-off-by: Platform Engineering Bot <[email protected]>
Fixes containers#255. Allows users to create files within /usr/lib/bootc/kargs.d with kernel arguments. These arguments can now be applied on a switch, upgrade, or edit.

General process:
- use ostree commit of fetched container image to return
the file tree
- navigate to /usr/lib/bootc/kargs.d
- read each file within the directory
- calculate the diff between the booted and fetched kargs in kargs.d
- apply the diff to the kargs currently on the running system
- pass the kargs to the stage() function

Signed-off-by: Luke Yang <[email protected]>
@lukewarmtemp lukewarmtemp force-pushed the kargs-integration-test branch from 69a7e33 to 97a5d1b Compare June 10, 2024 19:09
@github-actions github-actions bot added the documentation Improvements or additions to documentation label Jun 10, 2024
lukewarmtemp pushed a commit that referenced this pull request Jun 19, 2024
This is definitely the #1 confusion many people have to start;
it's very understandable!  Because in theory we *could* have
a runtime and a larger role, but it was an explicit choice
not to do that.

Signed-off-by: Colin Walters <[email protected]>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
area/install documentation Improvements or additions to documentation
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants