Skip to content

RANGER-4746: Fix ranger-jdk11 build profile #301

Merged
kumaab merged 2 commits intoapache:masterfrom
kumaab:RANGER-4746
Mar 22, 2024
Merged

RANGER-4746: Fix ranger-jdk11 build profile #301
kumaab merged 2 commits intoapache:masterfrom
kumaab:RANGER-4746

Conversation

@kumaab
Copy link
Copy Markdown
Contributor

@kumaab kumaab commented Mar 13, 2024

What changes were proposed in this pull request?

The ranger profile ranger-jdk11 has been added to build trino with jdk 11 as currently trino plugin is the only plugin to have a compile time dependency on jdk 11. However, it currently builds the full ranger repo. The PR aims to fix this. When modules have a jdk 11 requirement they can be added to this profile.

How was this patch tested?

  • Bring up the containers
    • Build with JDK 11: mvn clean package -DskipTests -P ranger-jdk11
    • cp target/ranger-admin.tar.gz dev-support/ranger-docker/dist
    • cp target/ranger-trino-plugin.tar.gz dev-support/ranger-docker/dist/
    • cd dev-support/ranger-docker
    • docker-compose -f docker-compose.ranger.yml -f docker-compose.ranger-postgres.yml -f docker-compose.ranger-trino.yml build
    • docker-compose -f docker-compose.ranger.yml -f docker-compose.ranger-postgres.yml up -d
    • docker-compose -f docker-compose.ranger.yml -f docker-compose.ranger-postgres.yml -f docker-compose.ranger-trino.yml up -d
  • Verified ranger and ranger-trino are successfully up in containers.
  • Verified ranger policy creation, policies created for trino are working.
  • Verified audits to Solr are visible in ranger UI.

@kumaab kumaab changed the title RANGER-4746: Fix ranger-jdk11 profile to only have the required modules RANGER-4746: Fix ranger-jdk11 build profile Mar 13, 2024
@kumaab kumaab self-assigned this Mar 17, 2024
@kumaab
Copy link
Copy Markdown
Contributor Author

kumaab commented Mar 22, 2024

Verified that all tests pass with : mvn clean package -P ranger-jdk11

@kumaab kumaab added the testing complete PR testing is complete label Mar 22, 2024
@kumaab kumaab merged commit a1dc832 into apache:master Mar 22, 2024
@kumaab kumaab deleted the RANGER-4746 branch March 22, 2024 16:41
mneethiraj pushed a commit that referenced this pull request Jun 20, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

testing complete PR testing is complete

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants