This is an automated email from the ASF dual-hosted git repository.
dataroaring pushed a commit to branch branch-3.0
in repository https://gitbox.apache.org/repos/asf/doris.git
The following commit(s) were added to refs/heads/branch-3.0 by this push:
new c02d8d72dbb branch-3.0: [fix](regression) Set `be-cluster-id` default
to false #53643 (#53813)
c02d8d72dbb is described below
commit c02d8d72dbba966fca71239485f005ecbae83e92
Author: github-actions[bot]
<41898282+github-actions[bot]@users.noreply.github.com>
AuthorDate: Tue Aug 12 11:04:08 2025 +0800
branch-3.0: [fix](regression) Set `be-cluster-id` default to false #53643
(#53813)
Cherry-picked from #53643
Co-authored-by: deardeng <[email protected]>
---
docker/runtime/doris-compose/command.py | 20 +++++++-------------
.../doris/regression/suite/SuiteCluster.groovy | 4 ++--
2 files changed, 9 insertions(+), 15 deletions(-)
diff --git a/docker/runtime/doris-compose/command.py
b/docker/runtime/doris-compose/command.py
index 57f302dcc45..84febcf415f 100644
--- a/docker/runtime/doris-compose/command.py
+++ b/docker/runtime/doris-compose/command.py
@@ -480,19 +480,12 @@ class UpCommand(Command):
"Do not set BE meta service endpoint in conf. Default is
False."
)
- if self._support_boolean_action():
- parser.add_argument(
- "--be-cluster-id",
- default=False,
- action=self._get_parser_bool_action(False),
- help="Do not set BE cluster ID in conf. Default is False.")
- else:
- parser.add_argument(
- "--no-be-cluster-id",
- dest='be_cluster_id',
- default=False,
- action=self._get_parser_bool_action(False),
- help="Do not set BE cluster ID in conf. Default is False.")
+ # if default==False, use this style to parser, like --be-cluster-id
+ parser.add_argument(
+ "--be-cluster-id",
+ default=False,
+ action=self._get_parser_bool_action(True),
+ help="Do not set BE cluster ID in conf. Default is False.")
parser.add_argument(
"--fdb-version",
@@ -500,6 +493,7 @@ class UpCommand(Command):
default="7.1.26",
help="fdb image version. Only use in cloud cluster.")
+ # if default==True, use this style to parser, like --detach
if self._support_boolean_action():
parser.add_argument(
"--detach",
diff --git
a/regression-test/framework/src/main/groovy/org/apache/doris/regression/suite/SuiteCluster.groovy
b/regression-test/framework/src/main/groovy/org/apache/doris/regression/suite/SuiteCluster.groovy
index 61758f9c5dd..aa7d3e7afe7 100644
---
a/regression-test/framework/src/main/groovy/org/apache/doris/regression/suite/SuiteCluster.groovy
+++
b/regression-test/framework/src/main/groovy/org/apache/doris/regression/suite/SuiteCluster.groovy
@@ -355,8 +355,8 @@ class SuiteCluster {
if (!options.beMetaServiceEndpoint) {
cmd += ['--no-be-metaservice-endpoint']
}
- if (!options.beClusterId) {
- cmd += ['--no-be-cluster-id']
+ if (options.beClusterId) {
+ cmd += ['--be-cluster-id']
}
cmd += ['--wait-timeout', String.valueOf(options.waitTimeout)]
---------------------------------------------------------------------
To unsubscribe, e-mail: [email protected]
For additional commands, e-mail: [email protected]
