Skip to content

HDDS-15749. Run specific JUnit tests if possible#10671

Draft
adoroszlai wants to merge 2 commits into
apache:masterfrom
adoroszlai:HDDS-15749
Draft

HDDS-15749. Run specific JUnit tests if possible#10671
adoroszlai wants to merge 2 commits into
apache:masterfrom
adoroszlai:HDDS-15749

Conversation

@adoroszlai

Copy link
Copy Markdown
Contributor

What changes were proposed in this pull request?

If only JUnit test classes have changed, limit tests run in pull request CI to that set. This adds a new check specific-tests (with display name integration), which is the same as integration, but only runs specific tests instead of profiles in a matrix.

Example: f5a5f0f only changed TestContainerBalancerDatanodeNodeLimit, no need to run all other integration tests.

Counterexample: 10a9437 changed test helper code (AbstractOzoneManagerHATest and MiniOzoneHAClusterImpl), need to run all integration tests.

Note: there are many test helpers and parent classes that do not follow naming convention. We'll need to rename these to ensure proper coverage. Creating as draft until we address that.

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

How was this patch tested?

Updated bats test. Also tested in fork:

Push build

  • integration with specific tests (skipped)
  • ✔️ profile-based integration

Pull requests

  1. test-only change: HDDS-15749. change some tests adoroszlai/ozone#62
    • ✔️ integration with specific tests (run)
      [INFO] Tests run: 336, Failures: 0, Errors: 0, Skipped: 0, Time elapsed: 33.89 s -- in org.apache.hadoop.hdds.scm.container.balancer.TestContainerBalancerDatanodeNodeLimit
      [INFO] Tests run: 5, Failures: 0, Errors: 0, Skipped: 0, Time elapsed: 1.719 s -- in org.apache.hadoop.ozone.om.TestAuthorizerLockImpl:w
      [INFO] Total time:  03:49 min
      
    • ❌ profile-based integration
  2. non-test change: HDDS-15749. change test helper, too adoroszlai/ozone#63
    • integration with specific tests (skipped)
    • ✔️ profile-based integration

@adoroszlai adoroszlai self-assigned this Jul 5, 2026
@adoroszlai adoroszlai added the CI label Jul 5, 2026
@jojochuang jojochuang requested a review from spacemonkd July 6, 2026 16:15
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.

1 participant