Fix addon devcontainer not being able to build addons#135
Fix addon devcontainer not being able to build addons#135agners merged 2 commits intohome-assistant:mainfrom
Conversation
be04aa4 to
9c23b02
Compare
|
@felipecrs is it possible to take this for a spin while waiting for the PR to be reviewed and merged? |
Yes, swap your devcontainer image with |
Thank you! Can confirm it works on my Apple M2. |
5ff3f9a to
fe17f2e
Compare
|
Yeah, I have been getting that in the It's not related to this PR though. |
|
I can't see avenues to raising issues about the devcontainers. Issues isn't enabled on this repo. I'm not having much luck with discord. HA community just gets a few "me too" comments. How to get attention of people behind this part of HA? |
|
I'd open an issue in Supervisor repository, after all that devcontainer is borked too. |
|
@agners are issues intentionally disabled for this repository? Just checking, maybe it was just a miss. |
I don't think so. I can't enable it, but I'll check with the team. |
|
Speaking of this PR, should we maybe add these two lines to the template in this repository ( |
|
@felipecrs uh right, I did not update my local devcontainer repo 🙈 . Sorry for the noise! |

Coming from the warning?
If you came here because of a warning like this:
Then all you need to do is add these two lines to your devcontainer.json:
PR description
After home-assistant/supervisor#5974 was merged, the add-on devcontainer was not able to build addons.
Now that Supervisor delegates the build to a container, we need to ensure that the add-on directory is within Supervisor's data directory on the host, so that Supervisor can properly mount it within the builder container.
Note that this PR is backwards compatible, i.e. it will work even against Supervisor versions prior to home-assistant/supervisor#5974.
I tested this and confirmed it works.
Closes home-assistant/supervisor#5991
/cc @agners