HA config implementation based on proposal in PR 1730

[ramr]

Initial cut of the HA config implementation based on the proposal in #1730

Example usage:

#  label some nodes router=us-west-ha
openshift kube label node minion-1 "router=us-west-ha"  
openshift kube label node minion-2 "router=us-west-ha"  

#  create router and ha sidecar on the labelled nodes.  
openshift ex router --create --labels="router=us-west-ha" --credentials="${KUBECONFIG}" --replicas=2  
openshift ex ipfailover har --virtual-ips=10.0.2.100-102 --credentials=$KUBECONFIG  --replicas=2 -w 80 --latest-images --selector="router=us-west-ha" --create  

FYI, @smarterclayton @pweil- PTAL and let me know if this approach seems okay.

This PR is dependent on the proposal PR - so don't merge till #1730 is merged in.
So there's probably going to be duplicate cruft in here (the cleaned up version of the keepalived demo pieces).

Also there is more work to do to handle nodes joining/leaving the cluster and on their labels changing.

Edit: updated command name from haconfig to ipfailover

[smarterclayton]

Would be better to do a go echo server here

[ramr]

Yeah, this was just to test and verify the image is okay before I had the integration code in place. The image verification test monitors the echo server listen port.

[smarterclayton]

K, just noticed it while i was going through.

----- Original Message -----

@@ -0,0 +1,36 @@
+#!/usr/bin/env python
+

Yeah, this was just to test and verify the image is okay before I had the
integration code in place. The image verification test monitors the echo
server listen port.

---

Reply to this email directly or view it on GitHub:
https://github.com/openshift/origin/pull/1807/files#r28801451

[ramr]

made all fixes as per review comments (and rebased) @smarterclayton PTAL. Thx

[smarterclayton]

You don't need a service - or at least, I can't think of a reason you would need one.

[ramr]

Ok - will remove it.

[ramr]

@smarterclayton made fixes as per your review comments.

[smarterclayton]

LGTM [merge] - we can track other items in later pulls. Thanks

[openshift-bot]

continuous-integration/openshift-jenkins/merge SUCCESS (https://ci.openshift.redhat.com/jenkins/job/merge_pull_requests_openshift3/1686/) (Image: devenv-fedora_1370)

[openshift-bot]

Evaluated for origin up to de67a26