Skip to content

Handle empty JVM_OPTIONS correctly on dynamic/container agents#7351

Draft
andrew-m-leonard with Copilot wants to merge 2 commits into
masterfrom
copilot/fix-system-tests-playlist-variation-mode
Draft

Handle empty JVM_OPTIONS correctly on dynamic/container agents#7351
andrew-m-leonard with Copilot wants to merge 2 commits into
masterfrom
copilot/fix-system-tests-playlist-variation-mode

Conversation

Copilot AI commented Jul 14, 2026

Copy link
Copy Markdown
Contributor

Fixes: adoptium/TKG#862
Fixes: #6835

System tests on dynamic Azure agents were not honoring playlist variation JVM args because containerized runs propagated JVM_OPTIONS="" into the shell, which suppresses Makefile defaults defined via JVM_OPTIONS?=.

  • Problem context

    • Static-node runs effectively had JVM_OPTIONS unset, so variation-derived options were applied.
    • Dynamic/container runs had JVM_OPTIONS explicitly set to empty, causing an unintended empty override.
  • Pipeline behavior update (buildenv/jenkins/JenkinsfileBase)

    • In makeTest(...), conditionally prepend unset JVM_OPTIONS; before invoking make when env.JVM_OPTIONS is null/empty/whitespace.
    • In genParallelList(...), apply the same conditional unset JVM_OPTIONS; before make genParallelList for consistency in dynamic parallel flows.
  • Behavioral impact

    • Empty/unset job parameter now behaves as “no override,” allowing playlist variation args to flow through.
    • Non-empty JVM_OPTIONS continues to intentionally override variation/default options.
if (!env.JVM_OPTIONS?.trim()) {
    makeTestCmd = "unset JVM_OPTIONS; $makeTestCmd"
}

Copilot AI changed the title [WIP] Fix SystemTests to honour playlist variation mode on dynamic test agents Handle empty JVM_OPTIONS correctly on dynamic/container agents Jul 14, 2026
Copilot AI requested a review from andrew-m-leonard July 14, 2026 09:06
Comment thread buildenv/jenkins/JenkinsfileBase
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

SystemTests failing to honour playlist variation mode on Dynamic Test Agents TestJlm* testcase failures due to CertificateException

2 participants