Skip to content
Merged
Changes from all commits
Commits
File filter

Filter by extension

Filter by extension


Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
17 changes: 10 additions & 7 deletions .github/workflows/tests.yml
Original file line number Diff line number Diff line change
Expand Up @@ -672,7 +672,12 @@ jobs:

extension_e2e_tests:
name: Run VS Code extension E2E tests
if: ${{ needs.setup_for_tests.outputs.run_extension_e2e == 'true' }}
# Temporarily disabled: the VS Code extension E2E shards are flaky and currently failing in CI
# (intermittent teardown AggregateError, see https://github.com/microsoft/aspire/issues/18098),
# so they block unrelated PRs. Keep the job defined (it stays in the aggregate gate's needs) but
# force it to skip until the flakiness is fixed. Re-enable by removing the `false &&` guard below.
# Tracked by https://github.com/microsoft/aspire/issues/18412.
if: ${{ false && needs.setup_for_tests.outputs.run_extension_e2e == 'true' }}
uses: ./.github/workflows/extension-e2e-tests.yml
needs: [setup_for_tests, build_packages, build_cli_archive_linux, build_cli_archive_windows, extension_tests_win, extension_bootstrap_linux]

Expand Down Expand Up @@ -789,8 +794,10 @@ jobs:
# bucket actually had work. Before selective CI these were always populated on a full run.
# - cli_starter_validation_windows: this job only runs for pull requests and is expected to
# be skipped for other workflow events.
# - extension_e2e_tests: this job only runs when the PR/push changes the VS Code extension,
# Aspire CLI, or the extension E2E workflow wiring.
# - extension_e2e_tests: temporarily disabled (always skipped) because the E2E shards are
# flaky/failing; see https://github.com/microsoft/aspire/issues/18412. While
# disabled, a 'skipped' result must NOT fail this gate, so the run_extension_e2e skip checks
# were removed below. Restore them when re-enabling the job.
# - polyglot_validation: this job runs for pull requests and main branch builds; it is
# expected to be skipped for other workflow events.
# All other jobs in this gate are required, and a 'skipped' result is treated as a failure.
Expand All @@ -800,8 +807,6 @@ jobs:
contains(needs.*.result, 'cancelled') ||
(github.event_name == 'pull_request' &&
((needs.extension_tests_win.result == 'skipped' && (needs.setup_for_tests.outputs.run_extension_unit == 'true' || needs.setup_for_tests.outputs.run_extension_e2e == 'true')) ||
(needs.setup_for_tests.outputs.run_extension_e2e == 'true' &&
needs.extension_e2e_tests.result == 'skipped') ||
(needs.cli_starter_validation_windows.result == 'skipped' && (needs.setup_for_tests.outputs.run_cli_starter == 'true')) ||
(needs.typescript_sdk_tests.result == 'skipped' &&
(needs.setup_for_tests.outputs.run_typescript_sdk == 'true')) ||
Expand All @@ -825,8 +830,6 @@ jobs:
(needs.polyglot_validation.result == 'skipped' && (needs.setup_for_tests.outputs.run_polyglot == 'true')))) ||
(github.event_name != 'pull_request' &&
((needs.extension_tests_win.result == 'skipped' && (needs.setup_for_tests.outputs.run_extension_unit == 'true' || needs.setup_for_tests.outputs.run_extension_e2e == 'true')) ||
(needs.setup_for_tests.outputs.run_extension_e2e == 'true' &&
needs.extension_e2e_tests.result == 'skipped') ||
(needs.typescript_sdk_tests.result == 'skipped' &&
(needs.setup_for_tests.outputs.run_typescript_sdk == 'true')) ||
needs.build_cli_archive_macos_x64.result == 'skipped' ||
Expand Down
Loading