Skip to content

Comments

Переход на s6-overlay#90

Draft
kuzja086 wants to merge 2 commits intofirstBitMarksistskaya:feature/first-bitfrom
kuzja086:onec-vnc
Draft

Переход на s6-overlay#90
kuzja086 wants to merge 2 commits intofirstBitMarksistskaya:feature/first-bitfrom
kuzja086:onec-vnc

Conversation

@kuzja086
Copy link

#88
Посмотрел опыт от @astrizhachuk
У меня не получилось сделать с использованием разделения отдельно на клиент и на клиент с vnc, почему-то терялись библиотеки платформы, при сборке образа с vnc, поэтому сделал в одном файле.
Ну и нужен ли отдельно клиент без vnc?

@coderabbitai
Copy link

coderabbitai bot commented Jan 26, 2026

Important

Review skipped

Draft detected.

Please check the settings in the CodeRabbit UI or the .coderabbit.yaml file in this repository. To trigger a single review, invoke the @coderabbitai review command.

You can disable this status message by setting the reviews.review_status to false in the CodeRabbit configuration file.

  • 🔍 Trigger a full review

Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out.

❤️ Share

Comment @coderabbitai help to get the list of available commands and usage tips.

@nixel2007
Copy link
Member

Я бы вообще vnc переделал в нормальный слой поверх client, а не копировал из базового образа, как сейчас. Мне никогда не была понятна эта архитектура от изначального автора. Ещё и куски докерфайла копируются и теряются постоянно.

@astrizhachuk
Copy link

astrizhachuk commented Jan 26, 2026

Так же непонятна идея отдельного базового слоя и в чем было его предназначение. Хочу подчеркнуть еще раз, что мой вариант предназначен и работал с последней версией 8.3.27 и для сборок старых версий может не работать (проблемы со скриптами скачивания и установки из дистрибутивов).

@Stepa86
Copy link

Stepa86 commented Jan 27, 2026

Попробовал эту ветку, попробовал на s6 перейти самостоятельно по советам в телеге и ИИшниц. Итог один: не работает образ base-jenkins-coverage-agent

1Ска падает с:
*** Unable to initialize GTK+ or connect to the windowing system. Is DISPLAY set properly?

В логе старта образа:
s6-overlay-suexec: fatal: can only run as pid 1

ИИшница говорит, что s6 нельзя стартовать в фоне, как это делается сейчас

#!/bin/sh

if [ -f "/init" ]; then
    /init &
fi

wget -O agent.jar $DOCKER_SWARM_PLUGIN_JENKINS_AGENT_JAR_URL && java -jar agent.jar -jnlpUrl $DOCKER_SWARM_PLUGIN_JENKINS_AGENT_JNLP_URL -secret $DOCKER_SWARM_PLUGIN_JENKINS_AGENT_SECRET -noReconnect -workDir /home/jenkins

И нужно сильно все переделывать. Как то много правок для поднятия версии s6

@kuzja086
Copy link
Author

Тоже дошел до того, что образ с клиентом и vnc запускается корректно, а base-jenkins-agent выдает ошибку как у @Stepa86
Есть идеи?)

@nixel2007
Copy link
Member

Чатжпт говорит, что единственный вариант - делать сервис в рамках s6 в который передавать параметры из entrypoint wrapper, а в самом скрипте оставлять /init как pid 1.

https://chatgpt.com/share/6978f1bd-a234-800e-b6d9-ce024901dd57

@kuzja086
Copy link
Author

У меня получилось поправить запуск base-jenkins-agent
На моем стенде проверял на версии Jenkins: 2.541
Также использовал новый swarm плагин https://github.com/jenkinsci/swarm-agents-cloud-plugin/tree/master от @AronMav

В плагине пришлось кое-что поправить, но я не думаю, что это ошибка. Возможно настройкой template это можно переопределить, чуть позже с Андреем обсудим ещё этот нюанс.

@goodwinvu
Copy link

У меня получилось поправить запуск base-jenkins-agent На моем стенде проверял на версии Jenkins: 2.541 Также использовал новый swarm плагин https://github.com/jenkinsci/swarm-agents-cloud-plugin/tree/master от @AronMav

В плагине пришлось кое-что поправить, но я не думаю, что это ошибка. Возможно настройкой template это можно переопределить, чуть позже с Андреем обсудим ещё этот нюанс.

А можно поподробней, про плагин ? какие настройки шаблона агента нужны и/или что там нужно испрвлять?
присматриваю тоже с неподдерживаемого сварм плагина слезть

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