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 requested a review from Landrash November 12, 2018 22:35
@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.

3 participants