Skip to content

Commit 1cefd43

Browse files
cron: remove local OSV client experiment (#4840)
Using a local, downloaded database was significantly slower at processing the daily release batch of repos. Additionally, there was a 10x increase in CPU usage (from 0.1 to 1.0 vCPU per worker). Signed-off-by: Spencer Schrock <[email protected]>
1 parent ae20577 commit 1cefd43

File tree

2 files changed

+1
-9
lines changed

2 files changed

+1
-9
lines changed

cron/internal/worker/main.go

Lines changed: 1 addition & 7 deletions
Original file line numberDiff line numberDiff line change
@@ -23,7 +23,6 @@ import (
2323
"fmt"
2424
"net/http"
2525
_ "net/http/pprof" //nolint:gosec
26-
"os"
2726
"strings"
2827

2928
"go.opencensus.io/stats/view"
@@ -130,12 +129,7 @@ func newScorecardWorker() (*ScorecardWorker, error) {
130129
if sw.ossFuzzRepoClient, err = ossfuzz.CreateOSSFuzzClientEager(ossfuzz.StatusURL); err != nil {
131130
return nil, fmt.Errorf("ossfuzz.CreateOSSFuzzClientEager: %w", err)
132131
}
133-
134-
if _, enabled := os.LookupEnv("SCORECARD_LOCAL_OSV"); enabled {
135-
sw.vulnsClient = clients.ExperimentalLocalOSVClient()
136-
} else {
137-
sw.vulnsClient = clients.DefaultVulnerabilitiesClient()
138-
}
132+
sw.vulnsClient = clients.DefaultVulnerabilitiesClient()
139133

140134
if sw.exporter, err = startMetricsExporter(); err != nil {
141135
return nil, fmt.Errorf("startMetricsExporter: %w", err)

cron/k8s/worker.release.yaml

Lines changed: 0 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -59,8 +59,6 @@ spec:
5959
key: auth_token
6060
- name: "SCORECARD_API_RESULTS_BUCKET_URL"
6161
value: "gs://ossf-scorecard-cron-releasetest-results"
62-
- name: SCORECARD_LOCAL_OSV
63-
value: "1"
6462
volumeMounts:
6563
- name: config-volume
6664
mountPath: /etc/scorecard

0 commit comments

Comments
 (0)