[GitHub] [beam] TSultanov commented on a diff in pull request #24946: [Playground] Resolve issue with SCIO examples failing on start due to a timeout

[GitBox <gi...@apache.org>]

TSultanov commented on code in PR #24946:
URL: https://github.com/apache/beam/pull/24946#discussion_r1082318716


##########
playground/backend/containers/scio/Dockerfile:
##########
@@ -84,4 +83,18 @@ RUN chown -R appuser:appgroup /opt/playground/backend/executable_files/ \
 # Switch to appuser
 USER appuser
 
+# Let sbt download files from Maven
+RUN mkdir -p /tmp/sbt-initialize
+WORKDIR /tmp/sbt-initialize
+RUN /opt/playground/backend/new_scio_project.sh
+WORKDIR /tmp/sbt-initialize/scio
+RUN sbt "+compile"
+WORKDIR /
+RUN rm -r /tmp/sbt-initialize
+
+# Enable mitmproxy
+ENV HTTP_PROXY="http://127.0.0.1:8081"
+ENV HTTPS_PROXY="http://127.0.0.1:8081"
+ENV SBT_OPTS="-Xmx512M -XX:+UseG1GC -XX:+UseStringDeduplication"

Review Comment:
   It's being used to filter out which Google Cloud Storage buckets are accessible by the examples.
   
   However, after investigating a little bit it seems that this is not the best place to have it. We discussed this with @MakarkinSAkvelon and think that the best solution would be to have a single proxy node which would act as a reverse proxy for letting external connections to the runners and also as a single point though which runners would be able to only access the allowlisted hosts. 
   
   This approach with a single proxy node will simplify origin policy for frontend and we will be able to get rid of [the need to generate](https://github.com/apache/beam/pull/25022) `config.g.dart` completely.



-- 
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: github-unsubscribe@beam.apache.org

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