Skip to content

RANGER-5775: Fix docker compose warnings - #1241

Open
kumaab wants to merge 1 commit into
apache:masterfrom
kumaab:RANGER-5775-docker-env-warnings
Open

kumaab wants to merge 1 commit into
apache:masterfrom
kumaab:RANGER-5775-docker-env-warnings

Conversation

@kumaab

@kumaab kumaab commented Sep 20, 2026

Copy link
Copy Markdown
Contributor

What changes were proposed in this pull request?

RANGER_JVM_METASPACE and RANGER_JVM_MAX_METASPACE are documented in dev-support/ranger-docker/.env as optional tuning knobs and are left commented out, so the services fall back to the upstream defaults (100m / 200m).

docker-compose.ranger-audit-service.yml interpolated them as ${RANGER_JVM_METASPACE} without a default, so every compose invocation printed:

level=warning msg="The "RANGER_JVM_METASPACE" variable is not set.
Defaulting to a blank string."

Ten such warnings appeared on each CI run (5 per variable: the audit-ingestor service, the x-audit-dispatcher-env-common anchor and the three dispatcher services that merge it).

The other compose files pass these variables through with the list form (- RANGER_JVM_METASPACE), which does not interpolate and therefore does not warn. Use ${VAR:-} here so the optional-knob semantics are kept without the warning.

How was this patch tested?

No warnings seen in CI stage: services-docker-build: https://github.com/kumaab/ranger/actions/runs/35481593938/job/106003841221

…CE variables

RANGER_JVM_METASPACE and RANGER_JVM_MAX_METASPACE are documented in
dev-support/ranger-docker/.env as optional tuning knobs and are left
commented out, so the services fall back to the upstream defaults
(100m / 200m). docker-compose.ranger-audit-service.yml interpolated them
as ${RANGER_JVM_METASPACE} without a default, so every compose
invocation printed:

  level=warning msg="The \"RANGER_JVM_METASPACE\" variable is not set.
  Defaulting to a blank string."

Ten such warnings appeared on each CI run (5 per variable: the
audit-ingestor service, the x-audit-dispatcher-env-common anchor and the
three dispatcher services that merge it).

The other compose files pass these variables through with the list form
(- RANGER_JVM_METASPACE), which does not interpolate and therefore does
not warn. Use ${VAR:-} here so the optional-knob semantics are kept
without the warning. Verified that "docker compose config" output is
byte-identical before and after, for both the core and the plugin stacks
used by CI, and that both now run with zero warnings.
@kumaab kumaab self-assigned this Sep 20, 2026
@kumaab
kumaab requested a review from mneethiraj September 20, 2026 02:45
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.

2 participants