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

chore: change the hostname of mongodb #24

Merged
merged 1 commit into from
Jul 5, 2023
Merged

Conversation

benoitf
Copy link
Contributor

@benoitf benoitf commented Jun 26, 2023

Description

podman is not able to reach mongo host

But as mongodb container is started with the name microcks-mongodb, reuse the container name as hostname so other containers on the same network can also reach it

Please check on your side on Docker

Related issue(s)

podman is not able to reach mongo host

Change-Id: Ie4d8070ed5e1edb1ac9520ad8e4d6e9760338ee2

Signed-off-by: Florent Benoit <[email protected]>
@hguerrero
Copy link
Member

It looks like Docker is able to find both hostnames, mongo and microcks-mongodb. How does Podman reference the hostnames and container names in the same network?

@benoitf
Copy link
Contributor Author

benoitf commented Jun 26, 2023

podman enables a 'dns' for the user created networks and then use the container name

@benoitf benoitf changed the title chore: change the host name of mongodb chore: change the hostname of mongodb Jun 26, 2023
@hguerrero
Copy link
Member

podman enables a 'dns' for the user created networks and then use the container name

If we use the same name for the container and the hostname then it should work with podman correct?

@benoitf
Copy link
Contributor Author

benoitf commented Jun 27, 2023

yes

@benoitf
Copy link
Contributor Author

benoitf commented Jul 5, 2023

@hguerrero do you want that I change something in my pull request ?

@hguerrero
Copy link
Member

No, thanks, @benoitf

I'm checking that we must change the remaining containers to have the same behavior to work with podman. I will merge this and continue working on the remaining containers.

@hguerrero hguerrero merged commit 957101e into microcks:main Jul 5, 2023
@benoitf
Copy link
Contributor Author

benoitf commented Jul 5, 2023

FYI the bug in podman is containers/podman#17370

@benoitf benoitf deleted the podman-3 branch July 5, 2023 19:49
@benoitf
Copy link
Contributor Author

benoitf commented Jul 5, 2023

@hguerrero when is planned a 0.2.2 release ?

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.

2 participants