diff options
| author | Don Brady <[email protected]> | 2023-10-20 10:29:02 -0600 |
|---|---|---|
| committer | Tony Hutter <[email protected]> | 2023-11-06 16:45:14 -0800 |
| commit | 0bcd1151f0c268f826e81438ceaa5ec1761f6baf (patch) | |
| tree | ca631664239b351a88389d03f94fe3681b27949e /include | |
| parent | 78fd79eacdc98f5452f69d62b55ba3f6c4d8018c (diff) | |
Fix ZED auto-replace for VDEVs using by-id paths
The change is simple -- restore the original code so that the VDEV
path is updated when using by-id paths. The more challenging part
was to devise a second ZTS test, that would test auto-replace for
'by-id' and help prevent a future regression.
With that new test, we can now do an A|B test with , and without,
the fix to confirm that auto-replace for by-id paths works. The
existing auto-replace test, functional/fault/auto_replace_001_pos,
will confirm that we didn't break auto-replace for 'by-vdev' paths.
In the original functional/fault/auto_replace_001_pos test, the disk
wipe (using dd) was not effective in removing the partitioning since
the kernel was never informed of the wipe.
Added a call to wipefs(8) so that the kernel is informed and ZED will
re-partition the device.
Added a validation step that the re-partitioning occurred by
confirming that the GPT partition UUID changes.
Sponsored-By: OpenDrives Inc.
Sponsored-By: Klara Inc.
Reviewed-by: Rob Norris <[email protected]>
Reviewed-by: Tony Hutter <[email protected]>
Signed-off-by: Don Brady <[email protected]>
Closes #15363
Diffstat (limited to 'include')
| -rw-r--r-- | include/libzutil.h | 2 |
1 files changed, 1 insertions, 1 deletions
diff --git a/include/libzutil.h b/include/libzutil.h index 237ff976b..053b1ed4b 100644 --- a/include/libzutil.h +++ b/include/libzutil.h @@ -34,7 +34,7 @@ extern "C" { #endif /* - * Default wait time for a device name to be created. + * Default wait time in milliseconds for a device name to be created. */ #define DISK_LABEL_WAIT (30 * 1000) /* 30 seconds */ |