Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Add dedicated user #10

Merged
merged 2 commits into from
Jan 4, 2023
Merged

Add dedicated user #10

merged 2 commits into from
Jan 4, 2023

Conversation

areedv
Copy link
Contributor

@areedv areedv commented Jan 4, 2023

No description provided.

Copy link
Contributor Author

@areedv areedv left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Dette funker fint hos meg, også med en liten justering ihht kommentar i koden

Dockerfile Outdated
@@ -27,7 +27,9 @@ RUN install2.r --error --skipinstalled --ncpus -1 \

EXPOSE 3838

RUN chown -R 1000:1000 /app/R && \
RUN adduser --uid "1000" --disabled-password --no-create-home rapporteket && \
Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Hvis man kutter ut --no-create-home (slik at /home/rapporteket faktisk opprettes) så slipper man advarselen

Warning in dir.create(cache_dir, recursive = TRUE) :
cannot create dir '/home/rapporteket', reason 'Permission denied'

Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Tok vekk --no-create-home fra min fork

@areedv areedv merged commit 5408317 into Rapporteket:main Jan 4, 2023
sighanse added a commit to sighanse/rapadm that referenced this pull request Jan 4, 2023
Merge pull request Rapporteket#10 from sighanse/main
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants