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

Fix docker build readme and shell #64

Open
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

inductor
Copy link

Docker pullのイメージ名が古いままだったのと、Linux上でbuildするとコンテナ内ユーザーのrootでファイルが書き換わってしまい、ホスト側で困ったのでそれを修正する対応を入れています。

本質的にはDockerイメージで実行ユーザーをnon-rootでやるべきなのですが、ちょっと根が深そうなのでshellの中できる範囲で対応を入れました。

macOSでの動作は確認できていない(手元にmacがない)ため、ご確認いただけると幸いです 🙏

@kmuto
Copy link
Collaborator

kmuto commented Dec 25, 2020

私は1 ContributorなのでDeveloper判断に委ねますが、
「dockerグループには入っているのでdocker起動はできるけれども、sudo権限はない」というケースの場合はどうするのしょう?

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