Docs: Add osbuild to documentation #4413
Docs: Add osbuild to documentation #4413angelcerveraroldan wants to merge 2 commits intocoreos:mainfrom
osbuild to documentation #4413Conversation
There was a problem hiding this comment.
Code Review
This pull request updates the documentation to reflect the introduction of the osbuild command for creating images, replacing the older buildextend commands. The changes in docs/cosa.md correctly add osbuild and remove buildextend. In docs/building-fcos.md, the build process is updated to use cosa osbuild qemu. However, I've noted one issue where the cosa fetch command is removed, which will cause an error for new users. My review includes a comment to address this.
I am having trouble creating individual review comments. Click here to see my feedback.
docs/building-fcos.md (188-195)
This change removes the explicit cosa fetch step. However, cosa build will fail on a fresh checkout if cosa fetch has not been run first, leading to a "Must fetch before building" error. This will break the workflow for new users following this guide. Please consider keeping this section, or at least mentioning that cosa fetch is a prerequisite to cosa build.
|
/test ci/prow/rhcos |
|
@dustymabe: The specified target(s) for DetailsIn response to this:
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-sigs/prow repository. |
|
/test rhcos |
|
NOTE: I'm running the rhcos test not because I suspect this change needs to be tested, but for another unrelated change where I want to see if CI is fixed now |
|
hmm. I think I'd prefer for the 4 commits to be combined into one with maybe each current commit title as a bullet point in the body of the commit. |
|
@angelcerveraroldan: The following test failed, say
Full PR test history. Your PR dashboard. DetailsInstructions 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-sigs/prow repository. I understand the commands that are listed here. |
jbtrystram
left a comment
There was a problem hiding this comment.
Agreed with Dusty, this should be one commit
a907b23 to
bd446bf
Compare
jbtrystram
left a comment
There was a problem hiding this comment.
Thanks ! Some small suggestions
- Run `cosa osbuild qemu` in build docs - Add `osbuild` to the list of main commands - Replace `buildextend-*` commands for `osbuild`
bd446bf to
d695ce1
Compare
osbuildto the list of main commands, and removebuildextendcommands.cosa osbuild qemuas a step in the build docs.Closes: #4304