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

doc: Add section about ostree and bootloaders #3116

Merged
merged 1 commit into from
Dec 14, 2023

Conversation

jmarrero
Copy link
Member

@jmarrero jmarrero commented Dec 8, 2023

No description provided.

Copy link

openshift-ci bot commented Dec 8, 2023

Skipping CI for Draft Pull Request.
If you want CI signal for your change, please convert it to an actual PR.
You can still manually trigger a test run with /test all

Copy link
Collaborator

@ericcurtin ericcurtin left a comment

Choose a reason for hiding this comment

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

LGTM, I might expand this to speak about Androidy Bootloaders found in ARM Automotive platforms


## GRUB and os-prober

A specific component of GRUB that can significantly impede the reliabilty of OS updates is the `os-prober` aspect, which scans all system block devices. If one doesn't care about dual booting, avoiding this is a good idea.
Copy link
Collaborator

Choose a reason for hiding this comment

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

s/reliabilty/reliability/g


## Anaconda

Until very recently the Anaconda project only supported setting up the bootloader (e.g. GRUB) on its own, which requires `grub2-mkconfig` etc. As of recently, Anaconda now [supports bootupd](https://github.com/rhinstaller/anaconda/pull/5298).
Copy link
Collaborator

Choose a reason for hiding this comment

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

s/Until very recently the Anaconda/Until very recently, the Anaconda/g


## bootupd

As of recently [the bootupd project](https://github.com/coreos/bootupd/) ships [static grub configs](https://github.com/coreos/bootupd/tree/main/src/grub2) and in this case, the `sysroot.bootloader` should be set to `none` (except on s390x).
Copy link
Collaborator

Choose a reason for hiding this comment

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

s/As of recently/As of recently,/g

@ericcurtin
Copy link
Collaborator

I found some super minor things, but happy to merge this as is, just say the word

@jmarrero
Copy link
Member Author

I found some super minor things, but happy to merge this as is, just say the word

thanks! and yes please expand this is a needed for auto.

@ericcurtin
Copy link
Collaborator

/test all

@ericcurtin
Copy link
Collaborator

I only have merge if everything is green permissions, trying to get this build:

ci/prow/fcos-e2e

from orange to green, it seems stuck or something.

@cgwalters
Copy link
Member

It doesn't run for docs-only PRs
/override ci/prow/fcos-e2e

Copy link

openshift-ci bot commented Dec 14, 2023

@cgwalters: Overrode contexts on behalf of cgwalters: ci/prow/fcos-e2e

In response to this:

It doesn't run for docs-only PRs
/override ci/prow/fcos-e2e

Instructions for interacting with me using PR comments are available here. If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository.

@ericcurtin ericcurtin merged commit 5c7f42b into ostreedev:main Dec 14, 2023
24 checks passed
@cgwalters cgwalters mentioned this pull request Jan 19, 2024
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.

3 participants