Skip to content

Conversation

@adoroszlai
Copy link
Contributor

What changes were proposed in this pull request?

Backport the following fixes from master to ozone-1.4 for 1.4.1.

If approved, I'll push to the ozone-1.4 branch directly.

How was this patch tested?

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

vtutrinov and others added 17 commits March 5, 2024 10:55
@adoroszlai adoroszlai self-assigned this Mar 5, 2024
@adoroszlai
Copy link
Contributor Author

/pending DO NOT MERGE

Copy link

@github-actions github-actions bot left a comment

Choose a reason for hiding this comment

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

Marking this issue as un-mergeable as requested.

Please use /ready comment when it's resolved.

Please note that the PR will be closed after 21 days of inactivity from now. (But can be re-opened anytime later...)

DO NOT MERGE

@adoroszlai
Copy link
Contributor Author

@ashishkumar50 @ArafatKhan2198 @aswinshakil @ChenSammi @duongkame @ivandika3 @SaketaChalamchala @Tejaskriya @vtutrinov please check your own commit(s) being backported here

Copy link
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.

LGTM +1 for my patch.

Copy link
Contributor

@ashishkumar50 ashishkumar50 left a comment

Choose a reason for hiding this comment

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

@adoroszlai Thanks for this, LGTM for my change.

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.

Thanks @adoroszlai LGTM !

@vtutrinov
Copy link
Contributor

Thanks @adoroszlai , LGTM, +1

@Tejaskriya
Copy link
Contributor

LGTM +1 for my patch. Thank you @adoroszlai

@adoroszlai adoroszlai merged commit 196462a into apache:ozone-1.4 Mar 11, 2024
@adoroszlai
Copy link
Contributor Author

Thanks @ashishkumar50, @ArafatKhan2198, @ivandika3, @Tejaskriya, @vtutrinov for the review.

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.