Skip to content

Commit

Permalink
Merge pull request #5 from ExposuresProvider/add-user
Browse files Browse the repository at this point in the history
Sterling had been having some permission issues when trying to run cam-pipeline earlier this month. I'm not sure what caused it, but adding an nru (non-root user) and changing the permissions on the PVC so that /current and /success are both owned by this user seems to have fixed it. Also, running this job as a non-root user is better from a security perspective.
  • Loading branch information
gaurav authored Jan 27, 2023
2 parents 1a58115 + 214494a commit 2d8123c
Showing 1 changed file with 4 additions and 0 deletions.
4 changes: 4 additions & 0 deletions Dockerfile
Original file line number Diff line number Diff line change
Expand Up @@ -51,3 +51,7 @@ RUN curl -O -L https://github.com/balhoff/ctd-to-owl/releases/download/v$CTD/ctd
&& tar -zxf ctd-to-owl-$CTD.tgz \
&& chmod +x /tools/ctd-to-owl-$CTD
ENV PATH "/tools/ctd-to-owl-$CTD/bin:$PATH"

## Set USER to nru
RUN useradd -m -u 1000 nru
USER nru

0 comments on commit 2d8123c

Please sign in to comment.