yesBad/pve-storage
1
0
Fork 0
Code Issues Pull Requests Actions Packages Projects Releases Wiki Activity

diskmanage: wipe blockdev: also change partition type

Browse Source 
when called with a partition. Since get_disks uses the partition type
(among other things) to detect LVM and ZFS volumes, such volumes would
still be seen as in-use after wiping. Thus, also change the partition
type and simply use 0x83 "Linux filesystem".

Signed-off-by: Fabian Ebner <f.ebner@proxmox.com>
This commit is contained in:
Fabian Ebner
2021-10-06 11:18:42 +02:00
committed by Thomas Lamprecht
parent e8df8fb179
commit bd46e59b5d
1 changed files with 6 additions and 0 deletions
Download Patch File Download Diff File Expand all files Collapse all files

6
PVE/Diskmanage.pm
View File

@ -927,6 +927,7 @@ sub change_parttype {
} }
# Wipes all labels and the first 200 MiB of a disk/partition (or the whole if it is smaller). # Wipes all labels and the first 200 MiB of a disk/partition (or the whole if it is smaller).
# If called with a partition, also sets the partition type to 0x83 'Linux filesystem'.
# Expected to be called with a result of verify_blockdev_path(). # Expected to be called with a result of verify_blockdev_path().
sub wipe_blockdev { sub wipe_blockdev {
my ($devpath) = @_; my ($devpath) = @_;
@ -959,6 +960,11 @@ sub wipe_blockdev {
['dd', 'if=/dev/zero', "of=${devpath}", 'bs=1M', 'conv=fdatasync', "count=${count}"], ['dd', 'if=/dev/zero', "of=${devpath}", 'bs=1M', 'conv=fdatasync', "count=${count}"],
errmsg => "error wiping '${devpath}'", errmsg => "error wiping '${devpath}'",
); );
if (is_partition($devpath)) {
eval { change_parttype($devpath, '8300'); };
warn $@ if $@;
}
} }
1; 1;