xapi-project / nbd / 350 / 1
75%
master: 65%

LAST BUILD BRANCH: docfix
DEFAULT BRANCH: master
Ran
Files 5
Run time 2s
Badge
Embed ▾
README BADGES
x

If you need to use a raster PNG badge, change the '.svg' to '.png' in the link

Markdown

Textile

RDoc

HTML

Rst
coverage: 74.643% (+3.6%) from 71.058%
PACKAGE=nbd DISTRO="debian-unstable" OCAML_VERSION=4.06 \ TEST=false \ POST_INSTALL_HOOK="opam install alcotest alcotest-lwt io-page-unix; env TRAVIS=$TRAVIS TRAVIS_JOB_ID=$TRAVIS_JOB_ID bash -ex coverage.sh"

Pull #131

travis-ci

web-flow 
CP-29049: Fix list and abort behaviour and expand protocol tests

Various fixes to make the client and server conform more to the NBD
protocol:
* Ensure Client.list terminates the option haggling by sending an abort.
  This change should not cause problems, as currently we don't seem to
  use Client.list anywhere.
* Fix the server's response to NBD_OPT_LIST - previously it did not send
  the lenghts to the client correctly, and therefore qemu-img and
  nbd-client could not list the exports.
* Check that the lengths the export names in the optional ~offer param
  passed to Server.connect are below the maximum limit allowed by the NBD
  protocol.
* Ensure the server responds to an abort with an ack. This change
  shouldn't cause issues either, because we gracefully handle the case
  without failing where the client disconnects without waiting for the
  ack.
* Clean up existing protocol unit tests, add more logging, and more
  tests for these corner cases.
* Add a new client<->server interaction test to check that NBD_OPT_LIST
  is implemented properly in both.

Signed-off-by: Gabor Igloi <gabor.igloi@citrix.com>
Pull Request #131: CP-29049: Fix list and abort behaviour and expand protocol tests

418 of 560 relevant lines covered (74.64%)

3396.83 hits per line

Source Files on job 350.1 (PACKAGE=nbd DISTRO="debian-unstable" OCAML_VERSION=4.06 \ TEST=false \ POST_INSTALL_HOOK="opam install alcotest alcotest-lwt io-page-unix; env TRAVIS=$TRAVIS TRAVIS_JOB_ID=$TRAVIS_JOB_ID bash -ex coverage.sh")