Slack notification owners

tapspatel commented 3 months ago

Hi folks, we are enabling slack notifications for regression tests. Here is a list I have of each t3000 test owner for each of the models. Please let me know if something is incorrect or should be reassigned. @dimitri-tenstorrent I'm not sure if we should have one owner per workflow or each individual test (if the latter, we will need to separate the pipelines in the workflow file)

[x] T3000 unit tests: https://github.com/tenstorrent/tt-metal/blob/main/.github/workflows/t3000-unit-tests.yaml
run_t3000_ttmetal_tests @SeanNijjar
run_t3000_ttnn_tests @arakhmati
run_t3000_falcon7b_tests @cfjchu
run_t3000_falcon40b_tests @johanna-rock-tt
run_t3000_mixtral_tests @mtairum
[x] T3000 profiler tests: https://github.com/tenstorrent/tt-metal/blob/main/.github/workflows/t3000-profiler-tests.yaml
@mo-tenstorrent
[x] T3000 model perf tests: https://github.com/tenstorrent/tt-metal/blob/main/.github/workflows/t3000-model-perf-tests.yaml
run_t3000_falcon7b_tests @skhorasganiTT @djordje-tt @s-jovic @pavlepopovic @pavlejosipovic
run_t3000_mixtral_tests @mtairum
run_t3000_llama2_70b_tests @cglagovich
[x] T3000 frequent tests: https://github.com/tenstorrent/tt-metal/blob/main/.github/workflows/t3000-frequent-tests.yaml
run_t3000_ethernet_tests @SeanNijjar
run_t3000_tteager_tests @SeanNijjar
run_t3000_falcon40b_tests @johanna-rock-tt @s-jovic @pavlepopovic @pavlejosipovic
run_t3000_llama2_70b_tests @cglagovich
run_t3000_mixtral_tests @mtairum
[x] T3000 demo tests: https://github.com/tenstorrent/tt-metal/blob/main/.github/workflows/t3000-demo-tests.yaml
run_t3000_falcon40b_tests @johanna-rock-tt @skhorasganiTT @s-jovic @pavlepopovic
run_t3000_falcon7b_tests @skhorasganiTT @djordje-tt @s-jovic @pavlepopovic @pavlejosipovic
run_t3000_mixtral_tests @mtairum

cc: @tt-rkim

tt-rkim commented 3 months ago

cc: @dimitri-tenstorrent Should we start splitting up these workflows into smaller jobs ? Like https://github.com/tenstorrent/tt-metal/blob/main/.github/workflows/fast-dispatch-full-regressions-and-models.yaml so that organizing this is easier

skhorasganiTT commented 3 months ago

[T3000 model perf tests / run_t3000_falcon7b_tests] and [T3000 demo tests / run_t3000_falcon7b_tests] should be reassigned to myself and @djordje-tt. Also I can be removed from the Falcon40 unit/frequent tests.

s-jovic commented 3 months ago

Could you add @pavlejosipovic @pavlepopovic and me for all Falcon 7b (except T3000 unit tests, those should stay with Joseph) and all Falcon 40b tests?

dimitri-tenstorrent commented 3 months ago

cc: @dimitri-tenstorrent Should we start splitting up these workflows into smaller jobs ? Like https://github.com/tenstorrent/tt-metal/blob/main/.github/workflows/fast-dispatch-full-regressions-and-models.yaml so that organizing this is easier

I am a big believer that the test ownership should be by a small section of the test suite not the full pipeline. I believe @TT-billteng advocated for the workflow owners as the first step just to get things off the ground and then focus on running smaller test suites where we can easily identify the owners if the suite fails and have a very targeted message.

tapspatel commented 3 months ago

@dimitri-tenstorrent sounds good. I'll let you decide on who to put as owners for each pipeline based on the owners in the ticket.

tenstorrent / tt-metal

Slack notification owners #9082