Skip to content

Conversation

@prudhvigodithi
Copy link
Member

@prudhvigodithi prudhvigodithi commented Sep 1, 2025

Description

Seeing https://github.com/diffplug/spotless/blob/main/plugin-gradle/README.md#eclipse-jdt Attempt to fix the jdt formatter error which is seen failing on recent open PR's.

https://github.com/opensearch-project/OpenSearch/actions/runs/17335154878/job/49219860173?pr=19188

* What went wrong:
A problem occurred configuring project ':build-tools'.
> Could not create task ':build-tools:spotlessJava'.
   > java.io.IOException: Failed to load eclipse jdt formatter: java.lang.RuntimeException: java.net.SocketException: Network is unreachable

The issue started because the https://mirror.umd.edu/eclipse/eclipse/updates/4.29 mirror is not accessible.

Related Issues

Seeing on all open PR's #19188

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.

Signed-off-by: Prudhvi Godithi <pgodithi@amazon.com>
@prudhvigodithi prudhvigodithi self-assigned this Sep 1, 2025
@prudhvigodithi
Copy link
Member Author

Adding @reta @cwperks @getsaurabh02 to please take a look. Thanks

Signed-off-by: Prudhvi Godithi <pgodithi@amazon.com>
@prudhvigodithi prudhvigodithi added the CI CI related label Sep 1, 2025
Signed-off-by: Prudhvi Godithi <pgodithi@amazon.com>
Signed-off-by: Prudhvi Godithi <pgodithi@amazon.com>
@prudhvigodithi
Copy link
Member Author

prudhvigodithi commented Sep 1, 2025

@github-actions
Copy link
Contributor

github-actions bot commented Sep 1, 2025

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

@prudhvigodithi prudhvigodithi marked this pull request as ready for review September 1, 2025 15:06
@prudhvigodithi prudhvigodithi requested a review from a team as a code owner September 1, 2025 15:06
@github-actions
Copy link
Contributor

github-actions bot commented Sep 1, 2025

❌ Gradle check result for 7561fe2: 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: Prudhvi Godithi <pgodithi@amazon.com>
Signed-off-by: Prudhvi Godithi <pgodithi@amazon.com>
Signed-off-by: Prudhvi Godithi <pgodithi@amazon.com>
Signed-off-by: Prudhvi Godithi <pgodithi@amazon.com>
@github-actions
Copy link
Contributor

github-actions bot commented Sep 1, 2025

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

)

eclipse().withP2Mirrors(Map.of("https://download.eclipse.org/", "https://mirror.umd.edu/eclipse/")).configFile rootProject.file('buildSrc/formatterConfig.xml')
eclipse().withP2Mirrors(['https://download.eclipse.org/eclipse/updates/4.29/':'https://ftp.osuosl.org/pub/eclipse/eclipse/updates/4.29/']).configFile rootProject.file('buildSrc/formatterConfig.xml')
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

This can be?

Suggested change
eclipse().withP2Mirrors(['https://download.eclipse.org/eclipse/updates/4.29/':'https://ftp.osuosl.org/pub/eclipse/eclipse/updates/4.29/']).configFile rootProject.file('buildSrc/formatterConfig.xml')
eclipse().withP2Mirrors(Map.of("https://download.eclipse.org/eclipse/updates/4.29/", "https://ftp.osuosl.org/pub/eclipse/eclipse/updates/4.29/")).configFile rootProject.file('buildSrc/formatterConfig.xml')

Copy link
Member Author

@prudhvigodithi prudhvigodithi Sep 2, 2025

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Initially we use Map.of (#17172) which works, but just following the instructions in https://github.com/diffplug/spotless/blob/main/plugin-gradle/README.md#eclipse-jdt

Copy link
Member

@owaiskazi19 owaiskazi19 Sep 2, 2025

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

@prudhvigodithi Map worked too https://github.com/opensearch-project/neural-search/pull/1540/files.
Nvm, I see mirrors are completely removed

@reta
Copy link
Contributor

reta commented Sep 1, 2025

I believe this is not new issue [1], has been surfacing since spotless 7.0.0 release (if I am not mistaken), probably the best option is to downgrade spotless to 6.x

[1] diffplug/spotless#2515

@github-actions
Copy link
Contributor

github-actions bot commented Sep 1, 2025

❌ Gradle check result for 6a74ac5: 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

github-actions bot commented Sep 2, 2025

❌ Gradle check result for 6a74ac5: 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

cwperks commented Sep 2, 2025

I'm still seeing:

 [3.1.0] FAILURE: Build failed with an exception.
 [3.1.0] 
 [3.1.0] * Where:
 [3.1.0] Build file '/var/jenkins/workspace/gradle-check/search/distribution/bwc/staged/build/bwc/checkout-3.1/buildSrc/build.gradle' line: 275
 [3.1.0] 
 [3.1.0] * What went wrong:
 [3.1.0] A problem occurred configuring project ':build-tools'.
 [3.1.0] > Could not create task ':build-tools:spotlessJava'.
 [3.1.0]    > java.io.IOException: Failed to load eclipse jdt formatter: java.lang.RuntimeException: java.net.NoRouteToHostException: No route to host

@cwperks
Copy link
Member

cwperks commented Sep 2, 2025

@prudhvigodithi do we potentially need to add this to other branches?

@prudhvigodithi
Copy link
Member Author

prudhvigodithi commented Sep 2, 2025

I'm thinking to completely remove withP2Mirrors and allow eclipse() to default use https://download.eclipse.org/. As seeing the instructions the mirrors https://github.com/diffplug/spotless/blob/main/plugin-gradle/README.md#eclipse-jdt are only for restricted scenarios and also withP2Mirrors setting defaults to mirror. It would be nice if there is a setting 1st it would try https://download.eclipse.org/ and then fallback to mirror.

Signed-off-by: Prudhvi Godithi <pgodithi@amazon.com>
@prudhvigodithi
Copy link
Member Author

@prudhvigodithi do we potentially need to add this to other branches?

Yes @cwperks we need to backport to the relevant branches , IMO we need to force merge and backport as the configured mirror is completely down https://mirror.umd.edu/eclipse/eclipse/updates/4.29.

@github-actions
Copy link
Contributor

github-actions bot commented Sep 2, 2025

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

cwperks commented Sep 2, 2025

Overriding and merging on this PR as the build is expected to fail until its backported:

> Task :distribution:bwc:minor:buildBwcLinuxTar
 [3.2.0] 
 [3.2.0] 
 [3.2.0] FAILURE: Build failed with an exception.
 [3.2.0] 
 [3.2.0] * Where:
 [3.2.0] Build file '/var/jenkins/workspace/gradle-check/search/distribution/bwc/minor/build/bwc/checkout-3.2/buildSrc/build.gradle' line: 275
 [3.2.0] 
 [3.2.0] * What went wrong:
 [3.2.0] [Incubating] Problems report is available at: file:///var/jenkins/workspace/gradle-check/search/distribution/bwc/minor/build/bwc/checkout-3.2/build/reports/problems/problems-report.html
 [3.2.0] A problem occurred configuring project ':build-tools'.
 [3.2.0] > Could not create task ':build-tools:spotlessJava'.
 [3.2.0]    > java.io.IOException: Failed to load eclipse jdt formatter: java.lang.RuntimeException: java.net.SocketTimeoutException: Connect timed out
 [3.2.0] 

@cwperks cwperks merged commit 4fc44d1 into opensearch-project:main Sep 2, 2025
42 of 45 checks passed
opensearch-trigger-bot bot pushed a commit that referenced this pull request Sep 2, 2025
* Attempt to fix the jdt formatter error

Signed-off-by: Prudhvi Godithi <pgodithi@amazon.com>

* Fix the changelog

Signed-off-by: Prudhvi Godithi <pgodithi@amazon.com>

* Update to 4.36

Signed-off-by: Prudhvi Godithi <pgodithi@amazon.com>

* Update to 4.29

Signed-off-by: Prudhvi Godithi <pgodithi@amazon.com>

* Update to 4.36

Signed-off-by: Prudhvi Godithi <pgodithi@amazon.com>

* Update Syntax

Signed-off-by: Prudhvi Godithi <pgodithi@amazon.com>

* Update Syntax

Signed-off-by: Prudhvi Godithi <pgodithi@amazon.com>

* Update Syntax

Signed-off-by: Prudhvi Godithi <pgodithi@amazon.com>

* Remove mirrors

Signed-off-by: Prudhvi Godithi <pgodithi@amazon.com>

---------

Signed-off-by: Prudhvi Godithi <pgodithi@amazon.com>
(cherry picked from commit 4fc44d1)
Signed-off-by: github-actions[bot] <github-actions[bot]@users.noreply.github.com>
opensearch-trigger-bot bot pushed a commit that referenced this pull request Sep 2, 2025
* Attempt to fix the jdt formatter error

Signed-off-by: Prudhvi Godithi <pgodithi@amazon.com>

* Fix the changelog

Signed-off-by: Prudhvi Godithi <pgodithi@amazon.com>

* Update to 4.36

Signed-off-by: Prudhvi Godithi <pgodithi@amazon.com>

* Update to 4.29

Signed-off-by: Prudhvi Godithi <pgodithi@amazon.com>

* Update to 4.36

Signed-off-by: Prudhvi Godithi <pgodithi@amazon.com>

* Update Syntax

Signed-off-by: Prudhvi Godithi <pgodithi@amazon.com>

* Update Syntax

Signed-off-by: Prudhvi Godithi <pgodithi@amazon.com>

* Update Syntax

Signed-off-by: Prudhvi Godithi <pgodithi@amazon.com>

* Remove mirrors

Signed-off-by: Prudhvi Godithi <pgodithi@amazon.com>

---------

Signed-off-by: Prudhvi Godithi <pgodithi@amazon.com>
(cherry picked from commit 4fc44d1)
Signed-off-by: github-actions[bot] <github-actions[bot]@users.noreply.github.com>
opensearch-trigger-bot bot pushed a commit that referenced this pull request Sep 2, 2025
* Attempt to fix the jdt formatter error

Signed-off-by: Prudhvi Godithi <pgodithi@amazon.com>

* Fix the changelog

Signed-off-by: Prudhvi Godithi <pgodithi@amazon.com>

* Update to 4.36

Signed-off-by: Prudhvi Godithi <pgodithi@amazon.com>

* Update to 4.29

Signed-off-by: Prudhvi Godithi <pgodithi@amazon.com>

* Update to 4.36

Signed-off-by: Prudhvi Godithi <pgodithi@amazon.com>

* Update Syntax

Signed-off-by: Prudhvi Godithi <pgodithi@amazon.com>

* Update Syntax

Signed-off-by: Prudhvi Godithi <pgodithi@amazon.com>

* Update Syntax

Signed-off-by: Prudhvi Godithi <pgodithi@amazon.com>

* Remove mirrors

Signed-off-by: Prudhvi Godithi <pgodithi@amazon.com>

---------

Signed-off-by: Prudhvi Godithi <pgodithi@amazon.com>
(cherry picked from commit 4fc44d1)
Signed-off-by: github-actions[bot] <github-actions[bot]@users.noreply.github.com>
@opensearch-trigger-bot
Copy link
Contributor

The backport to 2.18 failed:

The process '/usr/bin/git' failed with exit code 128

To backport manually, run these commands in your terminal:

# Navigate to the root of your repository
cd $(git rev-parse --show-toplevel)
# Fetch latest updates from GitHub
git fetch
# Create a new working tree
git worktree add ../.worktrees/OpenSearch/backport-2.18 2.18
# Navigate to the new working tree
pushd ../.worktrees/OpenSearch/backport-2.18
# Create a new branch
git switch --create backport/backport-19199-to-2.18
# Cherry-pick the merged commit of this pull request and resolve the conflicts
git cherry-pick -x --mainline 1 4fc44d10caa18c9280cf5ad8377dc1f1cdfa4599
# Push it to GitHub
git push --set-upstream origin backport/backport-19199-to-2.18
# Go back to the original working tree
popd
# Delete the working tree
git worktree remove ../.worktrees/OpenSearch/backport-2.18

Then, create a pull request where the base branch is 2.18 and the compare/head branch is backport/backport-19199-to-2.18.

cwperks pushed a commit that referenced this pull request Sep 2, 2025
* Attempt to fix the jdt formatter error



* Fix the changelog



* Update to 4.36



* Update to 4.29



* Update to 4.36



* Update Syntax



* Update Syntax



* Update Syntax



* Remove mirrors



---------


(cherry picked from commit 4fc44d1)

Signed-off-by: Prudhvi Godithi <pgodithi@amazon.com>
Signed-off-by: github-actions[bot] <github-actions[bot]@users.noreply.github.com>
Co-authored-by: github-actions[bot] <github-actions[bot]@users.noreply.github.com>
cwperks pushed a commit that referenced this pull request Sep 2, 2025
* Attempt to fix the jdt formatter error



* Fix the changelog



* Update to 4.36



* Update to 4.29



* Update to 4.36



* Update Syntax



* Update Syntax



* Update Syntax



* Remove mirrors



---------


(cherry picked from commit 4fc44d1)

Signed-off-by: Prudhvi Godithi <pgodithi@amazon.com>
Signed-off-by: github-actions[bot] <github-actions[bot]@users.noreply.github.com>
Co-authored-by: github-actions[bot] <github-actions[bot]@users.noreply.github.com>
prudhvigodithi pushed a commit that referenced this pull request Sep 2, 2025
* Attempt to fix the jdt formatter error



* Fix the changelog



* Update to 4.36



* Update to 4.29



* Update to 4.36



* Update Syntax



* Update Syntax



* Update Syntax



* Remove mirrors



---------


(cherry picked from commit 4fc44d1)

Signed-off-by: Prudhvi Godithi <pgodithi@amazon.com>
Signed-off-by: github-actions[bot] <github-actions[bot]@users.noreply.github.com>
Co-authored-by: github-actions[bot] <github-actions[bot]@users.noreply.github.com>
@prudhvigodithi
Copy link
Member Author

#19199 (comment) we dont need to backport to 2.18 as I dont see the mirrors added to 2.18 https://github.com/opensearch-project/OpenSearch/blob/2.18/gradle/formatting.gradle#L85.

@vinaykpud vinaykpud moved this from Todo to Done in Performance Roadmap Sep 4, 2025
kh3ra pushed a commit to kh3ra/OpenSearch that referenced this pull request Sep 5, 2025
* Attempt to fix the jdt formatter error

Signed-off-by: Prudhvi Godithi <pgodithi@amazon.com>

* Fix the changelog

Signed-off-by: Prudhvi Godithi <pgodithi@amazon.com>

* Update to 4.36

Signed-off-by: Prudhvi Godithi <pgodithi@amazon.com>

* Update to 4.29

Signed-off-by: Prudhvi Godithi <pgodithi@amazon.com>

* Update to 4.36

Signed-off-by: Prudhvi Godithi <pgodithi@amazon.com>

* Update Syntax

Signed-off-by: Prudhvi Godithi <pgodithi@amazon.com>

* Update Syntax

Signed-off-by: Prudhvi Godithi <pgodithi@amazon.com>

* Update Syntax

Signed-off-by: Prudhvi Godithi <pgodithi@amazon.com>

* Remove mirrors

Signed-off-by: Prudhvi Godithi <pgodithi@amazon.com>

---------

Signed-off-by: Prudhvi Godithi <pgodithi@amazon.com>
jainankitk pushed a commit to jainankitk/OpenSearch that referenced this pull request Sep 22, 2025
* Attempt to fix the jdt formatter error

Signed-off-by: Prudhvi Godithi <pgodithi@amazon.com>

* Fix the changelog

Signed-off-by: Prudhvi Godithi <pgodithi@amazon.com>

* Update to 4.36

Signed-off-by: Prudhvi Godithi <pgodithi@amazon.com>

* Update to 4.29

Signed-off-by: Prudhvi Godithi <pgodithi@amazon.com>

* Update to 4.36

Signed-off-by: Prudhvi Godithi <pgodithi@amazon.com>

* Update Syntax

Signed-off-by: Prudhvi Godithi <pgodithi@amazon.com>

* Update Syntax

Signed-off-by: Prudhvi Godithi <pgodithi@amazon.com>

* Update Syntax

Signed-off-by: Prudhvi Godithi <pgodithi@amazon.com>

* Remove mirrors

Signed-off-by: Prudhvi Godithi <pgodithi@amazon.com>

---------

Signed-off-by: Prudhvi Godithi <pgodithi@amazon.com>
jainankitk pushed a commit to jainankitk/OpenSearch that referenced this pull request Sep 22, 2025
* Attempt to fix the jdt formatter error

Signed-off-by: Prudhvi Godithi <pgodithi@amazon.com>

* Fix the changelog

Signed-off-by: Prudhvi Godithi <pgodithi@amazon.com>

* Update to 4.36

Signed-off-by: Prudhvi Godithi <pgodithi@amazon.com>

* Update to 4.29

Signed-off-by: Prudhvi Godithi <pgodithi@amazon.com>

* Update to 4.36

Signed-off-by: Prudhvi Godithi <pgodithi@amazon.com>

* Update Syntax

Signed-off-by: Prudhvi Godithi <pgodithi@amazon.com>

* Update Syntax

Signed-off-by: Prudhvi Godithi <pgodithi@amazon.com>

* Update Syntax

Signed-off-by: Prudhvi Godithi <pgodithi@amazon.com>

* Remove mirrors

Signed-off-by: Prudhvi Godithi <pgodithi@amazon.com>

---------

Signed-off-by: Prudhvi Godithi <pgodithi@amazon.com>
Signed-off-by: Ankit Jain <jainankitk@apache.org>
jainankitk pushed a commit to jainankitk/OpenSearch that referenced this pull request Sep 22, 2025
* Attempt to fix the jdt formatter error

Signed-off-by: Prudhvi Godithi <pgodithi@amazon.com>

* Fix the changelog

Signed-off-by: Prudhvi Godithi <pgodithi@amazon.com>

* Update to 4.36

Signed-off-by: Prudhvi Godithi <pgodithi@amazon.com>

* Update to 4.29

Signed-off-by: Prudhvi Godithi <pgodithi@amazon.com>

* Update to 4.36

Signed-off-by: Prudhvi Godithi <pgodithi@amazon.com>

* Update Syntax

Signed-off-by: Prudhvi Godithi <pgodithi@amazon.com>

* Update Syntax

Signed-off-by: Prudhvi Godithi <pgodithi@amazon.com>

* Update Syntax

Signed-off-by: Prudhvi Godithi <pgodithi@amazon.com>

* Remove mirrors

Signed-off-by: Prudhvi Godithi <pgodithi@amazon.com>

---------

Signed-off-by: Prudhvi Godithi <pgodithi@amazon.com>
Signed-off-by: Ankit Jain <jainankitk@apache.org>
asimmahmood1 pushed a commit to jainankitk/OpenSearch that referenced this pull request Sep 23, 2025
* Attempt to fix the jdt formatter error

Signed-off-by: Prudhvi Godithi <pgodithi@amazon.com>

* Fix the changelog

Signed-off-by: Prudhvi Godithi <pgodithi@amazon.com>

* Update to 4.36

Signed-off-by: Prudhvi Godithi <pgodithi@amazon.com>

* Update to 4.29

Signed-off-by: Prudhvi Godithi <pgodithi@amazon.com>

* Update to 4.36

Signed-off-by: Prudhvi Godithi <pgodithi@amazon.com>

* Update Syntax

Signed-off-by: Prudhvi Godithi <pgodithi@amazon.com>

* Update Syntax

Signed-off-by: Prudhvi Godithi <pgodithi@amazon.com>

* Update Syntax

Signed-off-by: Prudhvi Godithi <pgodithi@amazon.com>

* Remove mirrors

Signed-off-by: Prudhvi Godithi <pgodithi@amazon.com>

---------

Signed-off-by: Prudhvi Godithi <pgodithi@amazon.com>
vinaykpud pushed a commit to vinaykpud/OpenSearch that referenced this pull request Sep 26, 2025
* Attempt to fix the jdt formatter error

Signed-off-by: Prudhvi Godithi <pgodithi@amazon.com>

* Fix the changelog

Signed-off-by: Prudhvi Godithi <pgodithi@amazon.com>

* Update to 4.36

Signed-off-by: Prudhvi Godithi <pgodithi@amazon.com>

* Update to 4.29

Signed-off-by: Prudhvi Godithi <pgodithi@amazon.com>

* Update to 4.36

Signed-off-by: Prudhvi Godithi <pgodithi@amazon.com>

* Update Syntax

Signed-off-by: Prudhvi Godithi <pgodithi@amazon.com>

* Update Syntax

Signed-off-by: Prudhvi Godithi <pgodithi@amazon.com>

* Update Syntax

Signed-off-by: Prudhvi Godithi <pgodithi@amazon.com>

* Remove mirrors

Signed-off-by: Prudhvi Godithi <pgodithi@amazon.com>

---------

Signed-off-by: Prudhvi Godithi <pgodithi@amazon.com>
sokdak pushed a commit to sokdak/OpenSearch that referenced this pull request Oct 15, 2025
…ensearch-project#19217)

* Attempt to fix the jdt formatter error



* Fix the changelog



* Update to 4.36



* Update to 4.29



* Update to 4.36



* Update Syntax



* Update Syntax



* Update Syntax



* Remove mirrors



---------


(cherry picked from commit 4fc44d1)

Signed-off-by: Prudhvi Godithi <pgodithi@amazon.com>
Signed-off-by: github-actions[bot] <github-actions[bot]@users.noreply.github.com>
Co-authored-by: github-actions[bot] <github-actions[bot]@users.noreply.github.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

Status: Done

Development

Successfully merging this pull request may close these issues.

4 participants