Skip to content

Commit 95f1757

Browse files
trasktrask
authored
Remove flaky test reporter (open-telemetry#18329)
1 parent d9ceeff commit 95f1757

9 files changed

Lines changed: 1 addition & 452 deletions

File tree

‎.github/repository-settings.md‎

Lines changed: 0 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -24,10 +24,6 @@ Secrets:
2424

2525
## Secrets and variables > Actions
2626

27-
### Repository secrets
28-
29-
- `FLAKY_TEST_REPORTER_ACCESS_KEY` - owned by [@laurit](https://github.com/laurit)
30-
3127
### Organization secrets
3228

3329
- `DEVELOCITY_ACCESS_KEY` (scoped only to Java repos)

‎.github/workflows/build-common.yml‎

Lines changed: 0 additions & 44 deletions
Original file line numberDiff line numberDiff line change
@@ -18,8 +18,6 @@ on:
1818
secrets:
1919
DEVELOCITY_ACCESS_KEY:
2020
required: false
21-
FLAKY_TEST_REPORTER_ACCESS_KEY:
22-
required: false
2321

2422
permissions:
2523
contents: read
@@ -338,48 +336,6 @@ jobs:
338336
if: ${{ !cancelled() && hashFiles('build-scan.txt') != '' }}
339337
run: cat build-scan.txt
340338

341-
- name: Get current job url
342-
id: jobs
343-
if: ${{ !cancelled() }}
344-
uses: actions/github-script@3a2844b7e9c422d3c10d287c895573f7108da1b3 # v9.0.0
345-
env:
346-
matrix: ${{ toJson(matrix) }}
347-
with:
348-
result-encoding: string
349-
script: |
350-
try {
351-
const matrix = JSON.parse(process.env.matrix);
352-
const job_name = `common / test${ matrix['test-partition'] } (${ matrix['test-java-version'] }, ${ matrix.vm }, indy ${ matrix['test-indy'] })`;
353-
const workflow_jobs_nested = await github.paginate(
354-
github.rest.actions.listJobsForWorkflowRun,
355-
{
356-
owner: context.repo.owner,
357-
repo: context.repo.repo,
358-
run_id: context.runId,
359-
per_page: 100
360-
},
361-
(response) => {
362-
return response.data;
363-
},
364-
);
365-
const job = workflow_jobs_nested.flat().find((job) => job.name === job_name);
366-
return job ? job.html_url : '';
367-
} catch (e) {
368-
core.warning(`Failed to get current job url: ${e}`);
369-
return '';
370-
}
371-
372-
- name: Flaky test report
373-
if: ${{ !cancelled() }}
374-
env:
375-
FLAKY_TEST_REPORTER_ACCESS_KEY: ${{ secrets.FLAKY_TEST_REPORTER_ACCESS_KEY }}
376-
JOB_URL: ${{ steps.jobs.outputs.result }}
377-
run: |
378-
if [ -s build-scan.txt ]; then
379-
export BUILD_SCAN_URL=$(cat build-scan.txt)
380-
fi
381-
./gradlew :test-report:reportFlakyTests
382-
383339
- name: Upload deadlock detector artifacts if any
384340
if: failure()
385341
uses: actions/upload-artifact@043fb46d1a93c77aae656e7c1c64a875d1fc6a0a # v7.0.1

‎.github/workflows/build-daily-no-build-cache.yml‎

Lines changed: 0 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -16,15 +16,13 @@ jobs:
1616
no-build-cache: true
1717
secrets:
1818
DEVELOCITY_ACCESS_KEY: ${{ secrets.DEVELOCITY_ACCESS_KEY }}
19-
FLAKY_TEST_REPORTER_ACCESS_KEY: ${{ secrets.FLAKY_TEST_REPORTER_ACCESS_KEY }}
2019

2120
test-latest-deps:
2221
uses: ./.github/workflows/reusable-test-latest-deps.yml
2322
with:
2423
no-build-cache: true
2524
secrets:
2625
DEVELOCITY_ACCESS_KEY: ${{ secrets.DEVELOCITY_ACCESS_KEY }}
27-
FLAKY_TEST_REPORTER_ACCESS_KEY: ${{ secrets.FLAKY_TEST_REPORTER_ACCESS_KEY }}
2826

2927
# muzzle is not included here because it doesn't use gradle cache anyway and so is already covered
3028
# by the normal daily build

‎.github/workflows/build-daily.yml‎

Lines changed: 0 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -14,13 +14,11 @@ jobs:
1414
uses: ./.github/workflows/build-common.yml
1515
secrets:
1616
DEVELOCITY_ACCESS_KEY: ${{ secrets.DEVELOCITY_ACCESS_KEY }}
17-
FLAKY_TEST_REPORTER_ACCESS_KEY: ${{ secrets.FLAKY_TEST_REPORTER_ACCESS_KEY }}
1817

1918
test-latest-deps:
2019
uses: ./.github/workflows/reusable-test-latest-deps.yml
2120
secrets:
2221
DEVELOCITY_ACCESS_KEY: ${{ secrets.DEVELOCITY_ACCESS_KEY }}
23-
FLAKY_TEST_REPORTER_ACCESS_KEY: ${{ secrets.FLAKY_TEST_REPORTER_ACCESS_KEY }}
2422

2523
muzzle:
2624
uses: ./.github/workflows/reusable-muzzle.yml

‎.github/workflows/build.yml‎

Lines changed: 0 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -15,7 +15,6 @@ jobs:
1515
uses: ./.github/workflows/build-common.yml
1616
secrets:
1717
DEVELOCITY_ACCESS_KEY: ${{ secrets.DEVELOCITY_ACCESS_KEY }}
18-
FLAKY_TEST_REPORTER_ACCESS_KEY: ${{ secrets.FLAKY_TEST_REPORTER_ACCESS_KEY }}
1918

2019
test-latest-deps:
2120
# release branches are excluded
@@ -25,7 +24,6 @@ jobs:
2524
uses: ./.github/workflows/reusable-test-latest-deps.yml
2625
secrets:
2726
DEVELOCITY_ACCESS_KEY: ${{ secrets.DEVELOCITY_ACCESS_KEY }}
28-
FLAKY_TEST_REPORTER_ACCESS_KEY: ${{ secrets.FLAKY_TEST_REPORTER_ACCESS_KEY }}
2927

3028
muzzle:
3129
# release branches are excluded

‎.github/workflows/reusable-test-latest-deps.yml‎

Lines changed: 0 additions & 43 deletions
Original file line numberDiff line numberDiff line change
@@ -12,8 +12,6 @@ on:
1212
secrets:
1313
DEVELOCITY_ACCESS_KEY:
1414
required: false
15-
FLAKY_TEST_REPORTER_ACCESS_KEY:
16-
required: false
1715

1816
permissions:
1917
contents: read
@@ -88,47 +86,6 @@ jobs:
8886
if: ${{ !cancelled() && hashFiles('build-scan.txt') != '' }}
8987
run: cat build-scan.txt
9088

91-
- name: Get current job url
92-
id: jobs
93-
if: ${{ !cancelled() }}
94-
uses: actions/github-script@3a2844b7e9c422d3c10d287c895573f7108da1b3 # v9.0.0
95-
env:
96-
matrix: ${{ toJson(matrix) }}
97-
with:
98-
result-encoding: string
99-
script: |
100-
try {
101-
const job_name = `test-latest-deps / testLatestDeps${ matrix['test-partition'] }`;
102-
const workflow_jobs_nested = await github.paginate(
103-
github.rest.actions.listJobsForWorkflowRun,
104-
{
105-
owner: context.repo.owner,
106-
repo: context.repo.repo,
107-
run_id: context.runId,
108-
per_page: 100
109-
},
110-
(response) => {
111-
return response.data;
112-
},
113-
);
114-
const job = workflow_jobs_nested.flat().find((job) => job.name === job_name);
115-
return job ? job.html_url : '';
116-
} catch (e) {
117-
core.warning(`Failed to get current job url: ${e}`);
118-
return '';
119-
}
120-
121-
- name: Flaky test report
122-
if: ${{ !cancelled() }}
123-
env:
124-
FLAKY_TEST_REPORTER_ACCESS_KEY: ${{ secrets.FLAKY_TEST_REPORTER_ACCESS_KEY }}
125-
JOB_URL: ${{ steps.jobs.outputs.result }}
126-
run: |
127-
if [ -s build-scan.txt ]; then
128-
export BUILD_SCAN_URL=$(cat build-scan.txt)
129-
fi
130-
./gradlew :test-report:reportFlakyTests
131-
13289
- name: Upload deadlock detector artifacts if any
13390
if: failure()
13491
uses: actions/upload-artifact@043fb46d1a93c77aae656e7c1c64a875d1fc6a0a # v7.0.1

‎settings.gradle.kts‎

Lines changed: 1 addition & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -95,9 +95,7 @@ develocity {
9595
fileFingerprints = true
9696
}
9797

98-
if (!gradle.startParameter.taskNames.contains("listTestsInPartition") &&
99-
!gradle.startParameter.taskNames.contains(":test-report:reportFlakyTests")
100-
) {
98+
if (!gradle.startParameter.taskNames.contains("listTestsInPartition")) {
10199
buildScanPublished {
102100
File("build-scan.txt").printWriter().use { writer ->
103101
writer.println(buildScanUri)
@@ -150,7 +148,6 @@ include(":instrumentation-annotations-support-testing")
150148
// misc
151149
include(":dependencyManagement")
152150
include(":instrumentation-docs")
153-
include(":test-report")
154151
include(":testing:agent-exporter")
155152
include(":testing:agent-for-testing")
156153
include(":testing:dependencies-shaded-for-testing")

‎test-report/build.gradle.kts‎

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

0 commit comments

Comments
 (0)