Убийство разработки – опыт Тиньков Страхование

в 14:14, , рубрики: CICD, тинькофф, управление проектами, управление разработкой

Вечером зашел на Хабр, отсортировал лучшие по суткам, наткнулся на статью «Как убить разработку в три шага и на четвертый навсегда похоронить».

Хтонический ужас провисел почти сутки, не заминусованный. Я судорожно вышел из ридонли. Мне хочется кричать, но я бью по клавишам, пишу, что рвется наружу, на этот лист.

Автор статьи – irintus (имя изменено), в подробностях рассказала зачем и как она убила процесс разработки, куда и какие зонды вставила, но забыла написать, чего этим добивалась. Давайте я это исправлю.

Убийство разработки – опыт Тиньков Страхование - 1

Почему единые метрики качества — это важно для денег

Убийство разработки – опыт Тиньков Страхование - 2

Если на вопрос:

Может просто CI/CD сделаете?
Может просто CI/CD сделаете?

То вы - манагер. А если вы манагер, значит, софт вас не интересует, вас интересуют отчеты. Только зачем нужны отчеты?

Не завидую вам, ребята
Не завидую вам, ребята

Понятно, работников будут финансово стимулировать. А отрицательное финансовое стимулирование, как известно, тоже стимулирование.

Доктор Джордан Питерсон однажды обозначил различие между злом и трагедией. Ото зла трагедию отличает факт того, что она случается сама. Мы не можем обвинять град в том, что он побил нам помидоры. Побитые градом помидоры это - трагедия, а вот если ваши помидоры побил сосед, то это уже зло.

Но как будут собираться метрики? 

Шаг 1: находим все, что дорого

Шутка про олега
Шутка про олега

Как и любой другой манагер, Irintus первым делом начала смотреть на то, что уже работает. Ведь зачем приходить в компанию с набором готовых, рабочих практик, когда можно уничтожить существующие? 

Да, это называется CI/CD
Да, это называется CI/CD
Убийство разработки – опыт Тиньков Страхование - 7

CI/CD и тестирование на продакшене это стихийный процесс. Стопроцентное покрытие кода – стихийный процесс. Изящная автоматизация процесса разработки через пайплайн это – стихийный процесс. Вдумайтесь в безумие этой фразы.

Доказательства отсутствия проблем через тестирование больше недостаточно.  Даешь SLA! Что такое SLA? Это KPI. Не достал до KPI – не поел.

Убийство разработки – опыт Тиньков Страхование - 8

Юнит тесты, интеграционные тесты и тесты на продакшене, все слиплось…   пелена на глазах… трудно дышать…

Убийство разработки – опыт Тиньков Страхование - 9

Удобство, интеграция с IDE, CI/CD. Незнакомо… непонятно... Сжечь, уничтожить, заменить.

Шаг 2: искажаем, заменяем, уничтожаем

Орал как в последний день.
Орал как в последний день.

Irintus, манагеры, пожар, ураган или наводнение, это феномены, в них нет ни жалости, ни корысти. Не нужно винить огонь в том, что он поглощает топливо, их приход - трагедия. Но коптящий огонь инквизиции заговорил. И рассказал, что сделал, как уничтожил, подробно, цинично, по пунктам. Вот вам, краткий дайджест:

1. Заставить молчать

Пт. Количество заведенных и закрытых дефектов & пт. Соотношение дефектов и задач

«График закрытых дефектов должен быть приближен к графику заведенных. Обратное говорит о том, что команда не успевает исправлять все заведенные дефекты.»

Гит заменяется на джиру, больше нельзя заводить TODO, и обсуждения, нельзя писать RFC, иначе графики не совпадут. Теперь только баг репорты.

Конечно, чтобы подстроиться под KPI, люди кое-чо придумают, но пусть Irintus узнает об этом сама.

2. Запретить вопросы

Пт. Количество отмененных дефектов по месяцам для теста и прода

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

Новичок задает вопросы? Так делать нельзя, график вверх – плохо. Разработчики, новенькие и техническая поддержка не должны общаться напрямую, это поднимает линию вверх, а вверх это плохо.

Это очень эффективная мера по подавлению инициативы, у компании не будет будущего, если новички не будут вовлечены в процесс. 

3. Запретить тестирование

Пт. Состав релизов

«Большой объем дефектов тестирования может означать низкое качество разработки и/или требований, что негативно сказывается на скорости тестирования.»

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

Если нам необходимо подавить активную разработку, то почему бы и нет, инициатива хорошая.

4. Запретить релизы

Пт. Коэффициент ошибок, пропущенных на прод

«показывает качество тестирования, проработки требований и эффективность обнаружения ошибок — какая доля дефектов была отфильтрована, а какая прошла на прод.»

Напоминает на внедрение плохих врачебных практик в США. Лечение там опирается на строгие критерии от FDA.

Бывает, врач знает, что вот это лекарство, если дать его прямо сейчас, спасет человеку жизнь, но врач пациенту его не даст, потому что он нарушит гайдлайны FDA.

«Пойду домой, приму решение, может быть риск того и стоит, утро вечера мудренее». Подумал доктор. На следующий день пациент умер, проблема решена, совесть чиста, виновен FDA, Rinse and repeat.

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

Это закономерно породит цикл обвинений как на знаменитой диаграмме «заказчик-вендор-интегратор», а также разорвет фидбек с клиентами. Отличный результат!

5. Разрушить пайплайн

Пт. Распределение дефектов по приоритетам для теста и прода

«показывает, как соотносятся между собой количество дефектов с разными приоритетами для прода и тестовых сред.»

Вся сила CI/CD в том, что, работая в команде мы хотим поддерживать всего один бранч, main, и вместе весело пушить туда все что только можно. Все постоянно фетчат все изменения и всегда в курсе последних изменений, это удобно.

Сделал тесты, написал код. Проверил, все ли работает, прогнал тесты – запушил.
На сервере сборки началось тестирование – все хорошо, все прошло, все тесты зеленые -можно в релиз.

Это очень мощный инструмент, ускоряющий разработку и облегчающий жизнь всем.

Однако, если мы хотим это разрушить, то давайте заведем в джире отдельный стак ишшуев для дев и для прод. Отличная идея!

Такой подход неявно предполагает, что прод и дев это два разных пайпа или бранча, которые отдельно поддерживаются. Двойной оверхед при разработке.

Шаг 3: активные меры по подавлению разработки

Оставлять разработчиков наедине нельзя, иначе все обретет рациональные формы и станут звучать рациональные вопросы.

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

- Если есть рабочий CI/CD пайплайн, почему бы не взять его за основную метрику?

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

Шаг 4: похороны

Похороны процесса разработки сопровождались такими церемониями как «встречи-презентации» и «получение консультаций». В тинькове они, кстати, обрели ритуалистический характер. Успение разработки теперь отмечается у них постоянно.

Аминь
Аминь

Такой процесс я называю «End to end похороны», когда ни работники, ни начальство не понимает, зачем нужны метрики и чем одна от другой отличается.

Саботаж?

Шутка про Олега, хехе
Шутка про Олега, хехе

Похоже на активный саботаж. В своем обзоре я выделил только интересные участки. В целом, тут пробы негде ставить почитайте сами. Задачи, которые были достигнуты, походу, были поставлены так:

  1. Установление токсичной атмосферы, где сотрудники компании будут ожидать «ножа в джиру»

  2. Как можно чаще вытаскивать людей из интегрированных сред для разработки

  3. Заставить умалчивать проблемы

  4. Отбить новичкам желание задавать вопросы

  5. Задерживать новые релизы

  6. Поощрять code bloat, заставить писать mockery вместо тестов

  7. Демонтировать CI/CD пайплайн

  8. Уменьшить социализацию внутри команды

Но это еще не все, у них в планах:

1.       «Построение еще одного дашборда качества для руководителей, с помощью которого будет возможно быстро проанализировать и сравнить несколько команд в рамках одного департамента.»

Т.е. Посеять раздор и нездоровую конкуренция между командами

2.       «Прислушиваемся к хотелкам команд и развиваем текущие дашборды: делаем новые графики, улучшаем визуализацию ранее созданных графиков, добавляем фильтры и новые возможности для удобства использования.»

Т.е. поощрять итальянские забастовки и подрывную деятельность между командами

Заключение | Imprisonment

Убийство разработки – опыт Тиньков Страхование - 13

Учитывая весь контекст и основываясь на данных из статьи и новостных источников, делаю вывод – это саботаж и попытка устранения конкурента с помощью найма вот таких кадров.

Вдогонку лишь скажу, что как же хорошо, irintus, что я работаю не с вами.

Вперед, сначала банк, потом страхование. Так победите!

Автор:
nneeoo

Источник

* - обязательные к заполнению поля


https://ajax.googleapis.com/ajax/libs/jquery/3.4.1/jquery.min.js