qemu-commits
[Top][All Lists]
Advanced

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

[Qemu-commits] [qemu/qemu] bf2295: qemu-iotests: remove unused variable


From: GitHub
Subject: [Qemu-commits] [qemu/qemu] bf2295: qemu-iotests: remove unused variable 'here'
Date: Tue, 20 Nov 2018 02:12:37 -0800

  Branch: refs/heads/master
  Home:   https://github.com/qemu/qemu
  Commit: bf22957309369cf6f642e715ff6c470671920e7e
      
https://github.com/qemu/qemu/commit/bf22957309369cf6f642e715ff6c470671920e7e
  Author: Mao Zhongyi <address@hidden>
  Date:   2018-11-19 (Mon, 19 Nov 2018)

  Changed paths:
    M tests/qemu-iotests/001
    M tests/qemu-iotests/002
    M tests/qemu-iotests/003
    M tests/qemu-iotests/004
    M tests/qemu-iotests/005
    M tests/qemu-iotests/007
    M tests/qemu-iotests/008
    M tests/qemu-iotests/009
    M tests/qemu-iotests/010
    M tests/qemu-iotests/011
    M tests/qemu-iotests/012
    M tests/qemu-iotests/013
    M tests/qemu-iotests/014
    M tests/qemu-iotests/015
    M tests/qemu-iotests/017
    M tests/qemu-iotests/018
    M tests/qemu-iotests/019
    M tests/qemu-iotests/020
    M tests/qemu-iotests/021
    M tests/qemu-iotests/022
    M tests/qemu-iotests/023
    M tests/qemu-iotests/024
    M tests/qemu-iotests/025
    M tests/qemu-iotests/026
    M tests/qemu-iotests/027
    M tests/qemu-iotests/028
    M tests/qemu-iotests/029
    M tests/qemu-iotests/031
    M tests/qemu-iotests/032
    M tests/qemu-iotests/033
    M tests/qemu-iotests/034
    M tests/qemu-iotests/035
    M tests/qemu-iotests/036
    M tests/qemu-iotests/037
    M tests/qemu-iotests/038
    M tests/qemu-iotests/039
    M tests/qemu-iotests/042
    M tests/qemu-iotests/043
    M tests/qemu-iotests/046
    M tests/qemu-iotests/047
    M tests/qemu-iotests/049
    M tests/qemu-iotests/050
    M tests/qemu-iotests/051
    M tests/qemu-iotests/052
    M tests/qemu-iotests/053
    M tests/qemu-iotests/054
    M tests/qemu-iotests/058
    M tests/qemu-iotests/059
    M tests/qemu-iotests/060
    M tests/qemu-iotests/061
    M tests/qemu-iotests/062
    M tests/qemu-iotests/063
    M tests/qemu-iotests/064
    M tests/qemu-iotests/066
    M tests/qemu-iotests/067
    M tests/qemu-iotests/068
    M tests/qemu-iotests/069
    M tests/qemu-iotests/070
    M tests/qemu-iotests/071
    M tests/qemu-iotests/072
    M tests/qemu-iotests/073
    M tests/qemu-iotests/075
    M tests/qemu-iotests/076
    M tests/qemu-iotests/077
    M tests/qemu-iotests/078
    M tests/qemu-iotests/079
    M tests/qemu-iotests/080
    M tests/qemu-iotests/081
    M tests/qemu-iotests/082
    M tests/qemu-iotests/083
    M tests/qemu-iotests/084
    M tests/qemu-iotests/085
    M tests/qemu-iotests/086
    M tests/qemu-iotests/087
    M tests/qemu-iotests/088
    M tests/qemu-iotests/089
    M tests/qemu-iotests/090
    M tests/qemu-iotests/091
    M tests/qemu-iotests/092
    M tests/qemu-iotests/094
    M tests/qemu-iotests/095
    M tests/qemu-iotests/097
    M tests/qemu-iotests/098
    M tests/qemu-iotests/099
    M tests/qemu-iotests/101
    M tests/qemu-iotests/102
    M tests/qemu-iotests/103
    M tests/qemu-iotests/104
    M tests/qemu-iotests/105
    M tests/qemu-iotests/106
    M tests/qemu-iotests/107
    M tests/qemu-iotests/108
    M tests/qemu-iotests/109
    M tests/qemu-iotests/110
    M tests/qemu-iotests/111
    M tests/qemu-iotests/112
    M tests/qemu-iotests/113
    M tests/qemu-iotests/114
    M tests/qemu-iotests/115
    M tests/qemu-iotests/116
    M tests/qemu-iotests/117
    M tests/qemu-iotests/119
    M tests/qemu-iotests/120
    M tests/qemu-iotests/121
    M tests/qemu-iotests/122
    M tests/qemu-iotests/123
    M tests/qemu-iotests/125
    M tests/qemu-iotests/126
    M tests/qemu-iotests/127
    M tests/qemu-iotests/128
    M tests/qemu-iotests/130
    M tests/qemu-iotests/131
    M tests/qemu-iotests/133
    M tests/qemu-iotests/134
    M tests/qemu-iotests/135
    M tests/qemu-iotests/137
    M tests/qemu-iotests/138
    M tests/qemu-iotests/140
    M tests/qemu-iotests/141
    M tests/qemu-iotests/142
    M tests/qemu-iotests/143
    M tests/qemu-iotests/144
    M tests/qemu-iotests/145
    M tests/qemu-iotests/146
    M tests/qemu-iotests/150
    M tests/qemu-iotests/153
    M tests/qemu-iotests/154
    M tests/qemu-iotests/156
    M tests/qemu-iotests/157
    M tests/qemu-iotests/158
    M tests/qemu-iotests/159
    M tests/qemu-iotests/160
    M tests/qemu-iotests/162
    M tests/qemu-iotests/170
    M tests/qemu-iotests/171
    M tests/qemu-iotests/172
    M tests/qemu-iotests/173
    M tests/qemu-iotests/174
    M tests/qemu-iotests/175
    M tests/qemu-iotests/176
    M tests/qemu-iotests/177
    M tests/qemu-iotests/178
    M tests/qemu-iotests/179
    M tests/qemu-iotests/181
    M tests/qemu-iotests/182
    M tests/qemu-iotests/183
    M tests/qemu-iotests/184
    M tests/qemu-iotests/185
    M tests/qemu-iotests/186
    M tests/qemu-iotests/187
    M tests/qemu-iotests/188
    M tests/qemu-iotests/189
    M tests/qemu-iotests/190
    M tests/qemu-iotests/191
    M tests/qemu-iotests/192
    M tests/qemu-iotests/195
    M tests/qemu-iotests/197
    M tests/qemu-iotests/198
    M tests/qemu-iotests/200
    M tests/qemu-iotests/204
    M tests/qemu-iotests/214
    M tests/qemu-iotests/215
    M tests/qemu-iotests/221
    M tests/qemu-iotests/223
    M tests/qemu-iotests/225
    M tests/qemu-iotests/226
    M tests/qemu-iotests/227
    M tests/qemu-iotests/229
    M tests/qemu-iotests/231
    M tests/qemu-iotests/232

  Log Message:
  -----------
  qemu-iotests: remove unused variable 'here'

Running
git grep '\$here' tests/qemu-iotests

has 0 hits, which means we are setting a variable that has
no use.  It appears that commit e8f8624d removed the last
use.  So execute the following cmd to remove all of
the 'here=...' lines as dead code.

sed -i '/^here=/d' $(git grep -l '^here=' tests/qemu-iotests)

Cc: address@hidden
Cc: address@hidden
Cc: address@hidden
Suggested-by: Eric Blake <address@hidden>
Signed-off-by: Mao Zhongyi <address@hidden>
Message-Id: <address@hidden>
Reviewed-by: Eric Blake <address@hidden>
[eblake: touch up commit message, reorder series, rebase to master]
Signed-off-by: Eric Blake <address@hidden>


  Commit: e8d81a61e1b9e28267164f751dee5b9b59444e71
      
https://github.com/qemu/qemu/commit/e8d81a61e1b9e28267164f751dee5b9b59444e71
  Author: Mao Zhongyi <address@hidden>
  Date:   2018-11-19 (Mon, 19 Nov 2018)

  Changed paths:
    M configure
    M scripts/coccinelle/tcg_gen_extract.cocci
    M tests/check-block.sh
    M tests/qemu-iotests/check
    M tests/qemu-iotests/common.config
    M tests/qemu-iotests/common.rc

  Log Message:
  -----------
  qemu-iotests: convert `pwd` and $(pwd) to $PWD

POSIX requires $PWD to be reliable, and we expect all
shells used by qemu scripts to be relatively close to
POSIX.  Thus, it is smarter to avoid forking the pwd
executable for something that is already available in
the environment.

So replace it with the following:

sed -i 's/\(`pwd`\|\$(pwd)\)/$PWD/g' $(git grep -l pwd)

Then delete a pointless line assigning PWD to itself.

Cc: address@hidden
Cc: address@hidden
Cc: address@hidden
Suggested-by: Eric Blake <address@hidden>
Signed-off-by: Mao Zhongyi <address@hidden>
Message-Id: <address@hidden>
Reviewed-by: Eric Blake <address@hidden>
[eblake: touch up commit message, reorder series, tweak a couple more files]
Signed-off-by: Eric Blake <address@hidden>


  Commit: 4a9e751f61522991c2fa94a4da8feda6a4d09c70
      
https://github.com/qemu/qemu/commit/4a9e751f61522991c2fa94a4da8feda6a4d09c70
  Author: Mao Zhongyi <address@hidden>
  Date:   2018-11-19 (Mon, 19 Nov 2018)

  Changed paths:
    M tests/qemu-iotests/check
    M tests/qemu-iotests/common.config

  Log Message:
  -----------
  qemu-iotests: Modern shell scripting (use $() instead of ``)

Various shell files contain a mix between obsolete ``
and modern $(); It would be nice to convert to using
$() everywhere.  For now, just do the qemu-iotests directory.

Cc: address@hidden
Cc: address@hidden
Cc: address@hidden
Suggested-by: Eric Blake <address@hidden>
Signed-off-by: Mao Zhongyi <address@hidden>
Message-Id: <address@hidden>
Reviewed-by: Eric Blake <address@hidden>
[eblake: tweak commit message]
Signed-off-by: Eric Blake <address@hidden>


  Commit: 0b0bb124bb797b2fc3389ffa1e933fc34fe6dacb
      
https://github.com/qemu/qemu/commit/0b0bb124bb797b2fc3389ffa1e933fc34fe6dacb
  Author: Daniel P. Berrangé <address@hidden>
  Date:   2018-11-19 (Mon, 19 Nov 2018)

  Changed paths:
    M nbd/server.c

  Log Message:
  -----------
  nbd: fix whitespace in server error message

A space was missing after the option number was printed:

  Option 0x8not permitted before TLS

becomes

  Option 0x8 not permitted before TLS

This fixes

  commit 3668328303429f3bc93ab3365c66331600b06a2d
  Author: Eric Blake <address@hidden>
  Date:   Fri Oct 14 13:33:09 2016 -0500

    nbd: Send message along with server NBD_REP_ERR errors

Signed-off-by: Daniel P. Berrangé <address@hidden>
Message-Id: <address@hidden>
Reviewed-by: Eric Blake <address@hidden>
[eblake: move lone space to next line]
Signed-off-by: Eric Blake <address@hidden>


  Commit: 3e99ebb9d3df15ce0ecf1b435816c9c46ee9a1ad
      
https://github.com/qemu/qemu/commit/3e99ebb9d3df15ce0ecf1b435816c9c46ee9a1ad
  Author: Eric Blake <address@hidden>
  Date:   2018-11-19 (Mon, 19 Nov 2018)

  Changed paths:
    M nbd/server.c

  Log Message:
  -----------
  nbd/server: Ignore write errors when replying to NBD_OPT_ABORT

Commit 37ec36f6 intentionally ignores errors when trying to reply
to an NBD_OPT_ABORT request for plaintext clients, but did not make
the same change for a TLS server.  Since NBD_OPT_ABORT is
documented as being a potential for an EPIPE when the client hangs
up without waiting for our reply, we don't need to pollute the
server's output with that failure.

Signed-off-by: Eric Blake <address@hidden>
Message-Id: <address@hidden>
Reviewed-by: Daniel P. Berrangé <address@hidden>


  Commit: a2458b6f6998c9a079f710ed7495d5c6f037e942
      
https://github.com/qemu/qemu/commit/a2458b6f6998c9a079f710ed7495d5c6f037e942
  Author: Daniel P. Berrangé <address@hidden>
  Date:   2018-11-19 (Mon, 19 Nov 2018)

  Changed paths:
    M crypto/tlssession.c
    M include/io/channel-tls.h
    M include/io/channel.h
    M io/channel-tls.c

  Log Message:
  -----------
  io: return 0 for EOF in TLS session read after shutdown

GNUTLS takes a paranoid approach when seeing 0 bytes returned by the
underlying OS read() function. It will consider this an error and
return GNUTLS_E_PREMATURE_TERMINATION instead of propagating the 0
return value. It expects apps to arrange for clean termination at
the protocol level and not rely on seeing EOF from a read call to
detect shutdown. This is to harden apps against a malicious 3rd party
causing termination of the sockets layer.

This is unhelpful for the QEMU NBD code which does have a clean
protocol level shutdown, but still relies on seeing 0 from the I/O
channel read in the coroutine handling incoming replies.

The upshot is that when using a plain NBD connection shutdown is
silent, but when using TLS, the client spams the console with

  Cannot read from TLS channel: Broken pipe

The NBD connection has, however, called qio_channel_shutdown()
at this point to indicate that it is done with I/O. This gives
the opportunity to optimize the code such that when the channel
has been shutdown in the read direction, the error code
GNUTLS_E_PREMATURE_TERMINATION gets turned into a '0' return
instead of an error.

Signed-off-by: Daniel P. Berrangé <address@hidden>
Message-Id: <address@hidden>
Reviewed-by: Eric Blake <address@hidden>
Signed-off-by: Eric Blake <address@hidden>


  Commit: e6d5d6fdd43484f60f678e66c21f0d1286749977
      
https://github.com/qemu/qemu/commit/e6d5d6fdd43484f60f678e66c21f0d1286749977
  Author: Daniel P. Berrangé <address@hidden>
  Date:   2018-11-19 (Mon, 19 Nov 2018)

  Changed paths:
    M tests/qemu-iotests/058
    A tests/qemu-iotests/common.nbd

  Log Message:
  -----------
  tests: pull qemu-nbd iotest helpers into common.nbd file

The helpers for starting/stopping qemu-nbd in 058 will be useful in
other test cases, so move them into a common.nbd file.

Signed-off-by: Daniel P. Berrangé <address@hidden>
Message-Id: <address@hidden>
Reviewed-by: Eric Blake <address@hidden>
[eblake: fix shell quoting]
Signed-off-by: Eric Blake <address@hidden>


  Commit: b39b58d5d0da3e7057d7d636641018b0fc25139b
      
https://github.com/qemu/qemu/commit/b39b58d5d0da3e7057d7d636641018b0fc25139b
  Author: Daniel P. Berrangé <address@hidden>
  Date:   2018-11-19 (Mon, 19 Nov 2018)

  Changed paths:
    M tests/qemu-iotests/common.nbd

  Log Message:
  -----------
  tests: check if qemu-nbd is still alive before waiting

If the qemu-nbd UNIX socket has not shown up, the tests will sleep a bit
and then check again repeatedly for up to 30 seconds. This is pointless
if the qemu-nbd process has quit due to an error, so check whether the
pid is still alive before waiting and retrying.

Signed-off-by: Daniel P. Berrangé <address@hidden>
Message-Id: <address@hidden>
Reviewed-by: Eric Blake <address@hidden>
Signed-off-by: Eric Blake <address@hidden>


  Commit: a46b68410669fa14c4a85d9284953fc0d42392d0
      
https://github.com/qemu/qemu/commit/a46b68410669fa14c4a85d9284953fc0d42392d0
  Author: Daniel P. Berrangé <address@hidden>
  Date:   2018-11-19 (Mon, 19 Nov 2018)

  Changed paths:
    A tests/qemu-iotests/common.tls

  Log Message:
  -----------
  tests: add iotests helpers for dealing with TLS certificates

Add helpers to common.tls for creating TLS certificates for a CA,
server and client.

Signed-off-by: Daniel P. Berrangé <address@hidden>
Message-Id: <address@hidden>
Reviewed-by: Eric Blake <address@hidden>
[eblake: spelling and quoting touchups]
Signed-off-by: Eric Blake <address@hidden>


  Commit: afcd1c2f2d438930a17eb87293c0ac2c377158fa
      
https://github.com/qemu/qemu/commit/afcd1c2f2d438930a17eb87293c0ac2c377158fa
  Author: Daniel P. Berrangé <address@hidden>
  Date:   2018-11-19 (Mon, 19 Nov 2018)

  Changed paths:
    A tests/qemu-iotests/233
    A tests/qemu-iotests/233.out
    M tests/qemu-iotests/common.nbd
    M tests/qemu-iotests/group

  Log Message:
  -----------
  tests: exercise NBD server in TLS mode

Add tests that validate it is possible to connect to an NBD server
running TLS mode. Also test mis-matched TLS vs non-TLS connections
correctly fail.

Signed-off-by: Daniel P. Berrangé <address@hidden>
Message-Id: <address@hidden>
Reviewed-by: Eric Blake <address@hidden>
Tested-by: Eric Blake <address@hidden>
[eblake: rebase to iotests shell cleanups, use ss instead of socat for
port probing, sanitize port number in expected output]
Signed-off-by: Eric Blake <address@hidden>


  Commit: bb39c47d70e84acf5066f79eba27ae5945b837be
      
https://github.com/qemu/qemu/commit/bb39c47d70e84acf5066f79eba27ae5945b837be
  Author: Eric Blake <address@hidden>
  Date:   2018-11-19 (Mon, 19 Nov 2018)

  Changed paths:
    M tests/qemu-iotests/233
    M tests/qemu-iotests/233.out

  Log Message:
  -----------
  iotests: Also test I/O over NBD TLS

Enhance test 233 to also perform I/O beyond the initial handshake.

Signed-off-by: Eric Blake <address@hidden>
Message-Id: <address@hidden>
Reviewed-by: Daniel P. Berrangé <address@hidden>


  Commit: 8cedcffdc195bc39aeb1373826ba0a45629741e0
      
https://github.com/qemu/qemu/commit/8cedcffdc195bc39aeb1373826ba0a45629741e0
  Author: Eric Blake <address@hidden>
  Date:   2018-11-19 (Mon, 19 Nov 2018)

  Changed paths:
    M tests/qemu-iotests/035
    M tests/qemu-iotests/037
    M tests/qemu-iotests/038
    M tests/qemu-iotests/046
    M tests/qemu-iotests/047
    M tests/qemu-iotests/049
    M tests/qemu-iotests/051
    M tests/qemu-iotests/067
    M tests/qemu-iotests/071
    M tests/qemu-iotests/077
    M tests/qemu-iotests/081
    M tests/qemu-iotests/082
    M tests/qemu-iotests/085
    M tests/qemu-iotests/086
    M tests/qemu-iotests/087
    M tests/qemu-iotests/099
    M tests/qemu-iotests/109
    M tests/qemu-iotests/112
    M tests/qemu-iotests/142
    M tests/qemu-iotests/153
    M tests/qemu-iotests/157
    M tests/qemu-iotests/172
    M tests/qemu-iotests/176
    M tests/qemu-iotests/177
    M tests/qemu-iotests/184
    M tests/qemu-iotests/186
    M tests/qemu-iotests/195
    M tests/qemu-iotests/204
    M tests/qemu-iotests/223
    M tests/qemu-iotests/227
    M tests/qemu-iotests/232
    M tests/qemu-iotests/common.nbd
    M tests/qemu-iotests/common.pattern
    M tests/qemu-iotests/common.qemu
    M tests/qemu-iotests/common.tls

  Log Message:
  -----------
  iotests: Drop use of bash keyword 'function'

Bash allows functions to be declared with or without the leading
keyword 'function'; but including the keyword does not comply with
POSIX syntax, and is confusing to ksh users where the use of the
keyword changes the scoping rules for functions.  Stick to the
POSIX form through iotests.

Done mechanically with:
  sed -i 's/^function //' $(git ls-files tests/qemu-iotests)

Signed-off-by: Eric Blake <address@hidden>
Message-Id: <address@hidden>
Reviewed-by: Daniel P. Berrangé <address@hidden>
Reviewed-by: Philippe Mathieu-Daudé <address@hidden>


  Commit: 393aac1629c8f82b7c448c8615745e89984899a8
      
https://github.com/qemu/qemu/commit/393aac1629c8f82b7c448c8615745e89984899a8
  Author: Peter Maydell <address@hidden>
  Date:   2018-11-19 (Mon, 19 Nov 2018)

  Changed paths:
    M configure
    M crypto/tlssession.c
    M include/io/channel-tls.h
    M include/io/channel.h
    M io/channel-tls.c
    M nbd/server.c
    M scripts/coccinelle/tcg_gen_extract.cocci
    M tests/check-block.sh
    M tests/qemu-iotests/001
    M tests/qemu-iotests/002
    M tests/qemu-iotests/003
    M tests/qemu-iotests/004
    M tests/qemu-iotests/005
    M tests/qemu-iotests/007
    M tests/qemu-iotests/008
    M tests/qemu-iotests/009
    M tests/qemu-iotests/010
    M tests/qemu-iotests/011
    M tests/qemu-iotests/012
    M tests/qemu-iotests/013
    M tests/qemu-iotests/014
    M tests/qemu-iotests/015
    M tests/qemu-iotests/017
    M tests/qemu-iotests/018
    M tests/qemu-iotests/019
    M tests/qemu-iotests/020
    M tests/qemu-iotests/021
    M tests/qemu-iotests/022
    M tests/qemu-iotests/023
    M tests/qemu-iotests/024
    M tests/qemu-iotests/025
    M tests/qemu-iotests/026
    M tests/qemu-iotests/027
    M tests/qemu-iotests/028
    M tests/qemu-iotests/029
    M tests/qemu-iotests/031
    M tests/qemu-iotests/032
    M tests/qemu-iotests/033
    M tests/qemu-iotests/034
    M tests/qemu-iotests/035
    M tests/qemu-iotests/036
    M tests/qemu-iotests/037
    M tests/qemu-iotests/038
    M tests/qemu-iotests/039
    M tests/qemu-iotests/042
    M tests/qemu-iotests/043
    M tests/qemu-iotests/046
    M tests/qemu-iotests/047
    M tests/qemu-iotests/049
    M tests/qemu-iotests/050
    M tests/qemu-iotests/051
    M tests/qemu-iotests/052
    M tests/qemu-iotests/053
    M tests/qemu-iotests/054
    M tests/qemu-iotests/058
    M tests/qemu-iotests/059
    M tests/qemu-iotests/060
    M tests/qemu-iotests/061
    M tests/qemu-iotests/062
    M tests/qemu-iotests/063
    M tests/qemu-iotests/064
    M tests/qemu-iotests/066
    M tests/qemu-iotests/067
    M tests/qemu-iotests/068
    M tests/qemu-iotests/069
    M tests/qemu-iotests/070
    M tests/qemu-iotests/071
    M tests/qemu-iotests/072
    M tests/qemu-iotests/073
    M tests/qemu-iotests/075
    M tests/qemu-iotests/076
    M tests/qemu-iotests/077
    M tests/qemu-iotests/078
    M tests/qemu-iotests/079
    M tests/qemu-iotests/080
    M tests/qemu-iotests/081
    M tests/qemu-iotests/082
    M tests/qemu-iotests/083
    M tests/qemu-iotests/084
    M tests/qemu-iotests/085
    M tests/qemu-iotests/086
    M tests/qemu-iotests/087
    M tests/qemu-iotests/088
    M tests/qemu-iotests/089
    M tests/qemu-iotests/090
    M tests/qemu-iotests/091
    M tests/qemu-iotests/092
    M tests/qemu-iotests/094
    M tests/qemu-iotests/095
    M tests/qemu-iotests/097
    M tests/qemu-iotests/098
    M tests/qemu-iotests/099
    M tests/qemu-iotests/101
    M tests/qemu-iotests/102
    M tests/qemu-iotests/103
    M tests/qemu-iotests/104
    M tests/qemu-iotests/105
    M tests/qemu-iotests/106
    M tests/qemu-iotests/107
    M tests/qemu-iotests/108
    M tests/qemu-iotests/109
    M tests/qemu-iotests/110
    M tests/qemu-iotests/111
    M tests/qemu-iotests/112
    M tests/qemu-iotests/113
    M tests/qemu-iotests/114
    M tests/qemu-iotests/115
    M tests/qemu-iotests/116
    M tests/qemu-iotests/117
    M tests/qemu-iotests/119
    M tests/qemu-iotests/120
    M tests/qemu-iotests/121
    M tests/qemu-iotests/122
    M tests/qemu-iotests/123
    M tests/qemu-iotests/125
    M tests/qemu-iotests/126
    M tests/qemu-iotests/127
    M tests/qemu-iotests/128
    M tests/qemu-iotests/130
    M tests/qemu-iotests/131
    M tests/qemu-iotests/133
    M tests/qemu-iotests/134
    M tests/qemu-iotests/135
    M tests/qemu-iotests/137
    M tests/qemu-iotests/138
    M tests/qemu-iotests/140
    M tests/qemu-iotests/141
    M tests/qemu-iotests/142
    M tests/qemu-iotests/143
    M tests/qemu-iotests/144
    M tests/qemu-iotests/145
    M tests/qemu-iotests/146
    M tests/qemu-iotests/150
    M tests/qemu-iotests/153
    M tests/qemu-iotests/154
    M tests/qemu-iotests/156
    M tests/qemu-iotests/157
    M tests/qemu-iotests/158
    M tests/qemu-iotests/159
    M tests/qemu-iotests/160
    M tests/qemu-iotests/162
    M tests/qemu-iotests/170
    M tests/qemu-iotests/171
    M tests/qemu-iotests/172
    M tests/qemu-iotests/173
    M tests/qemu-iotests/174
    M tests/qemu-iotests/175
    M tests/qemu-iotests/176
    M tests/qemu-iotests/177
    M tests/qemu-iotests/178
    M tests/qemu-iotests/179
    M tests/qemu-iotests/181
    M tests/qemu-iotests/182
    M tests/qemu-iotests/183
    M tests/qemu-iotests/184
    M tests/qemu-iotests/185
    M tests/qemu-iotests/186
    M tests/qemu-iotests/187
    M tests/qemu-iotests/188
    M tests/qemu-iotests/189
    M tests/qemu-iotests/190
    M tests/qemu-iotests/191
    M tests/qemu-iotests/192
    M tests/qemu-iotests/195
    M tests/qemu-iotests/197
    M tests/qemu-iotests/198
    M tests/qemu-iotests/200
    M tests/qemu-iotests/204
    M tests/qemu-iotests/214
    M tests/qemu-iotests/215
    M tests/qemu-iotests/221
    M tests/qemu-iotests/223
    M tests/qemu-iotests/225
    M tests/qemu-iotests/226
    M tests/qemu-iotests/227
    M tests/qemu-iotests/229
    M tests/qemu-iotests/231
    M tests/qemu-iotests/232
    A tests/qemu-iotests/233
    A tests/qemu-iotests/233.out
    M tests/qemu-iotests/check
    M tests/qemu-iotests/common.config
    A tests/qemu-iotests/common.nbd
    M tests/qemu-iotests/common.pattern
    M tests/qemu-iotests/common.qemu
    M tests/qemu-iotests/common.rc
    A tests/qemu-iotests/common.tls
    M tests/qemu-iotests/group

  Log Message:
  -----------
  Merge remote-tracking branch 'remotes/ericb/tags/pull-nbd-2018-11-19' into 
staging

nbd patches for 2018-11-19

Add iotest coverage for NBD connections using TLS, including
a couple of code fixes that it pointed out

- Mao Zhongyi: 0/3 Do some cleaning work in qemu-iotests
- Daniel P. Berrangé: io: return 0 for EOF in TLS session read after shutdown
- Daniel P. Berrangé: 0/6 Misc fixes to NBD
- Eric Blake: iotests: Drop use of bash keyword 'function'

# gpg: Signature made Mon 19 Nov 2018 17:43:32 GMT
# gpg:                using RSA key A7A16B4A2527436A
# gpg: Good signature from "Eric Blake <address@hidden>"
# gpg:                 aka "Eric Blake (Free Software Programmer) 
<address@hidden>"
# gpg:                 aka "[jpeg image of size 6874]"
# Primary key fingerprint: 71C2 CC22 B1C4 6029 27D2  F3AA A7A1 6B4A 2527 436A

* remotes/ericb/tags/pull-nbd-2018-11-19:
  iotests: Drop use of bash keyword 'function'
  iotests: Also test I/O over NBD TLS
  tests: exercise NBD server in TLS mode
  tests: add iotests helpers for dealing with TLS certificates
  tests: check if qemu-nbd is still alive before waiting
  tests: pull qemu-nbd iotest helpers into common.nbd file
  io: return 0 for EOF in TLS session read after shutdown
  nbd/server: Ignore write errors when replying to NBD_OPT_ABORT
  nbd: fix whitespace in server error message
  qemu-iotests: Modern shell scripting (use $() instead of ``)
  qemu-iotests: convert `pwd` and $(pwd) to $PWD
  qemu-iotests: remove unused variable 'here'

Signed-off-by: Peter Maydell <address@hidden>


Compare: https://github.com/qemu/qemu/compare/d304cf014bf9...393aac1629c8
      **NOTE:** This service has been marked for deprecation: 
https://developer.github.com/changes/2018-04-25-github-services-deprecation/

      Functionality will be removed from GitHub.com on January 31st, 2019.

reply via email to

[Prev in Thread] Current Thread [Next in Thread]