docs: Add docs for install reset to experimental section #1717
Conversation
![gemini-code-assist[bot]](https://avatars.githubusercontent.com/in/956858?s=60&v=4){kind=small}
There was a problem hiding this comment.
Code Review
This pull request adds documentation for the new experimental bootc install reset command. The new documentation is clear and well-structured, providing a good overview, usage examples, and details on the feature. I've identified one issue in a command example for cleaning up old stateroots and have provided a correction.
| ls -la /sysroot/ostree/deploy/ | ||
|
|
||
| # Remove the old stateroot (replace 'default' with your old stateroot name) | ||
| sudo ostree admin stateroot-init --delete default |
There was a problem hiding this comment.
The command provided for cleaning up an old stateroot appears to be incorrect. ostree admin stateroot-init is used for initializing a stateroot, not deleting one. The correct command to delete a stateroot is ostree admin stateroot-delete. Using the wrong command in the documentation could lead to user confusion and errors.
| sudo ostree admin stateroot-init --delete default | |
| sudo ostree admin stateroot-delete default |
ckyrouac
force-pushed
the
factory-reset-docs
branch
from
40a1cd9 to
0265615
Compare
Assisted-by: Claude Code Signed-off-by: ckyrouac <[email protected]>
ckyrouac
force-pushed
the
factory-reset-docs
branch
from
0265615 to
22d26bd
Compare
|
|
||
| # Remove the old deployment(s) by index | ||
| # The index is shown in the output (e.g., "1" for the second deployment) | ||
| ostree admin undeploy <index> |
There was a problem hiding this comment.
I think this is something we need to wrap in bootc more nicely...perhaps something like bootc stateroot rm <name> or so that would also remove all deployments associated with it?
3 participants
Assisted-by: Claude Code