Skip to content

Commit 222c48f

Browse files
committed
Remove MONAI user ownership
1 parent 9a5ca3f commit 222c48f

File tree

2 files changed

+15
-20
lines changed

2 files changed

+15
-20
lines changed

monai/deploy/packager/templates.py

Lines changed: 13 additions & 18 deletions
Original file line numberDiff line numberDiff line change
@@ -14,42 +14,37 @@
1414
1515
RUN pip install --no-cache-dir --upgrade setuptools==57.4.0 pip==21.2.4 wheel==0.37.0
1616
17-
RUN groupadd -g $MONAI_GID -o -r monai
18-
RUN useradd -g $MONAI_GID -u $MONAI_UID -m -o -r monai
19-
20-
RUN mkdir -p /etc/monai/ && chown -R monai:monai /etc/monai \\
21-
&& mkdir -p /opt/monai/ && chown -R monai:monai /opt/monai \\
22-
&& mkdir -p {working_dir} && chown -R monai:monai {working_dir} \\
23-
&& mkdir -p {app_dir} && chown -R monai:monai {app_dir} \\
24-
&& mkdir -p {executor_dir} && chown -R monai:monai {executor_dir} \\
25-
&& mkdir -p {full_input_path} && chown -R monai:monai {full_input_path} \\
26-
&& mkdir -p {full_output_path} && chown -R monai:monai {full_output_path} \\
27-
&& mkdir -p {models_dir} && chown -R monai:monai {models_dir}
17+
RUN mkdir -p /etc/monai/ \\
18+
&& mkdir -p /opt/monai/ \\
19+
&& mkdir -p {working_dir} \\
20+
&& mkdir -p {app_dir} \\
21+
&& mkdir -p {executor_dir} \\
22+
&& mkdir -p {full_input_path} \\
23+
&& mkdir -p {full_output_path} \\
24+
&& mkdir -p {models_dir}
2825
2926
{models_string}
3027
31-
COPY --chown=monai:monai ./pip/requirements.txt {map_requirements_path}
28+
COPY ./pip/requirements.txt {map_requirements_path}
3229
3330
RUN curl {executor_url} -o {executor_dir}/executor.zip \\
3431
&& unzip {executor_dir}/executor.zip -d {executor_dir}/executor_pkg \\
3532
&& mv {executor_dir}/executor_pkg/lib/native/linux-x64/* {executor_dir} \\
3633
&& rm -f {executor_dir}/executor.zip \\
3734
&& rm -rf {executor_dir}/executor_pkg \\
38-
&& chown -R monai:monai {executor_dir} \\
3935
&& chmod +x {executor_dir}/monai-exec
4036
41-
USER monai
4237
ENV PATH=/home/monai/.local/bin:$PATH
4338
4439
RUN pip install --no-cache-dir --upgrade -r {map_requirements_path}
4540
4641
# Override monai-deploy-app-sdk module
47-
COPY --chown=monai:monai ./monai-deploy-app-sdk /home/monai/.local/lib/python3.8/site-packages/monai/deploy/
42+
COPY ./monai-deploy-app-sdk /home/monai/.local/lib/python3.8/site-packages/monai/deploy/
4843
49-
COPY --chown=monai:monai ./map/app.json /etc/monai/
50-
COPY --chown=monai:monai ./map/pkg.json /etc/monai/
44+
COPY ./map/app.json /etc/monai/
45+
COPY ./map/pkg.json /etc/monai/
5146
52-
COPY --chown=monai:monai ./app {app_dir}
47+
COPY ./app {app_dir}
5348
5449
# Create bytecodes for monai and app's code. This would help speed up loading time a little bit.
5550
RUN python -m compileall -q -j 0 /home/monai/.local/lib/python3.8/site-packages/monai /opt/monai/app

monai/deploy/packager/util.py

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -194,8 +194,8 @@ def build_image(args: dict, temp_dir: str):
194194
else:
195195
shutil.copytree(model_path, dest_model_path)
196196

197-
models_string = f"RUN mkdir -p {models_dir} && chown -R monai:monai {models_dir}\n"
198-
models_string += f"COPY --chown=monai:monai ./models {models_dir}\n"
197+
models_string = f"RUN mkdir -p {models_dir}\n"
198+
models_string += f"COPY ./models {models_dir}\n"
199199

200200
# Dockerfile template
201201
template_params = {

0 commit comments

Comments
 (0)