network test script needs better output
Affects | Status | Importance | Assigned to | Milestone | |
---|---|---|---|---|---|
Checkbox (Legacy) |
Fix Released
|
Medium
|
Jeff Lane | ||
Checkbox Provider - Base |
Fix Released
|
Medium
|
Jeff Lane |
Bug Description
The network test script needs to provide better output to checkbox... the problem seems to be that the output from the FTP session is dumped to stdout, while the debug logging all goes to stderr.
Since checkbox is an either-or when grabbing output, this means that on a successful test, we get this:
*cmd* 'USER ubuntu'
*put* 'USER ubuntu\r\n'
*get* '331 Please specify the password.\n'
*resp* '331 Please specify the password.'
*cmd* 'PASS ******'
*put* 'PASS ******\r\n'
*get* '230 Login successful.\n'
*resp* '230 Login successful.'
*cmd* 'CWD .'
*put* 'CWD .\r\n'
*get* '250 Directory successfully changed.\n'
*resp* '250 Directory successfully changed.'
*cmd* 'TYPE I'
*put* 'TYPE I\r\n'
*get* '200 Switching to Binary mode.\n'
*resp* '200 Switching to Binary mode.'
*cmd* 'PASV'
*put* 'PASV\r\n'
*get* '227 Entering Passive Mode (172,16,
*resp* '227 Entering Passive Mode (172,16,
*cmd* 'STOR ftp_performance
*put* 'STOR ftp_performance
*get* '150 Ok to send data.\n'
*resp* '150 Ok to send data.'
*get* '226 Transfer complete.\n'
*resp* '226 Transfer complete.'
which doesn't inform us about the throughput speeds observed or any other data. All that logging data is only displayed if the test fails, however, it would be important to know what the throughput is even on successful tests.
Related branches
- Daniel Manrique (community): Approve
-
Diff: 34 lines (+10/-1)1 file modifiedproviders/plainbox-provider-checkbox/bin/network (+10/-1)
- Daniel Manrique (community): Approve
-
Diff: 13 lines (+3/-0)1 file modifiedcheckbox-old/scripts/network (+3/-0)
tags: | added: scripts |
affects: | checkbox → plainbox-provider-checkbox |
Changed in plainbox-provider-checkbox: | |
status: | New → Triaged |
Changed in checkbox-legacy: | |
status: | New → In Progress |
importance: | Undecided → Medium |
assignee: | nobody → Jeff Lane (bladernr) |
Changed in checkbox-legacy: | |
status: | In Progress → Fix Committed |
Changed in plainbox-provider-checkbox: | |
status: | In Progress → Fix Released |
status: | Fix Released → Fix Committed |
milestone: | none → 0.17 |
Changed in checkbox-legacy: | |
status: | Fix Committed → Fix Released |
Changed in plainbox-provider-checkbox: | |
status: | Fix Committed → Fix Released |
Even since moving to iperf, there are some issues. The output on Success is ok, but the output on failure is lacking:
DEBUG:root:Shutting down interface:eth0 Restoring interface:eth0
DEBUG:root:timeout 180 iperf -c 172.100.166.27 -n 1024M
WARNING:root:Poor network performance detected
DEBUG:root: