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

Docker compose #3

Open
wants to merge 3 commits into
base: master
Choose a base branch
from
Open

Docker compose #3

wants to merge 3 commits into from

Conversation

zepouet
Copy link

@zepouet zepouet commented Jun 10, 2017

Hello

I just rewrite your kubernetes demo for Docker Swarm.
I will do it with Docker Swarm soon.

Nicolas

@waxzce
Copy link
Member

waxzce commented Jun 22, 2017


On OSX, you can use Docker for Mac with success.

We have to add the domain names to our host file. We need to know the machine IP and add it to the /etc/hosts file
Copy link
Member

Choose a reason for hiding this comment

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

the doc about ip is confusing, plz fix (use localhost or something)

Copy link
Author

Choose a reason for hiding this comment

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

You are right. It is confusing. I will try to be more short and concise.
Thanks

* [pikachu.local](http://pikachu.local)
* [mewtwo.local](http://mewtwo.local)
* [nidoqueen.local](http://nidoqueen.local)

Copy link
Member

Choose a reason for hiding this comment

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

can you provide a demo with changing domain name

Copy link
Author

Choose a reason for hiding this comment

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

Can you give me more information ? Do you prefer this other domain name ?
I wanted to use the same as yours with k8s.
Maybe i-like-pikachu.local ?

Thanks

@waxzce waxzce requested a review from NotBad4U April 9, 2018 17:52
@waxzce
Copy link
Member

waxzce commented Apr 9, 2018

@NotBad4U can you step in?

@NotBad4U
Copy link

@waxzce @zepouet Sozu can push his metrics on any service that implement the statsd protocol. I think we can add a metrics collector service (telegraf, cernan, ..) and another service to visualize them.
NOTE:

  • That can be done in another PR.
  • timeseries metrics will be release soon in sozu.

@waxzce
Copy link
Member

waxzce commented Apr 10, 2018

@NotBad4U what metrics does here.... The goal is to provide a demo using sozu and another soft. look others demo on this repo plz

@NotBad4U
Copy link

NotBad4U commented Apr 10, 2018

Okai I didn't understand we were focused on only one application per forlder.
The sozu configuration has a bit changed. I'll try to find some time to fix that.


[proxies]

[proxies.HTTP]

Choose a reason for hiding this comment

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

The configuration now use a Application table.

address = "192.168.59.103"
port = 8125

[proxies]

Choose a reason for hiding this comment

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

The table is now call applications


[proxies.HTTP]
address = "0.0.0.0"
proxy_type = "HTTP"

Choose a reason for hiding this comment

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

You will add this in a protocol part. ([http] table)

@@ -0,0 +1,48 @@
version: "3"

services:

Choose a reason for hiding this comment

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

I think that can be good to add a custom bridge network and avoid to use the default bridge.

services:

sozu:
image: geal/sozu

Choose a reason for hiding this comment

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

Please use this image instead: clevercloud/sozu

@waxzce
Copy link
Member

waxzce commented Apr 11, 2018

Can you test it on a small screen cast also ?

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.

3 participants