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

tkt-42090: Improved arcstat. Added -a option to automatically print all valid st… #1702

Merged
merged 2 commits into from Aug 20, 2018

Conversation

powernap
Copy link
Contributor

…atistics. Added -p option to suppress scaling of printed data. Changed printing so that there is no longer a trailing seperator character at the end of every line.

Ticket: #42090

…atistics. Added -p option to suppress scaling of printed data. Changed printing so that there is no longer a trailing seperator character at the end of every line.

Ticket: #42090
@ghost ghost assigned powernap Aug 17, 2018
@ghost ghost added the review label Aug 17, 2018
@@ -224,26 +227,48 @@ def prettynum(sz, scale, num=0):
else:
return "%*d%s" % (sz - 1, num, suffix[index])

def print_with_sep(value, sep, first):
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Need one more blank line before and after this method to make PEP8 happy.

Copy link
Contributor

@amotin amotin left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

It would be OK for me if style checker would not be so loud about it. Could you please look on the CI failure messages to make it happy?

@bugclerk bugclerk changed the title Improved arcstat. Added -a option to automatically print all valid st… tkt-42317: Improved arcstat. Added -a option to automatically print all valid st… Aug 20, 2018
@william-gr william-gr removed the jira label Aug 20, 2018
@william-gr william-gr changed the title tkt-42317: Improved arcstat. Added -a option to automatically print all valid st… tkt-42090: Improved arcstat. Added -a option to automatically print all valid st… Aug 20, 2018
@amotin amotin merged commit 9671def into truenas:master Aug 20, 2018
@ghost ghost removed the review label Aug 20, 2018
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
3 participants