Skip to content

[3.4] Ensure all modules are included in INTEG_TEST testcluster distribution#20243

Closed
cwperks wants to merge 2 commits intoopensearch-project:3.4from
cwperks:use-all-modules-3.4
Closed

[3.4] Ensure all modules are included in INTEG_TEST testcluster distribution#20243
cwperks wants to merge 2 commits intoopensearch-project:3.4from
cwperks:use-all-modules-3.4

Conversation

@cwperks
Copy link
Member

@cwperks cwperks commented Dec 15, 2025

Description

This PR ensures that all modules are included when using the INTEG_TEST testcluster distribution. Currently, when using a testcluster with INTEG_TEST distribution, only modules matching transport-* are included and as a result many plugins cannot use this distribution adequately because its missing modules like autotagging-commons.

Related Issues

See discussion on #20229

Check List

  • Functionality includes testing.
  • API changes companion pull request created, if applicable.
  • Public documentation issue/PR created, if applicable.

By submitting this pull request, I confirm that my contribution is made under the terms of the Apache 2.0 license.
For more information on following Developer Certificate of Origin and signing off your commits, please check here.

Summary by CodeRabbit

  • Chores
    • Refactored build distribution to replace transport-specific module handling with an integrated test-focused workflow, streamlining how modules are assembled for test distributions.
  • Documentation
    • Changelog updated: noted change ensuring all modules are included in the INTEG_TEST testcluster distribution.

✏️ Tip: You can customize this high-level summary in your review settings.

Signed-off-by: Craig Perkins <cwperx@amazon.com>
Signed-off-by: Craig Perkins <cwperx@amazon.com>
@cwperks cwperks requested a review from a team as a code owner December 15, 2025 18:35
@coderabbitai
Copy link
Contributor

coderabbitai bot commented Dec 15, 2025

Important

Review skipped

Auto reviews are disabled on base/target branches other than the default branch.

Please check the settings in the CodeRabbit UI or the .coderabbit.yaml file in this repository. To trigger a single review, invoke the @coderabbitai review command.

You can disable this status message by setting the reviews.review_status to false in the CodeRabbit configuration file.

✨ Finishing touches
🧪 Generate unit tests (beta)
  • Create PR with unit tests
  • Post copyable unit tests in a comment

Comment @coderabbitai help to get the list of available commands and usage tips.

@github-actions
Copy link
Contributor

❌ Gradle check result for aaa1183: FAILURE

Please examine the workflow log, locate, and copy-paste the failure(s) below, then iterate to green. Is the failure a flaky test unrelated to your change?

@github-actions
Copy link
Contributor

❌ Gradle check result for aaa1183: FAILURE

Please examine the workflow log, locate, and copy-paste the failure(s) below, then iterate to green. Is the failure a flaky test unrelated to your change?

@github-actions
Copy link
Contributor

❌ Gradle check result for aaa1183: FAILURE

Please examine the workflow log, locate, and copy-paste the failure(s) below, then iterate to green. Is the failure a flaky test unrelated to your change?

@github-actions
Copy link
Contributor

❌ Gradle check result for aaa1183: FAILURE

Please examine the workflow log, locate, and copy-paste the failure(s) below, then iterate to green. Is the failure a flaky test unrelated to your change?

@cwperks
Copy link
Member Author

cwperks commented Dec 15, 2025

Closing the PR against 3.4 as this is getting fixed in the following ways in CCR:

The PR against main should be merged as a stable long term solution for running integ tests with testclusters for plugins

@cwperks cwperks closed this Dec 15, 2025
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