РИФ 2016, Внедрение контроля качества в большом web-проекте на примере Badoo
0 likes117 views
Документ описывает внедрение контроля качества в крупном web-проекте Badoo, включая переход на Git и стратегию автоматизации процессов разработки. Рассматриваются этапы улучшения, такие как внедрение юнит-тестов, процесс код-ревью и автоматизация релизов, что позволило значительно повысить качество кода и скорость разработки. Успешные изменения включают рост количества юнит-тестов и интеграцию инструментов для автоматизации тестирования и выкладки.
РИФ 2016, Внедрение контроля качества в большом web-проекте на примере Badoo
- 1. Внедрение контроля качества в большом web-проекте на примере Badoo
- 2. Badoo - сайт знакомств с новыми людьми, крупнейшая интернет-компания Западной Европы. • 300 000 000+ пользователей • 190 стран • 46 языков Badoo в наши дни
- 3. С технической точки зрения это • 150+ разработчиков и 50+ QA инженеров • 3 000 000+ строк кода • 60000+ юнит-тестам • 600+ selenium-тестам • 2 ежедневных релиза Badoo в наши дни
- 4. Как это было? Feature1 Feature2 Feature3 Bugfix1 Bugfix2 Разработка trunk commits release releasesvn
- 5. Переводы, тестирование и выкладка svn Staging QA + Translation Production Deploy Как это было? Devel
- 7. Преимущества • Скорость обсуждения задачи Как это было?
- 8. Преимущества • Скорость обсуждения задачи • Скорость разработки Как это было?
- 9. Преимущества • Скорость обсуждения задачи • Скорость разработки • Проверенные инструменты Как это было?
- 10. Преимущества • Скорость обсуждения задачи • Скорость разработки • Проверенные инструменты • Тесное общение Как это было?
- 12. Недостатки • Незавершенные задачи в выкладке Как это было?
- 13. Недостатки • Незавершенные задачи в выкладке • Тяжело отслеживать состояние кода Как это было?
- 14. Недостатки • Незавершенные задачи в выкладке • Тяжело отслеживать состояние кода • Невозможно откатить задачу Как это было?
- 15. Недостатки • Незавершенные задачи в выкладке • Тяжело отслеживать состояние кода • Невозможно откатить задачу • Работа отделов волнообразно зависит от выкладки Как это было?
- 16. Еще недостатки • Выкладка задач тормозится Как это было?
- 17. Еще недостатки • Выкладка задач тормозится • Выкладка 1-2 раза в неделю Как это было?
- 18. Еще недостатки • Выкладка задач тормозится • Выкладка 1-2 раза в неделю • Тяжелый процесс выкладки Как это было?
- 19. Еще недостатки • Выкладка задач тормозится • Выкладка 1-2 раза в неделю • Тяжелый процесс выкладки • С ростом команды растут проблемы с качеством Как это было?
- 20. Глобальные требования • Формализовать процесс разработки
- 21. Глобальные требования • Формализовать процесс разработки • Внедрить контроль качества на всех этапах
- 22. Глобальные требования • Формализовать процесс разработки • Внедрить контроль качества на всех этапах • Внедрить автоматизированное тестирование
- 23. Глобальные требования • Формализовать процесс разработки • Внедрить контроль качества на всех этапах • Внедрить автоматизированное тестирование • Не потерять в скорости разработки
- 24. Этап первый: упрощаем разработку Задача: спроектировать удобный процесс параллельной разработки?
- 25. Этап первый: упрощаем разработку Задача: спроектировать удобный процесс параллельной разработки Решение: заменить svn на git, который позволяет удобно осуществлять разработку в собственной версии кода каждому разработчику ? !
- 26. master build_2016_03_21_11am SRV-10637_add_sf_support build_2016_03_21_4pm BD-1786_spp_in_spain B0-435_cs_improvements release release Этап первый: упрощаем разработку git
- 27. Проектируем и пробуем Этап первый: упрощаем разработку +
- 28. Проектируем и пробуем • Используем оба хранилища - и svn, и git Этап первый: упрощаем разработку +
- 29. Проектируем и пробуем • Используем оба хранилища - и svn, и git • Синхронизация кода при помощи git svn Этап первый: упрощаем разработку +
- 30. Проектируем и пробуем • Используем оба хранилища - и svn, и git • Синхронизация кода при помощи git svn Переводим на новую схему одну из команд Этап первый: упрощаем разработку +
- 31. Анализируем и дополняем Этап первый: упрощаем разработку +
- 32. Анализируем и дополняем • Базовая инструкция дополнена четкими шагами Этап первый: упрощаем разработку +
- 33. Анализируем и дополняем • Базовая инструкция дополнена четкими шагами • Разработан прототип системы сборки релиза Этап первый: упрощаем разработку +
- 34. Анализируем и дополняем • Базовая инструкция дополнена четкими шагами • Разработан прототип системы сборки релиза Начинаем перевод остальных отделов Этап первый: упрощаем разработку +
- 35. Переходим полностью Этап первый: упрощаем разработку +
- 36. Переходим полностью • Презентация об основах работы с git Этап первый: упрощаем разработку +
- 37. Переходим полностью • Презентация об основах работы с git • Помощь в переходный период Этап первый: упрощаем разработку +
- 38. Переходим полностью • Презентация об основах работы с git • Помощь в переходный период • Специальная страничка в Confluence Этап первый: упрощаем разработку +
- 39. Переходим полностью • Презентация об основах работы с git • Помощь в переходный период • Специальная страничка в Confluence Торжественное закрытие svn Этап первый: упрощаем разработку +
- 40. Этап второй: unit-тесты Задача: дать разработчику быстрый ответ о качестве кода на ранних этапах разработки?
- 41. Этап второй: unit-тесты Задача: дать разработчику быстрый ответ о качестве кода на ранних этапах разработки Решение: unit-тесты, написанные самим разработчиком + регрессионные тесты, написанные коллегами ранее ? !
- 42. Внедрение культуры написания юнит-тестов Этап второй: unit-тесты +
- 43. Внедрение культуры написания юнит-тестов • Написание нескольких примеров Этап второй: unit-тесты +
- 44. Внедрение культуры написания юнит-тестов • Написание нескольких примеров • Организация семинаров Этап второй: unit-тесты +
- 45. Внедрение культуры написания юнит-тестов • Написание нескольких примеров • Организация семинаров • Показательная демонстрация полезности Этап второй: unit-тесты +
- 46. Внедрение культуры написания юнит-тестов • Написание нескольких примеров • Организация семинаров • Показательная демонстрация полезности • Поощрение первопроходцев Этап второй: unit-тесты +
- 48. Стимулируем процесс • Добавление инструментов Этап второй: unit-тесты +
- 49. Стимулируем процесс • Добавление инструментов • Добавление обязательного прогона unit-тестов Этап второй: unit-тесты +
- 50. Стимулируем процесс • Добавление инструментов • Добавление обязательного прогона unit-тестов Как результат - резкий рост количества юнит-тестов: с пары десятков до более 3000. Этап второй: unit-тесты +
- 51. Этап третий: code review Задача: повысить читаемость кода, добавив процесс ревью кода и удобный интрумент для него ?
- 52. Этап третий: code review Задача: повысить читаемость кода, добавив процесс ревью кода и удобный интрумент для него Решение: gitphp с надстройками для ревью и интеграцией с Jira ? !
- 53. Процесс - обязательный. Способ - вольный: Этап третий: code review +
- 54. Процесс - обязательный. Способ - вольный: • code review осуществляется руководителем отдела Этап третий: code review +
- 55. Процесс - обязательный. Способ - вольный: • code review осуществляется руководителем отдела • code review - внутри группы Этап третий: code review +
- 56. Процесс - обязательный. Способ - вольный: • code review осуществляется руководителем отдела • code review - внутри группы • задачи попадают в общую очередь на review Этап третий: code review +
- 57. Процесс - обязательный. Способ - вольный: • code review осуществляется руководителем отдела • code review - внутри группы • задачи попадают в общую очередь на review • review проводят от каждого отдела Этап третий: code review +
- 58. Этап четвертый: автоматизация процессов Задача: ускорить выкладку новых фич, не потеряв в качестве?
- 59. Этап четвертый: автоматизация процессов Задача: ускорить выкладку новых фич, не потеряв в качестве Решение: специальная команда release- инженеров для автоматизации сборки и выкладки билда ? !
- 60. Внедрение автоматизации Автовыкладка на стейджинг Code freeze git hooks AIDA CI + CD Selenium Шоты Этап четвертый: автоматизация процессов
- 61. Подзадача: автоматизировать часть code review Этап четвертый: автоматизация процессов ?
- 62. Подзадача: автоматизировать часть code review Решение: добавляются новые git-хуки, использующие в том числе анализаторы кода Этап четвертый: автоматизация процессов ? !
- 63. Что начинаем проверять + Этап четвертый: автоматизация процессов
- 64. Что начинаем проверять • Каждая ветка начинается с номера задачи + Этап четвертый: автоматизация процессов
- 65. Что начинаем проверять • Каждая ветка начинается с номера задачи • Наличие сопровождающего для кода + Этап четвертый: автоматизация процессов
- 66. Что начинаем проверять • Каждая ветка начинается с номера задачи • Наличие сопровождающего для кода • Состояние задачи + Этап четвертый: автоматизация процессов
- 67. Что начинаем проверять • Каждая ветка начинается с номера задачи • Наличие сопровождающего для кода • Состояние задачи • Соответствие кода принятым стандартам + Этап четвертый: автоматизация процессов
- 68. Подзадача: разумно организовать сборку билда Этап четвертый: автоматизация процессов ?
- 69. Подзадача: разумно организовать сборку билда Решение: уменьшаем человеческий фактор в процессе сборки Этап четвертый: автоматизация процессов ? !
- 71. Разгружаем релиз-инженеров • В помощь написана AIDA + Этап четвертый: автоматизация процессов
- 72. Разгружаем релиз-инженеров • В помощь написана AIDA • Автомерж каждые 5 минут + Этап четвертый: автоматизация процессов
- 73. Разгружаем релиз-инженеров • В помощь написана AIDA • Автомерж каждые 5 минут • Вводится code freeze перед выкладками + Этап четвертый: автоматизация процессов
- 74. Разгружаем релиз-инженеров • В помощь написана AIDA • Автомерж каждые 5 минут • Вводится code freeze перед выкладками Два релиза в день! + Этап четвертый: автоматизация процессов
- 75. Подзадача: ускорить unit-тесты (написано уже более 25000!) Этап четвертый: автоматизация процессов ?
- 76. Подзадача: ускорить unit-тесты (написано уже более 25000!) Решение: разделение unit-тестов на группы для асинхронного прогона на нескольких серверах Этап четвертый: автоматизация процессов ? !
- 78. suite 1 suite 3 suite 2 suite 4 suite 5 suite 6 Этап четвертый: автоматизация процессов Common suite +
- 79. suite 1 suite 3 suite 2 suite 4 suite 5 suite 6 Cloud computing Этап четвертый: автоматизация процессов Common suite +
- 80. Подзадача: упростить повторные проверки сложных сценариев Этап четвертый: автоматизация процессов ?
- 81. Подзадача: упростить повторные проверки сложных сценариев Решение: подключить к написанию selenium- тестов всех QA-инженеров Этап четвертый: автоматизация процессов ? !
- 83. Реализуем • Команда автоматизаторов тестов Этап четвертый: автоматизация процессов +
- 84. Реализуем • Команда автоматизаторов тестов • Цикл cеминаров Этап четвертый: автоматизация процессов +
- 85. Реализуем • Команда автоматизаторов тестов • Цикл cеминаров • Когда можно и нужно писать selenium-тесты Этап четвертый: автоматизация процессов +
- 86. Реализуем • Команда автоматизаторов тестов • Цикл cеминаров • Когда можно и нужно писать selenium-тесты Участвует каждый QA-инженер! Этап четвертый: автоматизация процессов +
- 87. Подзадача: уметь проверять задачу на боевых данных до выкладки. Этап четвертый: автоматизация процессов ?
- 88. Подзадача: уметь проверять задачу на боевых данных до выкладки. Решение: шоты - создание “стейджинга” для отдельной ветки (бонус - генерация шаблонов). Этап четвертый: автоматизация процессов ? !
- 89. Подзадача: дополнить автоматизированные проверки составных задач Этап четвертый: автоматизация процессов ?
- 90. Подзадача: дополнить автоматизированные проверки составных задач Решение: AIDA приобретает дополнительные возможности Этап четвертый: автоматизация процессов ? !
- 91. Дополнительные возможности Этап четвертый: автоматизация процессов +
- 92. Дополнительные возможности • Проверка дополнительных маркеров Этап четвертый: автоматизация процессов +
- 93. Дополнительные возможности • Проверка дополнительных маркеров • Запуск smoke-тестов Этап четвертый: автоматизация процессов +
- 94. Дополнительные возможности • Проверка дополнительных маркеров • Запуск smoke-тестов • Пересборка билда при откате проблемной задачи Этап четвертый: автоматизация процессов +
- 95. Дополнительные возможности • Проверка дополнительных маркеров • Запуск smoke-тестов • Пересборка билда при откате проблемной задачи • Нотификации разработчику и QA-инженеру Этап четвертый: автоматизация процессов +
- 96. Подзадача: уметь быстро определять и решать проблемы при выкладке Этап четвертый: автоматизация процессов ?
- 97. Подзадача: уметь быстро определять и решать проблемы при выкладке Решение: расширение мониторинга и карантин на завершение выкладки Этап четвертый: автоматизация процессов ? !
- 98. Контроль выкладки Этап четвертый: автоматизация процессов +
- 99. Контроль выкладки • Асинхронная выкладка, атомарное переключение Этап четвертый: автоматизация процессов +
- 100. Контроль выкладки • Асинхронная выкладка, атомарное переключение • Мониторинг ключевых показателей Этап четвертый: автоматизация процессов +
- 101. Контроль выкладки • Асинхронная выкладка, атомарное переключение • Мониторинг ключевых показателей • Быстрый “откат” при проблемах Этап четвертый: автоматизация процессов +
- 102. • Набирать высококлассных QA-инженеров • Подобные задачи требуют времени • Искать пути оптимизации Что же ещё?