Skip to content

Commit 04f4eee

Browse files
committed
Add focused workflow to verify OpAMP client fix
This workflow tests the specific fixed test across: - Java 17 and 21 - 10 iterations each (20 total runs) - Only the whenServerProvidesNewInstanceUid_useIt test This will prove the array comparison fix eliminates flakiness.
1 parent 7ebd072 commit 04f4eee

File tree

1 file changed

+59
-0
lines changed

1 file changed

+59
-0
lines changed
Lines changed: 59 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -0,0 +1,59 @@
1+
name: Verify OpAMP Client Fix
2+
3+
on:
4+
workflow_dispatch:
5+
push:
6+
branches: [ flaky ]
7+
8+
jobs:
9+
verify-fix:
10+
runs-on: ubuntu-latest
11+
strategy:
12+
matrix:
13+
java: [17, 21]
14+
iteration: [1, 2, 3, 4, 5, 6, 7, 8, 9, 10]
15+
16+
steps:
17+
- uses: actions/checkout@v4
18+
19+
- name: Set up JDK ${{ matrix.java }}
20+
uses: actions/setup-java@v4
21+
with:
22+
java-version: ${{ matrix.java }}
23+
distribution: 'temurin'
24+
25+
- name: Setup Gradle
26+
uses: gradle/gradle-build-action@v3
27+
with:
28+
cache-read-only: true
29+
30+
- name: Test the fixed OpAMP client test (Iteration ${{ matrix.iteration }})
31+
run: |
32+
echo "=== Testing OpAMP client - Java ${{ matrix.java }}, Iteration ${{ matrix.iteration }} ==="
33+
./gradlew opamp-client:test --tests "*whenServerProvidesNewInstanceUid_useIt*" --rerun-tasks --info
34+
echo "SUCCESS: Iteration ${{ matrix.iteration }} passed"
35+
36+
- name: Upload test results on failure
37+
if: failure()
38+
uses: actions/upload-artifact@v4
39+
with:
40+
name: test-results-java${{ matrix.java }}-iter${{ matrix.iteration }}
41+
path: |
42+
opamp-client/build/reports/
43+
opamp-client/build/test-results/
44+
retention-days: 3
45+
46+
summary:
47+
needs: verify-fix
48+
runs-on: ubuntu-latest
49+
if: always()
50+
steps:
51+
- name: Check results
52+
run: |
53+
if [ "${{ needs.verify-fix.result }}" == "success" ]; then
54+
echo "✅ ALL TESTS PASSED! The OpAMP client fix is verified."
55+
echo "🎉 Ran 20 iterations across Java 17 and 21 - no flakiness detected!"
56+
else
57+
echo "❌ Some tests failed. The fix may need more work."
58+
exit 1
59+
fi

0 commit comments

Comments
 (0)