Skip to content

Commit 89cd7bb

Browse files
committed
Remove completed Windows manager capture diagnostic
The original test_remote and all 82 manager tests completed in the retained hosted capture. Restore the unchanged canonical invocation and remove temporary capture upload/launcher. Assisted-by: OpenAI Codex: model identifier unavailable
1 parent b4b1f58 commit 89cd7bb

2 files changed

Lines changed: 1 addition & 77 deletions

File tree

‎.github/workflows/ci.yaml‎

Lines changed: 1 addition & 17 deletions
Original file line numberDiff line numberDiff line change
@@ -567,11 +567,7 @@ jobs:
567567
- name: Run CPython tests
568568
run: |
569569
# shellcheck disable=SC2086
570-
if [[ "$RUNNER_OS" == "Windows" ]]; then
571-
target/release/rustpython -u scripts/diagnostics/manager_9088/preserve_manager_stdout.py --slow-ci -j "$CORES" $EXTRA_TEST_ARGS $SKIPS
572-
else
573-
target/release/rustpython -u -m test --slow-ci -j "$CORES" $EXTRA_TEST_ARGS $SKIPS
574-
fi
570+
target/release/rustpython -u -m test --slow-ci -j "$CORES" $EXTRA_TEST_ARGS $SKIPS
575571
timeout-minutes: ${{ matrix.timeout }}
576572
shell: bash
577573
env:
@@ -644,18 +640,6 @@ jobs:
644640
shell: bash
645641
run: python -I scripts/whats_left.py --rustpython target/release/rustpython
646642

647-
# Temporary #9088 observation only. Remove with the capture launcher
648-
# after the existing canonical manager worker's stalled method is known.
649-
- name: Preserve canonical Windows manager worker output
650-
timeout-minutes: 2
651-
if: ${{ always() && runner.os == 'Windows' }}
652-
uses: actions/upload-artifact@043fb46d1a93c77aae656e7c1c64a875d1fc6a0a # v7.0.1
653-
with:
654-
name: windows-manager-capture-${{ github.run_id }}-${{ github.run_attempt }}
655-
path: scripts/diagnostics/manager_9088/capture/manager-worker-*.log
656-
if-no-files-found: warn
657-
retention-days: 7
658-
659643
clippy:
660644
# Named after the matrix entry rather than left to be named for it: a
661645
# generated name lists every value in the entry, so adding or removing one

‎scripts/diagnostics/manager_9088/preserve_manager_stdout.py‎

Lines changed: 0 additions & 60 deletions
This file was deleted.

0 commit comments

Comments
 (0)