Installing Koji kernel. Booting specific kernel by default. #396

Merged
ryanlerch merged 1 commit from master into master 2022-01-20 16:36:13 +00:00
Contributor

Added the following changes:

  • quickdoc for installing kernel from Koji
  • booting page and HowTo for booting a specific kernel by default
Added the following changes: - quickdoc for installing kernel from Koji - booting page and HowTo for booting a specific kernel by default
Contributor

Could you add an example here please? I mean show the output of ls /boot on your machine, and then what the command should be? (These docs are for users who will not be techies and so we need explicit instructions with examples to make sure they can follow steps). Also: does the command always need the x86_64 suffix, and is /boot the correct place for UEFI boots also?

Could you add an example here please? I mean show the output of `ls /boot` on your machine, and then what the command should be? (These docs are for users who will not be techies and so we need explicit instructions with examples to make sure they can follow steps). Also: does the command always need the `x86_64` suffix, and is `/boot` the correct place for UEFI boots also?
Contributor

I'd remove the Fedora 23 guide. Even if it's relevant, that version of Fedora is long gone and we don't want users to be looking at other information there assuming everything there still applies :/

I'd remove the Fedora 23 guide. Even if it's relevant, that version of Fedora is long gone and we don't want users to be looking at other information there assuming everything there still applies :/
Contributor

Is this heading required? The include has its own heading (and it's about changing the default not just "booting" in general?

Is this heading required? The include has its own heading (and it's about changing the default not just "booting" in general?
Contributor

I'd split each sentence into a line by itself. See https://sembr.org/

I'd split each sentence into a line by itself. See https://sembr.org/
Contributor

"If there is a need to install a kernel that is not currently available in the Fedora repositories"?

"If there is a need to install a kernel that is not currently available in the Fedora repositories"?
Contributor

package -> client ?

package -> client ?
Contributor

So, one needs to first check if a kernel with this NVR was actually ever built for Fedora, by going to the Koji web interface.

I think it'll be good to give two methods here: one by going to the koji web interface, finding the necessary build, and then downloading files using the browser; and then the second can be this cli based method which is for more advanced users.

So, one needs to first check if a kernel with this NVR was actually ever built for Fedora, by going to the Koji web interface. I think it'll be good to give two methods here: one by going to the koji web interface, finding the necessary build, and then downloading files using the browser; and then the second can be this cli based method which is for more advanced users.
Contributor

I think it's best to use - instead of n.nn and so on everywhere. That way the command is self explanatory?

I think it's best to use <version>-<release> instead of `n.nn` and so on everywhere. That way the command is self explanatory?
Contributor

Metadata Update from @ankursinha:

  • Request assigned
**Metadata Update from @ankursinha**: - Request assigned
Contributor

Metadata Update from @ankursinha:

  • Pull-request tagged with: improvement, needs changes
**Metadata Update from @ankursinha**: - Pull-request tagged with: improvement, needs changes
Contributor

Thanks very much for this @jetstream . I've done a first round of review here now ^

Thanks very much for this @jetstream . I've done a first round of review here now ^
Author
Contributor

Could you add an example here please? I mean show the output of ls /boot on your machine, and then what the command should be? (These docs are for users who will not be techies and so we need explicit instructions with examples to make sure they can follow steps). Also: does the command always need the x86_64 suffix, and is /boot the correct place for UEFI boots also?

Sure. Adding an example in the next push.

I reckon it can be generalised to <arch>. Including the change in the next update.

AFAIK, kernels are located in /boot by default (for BIOS as well as UEFI systems)

> Could you add an example here please? I mean show the output of `ls /boot` on your machine, and then what the command should be? (These docs are for users who will not be techies and so we need explicit instructions with examples to make sure they can follow steps). Also: does the command always need the `x86_64` suffix, and is `/boot` the correct place for UEFI boots also? Sure. Adding an example in the next push. I reckon it can be generalised to `<arch>`. Including the change in the next update. AFAIK, kernels are located in `/boot` by default (for BIOS as well as UEFI systems)
Author
Contributor

Is this heading required? The include has its own heading (and it's about changing the default not just "booting" in general?

I was trying to align the addition to the existing structure. Here's my understanding.

Introduced a page for Booting (hence the heading Booting) under the section Kernel and Booting. This page has one include (for booting a kernel by default) for now. Any additional quick-docs on booting could be added as includes to this page.

Is that moving away from the current layout?

> Is this heading required? The include has its own heading (and it's about changing the default not just "booting" in general? I was trying to align the addition to the existing structure. Here's my understanding. Introduced a page for `Booting` (hence the heading `Booting`) under the section `Kernel and Booting`. This page has one include (for booting a kernel by default) for now. Any additional quick-docs on booting could be added as includes to this page. Is that moving away from the current layout?
Author
Contributor

"If there is a need to install a kernel that is not currently available in the Fedora repositories"?

Not sure I understood. Did you want me to change the wording to the sentence you suggested? I thought Koji is a Fedora repository.

> "If there is a need to install a kernel that is not currently available in the Fedora repositories"? Not sure I understood. Did you want me to change the wording to the sentence you suggested? I thought Koji is a Fedora repository.
Contributor

"If there is a need to install a kernel that is not currently available in the Fedora repositories"?

Not sure I understood. Did you want me to change the wording to the sentence you suggested? I thought Koji is a Fedora repository.

Yes please. Koji is not a repository, it's the build system which archives a certain number of old builds. The repositories only include a version of software that was released with a new Fedora release in the fedora repository, and then another version in the updates repository---they won't contain all the different update versions that may be available on koji. dnf does not speak to koji, it only knows of the repositories.

Is this heading required? The include has its own heading (and it's about changing the default not just "booting" in general?

I was trying to align the addition to the existing structure. Here's my understanding.

Introduced a page for Booting (hence the heading Booting) under the section Kernel and Booting. This page has one include (for booting a kernel by default) for now. Any additional quick-docs on booting could be added as includes to this page.

Is that moving away from the current layout?

Let's leave this as is for the time being, it can be easily tweaked later.

Please drop a comment when you've made more changes. Thanks again

> > "If there is a need to install a kernel that is not currently available in the Fedora repositories"? > > Not sure I understood. Did you want me to change the wording to the sentence you suggested? I thought Koji is a Fedora repository. Yes please. Koji is not a repository, it's the build system which archives a certain number of old builds. The repositories only include a version of software that was released with a new Fedora release in the `fedora` repository, and then another version in the `updates` repository---they won't contain all the different update versions that may be available on koji. `dnf` does not speak to koji, it only knows of the repositories. > > Is this heading required? The include has its own heading (and it's about changing the default not just "booting" in general? > > I was trying to align the addition to the existing structure. Here's my understanding. > > Introduced a page for `Booting` (hence the heading `Booting`) under the section `Kernel and Booting`. This page has one include (for booting a kernel by default) for now. Any additional quick-docs on booting could be added as includes to this page. > > Is that moving away from the current layout? Let's leave this as is for the time being, it can be easily tweaked later. Please drop a comment when you've made more changes. Thanks again
Author
Contributor

1 new commit added

  • Updates to resolve review comments.
**1 new commit added** * ``Updates to resolve review comments.``
Author
Contributor

Apologies for the delay, @ankursinha I have made the changes and pushed another commit. Please can you have a look when you get a chance?

Apologies for the delay, @ankursinha I have made the changes and pushed another commit. Please can you have a look when you get a chance?
Contributor

Sure thing, on my list to do this week 👍

Sure thing, on my list to do this week :thumbsup:
Contributor

Looks great now. Merging! Thanks!

Looks great now. Merging! Thanks!
Contributor

Pull-Request has been merged by ankursinha

Pull-Request has been merged by ankursinha
Contributor

I've also added you to the quick-doc committers group now. Please feel free to review other tickets and so on :)

I've also added you to the quick-doc committers group now. Please feel free to review other tickets and so on :)
Sign in to join this conversation.
No reviewers
No milestone
No project
No assignees
2 participants
Notifications
Due date
The due date is invalid or out of range. Please use the format "yyyy-mm-dd".

No due date set.

Dependencies

No dependencies set.

Reference: Docs/quick-docs#396
No description provided.