Skip to content
This repository was archived by the owner on Sep 30, 2024. It is now read-only.

fix(sg/bazel-do): use ci.sourcegraph.bazelrc with bazel-do#63545

Merged
burmudar merged 2 commits into
mainfrom
wb/bazel-do/use-ci-sourcegraph-rc
Jun 28, 2024
Merged

fix(sg/bazel-do): use ci.sourcegraph.bazelrc with bazel-do#63545
burmudar merged 2 commits into
mainfrom
wb/bazel-do/use-ci-sourcegraph-rc

Conversation

@burmudar

Copy link
Copy Markdown
Contributor

Without ci.sourcegraph.bazelrc the bazel environment won't have the right credentials to access the db. This adds the rc to the bazel-do invocation.

For context - the ci.sourcegraph.bazelrc contains this following

# Needed for DB in CI
common --test_env=PGUSER=postgres
common --test_env=PGPASSWORD=postgres
common --test_env=PGSSLMODE=disable
common --test_env=PGDATABASE=postgres

Test plan

https://buildkite.com/sourcegraph/sourcegraph/builds/280332#01905ef3-1fce-4d76-bf5b-0530dc434cff

Changelog

  • sg - ensure bazel-do invocations use the ci sourcegraph bazelrc

@burmudar burmudar requested a review from a team June 28, 2024 13:08
@burmudar burmudar self-assigned this Jun 28, 2024
@cla-bot cla-bot Bot added the cla-signed label Jun 28, 2024
@burmudar burmudar changed the title fix(sg/bazel-d): use ci.sourcegraph.bazelrc with bazel-do fix(sg/bazel-do): use ci.sourcegraph.bazelrc with bazel-do Jun 28, 2024
@burmudar burmudar enabled auto-merge (squash) June 28, 2024 13:10
@burmudar burmudar merged commit 720b2ec into main Jun 28, 2024
@burmudar burmudar deleted the wb/bazel-do/use-ci-sourcegraph-rc branch June 28, 2024 13:14
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants