Skip to content

Commit

Permalink
Drop the CI test timeout back down to 120 seconds.
Browse files Browse the repository at this point in the history
Multi-minute YAML tests should not be running per-push in CI.
  • Loading branch information
bzbarsky-apple committed Jul 18, 2022
1 parent e4cfa0a commit 1663c61
Show file tree
Hide file tree
Showing 5 changed files with 348 additions and 947 deletions.
2 changes: 1 addition & 1 deletion .github/workflows/darwin-tests.yaml
Original file line number Diff line number Diff line change
Expand Up @@ -105,7 +105,7 @@ jobs:
--target-skip-glob '{TestGroupMessaging}' \
run \
--iterations 1 \
--test-timeout-seconds 200 \
--test-timeout-seconds 120 \
--all-clusters-app ./out/darwin-x64-all-clusters-${BUILD_VARIANT}/chip-all-clusters-app \
--lock-app ./out/darwin-x64-lock-${BUILD_VARIANT}/chip-lock-app \
--ota-provider-app ./out/darwin-x64-ota-provider-${BUILD_VARIANT}/chip-ota-provider-app \
Expand Down
4 changes: 2 additions & 2 deletions .github/workflows/tests.yaml
Original file line number Diff line number Diff line change
Expand Up @@ -97,7 +97,7 @@ jobs:
--chip-tool ./out/linux-x64-chip-tool${CHIP_TOOL_VARIANT}-${BUILD_VARIANT}/chip-tool \
run \
--iterations 1 \
--test-timeout-seconds 200 \
--test-timeout-seconds 120 \
--all-clusters-app ./out/linux-x64-all-clusters-${BUILD_VARIANT}/chip-all-clusters-app \
--lock-app ./out/linux-x64-lock-${BUILD_VARIANT}/chip-lock-app \
--ota-provider-app ./out/linux-x64-ota-provider-${BUILD_VARIANT}/chip-ota-provider-app \
Expand Down Expand Up @@ -199,7 +199,7 @@ jobs:
--target-skip-glob '{TestGroupMessaging,Test_TC_DGTHREAD_2_1,Test_TC_DGTHREAD_2_2,Test_TC_DGTHREAD_2_3,Test_TC_DGTHREAD_2_4}' \
run \
--iterations 1 \
--test-timeout-seconds 200 \
--test-timeout-seconds 120 \
--all-clusters-app ./out/darwin-x64-all-clusters-${BUILD_VARIANT}/chip-all-clusters-app \
--lock-app ./out/darwin-x64-lock-${BUILD_VARIANT}/chip-lock-app \
--ota-provider-app ./out/darwin-x64-ota-provider-${BUILD_VARIANT}/chip-ota-provider-app \
Expand Down
2 changes: 1 addition & 1 deletion src/app/tests/suites/tests.js
Original file line number Diff line number Diff line change
Expand Up @@ -247,6 +247,7 @@ function getManualTests() {
"Test_TC_CADMIN_1_23",
"Test_TC_CADMIN_1_24",
// Slow tests that should not run in CI because they take many minutes each
"Test_TC_CADMIN_1_3",
"Test_TC_CADMIN_1_5",
"Test_TC_CADMIN_1_6",
"Test_TC_CADMIN_1_9",
Expand Down Expand Up @@ -703,7 +704,6 @@ function getTests() {
];

const MultipleFabrics = [
"Test_TC_CADMIN_1_3",
"Test_TC_CADMIN_1_4",
];

Expand Down
Loading

0 comments on commit 1663c61

Please sign in to comment.