Skip to content

Fix docker build readme and shell#64

Open
inductor wants to merge 1 commit intoTechBooster:masterfrom
inductor:fix/docker
Open

Fix docker build readme and shell#64
inductor wants to merge 1 commit intoTechBooster:masterfrom
inductor:fix/docker

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