Skip to content

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

Merged
merged 8 commits into from
Jun 19, 2025
Merged

deploy_script_host #59

merged 8 commits into from
Jun 19, 2025

Conversation

NiksBrn
Copy link
Collaborator

@NiksBrn NiksBrn commented May 20, 2025

No description provided.

@NiksBrn NiksBrn linked an issue May 20, 2025 that may be closed by this pull request
@NiksBrn NiksBrn changed the title deploy_script deploy_script_host May 20, 2025
@AndrewGavril AndrewGavril requested a review from katyasots May 28, 2025 13:24
@AndrewGavril
Copy link
Collaborator

@katyasots протестируйте, пожалуйста

@katyasots
Copy link
Collaborator

katya@katya:~/qemu-riscv-cluster$ sudo ./deploy_service.sh start
chown: неверный пользователь: «service:service»

проблема с созданием директории run

@NiksBrn
Copy link
Collaborator Author

NiksBrn commented May 28, 2025

katya@katya:~/qemu-riscv-cluster$ sudo ./deploy_service.sh start
chown: неверный пользователь: «service:service»

проблема с созданием директории run

нужно создать пользователя
sudo useradd -u 1001 -g service service

@katyasots
Copy link
Collaborator

katya@katya:~/qemu-riscv-cluster$ sudo ./deploy_service.sh start
chown: неверный пользователь: «service:service»

проблема с созданием директории run

нужно создать пользователя sudo useradd -u 1001 -g service service

я вижу, что этот код закомментирован, а его нельзя внесни в скрипт?

@katyasots
Copy link
Collaborator

sudo useradd -u 1001 -g service service

katya@katya:~/qemu-riscv-cluster$ sudo useradd -u 1001 -g service service
useradd: группа «service» не существует

@NiksBrn
Copy link
Collaborator Author

NiksBrn commented May 28, 2025

sudo useradd -u 1001 -g service service

katya@katya:~/qemu-riscv-cluster$ sudo useradd -u 1001 -g service service
useradd: группа «service» не существует
sudo groupadd -g 1001 service
sudo useradd -u 1001 -g service service

@NiksBrn
Copy link
Collaborator Author

NiksBrn commented May 28, 2025

katya@katya:~/qemu-riscv-cluster$ sudo ./deploy_service.sh start
chown: неверный пользователь: «service:service»

проблема с созданием директории run

нужно создать пользователя sudo useradd -u 1001 -g service service

я вижу, что этот код закомментирован, а его нельзя внесни в скрипт?

ну, можно добавить опцию создания юзера... но будто это излишне

Comment on lines 26 to 30
# sudo useradd -u 1001 -g service service
# sudo useradd -u 1001 -g service service

mkdir -p /run/controller
chown service:service /run/controller
Copy link
Collaborator

Choose a reason for hiding this comment

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

Давайте вынесем создание директории в Dockerfile. Кажется что мы можем вполне задать нужные права и доступы к сокету на уровне докера, а не в скрипте хоста

Copy link
Collaborator

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"
Copy link
Collaborator

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"
Copy link
Collaborator

Choose a reason for hiding this comment

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

Аналогичный комментарий про порты

Copy link
Collaborator

Choose a reason for hiding this comment

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

И в целом про все порты в PR

@AndrewGavril
Copy link
Collaborator

AndrewGavril commented Jun 5, 2025

katya@katya:~/qemu-riscv-cluster$ sudo ./deploy_service.sh start
chown: неверный пользователь: «service:service»

проблема с созданием директории run

нужно создать пользователя sudo useradd -u 1001 -g service service

я вижу, что этот код закомментирован, а его нельзя внесни в скрипт?

ну, можно добавить опцию создания юзера... но будто это излишне

Раз мы условились работать в контейнере, то можно там делать пользователей и все необходимые действия для доступа к сокету

@AndrewGavril AndrewGavril merged commit 01f910f into main Jun 19, 2025
2 checks passed
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.

Скрипт для запуска системы на хосте
5 participants