Skip to content

RANGER-5583: Fix Ranger PDP startup and installation in docker#934

Merged
kumaab merged 1 commit intoapache:masterfrom
kumaab:ranger_5583
May 8, 2026
Merged

RANGER-5583: Fix Ranger PDP startup and installation in docker#934
kumaab merged 1 commit intoapache:masterfrom
kumaab:ranger_5583

Conversation

@kumaab
Copy link
Copy Markdown
Contributor

@kumaab kumaab commented May 8, 2026

What changes were proposed in this pull request?

Fixes ranger-pdp startup and installation in docker container.

How was this patch tested?

export RANGER_DB_TYPE=postgres
docker compose -f docker-compose.ranger.yml -f docker-compose.ranger-pdp.yml build
docker compose -f docker-compose.ranger.yml -f docker-compose.ranger-pdp.yml up -d
  • Verified no errors are seen in application logs for pdp server.

@kumaab kumaab requested a review from mneethiraj May 8, 2026 00:46
@kumaab kumaab self-assigned this May 8, 2026
@kumaab kumaab requested a review from pradeepagrawal8184 May 8, 2026 00:46
Copy link
Copy Markdown
Contributor

Copilot AI left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Pull request overview

Fixes ranger-pdp startup/installation in the Docker-based environment by ensuring required runtime libraries are available in the PDP build and assembled distribution, and by aligning the PDP compose network definition with the main Ranger compose setup.

Changes:

  • Add missing JAXB and Activation API dependencies to the ranger-pdp Maven module.
  • Ensure the PDP distro assembly explicitly includes the JAXB/Activation API JARs.
  • Adjust Docker Compose network configuration for the PDP compose file to avoid requiring an external pre-created network.

Reviewed changes

Copilot reviewed 4 out of 4 changed files in this pull request and generated no comments.

File Description
pdp/pom.xml Adds JAXB/Activation API dependencies needed for PDP runtime/classloading.
distro/src/main/assembly/pdp.xml Ensures JAXB/Activation API JARs are included in the PDP distribution lib/.
dev-support/ranger-docker/docker-compose.ranger-pdp.yml Removes external: true so the PDP compose setup works when combined with the main Ranger compose file.
agents-audit/dest-solr/pom.xml Adds an explicit commons-logging dependency for the Solr audit destination module.

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

@kumaab kumaab merged commit ba978cf into apache:master May 8, 2026
6 of 7 checks passed
@kumaab kumaab deleted the ranger_5583 branch May 8, 2026 01:41
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.

3 participants