[GitHub] [flink-kubernetes-operator] tamirsagi commented on a diff in pull request #536: [FLINK-31140] - Load additional dependencies in operator classpath

["tamirsagi (via GitHub)" <gi...@apache.org>]

tamirsagi commented on code in PR #536:
URL: https://github.com/apache/flink-kubernetes-operator/pull/536#discussion_r1121652649


##########
Dockerfile:
##########
@@ -36,13 +36,16 @@ RUN cd /app/tools/license; mkdir jars; cd jars; \
 # stage
 FROM eclipse-temurin:11-jre-jammy
 ENV FLINK_HOME=/opt/flink
-ENV FLINK_PLUGINS_DIR=/opt/flink/plugins
+ENV FLINK_PLUGINS_DIR=$FLINK_HOME/plugins
 ENV OPERATOR_VERSION=1.5-SNAPSHOT
 ENV OPERATOR_JAR=flink-kubernetes-operator-$OPERATOR_VERSION-shaded.jar
 ENV AUTOSCALER_JAR=flink-kubernetes-operator-autoscaler-$OPERATOR_VERSION.jar
 ENV WEBHOOK_JAR=flink-kubernetes-webhook-$OPERATOR_VERSION-shaded.jar
 ENV FLINK_KUBERNETES_SHADED_JAR=flink-kubernetes-standalone-$OPERATOR_VERSION-shaded.jar
 
+ENV USER_DEPENDENCIES_DIR=$FLINK_HOME/dependencies
+RUN mkdir -p $USER_DEPENDENCIES_DIR

Review Comment:
   @gyfora  what do you think about this?



-- 
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: issues-unsubscribe@flink.apache.org

For queries about this service, please contact Infrastructure at:
users@infra.apache.org