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

Added a command for getting the service #8189

Merged
merged 4 commits into from Feb 14, 2019

Conversation

@chilupa
Copy link
Contributor

commented Feb 8, 2019

Proposed changes

When you use docker service ls, it won't show you the list of services. Instead use docker stack services getstartedlab to show the service that was created and thereby getting the Service ID of it.

Added a command for getting the service
When you use `docker service ls`, it won't show you the list of services. Instead use `docker stack services getstartedlab` to show the service that was created and thereby getting the Service ID of it.
@GordonTheTurtle

This comment has been minimized.

Copy link

commented Feb 8, 2019

Deploy preview for docsdocker ready!

Built with commit 1e096d0

https://deploy-preview-8189--docsdocker.netlify.com

@ghost

This comment has been minimized.

Copy link

commented Feb 8, 2019

@thaJeztah PTAL

get-started/part3.md Outdated Show resolved Hide resolved
get-started/part3.md Outdated Show resolved Hide resolved

bermudezmt added some commits Feb 14, 2019

@bermudezmt bermudezmt requested a review from paigehargrave Feb 14, 2019

@bermudezmt

This comment has been minimized.

Copy link
Contributor

commented Feb 14, 2019

@thaJeztah I've added both suggested changes. Can you or @paigehargrave review for final validation before we merge?

paigehargrave
@paigehargrave

This comment has been minimized.

Copy link
Contributor

commented Feb 14, 2019

@thaJeztah - This looks good to me since you already reviewed. Please reopen it if I'm mistaken. Many thanks again for all your help with these PRs/Issues.

@paigehargrave paigehargrave merged commit f5266b3 into docker:master Feb 14, 2019

1 of 2 checks passed

deploy/netlify Deploy preview processing.
Details
continuous-integration/jenkins/pr-head This commit looks good
Details
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
5 participants
You can’t perform that action at this time.