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

Установка прав 755 для gitsync.exe #6

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

Conversation

igostv
Copy link
Contributor

@igostv igostv commented Dec 20, 2019

No description provided.

@igostv
Copy link
Contributor Author

igostv commented Dec 20, 2019

У файла по умолчанию права 700 и прав на запуск под не рутом не хватает (нет прав на чтение)

@jugatsu
Copy link
Owner

jugatsu commented Dec 20, 2019

А ведь по дефолту в образе нет непривилегированного пользователя. Под кем запускается и как?)

@nixel2007
Copy link
Collaborator

Образ используется для подключения в качестве агента дженкинса через yet another docker plugin/k8s

@jugatsu
Copy link
Owner

jugatsu commented Dec 21, 2019

chmod к сожалению пока никак не добавят к командам COPY/ADD moby/moby#34819

Может как раз под данный кейс и добавить создание пользователя gitsync по аналогии с https://github.com/jugatsu/onec-docker/blob/master/server/Dockerfile#L61

Для примера (код не тестил :) )

ARG app_uid="65000"
ARG app_gid="65000"

ENV APP_USER gitsync
ENV APP_GROUP gitsync
ENV APP_HOME /gitsync

# create user and group
RUN set -xe \
  && mkdir -p /home/$APP_USER \
  && groupadd --system $APP_GROUP --gid=$app_gid \
  && useradd --system --gid $APP_GROUP --uid=$app_uid \
    --shell /bin/false --home-dir=/home/$APP_USER $APP_USER \
  && chown $APP_USER:$APP_GROUP /home/$APP_USER

WORKDIR $APP_HOME

ADD --chown="$app_uid:$app_gid" https://github.com/oscript-library/gitsync/releases/download/$gitsync_ver/gitsync.exe $APP_HOME

USER $APP_USER

Что думаете, коллеги? Мне, если честно, решение с RUN chmod не очень нравится - как-то в лоб )

@jugatsu
Copy link
Owner

jugatsu commented Apr 28, 2020

@nixel2007 давай допинаем этот MR )

@nixel2007
Copy link
Collaborator

@jugatsu мы в итоге отказались от образа с gitsync в виде exe и перешли на единый оскриптовый (правда тоже с нуля написанный), так что если честно, не знаю, чем тут помочь :(

Посмотреть можно вот тут: https://github.com/firstBitSemenovskaya/onec-docker/blob/1c0d637810d84c2fe8b5aeb489725b214803b29b/base-jenkins-agent/Dockerfile

Даю пермалинк, т.к. я сейчас как раз этот докерфайл активно переделываю и часть оттуда выкину.

@jugatsu
Copy link
Owner

jugatsu commented Apr 28, 2020

Предлагаю как закончишь, сделай сюда PR на ревью. А это MR мы закроем.

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.

3 participants