jenkins-lib icon indicating copy to clipboard operation
jenkins-lib copied to clipboard

Добавить поддержку 1cedtcli для EDT >=2024.1

Open ovcharenko-di opened this issue 1 year ago • 5 comments

Предпосылки:

  • Jenkins LTS 2.479 требует Java 17 на агентах
  • EDT >=2024.1 работает на Java 17

BUT

  • начиная с EDT 2024.1 не поддерживается ring, только 1cedtcli

Нужно сделать так, чтобы при указании в конфиге версии EDT >=2024.1 вместо команд ring выполнялись команды 1cedtcli

ovcharenko-di avatar Nov 08 '24 09:11 ovcharenko-di

Либо сделать поддержку edtcli по умолчанию, а ring оставить в проперти (да, сломать обратную совместимость, но нет смысла делать дефолт на старую версию едт)

nixel2007 avatar Nov 08 '24 21:11 nixel2007

Потому что версия едт может быть и вообще не указана, а браться последняя с машины. Как вариант проверять доступность edtcli на машине и запускать её

nixel2007 avatar Nov 08 '24 21:11 nixel2007

зафиксирую тут проблему: на одной из моих конфигураций после 1cedtcli validate сохраняется пустой файл edt-validate.out

при этом в логе воркспейса пусто, вот последние три записи, между 1 и 2 записью проходит 10 минут, ошибок не возникает, но и файл не пишется

!ENTRY com._1c.g5.v8.dt.core 1 0 2024-11-20 14:30:53.779
!MESSAGE Project context is started: cf

!ENTRY com._1c.g5.v8.dt.core 1 0 2024-11-20 14:40:44.424
!MESSAGE Project context is being stopped: cf (IDE_SHUTDOWN)-(com._1c.g5.v8.dt.core.platform.workspace-project)

!ENTRY com._1c.g5.v8.dt.core 1 0 2024-11-20 14:40:44.701
!MESSAGE Project context is stopped: cf

UPD: к jenkins-lib это поведение не имеет отношения, проблема в EDT

ovcharenko-di avatar Nov 20 '24 16:11 ovcharenko-di

@nixel2007 что в итоге делаем?

сейчас реализовано так, что для использования шага edtValidate нужно указать версию EDT в конфиге, иначе он работать не будет, см. вот это исключение

ovcharenko-di avatar Nov 23 '24 08:11 ovcharenko-di

@ovcharenko-di может если версия не заполнена, использовать вариант с 1cedtcli как дефолт?

nixel2007 avatar Nov 23 '24 13:11 nixel2007