Skip to content

fix: Wire the CI remote cache into the nested joern/dicer build - #1953

Open
bor-p-s wants to merge 1 commit into
masterfrom
bor-p-s/joern-nested-remote-cache
Open

fix: Wire the CI remote cache into the nested joern/dicer build#1953
bor-p-s wants to merge 1 commit into
masterfrom
bor-p-s/joern-nested-remote-cache

Conversation

@bor-p-s

@bor-p-s bor-p-s commented Aug 27, 2026

Copy link
Copy Markdown
Collaborator

Description

downstream_test's nested bazel test builds joern/dicer cold every run. This adds the same remote cache CI's outer bazel already uses, gated on BUILDKITE=true (inert on a developer's machine), under its own cache-silo-key. --cache_test_results=no keeps the test targets (9 for joern, 163 for dicer) executing every time -- only the build-action time gets cached.

.bazelci/presubmit.yml adds --test_env=BUILDKITE to the ubuntu2004 task: Bazel gives test actions a fixed, curated environment regardless of sandboxing, so BUILDKITE reaches downstream_test_driver.sh only with an explicit forward.

Motivation

joern_test is the longest step in bazel test //...: ~1026-1062s, of which ~450-470s is build time for joern's own ~1030 actions, unaffected by #1938's caching.

@bor-p-s
bor-p-s force-pushed the bor-p-s/joern-nested-remote-cache branch from 74cb5fd to 523c049 Compare August 31, 2026 18:32
@bor-p-s
bor-p-s force-pushed the bor-p-s/joern-nested-remote-cache branch from 523c049 to b2dd81b Compare August 31, 2026 19:55
@bor-p-s bor-p-s changed the title experiment: Wire the CI remote cache into the nested joern/dicer build fix: Wire the CI remote cache into the nested joern/dicer build Aug 31, 2026
@bor-p-s
bor-p-s force-pushed the bor-p-s/joern-nested-remote-cache branch from b2dd81b to 98220a5 Compare August 31, 2026 21:04
@bor-p-s
bor-p-s force-pushed the bor-p-s/joern-nested-remote-cache branch from 98220a5 to 5f7275e Compare September 1, 2026 10:39
@bor-p-s
bor-p-s marked this pull request as ready for review September 1, 2026 10:42
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant