Skip to content
This repository has been archived by the owner on Dec 13, 2023. It is now read-only.

Doc: Updates docker-compose commands to start Database container #2465

Merged
merged 1 commit into from
Sep 20, 2021

Conversation

starkblaze01
Copy link
Contributor

Pull Request type

  • Bugfix
  • Feature
  • Refactoring (no functional changes, no api changes)
  • Build related changes
  • Other (please describe): Update docs to fix docker-compose commands according to the new changes

Changes in this PR

The docker-compose command is updated with the -f flag to start the database container also, and adds the --remove-orphans flag to ensure all the containers are removed properly

Issue #2464

Copy link
Contributor

@jxu-nflx jxu-nflx left a comment

Choose a reason for hiding this comment

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

lgtm

@jxu-nflx jxu-nflx merged commit feb1406 into Netflix:main Sep 20, 2021
@github-actions
Copy link
Contributor

Unit Test Results

   152 files  ±0     152 suites  ±0   8m 16s ⏱️ ±0s
1 115 tests ±0  1 115 ✔️ ±0  0 💤 ±0  0 ❌ ±0 

Results for commit feb1406. ± Comparison against base commit feb1406.

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants