Skip to content

Commit 51d83f8

Browse files
authored
Cleanup integration tests (#803)
build: cleanup integration tests
1 parent 6856b74 commit 51d83f8

File tree

2 files changed

+2
-63
lines changed

2 files changed

+2
-63
lines changed

.github/workflows/nightly_tests.yaml

Lines changed: 2 additions & 59 deletions
Original file line numberDiff line numberDiff line change
@@ -43,61 +43,6 @@ jobs:
4343
needs: [build_kjob, build_actions, build_wheel]
4444
uses: ./.github/workflows/integration_ray_cluster_create.yaml
4545
secrets: inherit
46-
47-
set-variables:
48-
runs-on: [ubuntu-22.04]
49-
concurrency:
50-
group: set-variables-${{ github.event.number}}
51-
cancel-in-progress: true
52-
outputs:
53-
cluster-name: ${{ steps.set-cluster-name.outputs.cluster-name }}
54-
cluster-name-dws: ${{ steps.set-cluster-name-dws.outputs.cluster-name-dws }}
55-
group-name: ${{ steps.set-group-name.outputs.group-name }}
56-
zone: ${{ steps.set-zone.outputs.zone }}
57-
tpu-type: ${{ steps.set-tpu-type.outputs.tpu-type }}
58-
tpu-type-topology: ${{ steps.set-tpu-type-topology.outputs.tpu-type-topology }}
59-
location: ${{steps.set-location.outputs.location}}
60-
run-id: ${{steps.set-run-id.outputs.run-id}}
61-
steps:
62-
- name: set run-id
63-
id: set-run-id
64-
run: |
65-
if [ "${{ github.event_name }}" == "workflow_dispatch" ]; then
66-
RUN_ID="dispatch"
67-
elif [ "${{ github.ref }}" == "refs/heads/main" ]; then
68-
RUN_ID="main"
69-
else
70-
RUN_ID="${{ github.event.number }}"
71-
fi
72-
echo run-id=$RUN_ID >> $GITHUB_OUTPUT
73-
- name: set cluster-name
74-
id: set-cluster-name
75-
run: |
76-
echo cluster-name=xpk-${{steps.set-run-id.outputs.run-id}} >> $GITHUB_OUTPUT
77-
- name: set cluster-name-dws
78-
id: set-cluster-name-dws
79-
run: |
80-
echo cluster-name-dws=xpk-dws-${{steps.set-run-id.outputs.run-id}} >> $GITHUB_OUTPUT
81-
- name: set group-name
82-
id: set-group-name
83-
run: |
84-
echo group-name=xpk-${{steps.set-run-id.outputs.run-id}} >> $GITHUB_OUTPUT
85-
- name: set zone
86-
id: set-zone
87-
run: |
88-
echo zone=us-central2-b >> $GITHUB_OUTPUT
89-
- name: set tpu-type
90-
id: set-tpu-type
91-
run: |
92-
echo tpu-type=v4-8 >> $GITHUB_OUTPUT
93-
- name: set tpu-type-topology
94-
id: set-tpu-type-topology
95-
run: |
96-
echo tpu-type-topology=v4-2x2x1 >> $GITHUB_OUTPUT
97-
- name: set location
98-
id: set-location
99-
run: |
100-
echo location=us-central2 >> $GITHUB_OUTPUT
10146
install-dependencies:
10247
needs: [set-variables]
10348
runs-on: ubuntu-22.04
@@ -137,12 +82,10 @@ jobs:
13782
${{env.pythonLocation}}
13883
key: xpk-deps-${{ matrix.python-version }}-${{github.run_id}}-${{github.run_attempt}}
13984
run-integration-tests:
140-
needs: [install-dependencies, set-variables]
85+
needs: [install-dependencies]
14186
uses: ./.github/workflows/reusable_integration_tests.yaml
142-
with:
143-
run-id: '${{needs.set-variables.outputs.run-id}}'
14487
concurrency: # We support one build or nightly test to run at a time currently.
145-
group: integration-tests-${{needs.set-variables.outputs.run-id}}
88+
group: integration-tests
14689
cancel-in-progress: true
14790
secrets: inherit
14891
storage-tests:

.github/workflows/reusable_integration_tests.yaml

Lines changed: 0 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -14,10 +14,6 @@
1414

1515
on:
1616
workflow_call:
17-
inputs:
18-
run-id:
19-
required: true
20-
type: string
2117

2218
jobs:
2319
integration-tests:

0 commit comments

Comments
 (0)