Skip to content

HDDS-14975. Move awssdk version definition to ozone-integration-test-s3#10042

Merged
adoroszlai merged 1 commit intoapache:masterfrom
adoroszlai:HDDS-14975
Apr 4, 2026
Merged

HDDS-14975. Move awssdk version definition to ozone-integration-test-s3#10042
adoroszlai merged 1 commit intoapache:masterfrom
adoroszlai:HDDS-14975

Conversation

@adoroszlai
Copy link
Copy Markdown
Contributor

What changes were proposed in this pull request?

awssdk (v2) is only used in ozone-integration-test-s3. Move it to that module to reduce CI for weekly version upgrades.

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

How was this patch tested?

https://github.com/adoroszlai/ozone/actions/runs/23978406293

@adoroszlai adoroszlai self-assigned this Apr 4, 2026
@adoroszlai adoroszlai added the build Pull request that modifies the build process label Apr 4, 2026
@adoroszlai adoroszlai requested a review from ivandika3 April 4, 2026 14:33
Copy link
Copy Markdown
Contributor

@Russole Russole left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thanks @adoroszlai for working on this. The changes look good, LGTM!

Copy link
Copy Markdown
Contributor

@ivandika3 ivandika3 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thanks for the improvement. LGTM +1.

Just for my understanding, this will make it such that the AWS S3 SDK upgrade only need to run the ozone-integration-test-s3 suites?

@adoroszlai
Copy link
Copy Markdown
Contributor Author

will make it such that the AWS S3 SDK upgrade only need to run the ozone-integration-test-s3 suites?

Not quite, but it will allow skipping acceptance and kubernetes checks.

@ivandika3
Copy link
Copy Markdown
Contributor

Ok thanks for the explanation. Sounds good.

@adoroszlai adoroszlai merged commit 93bf1d9 into apache:master Apr 4, 2026
44 checks passed
@adoroszlai adoroszlai deleted the HDDS-14975 branch April 4, 2026 17:11
@adoroszlai
Copy link
Copy Markdown
Contributor Author

Thanks @ivandika3, @Russole for the review.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

build Pull request that modifies the build process

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants