Skip to content

Commit 07f1582

Browse files
authored
Merge pull request #8492 from liranmauda/liran-update-dockerfiles
Update the Dockerfiles
2 parents 64336ec + 6ece4f9 commit 07f1582

File tree

5 files changed

+20
-19
lines changed

5 files changed

+20
-19
lines changed

src/deploy/NVA_build/Base.Dockerfile

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -1,4 +1,4 @@
1-
FROM noobaa-builder
1+
FROM noobaa-builder AS noobaa-base
22

33
######################################################################
44
# Layers:

src/deploy/NVA_build/NooBaa.Dockerfile

Lines changed: 12 additions & 12 deletions
Original file line numberDiff line numberDiff line change
@@ -1,5 +1,5 @@
11
ARG CENTOS_VER=9
2-
FROM noobaa-base as server_builder
2+
FROM noobaa-base AS server_builder
33

44
RUN mkdir -p /noobaa_init_files && \
55
cp -p ./build/Release/kube_pv_chown /noobaa_init_files
@@ -39,18 +39,18 @@ RUN tar \
3939
# Cache: Rebuild when any layer is changing
4040
##############################################################
4141

42-
FROM quay.io/centos/centos:stream${CENTOS_VER}
42+
FROM quay.io/centos/centos:stream${CENTOS_VER} AS noobaa
4343

4444
# The ports are overridden for Ceph Test later
45-
ENV container docker
46-
ENV PORT 8080
47-
ENV SSL_PORT 8443
48-
ENV ENDPOINT_PORT 6001
49-
ENV ENDPOINT_SSL_PORT 6443
50-
ENV WEB_NODE_OPTIONS ''
51-
ENV BG_NODE_OPTIONS ''
52-
ENV HOSTED_AGENTS_NODE_OPTIONS ''
53-
ENV ENDPOINT_NODE_OPTIONS ''
45+
ENV container=docker
46+
ENV PORT=8080
47+
ENV SSL_PORT=8443
48+
ENV ENDPOINT_PORT=6001
49+
ENV ENDPOINT_SSL_PORT=6443
50+
ENV WEB_NODE_OPTIONS=''
51+
ENV BG_NODE_OPTIONS=''
52+
ENV HOSTED_AGENTS_NODE_OPTIONS=''
53+
ENV ENDPOINT_NODE_OPTIONS=''
5454

5555
##############################################################
5656
# Layers:
@@ -153,7 +153,7 @@ EXPOSE 27000
153153
EXPOSE 26050
154154

155155
# Needs to be added only after installing jemalloc in dependencies section (our env section is before) - otherwise it will fail
156-
ENV LD_PRELOAD /usr/lib64/libjemalloc.so.2
156+
ENV LD_PRELOAD=/usr/lib64/libjemalloc.so.2
157157

158158
#RUN mkdir -p /nsfs/fs1/amitpb && chmod -R 777 /nsfs/
159159
#RUN mkdir -p /nsfsAA/fs1/amitpb && chmod -R 777 /nsfsAA/

src/deploy/NVA_build/Tests.Dockerfile

Lines changed: 5 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -1,9 +1,9 @@
1-
FROM noobaa
1+
FROM noobaa AS noobaa-tester
22

33
USER 0:0
44

5-
ENV container docker
6-
ENV TEST_CONTAINER true
5+
ENV container=docker
6+
ENV TEST_CONTAINER=true
77

88
##############################################################
99
# Layers:
@@ -13,6 +13,7 @@ ENV TEST_CONTAINER true
1313
##############################################################
1414

1515
RUN dnf config-manager --enable crb || true
16+
RUN dnf clean all
1617

1718
RUN dnf group install -y -q "Development Tools" && \
1819
dnf install -y -q --nogpgcheck vim \
@@ -62,7 +63,7 @@ COPY .eslintignore /root/node_modules/noobaa-core
6263
# Making mocha accessible
6364
RUN ln -s /root/node_modules/noobaa-core/node_modules/mocha/bin/mocha.js /usr/local/bin
6465

65-
ENV SPAWN_WRAP_SHIM_ROOT /data
66+
ENV SPAWN_WRAP_SHIM_ROOT=/data
6667
RUN mkdir -p /data && \
6768
chgrp -R 0 /data && \
6869
chmod -R g=u /data

src/deploy/NVA_build/builder.Dockerfile

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -1,5 +1,5 @@
11
ARG CENTOS_VER=9
2-
FROM quay.io/centos/centos:stream${CENTOS_VER}
2+
FROM quay.io/centos/centos:stream${CENTOS_VER} AS noobaa-builder
33
#Needs to reapply ARG, it was cleaned by FROM command.
44
ARG CENTOS_VER
55
LABEL maintainer="Liran Mauda (lmauda@redhat.com)"

src/deploy/NVA_build/dev.Dockerfile

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -2,7 +2,7 @@
22
ARG CENTOS_VER=9
33
FROM quay.io/centos/centos:stream${CENTOS_VER}
44

5-
ENV container docker
5+
ENV container=docker
66

77
RUN dnf update -y -q && \
88
dnf install -y -q \

0 commit comments

Comments
 (0)