Skip to content

[BUG] Fix ambiguous Node type reference in build.gradle - #22514

Open
revijay wants to merge 1 commit into
opensearch-project:mainfrom
revijay:fix-groovy-node-import
Open

[BUG] Fix ambiguous Node type reference in build.gradle#22514
revijay wants to merge 1 commit into
opensearch-project:mainfrom
revijay:fix-groovy-node-import

Conversation

@revijay

@revijay revijay commented Jul 20, 2026

Copy link
Copy Markdown
Contributor

Description

Fixes the ambiguous Node type reference in the root build.gradle.

Some Gradle environments may resolve Node to org.gradle.execution.plan.Node, causing a ClassCastException during Gradle execution. This change explicitly imports groovy.util.Node so the intended type is always used.

Related Issues

Resolves #19247

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.

@revijay
revijay requested a review from a team as a code owner July 20, 2026 11:02
@github-actions github-actions Bot added bug Something isn't working Build Build Tasks/Gradle Plugin, groovy scripts, build tools, Javadoc enforcement. labels Jul 20, 2026
@github-actions

Copy link
Copy Markdown
Contributor

❌ Gradle check result for 5125243: 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?

@revijay

revijay commented Jul 23, 2026

Copy link
Copy Markdown
Contributor Author

❌ Gradle check result for 5125243: 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?

I looked into the failure. It appears to be unrelated to this change.

The failure is in Netty4HttpRequestSizeLimitIT:

  • java.lang.AssertionError: Failed to get all expected responses: 150 left
  • SpanData validation failed for validator AllSpansAreEndedProperly

The Jenkins output indicates this test has been failing for the past 26 builds (Since Failed#82329), so it doesn't appear to have been introduced by this PR.

My change only updates the build.gradle import and does not touch the HTTP or Netty request handling code. Could a maintainer please confirm whether this is a known flaky/pre-existing CI failure?

@github-actions

Copy link
Copy Markdown
Contributor

❌ Gradle check result for 5125243: 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?

Signed-off-by: revijay <reshmi.vijayan@ibm.com>
@revijay
revijay force-pushed the fix-groovy-node-import branch from 5125243 to c3a9f68 Compare July 30, 2026 10:18
@github-actions

Copy link
Copy Markdown
Contributor

✅ Gradle check result for c3a9f68: SUCCESS

@codecov

codecov Bot commented Jul 30, 2026

Copy link
Copy Markdown

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 71.45%. Comparing base (03a4de3) to head (c3a9f68).

Additional details and impacted files
@@             Coverage Diff              @@
##               main   #22514      +/-   ##
============================================
+ Coverage     71.42%   71.45%   +0.02%     
- Complexity    76786    76849      +63     
============================================
  Files          6148     6148              
  Lines        357980   357980              
  Branches      52177    52177              
============================================
+ Hits         255689   255785      +96     
+ Misses        81937    81857      -80     
+ Partials      20354    20338      -16     

☔ View full report in Codecov by Harness.
📢 Have feedback on the report? Share it here.

🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.

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

Labels

bug Something isn't working Build Build Tasks/Gradle Plugin, groovy scripts, build tools, Javadoc enforcement.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

[BUG] A ClassCastException occurred after executing "./gradlew run" due to "groovy.util.Node"

1 participant