paketo-buildpacks/libjvm

BPL_JVM_HEAD_ROOM is not picked up

Closed this issue · 2 comments

dmikusa commented

What happened?

  • What were you attempting to do?

Set BPL_JVM_HEAD_ROOM to add headroom in the JVM

  • What did you expect to happen?

The buildpack should see this env variable and adjust its memory configuration.

  • What was the actual behavior? Please provide log output, if possible.

The buildpack does not see the environment variable.

It appears that libjvm is looking for BPL_JVM_HEADROOM (without the last underscore).

See https://github.com/paketo-buildpacks/libjvm/blob/main/helper/memory_calculator.go#L58

Build Configuration

  • What platform (pack, kpack, tekton buildpacks plugin, etc.) are you
    using? Please include a version.

pack

  • What buildpacks are you using? Please include versions.

bellsoft-liberica 7.1.0

  • What builder are you using? If custom, can you provide the output from pack inspect-builder <builder>?

default

  • Can you provide a sample app or relevant configuration (buildpack.yml,
    nginx.conf, etc.)?

Any

Checklist

  • I have included log output.
  • The log output includes an error message.
  • I have included steps for reproduction.
dmikusa commented

Or if we want to stick with BPL_JVM_HEAD_ROOM, then we need to update libjvm.