Skip to content

Conversation

@xichen01
Copy link
Contributor

@xichen01 xichen01 commented Oct 22, 2024

What changes were proposed in this pull request?

Backport the following commit from master to ozone-1.4 for 1.4.1. Including some bug fixes and Ratis upgrade and necessary dependencies.

HDDS-11333. Avoid hard-coded current version in upgrade/xcompat tests (#7089)
HDDS-10168. Add Ozone 1.4.0 to compatibility acceptance tests (#6040)
HDDS-11414. Key listing for FSO buckets fails with forward client (#7161)
HDDS-11570. Fix HDDS Docs build failure with Hugo v0.135.0 (#7337)
HDDS-11498. Improve SCM deletion efficiency. (#7249)
HDDS-11482. EC Checksum throws IllegalArgumentException because the buffer limit is negative (#7230)
HDDS-10465. Change ozone.client.bytes.per.checksum default to 16KB (#6331)
HDDS-10480. Avoid proto2 ByteString.toByteArray() calls. (#6342)
HDDS-11536. Bump macOS runner version to macos-13 (#7279)
HDDS-11504. Update Ratis to 3.1.1. (#7257)
HDDS-11472. Avoid recreating external access authorizer on OM state reload (#7238)
HDDS-11436. Minor update in Recon API handling. (#7178)
HDDS-11223. Fix iteration over ChunkBufferImplWithByteBufferList (#6999)
HDDS-11136. Some containers affected by HDDS-8129 may still be in the DELETING state incorrectly (#6967)
HDDS-10918. NPE in OM when leader transfers (#6735)
HDDS-10777. S3 Gateway error when parsing XML concurrently (#6609)
HDDS-10780. NullPointerException in watchForCommit (#6627)

How was this patch tested?

https://github.com/xichen01/ozone/actions/runs/11445199314

duongkame and others added 15 commits October 16, 2024 17:19
… DELETING state incorrectly (apache#6967)

(cherry picked from commit 69ba680)
@xichen01 xichen01 changed the base branch from master to ozone-1.4 October 22, 2024 06:36
@xichen01
Copy link
Contributor Author

For https://issues.apache.org/jira/browse/HDDS-11414, ListStatusLight is disabled in ozone-1.4 due to compatibility issues discussed in #7326
cc: @adoroszlai @sumitagrawl @tanvipenumudy

@xichen01
Copy link
Contributor Author

@duongkame, @guohao-rosicky @ChenSammi @siddhantsangwan @Cyrill @ArafatKhan2198 @devabhishekpal @jojochuang @adoroszlai @szetszwo @aswinshakil @slfan1989 @scolley31 @tanvipenumudy
please check that your commits are correctly cherry-picked.

Copy link
Contributor

@devabhishekpal devabhishekpal 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 effort @xichen01 .
My changes look good. +1

Copy link
Contributor

@ArafatKhan2198 ArafatKhan2198 left a comment

Choose a reason for hiding this comment

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

My change look good +1

Copy link
Contributor

@tanvipenumudy tanvipenumudy left a comment

Choose a reason for hiding this comment

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

For https://issues.apache.org/jira/browse/HDDS-11414, ListStatusLight is disabled in ozone-1.4 due to compatibility issues discussed in #7326

Thank you @xichen01, the changes LGTM +1

@adoroszlai
Copy link
Contributor

@xichen01 can you please cherry-pick the following two commits (in this order)?

@xichen01
Copy link
Contributor Author

@xichen01 can you please cherry-pick the following two commits (in this order)?

Done.

@xichen01
Copy link
Contributor Author

https://issues.apache.org/jira/browse/HDDS-11368
@devabhishekpal @szetszwo @adoroszlai Is this need to be backport? ASF's policy on obtaining MIT licenses appears to be inconclusive.

@adoroszlai adoroszlai merged commit b28b416 into apache:ozone-1.4 Oct 25, 2024
@adoroszlai
Copy link
Contributor

https://issues.apache.org/jira/browse/HDDS-11368
Is this need to be backport?

Let's try to include in next batch.

@adoroszlai
Copy link
Contributor

Thanks @xichen01 for the backports, @ArafatKhan2198, @devabhishekpal, @tanvipenumudy for the review.

@xichen01 xichen01 deleted the ozone-1.4-backport branch October 26, 2024 15:16
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.