Skip to content

Commit c33bafe

Browse files
authored
Disable the peformance comparision agaist baseline, keep just the perofrmance bench mark testing per PR (#836)
1 parent 317dfcf commit c33bafe

File tree

2 files changed

+4
-31
lines changed

2 files changed

+4
-31
lines changed

.github/workflows/performance-nightly.yml

Lines changed: 4 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -1,9 +1,10 @@
11
name: Nightly Performance Baseline
22

33
on:
4-
schedule:
5-
# Run at 3:00 AM UTC daily
6-
- cron: "0 3 * * *"
4+
# Disabled - baseline comparison not currently used in PR tests
5+
# schedule:
6+
# # Run at 3:00 AM UTC daily
7+
# - cron: "0 3 * * *"
78
workflow_dispatch: # Allow manual triggering
89

910
jobs:

.github/workflows/performance-test.yml

Lines changed: 0 additions & 28 deletions
Original file line numberDiff line numberDiff line change
@@ -24,8 +24,6 @@ jobs:
2424
steps:
2525
- name: Check out the repo
2626
uses: actions/checkout@v4
27-
with:
28-
fetch-depth: 0 # Need full history for baseline comparison
2927

3028
- name: Set up Go
3129
uses: actions/setup-go@v5
@@ -83,14 +81,6 @@ jobs:
8381
HF_HUB_DISABLE_TELEMETRY: 1
8482
run: make download-models
8583

86-
- name: Download performance baselines
87-
continue-on-error: true
88-
run: |
89-
mkdir -p perf/testdata/baselines
90-
git show main:perf/testdata/baselines/classification.json > perf/testdata/baselines/classification.json 2>/dev/null || echo '{"version":"v1.0.0","benchmarks":{}}' > perf/testdata/baselines/classification.json
91-
git show main:perf/testdata/baselines/decision.json > perf/testdata/baselines/decision.json 2>/dev/null || echo '{"version":"v1.0.0","benchmarks":{}}' > perf/testdata/baselines/decision.json
92-
git show main:perf/testdata/baselines/cache.json > perf/testdata/baselines/cache.json 2>/dev/null || echo '{"version":"v1.0.0","benchmarks":{}}' > perf/testdata/baselines/cache.json
93-
9484
- name: Run component benchmarks
9585
run: |
9686
mkdir -p reports
@@ -177,21 +167,3 @@ jobs:
177167
path: |
178168
reports/
179169
retention-days: 30
180-
181-
- name: Check for regressions (placeholder)
182-
id: regression_check
183-
continue-on-error: true
184-
run: |
185-
# In a real implementation, this would:
186-
# 1. Parse benchmark output
187-
# 2. Compare against baselines
188-
# 3. Calculate % changes
189-
# 4. Exit 1 if regressions exceed thresholds
190-
echo "No regressions detected (placeholder check)"
191-
192-
- name: Fail on regression
193-
if: steps.regression_check.outcome == 'failure'
194-
run: |
195-
echo "❌ Performance regressions detected!"
196-
echo "See benchmark results in artifacts for details"
197-
exit 1

0 commit comments

Comments
 (0)