Skip to content

docs: add troubleshooting section for Docker build and runtime issues#951

Merged
decyjphr merged 1 commit intomain-enterprisefrom
yadhav/add-docker-troubleshooting-docs
Mar 23, 2026
Merged

docs: add troubleshooting section for Docker build and runtime issues#951
decyjphr merged 1 commit intomain-enterprisefrom
yadhav/add-docker-troubleshooting-docs

Conversation

@decyjphr
Copy link
Collaborator

No description provided.

Copilot AI review requested due to automatic review settings March 23, 2026 17:33
@decyjphr decyjphr merged commit 241b5d9 into main-enterprise Mar 23, 2026
7 checks passed
Copy link
Contributor

Copilot AI left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Pull request overview

Adds a Docker troubleshooting runbook to help developers debug local builds vs published GHCR images, and links to it from the deployment docs.

Changes:

  • Add a new docs/docker-debugging.md runbook with step-by-step commands for building/running locally and comparing with GHCR.
  • Add a “Troubleshooting Docker Build and Runtime Issues” section to docs/deploy.md linking to the new runbook.

Reviewed changes

Copilot reviewed 2 out of 2 changed files in this pull request and generated 2 comments.

File Description
docs/docker-debugging.md New debugging runbook covering local build/run, GHCR pull/run, and common failure patterns.
docs/deploy.md Adds a troubleshooting pointer to the new Docker debugging runbook.

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

Comment on lines +39 to +43
--env-file ./.env \
--env NODE_ENV=development \
--env HOST=0.0.0.0 \
-p 3000:3000 \
-it safe-settings:local
Copy link

Copilot AI Mar 23, 2026

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

The command examples use tab characters for indentation (e.g., the --env-file / --env / -p lines). This repo’s .editorconfig specifies indent_style = space, and tabs often paste inconsistently across shells/editors. Replace the leading tabs with spaces throughout the code blocks to keep formatting consistent and copy/paste-friendly.

Suggested change
--env-file ./.env \
--env NODE_ENV=development \
--env HOST=0.0.0.0 \
-p 3000:3000 \
-it safe-settings:local
--env-file ./.env \
--env NODE_ENV=development \
--env HOST=0.0.0.0 \
-p 3000:3000 \
-it safe-settings:local

Copilot uses AI. Check for mistakes.
Comment on lines +68 to +70
```bash
docker pull ghcr.io/github/safe-settings:2.1.19
```
Copy link

Copilot AI Mar 23, 2026

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

The GHCR image tag is hard-coded to 2.1.19 in multiple commands. Other docs in this repo reference different tags (e.g., deploy.md mentions 2.1.13, awslambda.md mentions 2.1.17), so this is likely to become stale/confusing. Consider using a placeholder (e.g., $SAFE_SETTINGS_TAG) and/or pointing readers to where to find the current recommended tag, then reuse that consistently across docs.

Copilot uses AI. Check for mistakes.
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.

2 participants