Skip to content

HDDS-17501. OzoneContainer#stopContainerScrub do not stop all scanners - #11807

Merged
ivandika3 merged 1 commit into
apache:masterfrom
adoroszlai:HDDS-17501
Oct 28, 2026
Merged

ivandika3 merged 1 commit into
apache:masterfrom
adoroszlai:HDDS-17501

Conversation

@adoroszlai

Copy link
Copy Markdown
Contributor

What changes were proposed in this pull request?

Fix intermittent metrics leak in ContainerScannerIntegrationTests.

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

How was this patch tested?

10x10 passed:
https://github.com/adoroszlai/ozone/actions/runs/40585298431

CI (in progress):
https://github.com/adoroszlai/ozone/actions/runs/40587464213

@adoroszlai adoroszlai self-assigned this Oct 27, 2026

@ss77892 ss77892 left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

LGTM

@ivandika3 ivandika3 left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Thanks @adoroszlai for the patch and @Russole for finding out the issue. LGTM +1.

@ivandika3
ivandika3 marked this pull request as ready for review October 28, 2026 03:46

@smengcl smengcl left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Thanks @adoroszlai for posting this.

This has been causing a lot of CI flakiness lately.

@ivandika3
ivandika3 merged commit 2b26179 into apache:master Oct 28, 2026
73 checks passed
@ivandika3

Copy link
Copy Markdown
Contributor

Thanks @adoroszlai for the patch and @ss77892 @smengcl for the reviews.

@adoroszlai
adoroszlai deleted the HDDS-17501 branch October 28, 2026 05:38
@adoroszlai

Copy link
Copy Markdown
Contributor Author

Thanks @Russole for reporting the issue, @ivandika3 @smengcl @ss77892 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.

4 participants