Skip to content

Conversation

@mr-impossibru
Copy link
Contributor

@mr-impossibru mr-impossibru commented May 18, 2022

На образе будет повешен только один тег, в зависимости от ветки меняется его содержание :)

Документация:

@mr-impossibru mr-impossibru requested a review from vitaxa May 18, 2022 21:16
@mr-impossibru mr-impossibru marked this pull request as draft May 19, 2022 08:51
@mr-impossibru mr-impossibru marked this pull request as ready for review May 19, 2022 13:02
@vitaxa
Copy link
Contributor

vitaxa commented May 19, 2022

Я ваще думал, что мы как-то пересмотрим процессы с выкаткой epic веток. И почему они именно epic, так можно как угодно обзывать (типа rc - release candidate, а почему бы и нет 😄 ) Не обсуждали как-то детально это вопрос? Можем конечно жить как и жили, но может мы можем додумать эти процессы и давай какие-то осмысленные теги разным билдам.

@mr-impossibru
Copy link
Contributor Author

Вопрос обсуждал с Костей, совсем любые теги катить - плохая затея, люди всякую дичь будут пихать в теги :)
RC неплохая альтернатива

@mr-impossibru
Copy link
Contributor Author

Я ваще думал, что мы как-то пересмотрим процессы с выкаткой epic веток. И почему они именно epic, так можно как угодно обзывать (типа rc - release candidate, а почему бы и нет 😄 ) Не обсуждали как-то детально это вопрос? Можем конечно жить как и жили, но может мы можем додумать эти процессы и давай какие-то осмысленные теги разным билдам.

Copy link

@PospolitaNV PospolitaNV left a comment

Choose a reason for hiding this comment

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

У меня есть 2 комментария:

  1. Я бы поменял "epic" на "temp" - это больше отражает природу таких билдов. Они временные, для теста или для процесса миграции.
    А ещё стоит поменять contains на startsWith (если такое есть), чтобы избежать проблем вхождения четырёх букв в названия ветки :)
  2. Интересно узнать про юзкейс для таких контейнеров)

@mr-impossibru
Copy link
Contributor Author

Я бы поменял "epic" на "temp" - это больше отражает природу таких билдов. Они временные, для теста или для процесса миграции.

Тогда уж лучше rc, солиднее смотрится :)

А ещё стоит поменять contains на startsWith (если такое есть), чтобы избежать проблем вхождения четырёх букв в названия ветки :)

startsWith есть, но мне сложно представить ветку с названием epic :)

Интересно узнать про юзкейс для таких контейнеров)

Конкретно сейчас есть задача по оптимизации схемы newway, надо выкатить в параллель и проверить стоит ли игра свеч.

@ex01tus
Copy link

ex01tus commented May 23, 2022

Что касается названия – я вообще большого смысла менять его не вижу.

Все, что мы получим – это 10+ кейсов, когда кто-то создает ветку epic, у него не работает сборка, он полдня с этим разбирается, потом дергает девопсов, потом ему объясняют, что теперь надо rc/tmp/proto/unameit, он пересоздает ветку... Ну вы поняли, короче.

Если есть принципиальное желание поменять название и текущее кажется совсем неудачным – тогда можно голосовалку запилить, например.

@PospolitaNV
Copy link

Конкретно сейчас есть задача по оптимизации схемы newway, надо выкатить в параллель и проверить стоит ли игра свеч.

👍

но мне сложно представить ветку с названием epic :)

Ну я в целом за "rc"/"temp" и startsWIth, чтобы артефактов не возникло) раньше ветки начинались с epic/, но это бессмыслица.

он полдня с этим разбирается,

Сомневаюсь, что кто-то будет с этим полдня ковыряться. Название ветки с epic/ - уебанство))

@strug
Copy link
Contributor

strug commented May 23, 2022

я скорее солидарен с Сашей, тк мне вообще пох как ветки называются, но устоявшиеся привычки будут стрелять однозначно. Ну и epic вообще более логичен в нашем случае, тк у нас нет релизов и эти ветки не всегда попадают в прод.

@mr-impossibru mr-impossibru merged commit 33fc1aa into master May 23, 2022
@mr-impossibru mr-impossibru deleted the ft/IMP-31/add_epic_deploy branch May 23, 2022 15:21
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.

6 participants