Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Java LS builds fail randomly with error code 137(SIGKILL received?) #2226

Closed
snjeza opened this issue Sep 14, 2022 · 5 comments
Closed

Java LS builds fail randomly with error code 137(SIGKILL received?) #2226

snjeza opened this issue Sep 14, 2022 · 5 comments
Assignees

Comments

@snjeza
Copy link
Contributor

snjeza commented Sep 14, 2022

See https://ci.eclipse.org/ls/job/jdt-ls-pr/3566/

Caused by: org.apache.maven.plugin.MojoFailureException: An unexpected error occurred while launching the test runtime (process returned error code 137(SIGKILL received?)). 
@snjeza snjeza self-assigned this Sep 14, 2022
@snjeza snjeza changed the title Java Ls builds fail Java LS builds fail Sep 14, 2022
@rgrunber
Copy link
Contributor

Thanks, I was seeing the SIGKILL (137) as well. Any idea why we started running into this recently ?

@rgrunber rgrunber added this to the End September milestone Sep 14, 2022
@snjeza
Copy link
Contributor Author

snjeza commented Sep 14, 2022

I think the issue has been introduced by #2197. Our tests require more memory and jenkins/kernel/kubernetes kills JVM.
I can't reproduce the issue locally.

@snjeza snjeza changed the title Java LS builds fail Java LS builds fail randomly with error code 137(SIGKILL received?) Sep 14, 2022
@snjeza
Copy link
Contributor Author

snjeza commented Sep 15, 2022

See #2227 (comment)

@snjeza snjeza closed this as completed Sep 15, 2022
@rgrunber
Copy link
Contributor

Nice. I see we went with similar arguments to what are used in vscode-java, which is probably the right approach.

Do you know what requires java9.home ? I couldn't find it referenced anywhere in JDT-LS. I see java8fx.home for a test but nothing else.

@snjeza
Copy link
Contributor Author

snjeza commented Sep 15, 2022

Do you know what requires java9.home ? I couldn't find it referenced anywhere in JDT-LS

It isn't used. I have removed it from jdt-ls-pr and jdt.ls-master

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

Successfully merging a pull request may close this issue.

2 participants