Skip to content

HDDS-15347. Move virtual-host test out of smoketest/s3#10334

Open
adoroszlai wants to merge 2 commits into
apache:masterfrom
adoroszlai:HDDS-15347
Open

HDDS-15347. Move virtual-host test out of smoketest/s3#10334
adoroszlai wants to merge 2 commits into
apache:masterfrom
adoroszlai:HDDS-15347

Conversation

@adoroszlai
Copy link
Copy Markdown
Contributor

What changes were proposed in this pull request?

smoketest/s3/awss3virtualhost.robot requires special configuration (ozone.s3g.domain.name), so it does not work with most acceptance test environments. When running all tests in smoketest/s3, this test currently needs to be excluded, e.g.:

## Exclude virtual-host tests. This is tested separately as it requires additional config.
exclude="--exclude virtual-host"

We can avoid the need for exclusion by moving it to another directory.

https://issues.apache.org/jira/browse/HDDS-15347

How was this patch tested?

s3-virtual-host :: S3 gateway test with aws cli using virtual host style ad...
==============================================================================
File upload and directory list with virtual style addressing          | PASS |
------------------------------------------------------------------------------
s3-virtual-host :: S3 gateway test with aws cli using virtual host... | PASS |
1 test, 1 passed, 0 failed

https://github.com/adoroszlai/ozone/actions/runs/26282681788/job/77364249255#step:13:1357

CI:
https://github.com/adoroszlai/ozone/actions/runs/26282681788

@adoroszlai adoroszlai self-assigned this May 22, 2026
@adoroszlai adoroszlai added test s3 S3 Gateway code-cleanup Changes that aim to make code better, without changing functionality. labels May 22, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

code-cleanup Changes that aim to make code better, without changing functionality. s3 S3 Gateway test

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant