Skip to content

[8.18] Limit memory consumption in test on overflow (backport #17373)#17412

Merged
andsel merged 1 commit into8.18from
mergify/bp/8.18/pr-17373
Mar 31, 2025
Merged

[8.18] Limit memory consumption in test on overflow (backport #17373)#17412
andsel merged 1 commit into8.18from
mergify/bp/8.18/pr-17373

Conversation

@mergify
Copy link
Copy Markdown
Contributor

@mergify mergify bot commented Mar 26, 2025

Release notes

[rn:skip]

What does this PR do?

Updates only test code to be able to run a test that consumes big memory if:

  • the physical memory is bigger than the requested Java heap
  • JDK version is greater than or equal to 21.

The reason to limit the JDK version is that on 16GB machine the G1GC is more efficient than the one on previous JDKs and so let complete the test with 10GB heap, while in JDK 17 it consistently fails with OOM error.

Why is it important/What is the impact to the user?

N/A

Checklist

  • My code follows the style guidelines of this project
  • I have commented my code, particularly in hard-to-understand areas
  • [ ] I have made corresponding changes to the documentation
  • [ ] I have made corresponding change to the default configuration files (and/or docker env variables)
  • I have added tests that prove my fix is effective or that my feature works

Author's Checklist

  • tested on ubuntu 2024 16GB RAM VM with JDK 11, 17 and 21

How to test this PR locally

Related issues


This is an automatic backport of pull request #17373 done by [Mergify](https://mergify.com).

Updates only test code to be able to run a test that consumes big memory if:
- the physical memory is bigger than the requested Java heap
- JDK version is greater than or equal to 21.

The reason to limit the JDK version is that on 16GB machine the G1GC is more efficient than the one on previous JDKs and so let complete the test with 10GB heap, while in JDK 17 it consistently fails with OOM error.

(cherry picked from commit 075fdb4)
@mergify mergify bot added the backport label Mar 26, 2025
@mergify mergify bot assigned andsel Mar 26, 2025
@elastic-sonarqube
Copy link
Copy Markdown

@elasticmachine
Copy link
Copy Markdown

💚 Build Succeeded

cc @andsel

@mergify
Copy link
Copy Markdown
Contributor Author

mergify bot commented Mar 31, 2025

This pull request has not been merged yet. Could you please review and merge it @andsel? 🙏

@andsel andsel merged commit 01ec9ee into 8.18 Mar 31, 2025
7 checks passed
@andsel andsel deleted the mergify/bp/8.18/pr-17373 branch March 31, 2025 08:11
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.

2 participants