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 docker to user group & delete unnecessary lines #133

Merged
merged 3 commits into from
Aug 25, 2023
Merged

Conversation

helenwangjia
Copy link
Contributor

@helenwangjia helenwangjia commented Aug 25, 2023

close #131

  • ユーザーグループにdockerを加える
  • edge_tcpdumpの行を削除
  • sudo不要でビルドできるように変更

テストした結果:

sectu@sensor:~$ /usr/local/bin/docker-compose build
中略
Successfully built 7a50183aa581
Successfully tagged cron:revxxx
sectu@sensor:~/osect_sensor$ /usr/local/bin/docker-compose up -d
Recreating osect_sensor_edge_cron_1 ... done

通りました

@helenwangjia helenwangjia changed the title delete unnecessary sudo for docker build add docker to user group & delete unnecessary lines Aug 25, 2023
@@ -43,7 +43,8 @@ Docker CEをインストールします。

```bash
$ sudo apt install -y docker-ce
$ sudo docker container run --rm hello-world
$ sudo usermod -aG docker <username>
Copy link
Contributor Author

Choose a reason for hiding this comment

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

ユーザーグループにdockerを加える

Comment on lines -171 to -172
$ SK=`cat /dev/urandom | base64 | fold -w 64 | head -n 1`; sed -i -e 's@SECRET_KEY = ""@SECRET_KEY = "'$SK'"@g' ~/osect_sensor/Application/edge_tcpdump/sc_tcpdump/settings.py
(何も表示されません。)
Copy link
Contributor Author

Choose a reason for hiding this comment

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

edge_tcpdumpの行を削除

Comment on lines +208 to +209
$ /usr/local/bin/docker-compose build
$ /usr/local/bin/docker-compose up -d
Copy link
Contributor Author

Choose a reason for hiding this comment

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

sudo不要でビルドできるように変更

@takuma0121 takuma0121 merged commit c57daa8 into main Aug 25, 2023
@takuma0121 takuma0121 deleted the update-readme branch August 25, 2023 05:49
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.

READMEの修正
2 participants