-
Notifications
You must be signed in to change notification settings - Fork 0
deploy_script_host #59
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
Conversation
@katyasots протестируйте, пожалуйста |
проблема с созданием директории run |
нужно создать пользователя |
я вижу, что этот код закомментирован, а его нельзя внесни в скрипт? |
|
|
ну, можно добавить опцию создания юзера... но будто это излишне |
deploy_service.sh
Outdated
# sudo useradd -u 1001 -g service service | ||
# sudo useradd -u 1001 -g service service | ||
|
||
mkdir -p /run/controller | ||
chown service:service /run/controller |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Давайте вынесем создание директории в Dockerfile. Кажется что мы можем вполне задать нужные права и доступы к сокету на уровне докера, а не в скрипте хоста
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Почему не дописать этот docker-compose: https://github.com/moevm/grpc_server/blob/main/docker-compose.yml ?
И не плодить разные compose файлы
options: | ||
loki-timeout: 10s | ||
no-file: "true" | ||
loki-url: "http://localhost:3100/loki/api/v1/push" |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Хардкод портов -- плохая история, нужно вынести в env хотя бы внутри самого docker-compose файла
pushgateway: | ||
image: prom/pushgateway:latest | ||
ports: | ||
- "9091:9091" |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Аналогичный комментарий про порты
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
И в целом про все порты в PR
Раз мы условились работать в контейнере, то можно там делать пользователей и все необходимые действия для доступа к сокету |
No description provided.