Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

man zpool: get back iostat -y and -w description #7483

Merged
merged 1 commit into from
Apr 30, 2018

Conversation

gmelikov
Copy link
Member

Description

iostat -y and -w descriptions were left in cda0317 , get them back.

Closes #7479

Signed-off-by: George Melikov mail@gmelikov.ru

Motivation and Context

Fill the gap in man page.

How Has This Been Tested?

man ./man/man8/zpool.8

Types of changes

  • Bug fix (non-breaking change which fixes an issue)
  • New feature (non-breaking change which adds functionality)
  • Performance enhancement (non-breaking change which improves efficiency)
  • Code cleanup (non-breaking change which makes code smaller or more readable)
  • Breaking change (fix or feature that would cause existing functionality to change)
  • Documentation (a change to man pages or other documentation)

Checklist:

  • My code follows the ZFS on Linux code style requirements.
  • I have updated the documentation accordingly.
  • I have read the CONTRIBUTING document.
  • I have added tests to cover my changes.
  • All new and existing tests passed.
  • All commit messages are properly formatted and contain Signed-off-by.
  • Change has been approved by a ZFS on Linux member.

iostat -y and -w descriptions were left in cda0317
, get them back.

Closes openzfs#7479

Signed-off-by: George Melikov <mail@gmelikov.ru>
@codecov
Copy link

codecov bot commented Apr 28, 2018

Codecov Report

Merging #7483 into master will increase coverage by 0.26%.
The diff coverage is n/a.

Impacted file tree graph

@@            Coverage Diff             @@
##           master    #7483      +/-   ##
==========================================
+ Coverage   76.58%   76.84%   +0.26%     
==========================================
  Files         336      336              
  Lines      107137   107137              
==========================================
+ Hits        82047    82331     +284     
+ Misses      25090    24806     -284
Flag Coverage Δ
#kernel 77.39% <ø> (+0.45%) ⬆️
#user 66.23% <ø> (+0.41%) ⬆️

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update 089500e...81bc8c7. Read the comment docs.

@behlendorf behlendorf merged commit eb201f5 into openzfs:master Apr 30, 2018
tonyhutter pushed a commit to tonyhutter/zfs that referenced this pull request May 3, 2018
The iostat -y and -w descriptions were left in cda0317,
get them back.

Reviewed by: Matthew Ahrens <mahrens@delphix.com>
Reviewed-by: Tony Hutter <hutter2@llnl.gov>
Reviewed-by: Giuseppe Di Natale <dinatale2@llnl.gov>
Reviewed-by: Brian Behlendorf <behlendorf1@llnl.gov>
Signed-off-by: George Melikov <mail@gmelikov.ru>
Closes openzfs#7479
Closes openzfs#7483
tonyhutter pushed a commit to tonyhutter/zfs that referenced this pull request May 3, 2018
The iostat -y and -w descriptions were left in cda0317,
get them back.

Reviewed by: Matthew Ahrens <mahrens@delphix.com>
Reviewed-by: Tony Hutter <hutter2@llnl.gov>
Reviewed-by: Giuseppe Di Natale <dinatale2@llnl.gov>
Reviewed-by: Brian Behlendorf <behlendorf1@llnl.gov>
Signed-off-by: George Melikov <mail@gmelikov.ru>
Closes openzfs#7479
Closes openzfs#7483
tonyhutter pushed a commit to tonyhutter/zfs that referenced this pull request May 4, 2018
The iostat -y and -w descriptions were left in cda0317,
get them back.

Reviewed by: Matthew Ahrens <mahrens@delphix.com>
Reviewed-by: Tony Hutter <hutter2@llnl.gov>
Reviewed-by: Giuseppe Di Natale <dinatale2@llnl.gov>
Reviewed-by: Brian Behlendorf <behlendorf1@llnl.gov>
Signed-off-by: George Melikov <mail@gmelikov.ru>
Closes openzfs#7479
Closes openzfs#7483
tonyhutter pushed a commit that referenced this pull request May 10, 2018
The iostat -y and -w descriptions were left in cda0317,
get them back.

Reviewed by: Matthew Ahrens <mahrens@delphix.com>
Reviewed-by: Tony Hutter <hutter2@llnl.gov>
Reviewed-by: Giuseppe Di Natale <dinatale2@llnl.gov>
Reviewed-by: Brian Behlendorf <behlendorf1@llnl.gov>
Signed-off-by: George Melikov <mail@gmelikov.ru>
Closes #7479
Closes #7483
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.

zpool.8 manpage does not describe iostat -y or -w
5 participants