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

packer files for load balancer #36

Closed
wants to merge 22 commits into from
Closed

packer files for load balancer #36

wants to merge 22 commits into from

Conversation

wallies
Copy link
Contributor

@wallies wallies commented Mar 29, 2015

No description provided.

},
"post-processors": [{
"type": "atlas",
"token": "",
Copy link
Member

Choose a reason for hiding this comment

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

should this be removed? or reference ENV?

@@ -14,4 +14,5 @@ main() {
register_service "$node" docker
register_service "$node" consul
register_service "$node" dnsmasq
register_service "$node" haproxy
Copy link
Member

Choose a reason for hiding this comment

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

not sure we want haproxy running on every slave ?

@tayzlor
Copy link
Member

tayzlor commented Mar 30, 2015

Could this be hooked up to the wercker build (non amazon)?

@tayzlor
Copy link
Member

tayzlor commented Mar 31, 2015

This needs to be rebased on master after #45

@tayzlor tayzlor added this to the 0.1 milestone Apr 8, 2015
user = "ubuntu"
key_file = "${var.key_file}"
host = "${aws_instance.loadbalancer.public_ip}"
script_path = "/tmp/${element(aws_instance.loadbalancer.*.id, count.index)}.sh"
Copy link
Member

Choose a reason for hiding this comment

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

now 0.4.0 is out these can use ${self.id}

@tayzlor
Copy link
Member

tayzlor commented Apr 24, 2015

Needs a big refactor / rebase against master.

@tayzlor
Copy link
Member

tayzlor commented Apr 24, 2015

Outstanding tasks here -

  • Move haproxy / consul template provisioning (of config files / consul service definitions) to Ansible
  • Lock weave version
  • Update architecture diagram to reflect frontend proxy inclusion
  • Add support for provisioning frontend proxy in vagrant
  • Add support for provisioning frontend proxy in digitalocean
  • Fix wercker build yml file

@tayzlor
Copy link
Member

tayzlor commented Apr 27, 2015

I would be in favour of ditching this PR and replacing it with a docker container for haproxy/consul-template. This would be a bit more lightweight and has the benefit we could run it on a stock ubuntu (or other) AMI / droplet / whatever.

This looks promising -
https://github.com/CiscoCloud/haproxy-consul

or we could roll our own opinionated container borrowing some concepts from here too https://github.com/factorish/proxy

@enxebre
Copy link
Contributor

enxebre commented Apr 27, 2015

I find it kind of uncommon as we would have a mono-container dedicated vm but I am in favour too for the sake of portability and reusability across clouds/operating systems. Also this would allow us to test/switch between load-balancer approaches like the two links you suggest fairly easily

@tayzlor
Copy link
Member

tayzlor commented Apr 27, 2015

it would not necessarily have to sit on its own in a VM (a mono-container as you say). in fact it could sit anywhere. I think it might make sense to provision it on a separate instance though at least in the AWS setup for simplicity (at first anyway), but it could easily go on the NAT server (for example). Maybe it would be easier to just plonk it on the NAT server. we'd still need to think about what we do for vagrant/digitalocean though. Plus once we have #55 in (it wont just be on its own), plus we may add other services there for monitoring/logging and other stuff if we desire.

@tayzlor
Copy link
Member

tayzlor commented Apr 28, 2015

Going to close this in favour of #112 . Re-open if you have a burning desire to resurrect.

@tayzlor tayzlor closed this Apr 28, 2015
@wallies
Copy link
Contributor Author

wallies commented Apr 28, 2015

yeh im in favour of it going in a container. Could just setup the NAT to forward external requests to the loadbalanced haproxy container. It was in another one of the terraform examples

@tayzlor
Copy link
Member

tayzlor commented Apr 28, 2015

if you've got a link, drop it in #112

@wallies wallies deleted the 7-package-image-lb branch April 30, 2015 14:27
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