This is an automated email from the ASF dual-hosted git repository.
liaoxin pushed a commit to branch master
in repository https://gitbox.apache.org/repos/asf/doris.git
The following commit(s) were added to refs/heads/master by this push:
new e84d9b5a200 [fix](regression) Fix case
test_domain_connection_and_ak_sk_correction (#58572)
e84d9b5a200 is described below
commit e84d9b5a200ca8dba37b89889426e26ee7d15178
Author: Refrain <[email protected]>
AuthorDate: Tue Dec 2 22:53:06 2025 +0800
[fix](regression) Fix case test_domain_connection_and_ak_sk_correction
(#58572)
Related PR: #https://github.com/apache/doris/pull/56579
Problem Summary:
cuz we add the check : `Pattern STANDARD_ENDPOINT_PATTERN = Pattern
.compile("^(?:https?://)?(?:s3\\.)?oss-([a-z0-9-]+?)(?:-internal)?\\.aliyuncs\\.com$");`
the case will be invalid
---
.../broker_load/test_domain_connection_and_ak_sk_correction.groovy | 7 ++++---
1 file changed, 4 insertions(+), 3 deletions(-)
diff --git
a/regression-test/suites/load_p0/broker_load/test_domain_connection_and_ak_sk_correction.groovy
b/regression-test/suites/load_p0/broker_load/test_domain_connection_and_ak_sk_correction.groovy
index 786baaf2681..7ec6e8c4956 100644
---
a/regression-test/suites/load_p0/broker_load/test_domain_connection_and_ak_sk_correction.groovy
+++
b/regression-test/suites/load_p0/broker_load/test_domain_connection_and_ak_sk_correction.groovy
@@ -80,6 +80,7 @@ suite("test_domain_connection_and_ak_sk_correction",
"load_p0") {
logger.info("the first sql result is {}", result)
label = UUID.randomUUID().toString().replace("-", "")
+ def errorEndpoint = "oss-cn-hong123kong.aliyuncs.com"
try {
result = sql """
LOAD LABEL ${label}
@@ -91,7 +92,7 @@ suite("test_domain_connection_and_ak_sk_correction",
"load_p0") {
)
WITH S3
(
- "AWS_ENDPOINT" = "${getS3Endpoint()}1",
+ "AWS_ENDPOINT" = "${errorEndpoint}",
"AWS_ACCESS_KEY" = "${getS3AK()}",
"AWS_SECRET_KEY" = "${getS3SK()}",
"AWS_REGION" = "${getS3Region()}",
@@ -99,10 +100,10 @@ suite("test_domain_connection_and_ak_sk_correction",
"load_p0") {
);
"""
logger.info("the second sql result is {}", result)
- assertTrue(false. "The endpoint is wrong, so the connection test
should fale")
+ assertTrue(false. "The endpoint is wrong, so the connection test
should fail")
} catch (Exception e) {
logger.info("the second sql exception result is {}", e.getMessage())
- assertTrue(e.getMessage().contains("Failed to access object storage"),
e.getMessage())
+ assertTrue(e.getMessage().contains("${errorEndpoint}"), e.getMessage())
}
sql """ DROP TABLE IF EXISTS ${tableName} FORCE"""
---------------------------------------------------------------------
To unsubscribe, e-mail: [email protected]
For additional commands, e-mail: [email protected]
