GV

GeezerVison

The World Wide Web, restored to 800 pixels of glory.

Home|About|Gaming cabinet|Archives|How it works|Starter sites

On this site

  • Sign in
  • ActionsCI/CD and automation
  • CodespacesCloud development environments
  • All Features
  • Documentation
  • Training
  • Certifications
  • Build with Copilot
  • Customize Copilot
  • Integrations
  • Trending repos
  • GitHub Sponsors
  • Developer articles
  • Enterprises

This view

  • Original page
  • New address

GeezerVison is showing github.com — docker-compose: Start all three databases from db-only-migrate by marcleblanc2 · Pull Request #1485 · sourcegraph/deploy-sourcegraph-docker

Conversation

@marcleblanc2

@marcleblanc2 marcleblanc2 commented Nov 11, 2026 •
edited
Loading

Uh oh!

There was an error while loading. Please reload this page.

Copy link
Copy Markdown
Contributor

db-only-migrate.docker-compose.yaml starts only the databases, so nothing else is connected to them during a backup or restore. It had two problems:

  • It left out codeinsights-db. The backup and restore docs start this file, then run docker exec codeinsights-db ... pg_dump. That container was not running, so the step failed.
  • It had drifted. It was a hand-made copy of pgsql and codeintel-db, under a "MUST be kept in sync" comment, and no longer matched docker-compose.yaml.

It now uses extends for all three databases, so it cannot drift. extends copies one service's settings. It does not start any other service from docker-compose.yaml, so the file starts exactly three containers: pgsql, codeintel-db, and codeinsights-db.

What customers will see

codeinsights-db now starts, on the same codeinsights-db volume the full deployment uses.

pgsql and codeintel-db start with the same settings as in the full deployment:

Setting Before After
mem_limit 2g 4g
shm_size not set 1g
/conf mount missing ../<db>/conf:/conf/
Health check 3 retries, 15s start period 10 retries, 1h start period
stop_grace_period not set 120s

Risk: low to medium. Customers run this file during upgrades, backups, and restores. The databases get the settings they already have in the full deployment, on the same volumes.

Checklist

  • All images have a valid tag and SHA256 sum (no image changes)

Test plan

  • docker compose -f db-only-migrate.docker-compose.yaml config --services lists pgsql, codeintel-db, and codeinsights-db, and nothing else (Compose 2.18.1 and 5.7.1). On main it lists the first two

  • Each of the three resolves to the same configuration from this file and from docker-compose.yaml. On Compose 5.7.1 the output is identical; on 2.18.1 it lists each service's two mounts in the other order

  • Ran the backup steps from the docs on a test machine (Docker 29.9.3, Compose 5.7.1): a full deployment, down, then this file alone on the same volumes

    main This PR
    Containers started pgsql, codeintel-db pgsql, codeintel-db, codeinsights-db; healthy in 6 seconds, nothing else running
    Other clients connected to the databases 0 0
    The three pg_dump commands from the docs The third fails: No such container: codeinsights-db All three exit 0
    docker compose -f docker-compose.yaml run --rm --no-deps migrator Fails: lookup codeinsights-db ... no such host Schema(s) are up-to-date!
  • Restoring this PR's three dumps into empty volumes brings back the same row counts

  • Build: build 16870: passed, the smoke test job took 507 seconds. After the rebase onto the 8.2.0 images, build 16903 passed in 458 seconds

Closes FE-580

Sorry, something went wrong.

Uh oh!

There was an error while loading. Please reload this page.

All reactions
@github-actions

github-actions Bot commented Nov 11, 2026

Copy link
Copy Markdown

Open the Diff Tour for this PR in Sourcegraph

All reactions

Sorry, something went wrong.

Uh oh!

There was an error while loading. Please reload this page.

@marcleblanc2
marcleblanc2 added this pull request to stack #1491 November 11, 2026 10:45
This was referenced Nov 11, 2026
ci: Smoke test on a gcloud VM with a 1 hour TTL, a retry, Jaeger, and the executor #1477
Closed
ci: Check every Compose file and shell script, and test Jaeger and the executor #1489
Draft
@marcleblanc2
docker-compose: Start all three databases from db-only-migrate
7966f87
The file left out codeinsights-db, which the backup and restore docs dump, and its copies of pgsql and codeintel-db had drifted from docker-compose.yaml. It now extends all three services from that file.
@marcleblanc2
marcleblanc2 force-pushed the marc/db-only-migrate-three-databases branch from e1b6797 to 7966f87 Compare November 11, 2026 23:45
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

No reviews

Uh oh!

There was an error while loading. Please reload this page.

Assignees

No one assigned

Labels

None yet

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

Uh oh!

There was an error while loading. Please reload this page.

1 participant

@marcleblanc2
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
You are visitor 01402503 — GeezerVison does not store the pages you look up.
Best viewed with Netscape Navigator 4.7 or Internet Explorer 6 at 800×600. No cookies required.