zio command should also show any I/Os which are in the DONE stage

[grwilson]

No description provided.

[codecov-commenter]

Codecov Report

All modified and coverable lines are covered by tests ✅

Comparison is base (16267f9) 85.21% compared to head (ff5c520) 84.45%.

❗ Current head ff5c520 differs from pull request most recent head d3cd88f. Consider uploading reports for the commit d3cd88f to get more accurate results

❗ Your organization needs to install the Codecov GitHub app to enable full functionality.

Additional details and impacted files

@@             Coverage Diff             @@
##           develop     #343      +/-   ##
===========================================
- Coverage    85.21%   84.45%   -0.76%     
===========================================
  Files           67       67              
  Lines         3070     3068       -2     
===========================================
- Hits          2616     2591      -25     
- Misses         454      477      +23     

Files	Coverage Δ
sdb/commands/zfs/zio.py	85.29% <ø> (-7.57%)	⬇️

... and 4 files with indirect coverage changes

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

[prakashsurya]

do you know why we originally hit them? should we have a flag to show/hide them? just curious if we're regressing in some workflow; e.g. maybe there's "too many" in the "done" stage, making it more difficult to see the non-done IOs?

change LGTM, but I'm not sure I'm experienced enough to say if there'll be any adverse affects to this change or not.

we don't need to update any of the test cases? if not, perhaps that means we should add some tests for this command....

[grwilson]

I think the assumption was that a "DONE" zio is not relevant but if it's still around it may be calling a done func and you may want to be able to track down what it's doing. I don't see any reason why you wouldn't want to show it.

[prakashsurya]

cool, OK.. that makes sense to me.