Skip to content

justdoit: some bug fixes#8

Open
MatthewCroughan wants to merge 2 commits intocleverca22:masterfrom
MatthewCroughan:mc/bug-fixes
Open

justdoit: some bug fixes#8
MatthewCroughan wants to merge 2 commits intocleverca22:masterfrom
MatthewCroughan:mc/bug-fixes

Conversation

@MatthewCroughan
Copy link

  • justdoit: set bootType to vfat if uefi is true

    Without this, ext4 would always be used, even if uefi is true, which is incorrect

  • justdoit: set -f on zpool create

    This fixes the case where justdoit has previously been used on a disk before, where /dev/sda3 may contain an existing ZFS member signature/label for example. This would usually have to be cleared by zpool clearlabel, but passing -f to zpool create also works

    > + zpool create -o ashift=12 -o altroot=/mnt tank /dev/sda3
    > invalid vdev specification
    > use '-f' to override the following errors:
    > /dev/sda3 is part of potentially active pool 'tank'
    

This fixes the case where justdoit has previously been used on a disk
before, where /dev/sda3 may contain an existing ZFS member
signature/label for example. This would usually have to be cleared by
zpool clearlabel, but passing -f to zpool create also works

```
> + zpool create -o ashift=12 -o altroot=/mnt tank /dev/sda3
> invalid vdev specification
> use '-f' to override the following errors:
> /dev/sda3 is part of potentially active pool 'tank'
```
Without this, ext4 would always be used, even if uefi is true, which is incorrect
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.

1 participant