Skip to content

Correct container name for restart command #4021

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

Merged
merged 1 commit into from
Nov 20, 2017

Conversation

joegross
Copy link
Contributor

Description:
Using above instructions container name includes the dash.

shiny % docker ps
CONTAINER ID        IMAGE                          COMMAND                  CREATED             STATUS              PORTS                    NAMES
a9be19e6c76a        homeassistant/home-assistant   "python -m homeass..."   5 minutes ago       Up About a minute   0.0.0.0:8123->8123/tcp   home-assistant
shiny % docker restart homeassistant
Error response from daemon: No such container: homeassistant
[1]    67676 exit 1     docker restart homeassistant
shiny !1! % docker restart home-assistant
home-assistant

Checklist:

  • Branch: Fixes, changes and adjustments should be created against current. New documentation for platforms/components and features should go to next.
  • The documentation follow the standards.

Using above instructions container name includes the dash.

shiny % docker ps
CONTAINER ID        IMAGE                          COMMAND                  CREATED             STATUS              PORTS                    NAMES
a9be19e6c76a        homeassistant/home-assistant   "python -m homeass..."   5 minutes ago       Up About a minute   0.0.0.0:8123->8123/tcp   home-assistant
shiny % docker restart homeassistant
Error response from daemon: No such container: homeassistant
[1]    67676 exit 1     docker restart homeassistant
shiny !1! % docker restart home-assistant
home-assistant
@frenck
Copy link
Member

frenck commented Nov 20, 2017

@joegross Thank you for spotting this one! 🏅

@frenck frenck merged commit 16fd447 into home-assistant:current Nov 20, 2017
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