Add --json-immediate option.
#1605
Closed
Conversation
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This flag has three effects: * It implies `--json/-J` even if it is not also set. * It causes the JSON output to printedd output without whitespace formatting, so that a complete JSON structure appears on one output line. * It causes the JSON structure for each interval to be printed out at the end of that interval. The usual JSON output is still printed at the end of the test.
|
I've added this because it's useful for us and I'm perfectly happy to maintain it as a fork for our use, but wanted to contribute it back in case it's useful to others. |
|
Just noticed that this has considerable overlap with #1098, which is probably a better implementation of the same idea. It would be good to get some action on one of these, so that you don't keep getting PRs that duplicate the same work. |
|
Just merged the other one! Thank you for the pull request! |
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
Version of iperf3 (or development branch, such as
masteror3.1-STABLE) to which this pull request applies: masterIssues fixed (if any): This addresses easily parseable real time output option #444, though possibly not as the filer intended.
Brief description of code changes (suitable for use as a commit message):
This change adds a new boolean flag,
--json-immediate. This flag has three effects:--json/-Jeven if it is not also set.The motivation is, as requested in #444, to provide machine-readable output in real time.