Skip to content

HDDS-15621. TestSecureOzoneCluster fails locally#10550

Open
adoroszlai wants to merge 2 commits into
apache:masterfrom
adoroszlai:HDDS-15621
Open

HDDS-15621. TestSecureOzoneCluster fails locally#10550
adoroszlai wants to merge 2 commits into
apache:masterfrom
adoroszlai:HDDS-15621

Conversation

@adoroszlai

Copy link
Copy Markdown
Contributor

What changes were proposed in this pull request?

TestSecureOzoneCluster fails locally (also reproducible in flaky-test-check):

Expecting actual:
  "scm-sub@localhost"
to contain:
  "<hostname>" 
	at org.apache.hadoop.ozone.TestSecureOzoneCluster.validateCertificate(TestSecureOzoneCluster.java:1360)
	at org.apache.hadoop.ozone.TestSecureOzoneCluster.testSecureOmReInit(TestSecureOzoneCluster.java:883)

because SCM is configured to localhost:

but the test expects the actual hostname in certificate issuer name.

Also improve assertions by replacing assertTrue with assertThat.

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

How was this patch tested?

$ mvn -am -pl :ozone-integration-test -Dtest='TestSecureOzoneCluster' -DskipRecon clean test
...
Tests run: 16, Failures: 0, Errors: 0, Skipped: 0, Time elapsed: 232.2 s -- in org.apache.hadoop.ozone.TestSecureOzoneCluster
...

@adoroszlai adoroszlai requested review from ChenSammi and fapifta June 19, 2026 12:49
@adoroszlai adoroszlai self-assigned this Jun 19, 2026

@chungen0126 chungen0126 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.

Overall, looks good to me. Just a nit.

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.

2 participants