Skip to content

Update the devcontainer image version to v6 in devcontainer config - #197

Merged
agners merged 1 commit into
home-assistant:mainfrom
ricklambrechts:update-devcontainer-version
Sep 2, 2026
Merged

Update the devcontainer image version to v6 in devcontainer config#197
agners merged 1 commit into
home-assistant:mainfrom
ricklambrechts:update-devcontainer-version

Conversation

@ricklambrechts

@ricklambrechts ricklambrechts commented Sep 1, 2026

Copy link
Copy Markdown
Contributor

After using this example devcontainer file for a project the warning of #135 appeared.

The log output is:

Running the postStartCommand from devcontainer.json...

[8502 ms] Start: Run in container: /bin/sh -c bash devcontainer_bootstrap
WARNING: Mounting /mnt/supervisor/apps/local/<project> to /mnt/supervisor/addons/local/<project>.
To avoid this warning, see https://github.com/home-assistant/devcontainer/pull/135.
Done. Press any key to close the terminal.

When updating the image to ghcr.io/home-assistant/devcontainer:6-apps the warning is fixed!

The log output is:

Running the postStartCommand from devcontainer.json...

[8887 ms] Start: Run in container: /bin/sh -c bash devcontainer_bootstrap
Done. Press any key to close the terminal.

So I think the script inside the v5 image and the mounting location in de devcontainer did not match at that time the image was build but I did not fully dive into that.

@home-assistant home-assistant Bot left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Hi @ricklambrechts

It seems you haven't yet signed a CLA. Please do so here.

Once you do that we will be able to review and accept this pull request.

Thanks!

@home-assistant
home-assistant Bot marked this pull request as draft September 1, 2026 21:51
@home-assistant

home-assistant Bot commented Sep 1, 2026

Copy link
Copy Markdown

Please take a look at the requested changes, and use the Ready for review button when you are done, thanks 👍

Learn more about our pull request process.

@agners agners left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM!

@agners
agners merged commit 12854a5 into home-assistant:main Sep 2, 2026
3 checks passed
@ricklambrechts
ricklambrechts deleted the update-devcontainer-version branch September 2, 2026 10:31
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants