ydb icon indicating copy to clipboard operation
ydb copied to clipboard

Github wf analitycs

Open naspirato opened this issue 1 year ago • 2 comments

  1. base branch
  2. сейчас не понятно что мы показываем - только успешные или все - разобраться и сделать понятно 2.1) если сейчас все - то сделать селектор показывать только успешные для графика времени выполнения и количества
  3. просто точки по каждому запуску wh без агрегации
  4. steps
  5. кастомная стата - сделать модуль/либу аля google analitycs

https://datalens.yandex.cloud/wkptiaeyxz7qj-workflow-stat?tab=gD

naspirato avatar Oct 10 '24 16:10 naspirato

  • https://datalens.yandex/wkptiaeyxz7qj
  • base branch - done ( только для PR-check)
  • сейчас не понятно что мы показываем - только успешные или все - разобраться и сделать понятно теперь показываем вообще все, есть возможность фильтрации, на графике точек разные цвета для статусов "success/failure/canceled"
  • просто точки по каждому запуску wh без агрегации базовый график без агрегации

naspirato avatar Oct 15 '24 12:10 naspirato

После обсуждения решили:

  • по сути detail это "Job detail"
    • тут вьюхи по длительности и персентилям должны быть 1 графиком, который подвержен селекторам
    • возможно build type не нужен
  • по сути overview нужно сделать в "Workflow details", там
    • строить аналитику по всему wf, без селекторов и аггрегации джобов и build type
    • точно так же сделать как в джобах: точки, столбики ... -- по сути там то же самое что и там
  • conclusion заменить на например status (по аналогии с ui: https://github.com/ydb-platform/ydb/actions/runs/11385724390 -- здесь как пример "Status\nSuccess")

maximyurchuk avatar Oct 17 '24 13:10 maximyurchuk

  1. Чарт не найден image
  2. правильно понимаю, что overview удаляем в будущем вкладку? Если да, предлагаю вкладку с workflow сделать дефолтной -- на главной должна быть статистика понятная случайному человеку
  3. в job details не совсем понял семантику вот этого image
  4. мне кажется надо выставить правильные дефолты: это должен быть PR-Check и бранч main

maximyurchuk avatar Oct 21 '24 14:10 maximyurchuk

  1. еще посткоммитов не видно в стате, а они должны быть

maximyurchuk avatar Oct 21 '24 15:10 maximyurchuk

  1. нужно еще как-то отлавливать такие кейсы: Build and test release-asan "The runner has received a shutdown signal. This can happen when the runner service is stopped, or a manually started runner is canceled. "

maximyurchuk avatar Oct 21 '24 16:10 maximyurchuk