Thanks to visit codestin.com
Credit goes to github.com

Skip to content

Split CI into several jobs#5336

Closed
kan-bayashi wants to merge 32 commits intoespnet:masterfrom
kan-bayashi:feat/split-ci
Closed

Split CI into several jobs#5336
kan-bayashi wants to merge 32 commits intoespnet:masterfrom
kan-bayashi:feat/split-ci

Conversation

@kan-bayashi
Copy link
Member

@kan-bayashi kan-bayashi commented Jul 22, 2023

What?

  • Split CI into several jobs
    • test_python.sh -> test_python_espnet1.sh / test_python_espnet2.sh
    • test_shell.sh -> test_shell_espnet1.sh / test_shell_espnet2.sh
    • I found that the validation of configuraiton files in test_integration_espnet2.sh consume too much time so split it into test_integration_espnet2.sh and test_configuration_espnet2.sh.
  • Changed OS / version matrix
    • MacOS / Windows: check only installation
    • Cent / Debian: check except integration test
    • Ubuntu
      • ESPnet1 related job: pytest / bats / integration of espnet1
        • Run on only oldest and newest combiniation (3.7 + 1.10.2 & 3.19 + 1.13.1)
      • ESPnet2 related job: pytest / bats / integration of espnet2
        • Run on all supported pytorch with python 3.7
        • Run on all python version with newest pytorch 1.13.1
      • ESPnet2 configuration validation job: configuration check of espnet2
        • Run on only oldest and newest combiniation (3.7 + 1.10.2 & 3.19 + 1.13.1)

Why?

  • To reduce CI required time

See also

@mergify mergify bot added ESPnet2 CI Travis, Circle CI, etc labels Jul 22, 2023
@kan-bayashi kan-bayashi marked this pull request as draft July 22, 2023 04:28
@kan-bayashi kan-bayashi changed the title split ci Split CI into several jobs Jul 22, 2023
@mergify
Copy link
Contributor

mergify bot commented Jul 22, 2023

This pull request is now in conflict :(

@mergify mergify bot added the conflicts label Jul 22, 2023
@mergify mergify bot removed the conflicts label Jul 22, 2023
@kan-bayashi kan-bayashi deleted the feat/split-ci branch July 23, 2023 07:55
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

CI Travis, Circle CI, etc ESPnet2

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants