Skip to content
This repository has been archived by the owner on May 6, 2020. It is now read-only.

metrics: iperf3: improve error case output #963

Open
grahamwhaley opened this issue Apr 3, 2018 · 0 comments
Open

metrics: iperf3: improve error case output #963

grahamwhaley opened this issue Apr 3, 2018 · 0 comments
Assignees
@grahamwhaley

Comments

@grahamwhaley
Copy link
Contributor

Currently, if the server container fails we only print out a generic error, as we have already thrown away (not bothered to capture) the output from the 'test connection' command:

12:40:18 Status: Downloaded newer image for gabyct/network:latest
12:40:18 docker pull'd: gabyct/network
12:40:18 Iteration 1
12:40:24 ERROR: iperf server init fails
12:40:24 ERROR: result argument not supplied

See #960

Improve (a little at least), by capturing the output from the last 'test connection' attempt, and echoing it out in case of failure/timeout.
@grahamwhaley grahamwhaley self-assigned this Apr 3, 2018
mcastelino pushed a commit to mcastelino/tests that referenced this issue Jan 23, 2019
@chavafg
cri-o: build go-md2man from sources
ac6a056 
Now we should build and install go-md2man from sources instead of
using official repositories since centos removed the package
from its repos on centos 7.6.

Fixes: clearcontainers#963.

Signed-off-by: Salvador Fuentes <[email protected]>
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Assignees

@grahamwhaley grahamwhaley

Labels
None yet
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
1 participant
@grahamwhaley