Skip to content

Comments

HDDS-13738. Discovered 2 'junit-platform.properties' configuration files#9759

Open
echonesis wants to merge 1 commit intoapache:masterfrom
echonesis:HDDS-13738-dev
Open

HDDS-13738. Discovered 2 'junit-platform.properties' configuration files#9759
echonesis wants to merge 1 commit intoapache:masterfrom
echonesis:HDDS-13738-dev

Conversation

@echonesis
Copy link
Contributor

What changes were proposed in this pull request?

Exclude junit-platform.properties from test-jar artifacts to prevent duplicate configuration file warnings during test execution.

What is the link to the Apache JIRA

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

How was this patch tested?

> mvn clean install -DskipTests -DskipShade

> jar tf ~/.m2/repository/org/apache/ozone/ozone-manager/2.2.0-SNAPSHOT/ozone-manager-2.2.0-SNAPSHOT-tests.jar | grep junit-platform.properties

> ls hadoop-ozone/ozone-manager/target/test-classes/junit-platform.properties
hadoop-ozone/ozone-manager/target/test-classes/junit-platform.properties

@adoroszlai adoroszlai requested a review from smengcl February 12, 2026 10:33
@adoroszlai adoroszlai requested a review from szetszwo February 20, 2026 11:04
@adoroszlai
Copy link
Contributor

Thanks @echonesis for working on this. I would like to understand how to reproduce the warning on master in the first place.

I tried running some OM test without clean after packaging:

mvn -am -pl :ozone-manager clean package
mvn -am -pl :ozone-manager -Dtest='TestParentAcl' test

but didn't get the warning, nor is there a hadoop-ozone/ozone-manager/target/test-classes-jar-tests directory.

The jar does contain the properties file:

$ unzip -t hadoop-ozone/ozone-manager/target/ozone-manager-2.2.0-SNAPSHOT-tests.jar junit-platform.properties       
Archive:  hadoop-ozone/ozone-manager/target/ozone-manager-2.2.0-SNAPSHOT-tests.jar
    testing: junit-platform.properties   OK
No errors detected in hadoop-ozone/ozone-manager/target/ozone-manager-2.2.0-SNAPSHOT-tests.jar for the 1 file tested.

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

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants