[PATCH lkp-tests] daemon/multi-qemu: add a default value for the queues field
[why] For multi-qemu.yaml, if the queues field is not set, a default value is required. [how] If the multi-qemu.yaml not set queues field, there are two queues by default: "$tbox_group.$HOSTNAME, $tbox_group.$(arch)". Signed-off-by: Zhang Yu <2134782174@qq.com> --- daemon/multi-qemu | 1 + 1 file changed, 1 insertion(+) diff --git a/daemon/multi-qemu b/daemon/multi-qemu index 8d9dd38d..7cc625f4 100755 --- a/daemon/multi-qemu +++ b/daemon/multi-qemu @@ -6,6 +6,7 @@ # SPDX-License-Identifier: MulanPSL-2.0+ # Copyright (c) 2020 Huawei Technologies Co., Ltd. All rights reserved. +: ${queues:="$tbox_group.$HOSTNAME, $tbox_group.$(arch)"} multi_qemu() { export CCI_SRC=/c/compass-ci -- 2.23.0
On Thu, Nov 19, 2020 at 05:11:10PM +0800, Zhang Yu wrote:
[why] For multi-qemu.yaml, if the queues field is not set, a default value is required.
[how] If the multi-qemu.yaml not set queues field, there are two queues by default: "$tbox_group.$HOSTNAME, $tbox_group.$(arch)".
Looks good to me. Thanks, Yale
Signed-off-by: Zhang Yu <2134782174@qq.com> --- daemon/multi-qemu | 1 + 1 file changed, 1 insertion(+)
diff --git a/daemon/multi-qemu b/daemon/multi-qemu index 8d9dd38d..7cc625f4 100755 --- a/daemon/multi-qemu +++ b/daemon/multi-qemu @@ -6,6 +6,7 @@ # SPDX-License-Identifier: MulanPSL-2.0+ # Copyright (c) 2020 Huawei Technologies Co., Ltd. All rights reserved.
+: ${queues:="$tbox_group.$HOSTNAME, $tbox_group.$(arch)"} multi_qemu() { export CCI_SRC=/c/compass-ci -- 2.23.0
On Thu, Nov 19, 2020 at 05:11:10PM +0800, Zhang Yu wrote:
[why] For multi-qemu.yaml, if the queues field is not set, a default value is required.
[how] If the multi-qemu.yaml not set queues field, there are two queues by default: "$tbox_group.$HOSTNAME, $tbox_group.$(arch)".
Signed-off-by: Zhang Yu <2134782174@qq.com> --- daemon/multi-qemu | 1 + 1 file changed, 1 insertion(+)
diff --git a/daemon/multi-qemu b/daemon/multi-qemu index 8d9dd38d..7cc625f4 100755 --- a/daemon/multi-qemu +++ b/daemon/multi-qemu @@ -6,6 +6,7 @@ # SPDX-License-Identifier: MulanPSL-2.0+ # Copyright (c) 2020 Huawei Technologies Co., Ltd. All rights reserved.
+: ${queues:="$tbox_group.$HOSTNAME, $tbox_group.$(arch)"}
^^^ May be no need space after comma. Thanks, Baijing
multi_qemu() { export CCI_SRC=/c/compass-ci -- 2.23.0
On Fri, Nov 20, 2020 at 02:39:48PM +0800, Bai Jing wrote:
On Thu, Nov 19, 2020 at 05:11:10PM +0800, Zhang Yu wrote:
[why] For multi-qemu.yaml, if the queues field is not set, a default value is required.
[how] If the multi-qemu.yaml not set queues field, there are two queues by default: "$tbox_group.$HOSTNAME, $tbox_group.$(arch)".
Signed-off-by: Zhang Yu <2134782174@qq.com> --- daemon/multi-qemu | 1 + 1 file changed, 1 insertion(+)
diff --git a/daemon/multi-qemu b/daemon/multi-qemu index 8d9dd38d..7cc625f4 100755 --- a/daemon/multi-qemu +++ b/daemon/multi-qemu @@ -6,6 +6,7 @@ # SPDX-License-Identifier: MulanPSL-2.0+ # Copyright (c) 2020 Huawei Technologies Co., Ltd. All rights reserved.
+: ${queues:="$tbox_group.$HOSTNAME, $tbox_group.$(arch)"}
^^^ May be no need space after comma.
Good! Thanks, Zhangyu
Thanks, Baijing
multi_qemu() { export CCI_SRC=/c/compass-ci -- 2.23.0
participants (3)
-
Bai Jing -
Xu Xijian -
Zhang Yu