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
@adoroszlai
Copy link
Copy Markdown
Contributor Author

@SaketaChalamchala please review

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