search

weaveworks-guides / weave-net-legacy

Weave Net Old Guides
Apache License 2.0
88 stars 33 forks source link

Load-balance-simple updated for weave 1.1 #83

Closed abuehrle closed 9 years ago

abuehrle commented 9 years ago

Need someone to review this please. Recompiled sample app in Go, and then I created new docker images, retagged, and pushed them to the dockerhub.

Also, updated scripts and text. @pidster @davkal @errordeveloper

Do we need to come up with some conventions for pushing these images to DockerHub? i.e. what tags would you all like to see on an image.

Here are the options:

https://docs.docker.com/userguide/dockerimages/

Whoever has a few spare cycles to test out and review. thanks.

Also, bad branch name. Forgot to switch before I started working in here.

abuehrle commented 9 years ago

Question for @rade Should I be highlighting the proxy rather than ssh'ing onto the VMs or a combination thereof? Let me know what is the best practice here.

rade commented 9 years ago

I don't understand the question. Can you give an example of what you have in mind?

abuehrle commented 9 years ago

I meant should I install dockertoolbox on my mac and run the docker commands on the VMs from my local machine through the proxy. I think I just answered my own question, since I realized this adds complexity and may not be not necessary.

If you could assign someone to review this. Would be great.

abuehrle commented 9 years ago

I'm going to merge this, since the file that is online now, won't work at all if someone tries to use it.