Closed ghost closed 1 year ago
Yes. But someone who wish to destroy the BE would very likely miss the step where boot_dataset variable was set, which is currently at the very beginning of BE creation steps. So, we mention it again inside the BE destruction steps to avoid any confusion.
George Melikov @.***> writes:
@gmelikov commented on this pull request.
@@ -140,6 +140,8 @@ and other maintenance operations and is thus currently not supported. ::
set -x
- boot_dataset=$(df -P /boot | tail -n1 | cut -f1 -d' ' || true )
Looks like it already is here, line 146 in new variant.
-- Reply to this email directly or view it on GitHub: https://github.com/openzfs/openzfs-docs/pull/438#pullrequestreview-1429311640 You are receiving this because you authored the thread.
Message ID: @.***>
Thanks for reviewing and merging this! I see that the Sphinx build failed somehow; and the pages were not updated after merging this PR to master. Maybe we should trigger a manual rebuild of the GitHub pages.
@ne9z yep, http://cvsweb.netbsd.org/bsdweb.cgi/~checkout~/src/external/cddl/osnet/dist/cmd/zpool/zpool-features.7?content-type=text/plain&only_with_tag=MAIN... doesn't work now, I'll rerun CI when it will be available.
Closes #434
@gmelikov