Skip to content

Add Docker restart policy to deployment guides#1711

Merged
kmcginnes merged 1 commit intoaws:mainfrom
kmcginnes:docs/docker-restart-policy
Apr 23, 2026
Merged

Add Docker restart policy to deployment guides#1711
kmcginnes merged 1 commit intoaws:mainfrom
kmcginnes:docs/docker-restart-policy

Conversation

@kmcginnes
Copy link
Copy Markdown
Collaborator

Description

  • Add --restart unless-stopped to the docker run commands in the EC2 deployment guide and the getting-started Local Docker Setup. Without a restart policy, the Graph Explorer container won't come back up after a host reboot or crash — important for EC2, EKS, and long-running local deployments.
  • Add a tip in the EC2 guide linking to Docker's restart policy documentation for users who want different behavior.

Validation

Verified --restart unless-stopped is the recommended Docker restart policy for containers that should survive host reboots but respect manual stops.

Related Issues

Check List

  • I confirm that my contribution is made under the terms of the Apache 2.0 license.
  • I have verified pnpm checks passes with no errors.
  • I have verified pnpm test passes with no failures.
  • I have covered new added functionality with unit tests if necessary.
  • I have updated documentation if necessary.

@codecov
Copy link
Copy Markdown

codecov Bot commented Apr 23, 2026

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 72.69%. Comparing base (c5affc5) to head (dc4ce34).
⚠️ Report is 152 commits behind head on main.

Additional details and impacted files
@@             Coverage Diff             @@
##             main    #1711       +/-   ##
===========================================
+ Coverage   47.81%   72.69%   +24.88%     
===========================================
  Files         382      408       +26     
  Lines        8525    12287     +3762     
  Branches     3159     3529      +370     
===========================================
+ Hits         4076     8932     +4856     
+ Misses       3070     2337      -733     
+ Partials     1379     1018      -361     
Flag Coverage Δ
unittests 72.69% <ø> (+24.88%) ⬆️

Flags with carried forward coverage won't be shown. Click here to find out more.

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.
  • 📦 JS Bundle Analysis: Save yourself from yourself by tracking and limiting bundle sizes in JS merges.

@kmcginnes kmcginnes marked this pull request as ready for review April 23, 2026 22:13
@kmcginnes kmcginnes merged commit 1277a86 into aws:main Apr 23, 2026
4 of 5 checks passed
@kmcginnes kmcginnes deleted the docs/docker-restart-policy branch April 23, 2026 22:13
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.

Add guidance about Docker restart policy

1 participant