Инициализация БД и загрузка расширений #116
Conversation
Обзор измененийЭто обновление включает в себя изменение порядка выполнения этапов 'Инициализация ИБ' и 'Загрузка расширений в конфигурацию', а также соответствующее обновление условий и действий внутри этих этапов. Эти изменения направлены на оптимизацию процесса развертывания и повышение его эффективности. Изменения
Thank you for using CodeRabbit. We offer it for free to the OSS community and would appreciate your support in helping us grow. If you find it useful, would you consider giving us a shout-out on your favorite social media? TipsChatThere are 3 ways to chat with CodeRabbit:
Note: Be mindful of the bot's finite context window. It's strongly recommended to break down tasks such as reading entire modules into smaller chunks. For a focused discussion, use review comments to chat about specific files and their changes, instead of using the PR comments. CodeRabbit Commands (invoked as PR comments)
Additionally, you can add CodeRabbit Configration File (
|
![coderabbitai[bot]](https://avatars.githubusercontent.com/in/347564?s=60&v=4){kind=small}
There was a problem hiding this comment.
Review Status
Actionable comments generated: 0
Configuration used: CodeRabbit UI
Files selected for processing (1)
- vars/pipeline1C.groovy (1 hunks)
Additional comments: 1
vars/pipeline1C.groovy (1)
- 109-129: Перестановка стадий 'Инициализация ИБ' и 'Загрузка расширений в конфигурацию' выглядит логичной с точки зрения описания задачи PR. Однако, важно убедиться, что все зависимости и условия, связанные с этими стадиями, были корректно обновлены и не нарушают последовательность выполнения других стадий. Рекомендуется также добавить комментарии в коде, объясняющие причину изменения порядка для будущих разработчиков и облегчение поддержки кода.
|
boromir.jpg просто поменять местами стадии не получится, т.к. база создается впервые на шаге initInfobase. |
не, база создается на шаге stage('Создание ИБ') { А запуск первоначального заполнения на шаге stage('Инициализация ИБ') { |
|
Да, это логичное изменение. Особенно если разработка сделана только при помощи расширения и там тоже есть функции первого запуска - мы тогда получается вообще исключаем возможность их срабатывания если не поменяем порядок. У себя на проекте запустил, всё отработало штатно. Надо только в PR добавить в ридми картинку измененную с пайплайном в блю-оушен |
|
@Segate-ekb и правда, перепутал ну тогда "в путь" |
There was a problem hiding this comment.
Review Status
Actionable comments generated: 1
Configuration used: CodeRabbit UI
Files selected for processing (1)
- README.md (1 hunks)
Additional comments: 7
README.md (7)
- 70-70: Обновление ссылки на изображение в разделе "Внешний вид пайплайна в интерфейсе Blue Ocean" корректно отражает изменения, связанные с внешним видом пайплайна в интерфейсе Blue Ocean. Однако, стоит проверить доступность новой ссылки на изображение, чтобы убедиться, что она ведет на актуальное и корректное изображение.
- 67-73: > 📝 NOTE
This review was outside the diff hunks, and no overlapping diff hunk was found. Original lines [8-8]
Возможно, найдена орфографическая ошибка в слове "пайплайн". В русском языке принято использовать термин "конвейер".
- 67-73: > 📝 NOTE
This review was outside the diff hunks, and no overlapping diff hunk was found. Original lines [13-13]
Это предложение не начинается с заглавной буквы. Рекомендуется начинать каждый пункт списка с заглавной буквы для единообразия и соблюдения правил русского языка.
- 67-73: > 📝 NOTE
This review was outside the diff hunks, and no overlapping diff hunk was found. Original lines [44-44]
Возможно, найдена орфографическая ошибка в слове "oscript-приложений". Убедитесь, что термин "oscript" используется корректно и является частью специфической терминологии в контексте вашего проекта.
- 67-73: > 📝 NOTE
This review was outside the diff hunks, and no overlapping diff hunk was found. Original lines [66-66]
Возможно, найдена орфографическая ошибка в слове "пайплайн". В русском языке принято использовать термин "конвейер".
- 68-68: Возможно, найдена орфографическая ошибка в слове "пайплайна". В русском языке принято использовать термин "конвейера".
- 67-73: > 📝 NOTE
This review was outside the diff hunks, and no overlapping diff hunk was found. Original lines [123-123]
Возможно, найдена орфографическая ошибка в слове "тяжёлости". В данном контексте использование слова корректно, так как оно относится к сложности или интенсивности шага сборки.
|
|
||
| ## Внешний вид пайплайна в интерфейсе Blue Ocean | ||
|
|
||
|  | ||
|  | ||
|
|
||
| ## Конфигурирование | ||
|
|
There was a problem hiding this comment.
📝 NOTE
This review was outside the diff hunks, and no overlapping diff hunk was found. Original lines [110-110]
Возможно, найдена орфографическая ошибка в слове "По-умолчанию". Правильно писать слитно: "по умолчанию".
- По-умолчанию версия модуля EDT не заполнена
+ По умолчанию версия модуля EDT не заполнена
|
Мы тут с Никитой покумекали, и возник вопрос корректно ли так шаги перемещать. На данный момент загрузка расширения происходит не платформенным методом, а при помощи обработки т.е. требует запуска системы. Команда установки расширения: |
|
@Segate-ekb мяу |
|
Погонял тесты на разных конфигах, проблем не заметил. |
Иногда, разработка ведется так, что без загруженного расширения конфа не работает т.к. логика работы определенных механизмов переопределена именно в расширении.
Предлагаю поменять стадию загрузки расширения и стадию первоначальной инициализации базы местами.
Summary by CodeRabbit