| 8425f00c | 03-Jul-2024 |
Konrad Sztyber <konrad.sztyber@intel.com> |
nvme/tcp: respect the src_{addr,svcid} parameters
Previously, both were completely ignored.
Signed-off-by: Konrad Sztyber <konrad.sztyber@intel.com>
Change-Id: I9df0b50038742956e2be0b80ad239e80ac4d
nvme/tcp: respect the src_{addr,svcid} parameters
Previously, both were completely ignored.
Signed-off-by: Konrad Sztyber <konrad.sztyber@intel.com>
Change-Id: I9df0b50038742956e2be0b80ad239e80ac4dcba6
Reviewed-on: https://review.spdk.io/gerrit/c/spdk/spdk/+/24070
Reviewed-by: Tomasz Zawadzki <tomasz@tzawadzki.com>
Community-CI: Mellanox Build Bot
Tested-by: SPDK CI Jenkins <sys_sgci@intel.com>
Reviewed-by: Jim Harris <jim.harris@samsung.com>
show more ...
|
| b24df7cf | 09-May-2024 |
Michal Berger <michal.berger@intel.com> |
test: Drop superfluous calls to print_backtrace()
Since we use errtrace, the ERR trap is inherited and preserved in
each subshell, cmd substitution, etc. that we execute along the
way. This means th
test: Drop superfluous calls to print_backtrace()
Since we use errtrace, the ERR trap is inherited and preserved in
each subshell, cmd substitution, etc. that we execute along the
way. This means that each failure inside such subshell triggers
call to print_backtrace(). Depending on the context, this may not
be desired. Consider simple example:
# foo() { return 1; }
# [[ $(foo) == no_output_so_fail_me ]]
This assumes that from the test perspective we don't care if foo(),
internally, fails, we just look for a specific string it should
return. If it doesn't, errexit will be triggered at the [[ level
and fail the test. This is fine, but before this happens, extra
print_backtrace() call will be made from the $() context.
Changing logic above and we may start spamming these calls:
# foo() { return 1; }
# [[ $(foo) -eq 0 ]]
Here failure still triggers the print_backtrace() but logically
the [[ test succeeds so we end up with a useless call in the log
which may be downright confusing.
Cherry on top is the fact that we don't inherit errexit inside
the subshell. So this print_backtrace() call is entirely foobar
as it won't do anything at all.
Best case would be to enable inherit_errexit from autotest_common.sh
across all the test suites. This, however, will require proper
discipline in maintaining the tests as use of subshell environment
would have to be more explicit - in other words the ambiguity from
the first example (i.e. failure != no output) would not be allowed.
For now, refactor code a bit to get rid of these extra calls, next
step would be to test behavior of inherit_errexit.
Change-Id: Ice3f072008162c1713e78a6da99a833919bc716c
Signed-off-by: Michal Berger <michal.berger@intel.com>
Reviewed-on: https://review.spdk.io/gerrit/c/spdk/spdk/+/23078
Reviewed-by: Ben Walker <ben@nvidia.com>
Reviewed-by: Jim Harris <jim.harris@samsung.com>
Tested-by: SPDK CI Jenkins <sys_sgci@intel.com>
show more ...
|
| 5b4e69ae | 08-May-2024 |
Ankit Kumar <ankit.kumar@samsung.com> |
test/nvmf: add new tests for active-passive
Add 2 new tests for active/passive mode
1. For optimized optimized state verify that only first path is current.
2. For non-optimized non-optimized state
test/nvmf: add new tests for active-passive
Add 2 new tests for active/passive mode
1. For optimized optimized state verify that only first path is current.
2. For non-optimized non-optimized state verify that the first path is
current.
Change-Id: I95e67cfc44a6b7bc7790c85b5fd619049e541294
Signed-off-by: Ankit Kumar <ankit.kumar@samsung.com>
Reviewed-on: https://review.spdk.io/gerrit/c/spdk/spdk/+/23050
Reviewed-by: Shuhei Matsumoto <smatsumoto@nvidia.com>
Tested-by: SPDK CI Jenkins <sys_sgci@intel.com>
Reviewed-by: Jim Harris <jim.harris@samsung.com>
show more ...
|
| 3e4c5347 | 04-Mar-2024 |
Konrad Sztyber <konrad.sztyber@intel.com> |
bdev/nvme: pass DH-HMAC-CHAP controller key
Additionally, adjust the authentication tests to cover cases with
bidirectional authentication.
Signed-off-by: Konrad Sztyber <konrad.sztyber@intel.com>
bdev/nvme: pass DH-HMAC-CHAP controller key
Additionally, adjust the authentication tests to cover cases with
bidirectional authentication.
Signed-off-by: Konrad Sztyber <konrad.sztyber@intel.com>
Change-Id: I251260ac7e53ae0cf39da08941c669ee3ac2c58a
Reviewed-on: https://review.spdk.io/gerrit/c/spdk/spdk/+/22129
Reviewed-by: Jim Harris <jim.harris@samsung.com>
Reviewed-by: Ben Walker <ben@nvidia.com>
Tested-by: SPDK CI Jenkins <sys_sgci@intel.com>
Community-CI: Mellanox Build Bot
show more ...
|
| d34d8172 | 09-Apr-2024 |
Michal Berger <michal.berger@intel.com> |
test/nvmf: Hook get_notification_count() into waitforcondition()
Just to make sure notifications check is not racy.
Change-Id: I35b620a53f5792ded713fab9cb4e09cd620bbbeb
Signed-off-by: Michal Berger
test/nvmf: Hook get_notification_count() into waitforcondition()
Just to make sure notifications check is not racy.
Change-Id: I35b620a53f5792ded713fab9cb4e09cd620bbbeb
Signed-off-by: Michal Berger <michal.berger@intel.com>
Reviewed-on: https://review.spdk.io/gerrit/c/spdk/spdk/+/22694
Tested-by: SPDK CI Jenkins <sys_sgci@intel.com>
Reviewed-by: Konrad Sztyber <konrad.sztyber@intel.com>
Reviewed-by: Tomasz Zawadzki <tomasz.zawadzki@intel.com>
show more ...
|