BPL_JVM_HEAD_ROOM is not picked up
Closed this issue · 2 comments
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.
Also impacts https://github.com/paketo-buildpacks/amazon-corretto, https://github.com/paketo-buildpacks/azul-zulu, https://github.com/paketo-buildpacks/sap-machine, https://github.com/paketo-buildpacks/eclipse-openj9, https://github.com/paketo-buildpacks/adopt-openjdk, https://github.com/paketo-buildpacks/graalvm, and the web site https://github.com/paketo-buildpacks/paketo-website/blob/51d448d6e6bc70858898c4fe43d13760444eba53/content/docs/buildpacks/language-family-buildpacks/java.md
Or if we want to stick with BPL_JVM_HEAD_ROOM, then we need to update libjvm.