Skip to content

Commit 2948daf

Browse files
authored
[CHORE](ci): Gate GCS storage tests behind an input flag (#7512)
## Description of changes The test-storage-gcs job requires repository secrets that are not available to forked PRs, causing those runs to fail. Add a run_storage_gcs boolean input to the reusable rust-tests workflow, defaulting to false, and only execute the GCS job when it is set. The release-chromadb workflow enables the job when running on main, where the required secrets are present. ## Test plan CI ## Migration plan N/A ## Observability plan N/A ## Documentation Changes N/A Co-authored-by: AI
1 parent aa35638 commit 2948daf

2 files changed

Lines changed: 10 additions & 0 deletions

File tree

.github/workflows/_rust-tests.yml

Lines changed: 8 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -2,6 +2,12 @@ name: Rust tests
22

33
on:
44
workflow_call:
5+
inputs:
6+
run_storage_gcs:
7+
description: "Run GCS-backed storage tests"
8+
required: false
9+
type: boolean
10+
default: false
511

612
env:
713
FORCE_JAVASCRIPT_ACTIONS_TO_NODE24: "true"
@@ -46,6 +52,8 @@ jobs:
4652
run: cargo nextest run --profile ci_long_running
4753

4854
test-storage-gcs:
55+
# These tests require repository secrets that are unavailable to forked PRs.
56+
if: inputs.run_storage_gcs
4957
runs-on: blacksmith-8vcpu-ubuntu-2404
5058
env:
5159
CARGO_TERM_COLOR: always

.github/workflows/release-chromadb.yml

Lines changed: 2 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -70,6 +70,8 @@ jobs:
7070
name: Rust tests
7171
uses: ./.github/workflows/_rust-tests.yml
7272
secrets: inherit
73+
with:
74+
run_storage_gcs: ${{ github.ref == 'refs/heads/main' }}
7375

7476
go-tests:
7577
name: Go tests

0 commit comments

Comments
 (0)