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.

Sign up for GitHub

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

Jump to bottom

Not needed to mount /data as Home Assistant add-on #2800

Closed
pvizeli opened this issue Jul 9, 2024 · 2 comments
Closed

Not needed to mount /data as Home Assistant add-on #2800

pvizeli opened this issue Jul 9, 2024 · 2 comments
Labels
💡 enhancement-ideas New feature or change request platform/ha-addon Home Assistant Add-on platform 🌱 minor This is a issue/ticket which can be easily fixed
Milestone
3.77.6.20240720

Comments

@pvizeli
Copy link

pvizeli commented Jul 9, 2024

Describe the solution you'd like

It's not needed anymore to bind mount the /data partition to a different location. There is a new format that allow to define the path.

Describe alternatives you've considered

Well, currently it use the alternative and not the native way

Is your feature request related to a problem?

Need less permission and using mount inside a container is also not that nice

Additional information

home-assistant/developers.home-assistant#1974
@pvizeli pvizeli added the 💡 enhancement-ideas New feature or change request label Jul 9, 2024
@pvizeli pvizeli changed the title Not needed to mount /data with https://github.com/home-assistant/developers.home-assistant/pull/1974 Not needed to mount /data as Home Assistant add-on Jul 9, 2024
@jens-maus
Copy link
Owner

Wooooh. Completely missed that new possibility to specify a different mount path for the addon data directory and will test it ASAP! Thanks for the hint @pvizeli

@jens-maus jens-maus added 🌱 minor This is a issue/ticket which can be easily fixed platform/ha-addon Home Assistant Add-on platform labels Jul 9, 2024
@jens-maus jens-maus added this to the next release milestone Jul 9, 2024
jens-maus added a commit that referenced this issue Jul 9, 2024
@jens-maus
rework data path to /usr/local mapping by using a somewhat newer
3c028af 
possibility by HA to define the target path directly in config.yaml.
This allows to remove the S00DataMount init script to bind mount /data
to /usr/local as a workaround. This refs #2800.
@jens-maus
Copy link
Owner

latest dev version seems to work nicely with the updated config.yaml. Thus, closing issue as solved.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
💡 enhancement-ideas New feature or change request platform/ha-addon Home Assistant Add-on platform 🌱 minor This is a issue/ticket which can be easily fixed
Projects
None yet
Milestone
3.77.6.20240720
Development

No branches or pull requests
2 participants
@jens-maus @pvizeli