Skip to content

Commit

Permalink
nvme: do not print subsystem NQN to stdout
Browse files Browse the repository at this point in the history
The subsystem NQN might be changed from the default value, but
that shouldn't cause the tests to fail. So don't register the
subsystem NQN in the 'out' files to avoid a false positive.

Signed-off-by: Hannes Reinecke <hare@suse.de>
[Shin'ichiro: remove only subsystem NQN from nvme disconnect message]
Signed-off-by: Shin'ichiro Kawasaki <shinichiro.kawasaki@wdc.com>
  • Loading branch information
hreinecke authored and kawasaki committed Nov 24, 2023
1 parent ce61317 commit 424eaa3
Show file tree
Hide file tree
Showing 24 changed files with 42 additions and 41 deletions.
2 changes: 1 addition & 1 deletion tests/nvme/003.out
Original file line number Diff line number Diff line change
@@ -1,3 +1,3 @@
Running nvme/003
NQN:nqn.2014-08.org.nvmexpress.discovery disconnected 1 controller(s)
disconnected 1 controller(s)
Test complete
2 changes: 1 addition & 1 deletion tests/nvme/004.out
Original file line number Diff line number Diff line change
@@ -1,3 +1,3 @@
Running nvme/004
NQN:blktests-subsystem-1 disconnected 1 controller(s)
disconnected 1 controller(s)
Test complete
2 changes: 1 addition & 1 deletion tests/nvme/008.out
Original file line number Diff line number Diff line change
@@ -1,3 +1,3 @@
Running nvme/008
NQN:blktests-subsystem-1 disconnected 1 controller(s)
disconnected 1 controller(s)
Test complete
2 changes: 1 addition & 1 deletion tests/nvme/009.out
Original file line number Diff line number Diff line change
@@ -1,3 +1,3 @@
Running nvme/009
NQN:blktests-subsystem-1 disconnected 1 controller(s)
disconnected 1 controller(s)
Test complete
2 changes: 1 addition & 1 deletion tests/nvme/010.out
Original file line number Diff line number Diff line change
@@ -1,3 +1,3 @@
Running nvme/010
NQN:blktests-subsystem-1 disconnected 1 controller(s)
disconnected 1 controller(s)
Test complete
2 changes: 1 addition & 1 deletion tests/nvme/011.out
Original file line number Diff line number Diff line change
@@ -1,3 +1,3 @@
Running nvme/011
NQN:blktests-subsystem-1 disconnected 1 controller(s)
disconnected 1 controller(s)
Test complete
2 changes: 1 addition & 1 deletion tests/nvme/012.out
Original file line number Diff line number Diff line change
@@ -1,3 +1,3 @@
Running nvme/012
NQN:blktests-subsystem-1 disconnected 1 controller(s)
disconnected 1 controller(s)
Test complete
2 changes: 1 addition & 1 deletion tests/nvme/013.out
Original file line number Diff line number Diff line change
@@ -1,3 +1,3 @@
Running nvme/013
NQN:blktests-subsystem-1 disconnected 1 controller(s)
disconnected 1 controller(s)
Test complete
2 changes: 1 addition & 1 deletion tests/nvme/014.out
Original file line number Diff line number Diff line change
@@ -1,4 +1,4 @@
Running nvme/014
NVMe Flush: success
NQN:blktests-subsystem-1 disconnected 1 controller(s)
disconnected 1 controller(s)
Test complete
2 changes: 1 addition & 1 deletion tests/nvme/015.out
Original file line number Diff line number Diff line change
@@ -1,4 +1,4 @@
Running nvme/015
NVMe Flush: success
NQN:blktests-subsystem-1 disconnected 1 controller(s)
disconnected 1 controller(s)
Test complete
2 changes: 1 addition & 1 deletion tests/nvme/018.out
Original file line number Diff line number Diff line change
@@ -1,3 +1,3 @@
Running nvme/018
NQN:blktests-subsystem-1 disconnected 1 controller(s)
disconnected 1 controller(s)
Test complete
2 changes: 1 addition & 1 deletion tests/nvme/019.out
Original file line number Diff line number Diff line change
@@ -1,4 +1,4 @@
Running nvme/019
NVMe DSM: success
NQN:blktests-subsystem-1 disconnected 1 controller(s)
disconnected 1 controller(s)
Test complete
2 changes: 1 addition & 1 deletion tests/nvme/020.out
Original file line number Diff line number Diff line change
@@ -1,4 +1,4 @@
Running nvme/020
NVMe DSM: success
NQN:blktests-subsystem-1 disconnected 1 controller(s)
disconnected 1 controller(s)
Test complete
2 changes: 1 addition & 1 deletion tests/nvme/033.out
Original file line number Diff line number Diff line change
@@ -1,3 +1,3 @@
Running nvme/033
NQN:blktests-subsystem-1 disconnected 1 controller(s)
disconnected 1 controller(s)
Test complete
2 changes: 1 addition & 1 deletion tests/nvme/034.out
Original file line number Diff line number Diff line change
@@ -1,3 +1,3 @@
Running nvme/034
NQN:blktests-subsystem-1 disconnected 1 controller(s)
disconnected 1 controller(s)
Test complete
2 changes: 1 addition & 1 deletion tests/nvme/035.out
Original file line number Diff line number Diff line change
@@ -1,3 +1,3 @@
Running nvme/035
NQN:blktests-subsystem-1 disconnected 1 controller(s)
disconnected 1 controller(s)
Test complete
2 changes: 1 addition & 1 deletion tests/nvme/036.out
Original file line number Diff line number Diff line change
@@ -1,3 +1,3 @@
Running nvme/036
NQN:blktests-subsystem-1 disconnected 1 controller(s)
disconnected 1 controller(s)
Test complete
4 changes: 2 additions & 2 deletions tests/nvme/041.out
Original file line number Diff line number Diff line change
@@ -1,6 +1,6 @@
Running nvme/041
Test unauthenticated connection (should fail)
NQN:blktests-subsystem-1 disconnected 0 controller(s)
disconnected 0 controller(s)
Test authenticated connection
NQN:blktests-subsystem-1 disconnected 1 controller(s)
disconnected 1 controller(s)
Test complete
14 changes: 7 additions & 7 deletions tests/nvme/042.out
Original file line number Diff line number Diff line change
@@ -1,16 +1,16 @@
Running nvme/042
Testing hmac 0
NQN:blktests-subsystem-1 disconnected 1 controller(s)
disconnected 1 controller(s)
Testing hmac 1
NQN:blktests-subsystem-1 disconnected 1 controller(s)
disconnected 1 controller(s)
Testing hmac 2
NQN:blktests-subsystem-1 disconnected 1 controller(s)
disconnected 1 controller(s)
Testing hmac 3
NQN:blktests-subsystem-1 disconnected 1 controller(s)
disconnected 1 controller(s)
Testing key length 32
NQN:blktests-subsystem-1 disconnected 1 controller(s)
disconnected 1 controller(s)
Testing key length 48
NQN:blktests-subsystem-1 disconnected 1 controller(s)
disconnected 1 controller(s)
Testing key length 64
NQN:blktests-subsystem-1 disconnected 1 controller(s)
disconnected 1 controller(s)
Test complete
16 changes: 8 additions & 8 deletions tests/nvme/043.out
Original file line number Diff line number Diff line change
@@ -1,18 +1,18 @@
Running nvme/043
Testing hash hmac(sha256)
NQN:blktests-subsystem-1 disconnected 1 controller(s)
disconnected 1 controller(s)
Testing hash hmac(sha384)
NQN:blktests-subsystem-1 disconnected 1 controller(s)
disconnected 1 controller(s)
Testing hash hmac(sha512)
NQN:blktests-subsystem-1 disconnected 1 controller(s)
disconnected 1 controller(s)
Testing DH group ffdhe2048
NQN:blktests-subsystem-1 disconnected 1 controller(s)
disconnected 1 controller(s)
Testing DH group ffdhe3072
NQN:blktests-subsystem-1 disconnected 1 controller(s)
disconnected 1 controller(s)
Testing DH group ffdhe4096
NQN:blktests-subsystem-1 disconnected 1 controller(s)
disconnected 1 controller(s)
Testing DH group ffdhe6144
NQN:blktests-subsystem-1 disconnected 1 controller(s)
disconnected 1 controller(s)
Testing DH group ffdhe8192
NQN:blktests-subsystem-1 disconnected 1 controller(s)
disconnected 1 controller(s)
Test complete
8 changes: 4 additions & 4 deletions tests/nvme/044.out
Original file line number Diff line number Diff line change
@@ -1,10 +1,10 @@
Running nvme/044
Test host authentication
NQN:blktests-subsystem-1 disconnected 1 controller(s)
disconnected 1 controller(s)
Test invalid ctrl authentication (should fail)
NQN:blktests-subsystem-1 disconnected 0 controller(s)
disconnected 0 controller(s)
Test valid ctrl authentication
NQN:blktests-subsystem-1 disconnected 1 controller(s)
disconnected 1 controller(s)
Test invalid ctrl key (should fail)
NQN:blktests-subsystem-1 disconnected 0 controller(s)
disconnected 0 controller(s)
Test complete
2 changes: 1 addition & 1 deletion tests/nvme/045.out
Original file line number Diff line number Diff line change
Expand Up @@ -8,5 +8,5 @@ Change DH group to ffdhe8192
Re-authenticate with changed DH group
Change hash to hmac(sha512)
Re-authenticate with changed hash
NQN:blktests-subsystem-1 disconnected 1 controller(s)
disconnected 1 controller(s)
Test complete
2 changes: 1 addition & 1 deletion tests/nvme/048.out
Original file line number Diff line number Diff line change
@@ -1,3 +1,3 @@
Running nvme/048
NQN:blktests-subsystem-1 disconnected 1 controller(s)
disconnected 1 controller(s)
Test complete
3 changes: 2 additions & 1 deletion tests/nvme/rc
Original file line number Diff line number Diff line change
Expand Up @@ -430,7 +430,8 @@ _nvme_disconnect_ctrl() {
_nvme_disconnect_subsys() {
local subsysnqn="$1"

nvme disconnect -n "${subsysnqn}"
nvme disconnect -n "${subsysnqn}" |& tee -a "$FULL" |
grep -o "disconnected.*"
}

_nvme_connect_subsys() {
Expand Down

0 comments on commit 424eaa3

Please sign in to comment.