Clarify Update on Docker HA Core install

[fhoekstra]

Proposed change

I could not find any clear instructions in the docs on how to update a Docker install of Home Assistant Core. It seems to be recommended over a virtualenv install, but there are no instructions on how to update your install, and even a conflicting run command between the systemd service file and the install docs.
I want to save others the hassle of figuring this out.
If I put anything in a wrong place, or if what I figured out is incorrect, please tell me. I do believe this information should be in the documentation.

Type of change

• Spelling, grammar or other readability improvements (current branch).
• Adjusted missing or incorrect information in the current documentation (current branch).
• Added documentation for a new integration I'm adding to Home Assistant (next branch).
  • I've opened up a PR to add logo's and icons in Brands repository.
• Added documentation for a new feature I'm adding to Home Assistant (next branch).
• Removed stale or deprecated documentation.

Additional information

1. I consolidated the docker run command between Docker install and Docker systemd pages. Both now use homeassistant/home-assistant:stable
2. I added timezone binding (/etc/localtime). I am not sure if this is recommended, but it works for my install. Please confirm?
3. I added instructions on how to update a Docker install in the Docker installation page. Just like the update information is included in the virtualenv install page. I put it under Linux since I am not sure who this would work on other platforms. Maybe this should be structured differently?
4. I added the update procedure for HA Core running in a Docker as a systemd service on the same page as well, to demonstrate that whether you are running HA as a service impacts the way you should stop and start it and that the update script is not universal. Do you agree with my placement of the info in the update info instead of in the systemd page?

• Link to parent pull request in the codebase:
• Link to parent pull request in the Brands repository:
• This PR fixes or closes issue:

Checklist

• This PR uses the correct branch, based on one of the following:
  • I made a change to the existing documentation and used the current branch.
  • I made a change that is related to an upcoming version of Home Assistant and used the next branch.
• The documentation follows the Home Assistant documentation standards.

[frenck]

We should remove them altogether, as the proper way to handle this is using Docker itself, not systemd.

[fhoekstra]

Valid point. I didn't know that was built into Docker.

1. Shall I make an issue for that and still push these for now? This change helped me get my install working.
2. Shall I add a note that Docker has its own restart policy and this is preferred? Link to the Docker docs maybe?

[fhoekstra]

Should I say to include --restart=always in the docker run command? With this link: https://docs.docker.com/config/containers/start-containers-automatically/

[frenck]

Yes that is it.

[frenck]

I don't get the username part?

[fhoekstra]

That is the naming convention for systemd services: https://www.home-assistant.io/docs/autostart/systemd/#next-steps

[fhoekstra]

Okay, so perhaps it is better to do either or both of the following:

1. Add a 'HA Core in Docker container' part (either paragraph or page) to the autostart page, and say there how the docker run command should be changed, with a link.
2. Add this same info to the Docker installation page.

[fhoekstra]

I also added some 'Core' to mentions of Home Assistant.

[frenck]

The autostart documentation has been removed. Please rebase the PR to get rid of the changes made to that (or revert them).

Adding the restart=always to the Docker set up is a good replacement.

[fhoekstra]

Oof that was not pretty. I haven't really used git rebase outside a tutorial, but I verified with a git diff against an up-to-date current that I now pushed the proper version to this branch.

So you also agree on the change of mapping /etc/localtime instead of the New York string @frenck ? Only in the Linux instruction so far, haven't been able to test on Raspbian.

[frenck]

The mapping of local time seems ok. This PR however, has an issue now, with all kinds of unrelated commits. This needs to be cleaned up first 👍

[fhoekstra]

@frenck better?

[frenck]

No, if you look here: https://github.com/home-assistant/home-assistant.io/pull/13542/commits

You'll see all kind of commits that are not yours. You need to rebase it not merge branches.
I think the easiest right now is to create a new branch, cherry-pick your changes force put it onto this branch.

[fhoekstra]

Thanks for your repeated help, I'm learning advanced git here 😄 . I think I've done what you just said now @frenck .

[frenck]

Thanks, @fhoekstra 👍

[frenck]

Oh noticed one little thing just before I wanted to hit merge:

Our images have a init system onboard, so the --init isn't needed.

[fhoekstra]

Accepted, do you like being @'ed @frenck ?

[frenck]

Thanks, @fhoekstra! 👍 🎉