Removed all http_proxy reference from build. 43/97743/1
authorSebastien Premont-Tendland <sebastien.premont@bell.ca>
Tue, 29 Oct 2019 21:32:54 +0000 (17:32 -0400)
committerSebastien Premont-Tendland <sebastien.premont@bell.ca>
Wed, 30 Oct 2019 14:06:54 +0000 (10:06 -0400)
http_proxy is an environment configuration and should not be defined in
in the build script.
If you need to use a proxy to build docker images then add it to
the mvn command like this :

-Ddocker.buildArg.http_proxy=$PROXY -Ddocker.buildArg.https_proxy=$PROXY

Also adding http_proxy in the Dockerfile directly will add the
environment variable in the POD statically using the proxy the was
built with at runtime.

If a proxy is needed at runtime then the environment variables should be
injecting using helm charts and overrides.

Issue-ID: CCSDK-1877
Signed-off-by: Sebastien Premont-Tendland <sebastien.premont@bell.ca>
Change-Id: I2b99b277b594408c974cf29a5804384293776a22

ms/blueprintsprocessor/application/pom.xml
ms/blueprintsprocessor/application/src/main/docker/Dockerfile
ms/command-executor/src/main/docker/Dockerfile
ms/py-executor/docker/Dockerfile
ms/sdclistener/distribution/src/main/docker/Dockerfile

index 3a95e8c..0ee6ac3 100755 (executable)
@@ -38,7 +38,6 @@
         <name.space>org.onap.ccsdk.cds</name.space>
         <serviceArtifactName>blueprintsprocessor</serviceArtifactName>
         <image.name>onap/ccsdk-blueprintsprocessor</image.name>
-        <docker.buildArg.https_proxy>${https_proxy}</docker.buildArg.https_proxy>
         <docker.push.phase>deploy</docker.push.phase>
         <docker.verbose>true</docker.verbose>
         <ccsdk.project.version>${project.version}</ccsdk.project.version>
index dab0a4c..207cec5 100755 (executable)
@@ -1,8 +1,5 @@
 FROM omahoco1/alpine-java-python
 
-ENV HTTP_PROXY  ${HTTP_PROXY}
-ENV HTTPS_PROXY ${HTTPS_PROXY}
-
 # add entrypoint
 COPY run.source /etc/run.source
 COPY startService.sh /startService.sh
@@ -15,4 +12,4 @@ RUN tar -xzf /source.tar.gz -C /tmp \
  && rm -rf /source.tar.gz \
  && rm -rf /tmp/@project.build.finalName@
 
-ENTRYPOINT /startService.sh
\ No newline at end of file
+ENTRYPOINT /startService.sh
index c0458bd..b28e580 100644 (file)
@@ -1,8 +1,5 @@
 FROM python:3.6-slim
 
-ENV HTTP_PROXY  ${HTTP_PROXY}
-ENV HTTPS_PROXY ${HTTPS_PROXY}
-
 ENV GRPC_PYTHON_VERSION 1.20.0
 RUN python -m pip install --upgrade pip
 RUN pip install grpcio==${GRPC_PYTHON_VERSION} grpcio-tools==${GRPC_PYTHON_VERSION}
@@ -21,4 +18,4 @@ RUN tar -xzf /source.tar.gz -C /tmp \
 
 VOLUME /opt/app/onap/blueprints/deploy/
 
-ENTRYPOINT /opt/app/onap/start.sh
\ No newline at end of file
+ENTRYPOINT /opt/app/onap/start.sh
index b49daf6..9e86cc8 100644 (file)
@@ -1,8 +1,5 @@
 FROM python:3.7-slim
 
-ENV HTTP_PROXY  ${HTTP_PROXY}
-ENV HTTPS_PROXY ${HTTPS_PROXY}
-
 RUN mkdir -p /opt/app/onap/logs/ && touch /opt/app/onap/logs/application.log
 
 COPY @project.build.finalName@-@assembly.id@.tar.gz /source.tar.gz
@@ -15,4 +12,4 @@ RUN pip install --no-cache-dir -r /opt/app/onap/python/requirements.txt
 
 VOLUME /opt/app/onap/blueprints/deploy/
 
-ENTRYPOINT /opt/app/onap/python/start.sh
\ No newline at end of file
+ENTRYPOINT /opt/app/onap/python/start.sh
index e945188..34f6443 100755 (executable)
@@ -1,8 +1,5 @@
 FROM openjdk:8-jdk-alpine
 
-ENV HTTP_PROXY  ${HTTP_PROXY}
-ENV HTTPS_PROXY ${HTTPS_PROXY}
-
 # add entrypoint
 COPY startService.sh /startService.sh
 RUN chmod 751 /startService.sh