Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Update 02_our_app.md #18875

Closed
wants to merge 1 commit into from
Closed

Update 02_our_app.md #18875

wants to merge 1 commit into from

Conversation

torressam333
Copy link

Proposed changes

When going through the get started example I ran into an issue with running the container with the -d flag and the container would never actually run. Running the container in interactive mode helped for me to be able to run the container successfully. I added supplemental documentation to help other users by providing them a 2nd way to run a docker container if they run into an issue with the first method.

My machine specs: Darwin LAP09162TN 23.1.0 Darwin Kernel Version 23.1.0: Mon Oct 9 21:27:27 PDT 2023; root:xnu-10002.41.9~6/RELEASE_X86_64 x86_64

Related issues (optional)

N/A

When going through the get started example I ran into an issue with running the container with the -d flag and the container would never actually run. Running the container in interactive mode helped for me to be able to run the container successfully.

My machine specs: `Darwin LAP09162TN 23.1.0 Darwin Kernel Version 23.1.0: Mon Oct  9 21:27:27 PDT 2023; root:xnu-10002.41.9~6/RELEASE_X86_64 x86_64`
Copy link

netlify bot commented Dec 7, 2023

Deploy Preview for docsdocker ready!

Name Link
🔨 Latest commit 20e8589
🔍 Latest deploy log https://app.netlify.com/sites/docsdocker/deploys/65720fccc7ce110008fff7f3
😎 Deploy Preview https://deploy-preview-18875--docsdocker.netlify.app
📱 Preview on mobile
Toggle QR Code...

QR Code

Use your smartphone camera to open QR code link.

To edit notification comments on pull requests, go to your Netlify site configuration.

@github-actions github-actions bot added the area/get-started Relates to get started and onboarding docs label Dec 7, 2023
@dvdksn
Copy link
Collaborator

dvdksn commented Dec 7, 2023

What you encountered was actually the expected behavior - running a container in detached mode prints the containerid. The container still runs as you would see if you ran the docker ps command.

I do see how this can be confusing for someone getting started. But changing this behavior is probably not gonna happen, as I'm sure there are lots of scripts that rely on this. I guess we could rewrite it to be clearer that this is expected. //cc @craig-osterhout

@dvdksn dvdksn closed this Dec 7, 2023
@torressam333
Copy link
Author

@dvdksn Ah, gotcha. Thank you for the clarification.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
area/get-started Relates to get started and onboarding docs
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants