On Wed, 29 Apr 2026 08:52:59 GMT, Casper Norrbin <[email protected]> wrote:
> Hi everyone, > > `TestLimitsUpdating.java` tests updating container limits live during a JVMs > runtime. One of these updates is to the number of CPUs available, from 2 to > 3. However, if there are only 2 CPUs visible to the JVM (total available, or > restricted by other means), this update is ignored, and we are bounded to 2 > CPUs. The tests expects the new limit to be 3, and thus fails. > > I propose we lower the number from 2/3 -> 1/2 to allow more configurations to > run this tests. If we only have 1 CPU available, we should instead skip the > test as we can't update the number of CPUs in a meaningful way. > > Testing: > - Local testing with both low and high number of CPUs available > - Oracle tiers 1-5 > > --------- > - [x] I confirm that I make this contribution in accordance with the [OpenJDK > Interim AI Policy](https://openjdk.org/legal/ai). This pull request has now been integrated. Changeset: e6582b2e Author: Casper Norrbin <[email protected]> URL: https://git.openjdk.org/jdk/commit/e6582b2e80e31103bcda08d27fded72604f687d3 Stats: 30 lines in 2 files changed: 16 ins; 0 del; 14 mod 8377991: TestLimitsUpdating.java fails with runtime exception even after JDK-8370492 is fixed Reviewed-by: phubner, sgehwolf ------------- PR: https://git.openjdk.org/jdk/pull/30985
