Skip to content

HDDS-15827. Wait for SCM pipelines in TestDnRatisLogParser#10725

Merged
smengcl merged 1 commit into
apache:masterfrom
jojochuang:HDDS-15827-fix-dn-ratis-log-parser-test
Jul 11, 2026
Merged

HDDS-15827. Wait for SCM pipelines in TestDnRatisLogParser#10725
smengcl merged 1 commit into
apache:masterfrom
jojochuang:HDDS-15827-fix-dn-ratis-log-parser-test

Conversation

@jojochuang

@jojochuang jojochuang commented Jul 10, 2026

Copy link
Copy Markdown
Contributor

Summary

Jira

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

Problem

testRatisLogParsing uses a single-datanode mini-cluster and immediately indexes the first SCM pipeline. In CI, the pipeline list can still be empty when the test runs, even though the cluster is marked ready.

Test plan

  • checkstyle on ozone-integration-test module
  • mvn -pl :ozone-integration-test test -Dtest=TestDnRatisLogParser -DskipShade -DskipRecon -DskipDocs

Made with Cursor

Co-authored-by: Cursor <cursoragent@cursor.com>
Change-Id: I5849809e940c04cb6432a552f96ababa8db58251
Copilot AI review requested due to automatic review settings July 10, 2026 22:22

Copilot AI 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.

Pull request overview

This PR fixes an intermittent failure in the TestDnRatisLogParser integration test by waiting for SCM to report at least one pipeline before indexing getPipelines().get(0), preventing IndexOutOfBoundsException when pipeline creation lags behind cluster readiness.

Changes:

  • Add a GenericTestUtils.waitFor(...) guard to ensure the SCM pipeline list is non-empty before accessing the first pipeline ID.

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

@smengcl smengcl merged commit 4575cf9 into apache:master Jul 11, 2026
59 of 62 checks passed
@smengcl

smengcl commented Jul 11, 2026

Copy link
Copy Markdown
Contributor

Thanks @jojochuang for the fix.

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

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants