Hi Yi,
On 28/06/2023 12:21, Max Gurtovoy wrote:
On 28/06/2023 11:14, Sagi Grimberg wrote:
Yi,
Do you have hostnqn and hostid files in your /etc/nvme directory?
No, only one discovery.conf there.
# ls /etc/nvme/
discovery.conf
So the hostid is generated every time if it is not passed.
We should probably revert the patch and add it back when
blktests are passing.
Seems like the patch is doing exactly what it should do - fix wrong
behavior of users that override hostid.
Can we fix the tests instead ?
Right, I got confused between a provided host and the default host...
I think we need to add check that /etc/nvme/[hostnqn,hostid] exist
in the test cases.
Right.
And if one of the files doesn't exist, generate the value.
Should it go to tests/nvme/rc ?
Can you please try adding the bellow un-tested code to blktests and re-run:
[root@r-arch-stor03 blktests]# git diff
diff --git a/tests/nvme/rc b/tests/nvme/rc
index 191f3e2..88e6fa1 100644
--- a/tests/nvme/rc
+++ b/tests/nvme/rc
@@ -14,8 +14,23 @@ def_remote_wwnn="0x10001100aa000001"
def_remote_wwpn="0x20001100aa000001"
def_local_wwnn="0x10001100aa000002"
def_local_wwpn="0x20001100aa000002"
-def_hostnqn="$(cat /etc/nvme/hostnqn 2> /dev/null)"
-def_hostid="$(cat /etc/nvme/hostid 2> /dev/null)"
+
+if [ -f "/etc/nvme/hostid" ]; then
+ def_hostid="$(cat /etc/nvme/hostid 2> /dev/null)"
+else
+ def_hostid="$(uuidgen)"
+fi
+if [ -z "$def_hostid" ] ; then
+ def_hostid="0f01fb42-9f7f-4856-b0b3-51e60b8de349"
+fi
+
+if [ -f "/etc/nvme/hostnqn" ]; then
+ def_hostnqn="$(cat /etc/nvme/hostnqn 2> /dev/null)"
+fi
+if [ -z "$def_hostnqn" ] ; then
+ def_hostnqn="nqn.2014-08.org.nvmexpress:uuid:${def_hostid}"
+fi
+
nvme_trtype=${nvme_trtype:-"loop"}
nvme_img_size=${nvme_img_size:-"1G"}
nvme_num_iter=${nvme_num_iter:-"1000"}
