Skip to content

Fix satellite-admin to use Spanner with db selector#119

Open
Copilot wants to merge 2 commits intomainfrom
copilot/fix-admin-service-db-configuration
Open

Fix satellite-admin to use Spanner with db selector#119
Copilot wants to merge 2 commits intomainfrom
copilot/fix-admin-service-db-configuration

Conversation

Copy link

Copilot AI commented Feb 3, 2026

When running storj-up init db,core,admin,minimal, satellite-admin was connecting to CockroachDB while satellite-api and satellite-core correctly used Spanner. The db selector deploys Spanner, not CockroachDB.

Changes

Updated pkg/recipe/admin.yaml:

  • STORJ_DATABASE: cockroachspanner environment reference
  • STORJ_METAINFO_DATABASE_URL: cockroachspanner environment reference
  • Added missing SPANNER_EMULATOR_HOST configuration

This aligns satellite-admin with satellite-api and satellite-core. Using the explicit cockroach selector still works as expected.

Original prompt

This section details on the original issue you should resolve

<issue_title>Admin is still using cockroach DB by default</issue_title>
<issue_description>After running storj-up init db,core,admin,minimal admin service have crdb DB and rest of services is using Spanner.

Image</issue_description>

Comments on the Issue (you are @copilot in this section)


💬 We'd love your input! Share your thoughts on Copilot coding agent in our 2 minute survey.

Co-authored-by: mniewrzal <7313031+mniewrzal@users.noreply.github.com>
Copilot AI changed the title [WIP] Fix admin service to use Spanner instead of Cockroach DB Fix satellite-admin to use Spanner with db selector Feb 3, 2026
Copilot AI requested a review from mniewrzal February 3, 2026 12:26
@mniewrzal mniewrzal marked this pull request as ready for review February 3, 2026 12:31
@mniewrzal mniewrzal requested a review from elek as a code owner February 3, 2026 12:31
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.

Admin is still using cockroach DB by default

3 participants