Skip to content
This repository has been archived by the owner on Oct 30, 2019. It is now read-only.

Homeassistant: Set permissions before install. #220

Merged
merged 1 commit into from
Nov 13, 2018
Merged

Homeassistant: Set permissions before install. #220

merged 1 commit into from
Nov 13, 2018

Conversation

ludeeus
Copy link
Member

@ludeeus ludeeus commented Nov 12, 2018

Description:

The install_homeassistant.service now upgrades hassbian-scripts on first boot if there is an update, resulting in /srv/homeassistant being owned by root.

From first boot:

pi@hassbian:~ $ ls -ld /srv/homeassistant
drwxr-xr-x 3 root root 4096 Nov 12 22:11 /srv/homeassistant

And in journalctl

Nov 12 22:11:30 hassbian hassbian-config[1223]: Changing to the homeassistant user
Nov 12 22:11:30 hassbian hassbian-config[1223]: Creating Home Assistant venv
Nov 12 22:11:31 hassbian hassbian-config[1223]: Error: [Errno 13] Permission denied: '/srv/homeassistant/include'
Nov 12 22:11:31 hassbian hassbian-config[1223]: Changing to Home Assistant venv
Nov 12 22:11:31 hassbian hassbian-config[1223]: /bin/bash: line 6: /srv/homeassistant/bin/activate: No such file or directory

By explicit setting permissions before the install starts this will be corrected.
Added it to the upgrade script to for good measure.

Checklist (Required):

@ludeeus
Copy link
Member Author

ludeeus commented Nov 12, 2018

This should be released as a hotfix for 0.11.1 as soon as possible since all new installs now fail.

@ludeeus ludeeus changed the title WIP: Set permissions before install. Set permissions before install. Nov 12, 2018
@ludeeus ludeeus changed the title Set permissions before install. Homeassistant: Set permissions before install. Nov 12, 2018
@Landrash Landrash merged commit 78f456c into dev Nov 13, 2018
@ludeeus ludeeus deleted the permissions branch November 13, 2018 10:07
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

3 participants