Skip to content

Add consul client as a coprocess to mongo#10

Merged
geek merged 5 commits intoautopilotpattern:masterfrom
lucj:master
Mar 15, 2017
Merged

Add consul client as a coprocess to mongo#10
geek merged 5 commits intoautopilotpattern:masterfrom
lucj:master

Conversation

@lucj
Copy link
Copy Markdown
Contributor

@lucj lucj commented Mar 6, 2017

Hello,
I started to embed Consul client in mongodb container as a co-process of mongod.
I'm still working on a problem with the connection to the local consul though.

mongodb_1  | Service not registered, registering...
mongodb_1  | 2017/03/06 14:34:40 Service registration failed: Put http://localhost:8500/v1/agent/service/register: dial tcp [::1]:8500: getsockopt: connection refused

Regards,
Luc

@lucj
Copy link
Copy Markdown
Contributor Author

lucj commented Mar 6, 2017

The local consul agent is not correctly started, I cannot figure out why yet.

When I start it manually from inside the running container with the following command, mongo can register right away.

/usr/local/bin/consul agent -data-dir=/opt/consul/data -config-dir=/opt/consul/config -rejoin -retry-join consul

@lucj
Copy link
Copy Markdown
Contributor Author

lucj commented Mar 6, 2017

Seems to work fine switching to ContainerPilot 2.7.0.

@sberryman
Copy link
Copy Markdown
Contributor

What about bumping mongodb to 3.4 as well? It has been out since Nov 29 2016 so quite a long time...

@sberryman
Copy link
Copy Markdown
Contributor

thanks @lucj, that was quick! hopefully joyent will merge and push a new image soon.

@lucj
Copy link
Copy Markdown
Contributor Author

lucj commented Mar 6, 2017

@sberryman thanks, well spotted. It was planned but then forgotten. I've just changed it.

@geek
Copy link
Copy Markdown
Contributor

geek commented Mar 15, 2017

LGTM

@geek geek self-assigned this Mar 15, 2017
@geek geek added this to the 3.4.0-r1.1.0 milestone Mar 15, 2017
@geek geek merged commit cee31e6 into autopilotpattern:master Mar 15, 2017
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.

3 participants