Tony Hutter

Results 348 comments of Tony Hutter

Another rebase on top of master should fix the Fedora build errors.

@AttilaFueloep @lowjoel just checking in - are you guys pretty happy with where this PR is right now? Are you just waiting on more approvals?

> 2. switch from SCRUB to RESILVERING if a vdev requires resilvering I was under the impression that zfs will already cancel a scrub if a resilver is required. Is...

@Ramzec I know this is a really late ask, but could you add a test case to verify your fix? The test would be: 1. Fill the pool with data...

Sample output: ``` $ zpool status -vv pool: testpool state: ONLINE status: One or more devices has experienced an error resulting in data corruption. Applications may be affected. action: Restore...

Just the filenames (JSON): ``` $ zpool status -vj ... "errors": { ":": { "name": ":" }, "/testpool/4k/4k_file1": { "name": "/testpool/4k/4k_file1" }, "/testpool/4k/4k_file2": { "name": "/testpool/4k/4k_file2" }, "/testpool/1m/1m_file": { "name":...

Overall this is a really nice feature! I haven't looked at the code yet, but did kick the tires a little and have some comments/questions. 1. Regarding: > The size...

I also noticed that the anyraid TLD names don't include the parity level. They all just say "anyraid": ``` anyraid-0 ONLINE 0 0 0 ``` We should have it match...

1. The validation logic will need to be tweaked to allow differing numbers of vdevs per anyraid TLD: ``` $ truncate -s 30G file1_30g $ truncate -s 40G file2_40g $...

I did some more testing today looking for edge cases: 1. This is the error when creating an anyraid pool with devices that are too small: ``` $ sudo ./zpool...