mneethiraj commented on code in PR #11:
URL: https://github.com/apache/ranger-tools/pull/11#discussion_r3724622649


##########
docker/Dockerfile:
##########
@@ -17,35 +17,32 @@
 # see https://hub.docker.com/_/eclipse-temurin/tags
 ARG RANGER_BASE_JAVA_VERSION=8
 
-# Ubuntu 22.04 LTS
-FROM eclipse-temurin:${RANGER_BASE_JAVA_VERSION}-jdk-jammy
+# Ubuntu 26.04 LTS
+FROM eclipse-temurin:${RANGER_BASE_JAVA_VERSION}-jdk-resolute
 
-# Install packages
 RUN apt update -q \
     && DEBIAN_FRONTEND="noninteractive" apt install -y --no-install-recommends 
\
         bc \
         curl \
         iputils-ping \
         pdsh \
-        python3 \
-        python3-pip \
-        python-is-python3 \
         ssh \
         tzdata \
         vim \
         xmlstarlet \
         krb5-user \
-    && apt clean
-
-# Install Python modules
-RUN pip install apache-ranger requests \
-    && rm -rf ~/.cache/pip
+        python3-venv \
+    &&  apt clean \
+    && rm -rf /var/lib/apt/lists/*
 
 # Set environment variables
 ENV RANGER_DIST=/home/ranger/dist
 ENV RANGER_SCRIPTS=/home/ranger/scripts
 ENV RANGER_HOME=/opt/ranger
-ENV PATH=/usr/local/sbin:/usr/local/bin:/usr/sbin:/usr/bin:/sbin:/bin
+ENV 
PATH="/opt/venv/bin:${JAVA_HOME}/bin:/usr/local/sbin:/usr/local/bin:/usr/sbin:/usr/bin:/sbin:/bin"
+
+RUN python3 -m venv /opt/venv
+RUN /opt/venv/bin/pip install apache-ranger

Review Comment:
   Is it not necessary to install `requests` library? See line #41 in the 
original file.



-- 
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

To unsubscribe, e-mail: [email protected]

For queries about this service, please contact Infrastructure at:
[email protected]

Reply via email to