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.

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

Use host.docker.internal instead of local IP address in chrome.docker #474

Open
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

jamiepenney
Copy link

Resolves #473

Fixes an issue where the Chrome docker container cannot access Storybook running on the host machine via IP address, instead using the Docker host.docker.internal DNS entry to get access to it.

I'm not 100% convinced this is the right approach, but I wanted to put it out there as a solution to my problem. Is there a scenario where using host.docker.internal for accessing the host from the docker container won't work?

@patrik909
Copy link

This also works for me on 0.34.0! Would be great to have this merged :)

@sayonar2k
Copy link

it actually works, ty (:

@Vorbert-Kruk
Copy link

Same here, seems to be resolving the issue.
Would be great to see it merged and released!

Resolves oblador#473

Fixes an issue where the Chrome docker container cannot access Storybook running on the host machine via IP address, instead using the Docker `host.docker.internal` DNS entry to get access to it.
@jamiepenney jamiepenney force-pushed the bugfix/473-fix-chrome-docker-host-access branch from 05aef2e to 444871c Compare September 2, 2024 09:31
@Vorbert-Kruk
Copy link

Hey @techeverri, @oblador
In a convenient time, could you review this PR?

Pushing it forward would resolve CI issues with which people are currently struggling.
There are few issues related to it, but the most relevant probably would be #473.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Failed to load stories in chrome.docker, loki version 0.32.0
4 participants