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 — fix/observability: Prometheus read only root volume by marcleblanc2 · Pull Request #1479 · 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

Problem

Some customers need a read-only root filesystem on every container, and still need Alertmanager to work

The prometheus service couldn't be read_only, because Alertmanager wrote to the root filesystem

Solution

Add read_only: true to the prometheus service

sourcegraph/sourcegraph#18737 moves Alertmanager's files onto the /prometheus data volume.

Do not merge this PR until the pinned prometheus image includes it.

Test plan

Ran this branch with the image Buildkite built from sourcegraph/sourcegraph#18737:

  • prometheus starts read-only, with no read-only or permission errors
  • observability.alerts and observability.silenceAlerts reach Alertmanager, and a webhook receives the notifications
  • Upgrading from main with the released prometheus:8.1.4690 to this branch works on the same volume
  • docker compose stop takes 0.5 seconds, so the default 10 second grace period is enough
  • Control: the released prometheus:8.1.4690 fails with open /sg_config_prometheus/alertmanager.yml: read-only file system
  • The rest of the results are in sourcegraph/sourcegraph#18737

Linked PRs

Merge in this order, because the two deploy PRs need a prometheus image with the fix:

  1. Image fix: sourcegraph/sourcegraph#18737
  2. Helm: fix/observability: Prometheus read only root volume deploy-sourcegraph-helm#1000
  3. Docker Compose: fix/observability: Prometheus read only root volume #1479

Amp threads

  • Design, implementation, and the EKS upgrade test
  • Runtime test in Docker and kind, with commands and output
  • Earlier Helm-only workaround

Sorry, something went wrong.

Uh oh!

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

All reactions
@marcleblanc2 @ampagent
docker-compose: Run prometheus with a read-only root filesystem
533bfae
The prometheus image now keeps Alertmanager's configuration and state on
the /prometheus data volume, so nothing writes to the root filesystem.

Needs sourcegraph/sourcegraph#18737 in the pinned image.

Amp-Thread-ID: https://ampcode.com/threads/T-01a13cfb-9756-7111-8b29-381a841065e1
Co-authored-by: Amp <[email protected]>
@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 mentioned this pull request Nov 11, 2026
fix/observability: Prometheus read only root volume sourcegraph/deploy-sourcegraph-helm#1000
Open
@marcleblanc2 marcleblanc2 changed the title docker-compose: Run prometheus with a read-only root filesystem fix/observability: Prometheus read only root volume Nov 11, 2026
@marcleblanc2 marcleblanc2 mentioned this pull request Nov 11, 2026
docker-compose: Run services with a read-only root filesystem #1470
Draft
@marcleblanc2
marcleblanc2 marked this pull request as ready for review November 11, 2026 03:59
@marcleblanc2
marcleblanc2 requested a review from a team November 11, 2026 03:59
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.