Skip to content
This repository has been archived by the owner on Jul 27, 2023. It is now read-only.

Kubernetes by default #1330

Merged
merged 38 commits into from
Apr 6, 2016
Merged

Kubernetes by default #1330

merged 38 commits into from
Apr 6, 2016

Conversation

BrianHicks
Copy link
Contributor

  • Installs cleanly on a fresh build of most recent master branch
  • Upgrades cleanly from the most recent release
  • Updates documentation relevant to the changes

Add Kubernetes by default, incorporating many of @Zogg's changes. Docs are at 3af143e

Successor to #1302, obsoletes #1121 (since we no longer have to customize those variables)

@BrianHicks BrianHicks force-pushed the feature/k8s-by-default branch from 3af143e to 04642e4 Compare April 6, 2016 19:32
@BrianHicks
Copy link
Contributor Author

Just rebased onto lastest master.

@langston-barrett
Copy link
Contributor

After rebasing onto #1296, this upgrades cleanly from 1.0.3 👍

[centos@ip-10-1-1-148 ~]$ kubectl get pods  
NAME                                  READY     STATUS    RESTARTS   AGE
fluentd-elasticsearch-ip-10-1-1-148   1/1       Running   0          26m
[...]
kube-apiserver-ip-10-1-1-148          1/1       Running   0          25m
[...]
kube-podmaster-ip-10-1-1-148          2/2       Running   0          26m
[...]
kube-proxy-ip-10-1-1-148              1/1       Running   0          25m
[...]
[centos@ip-10-1-1-148 ~]$ kubectl get services
NAME                    CLUSTER-IP       EXTERNAL-IP   PORT(S)             AGE
elasticsearch-logging   10.254.54.16     <none>        9200/TCP            27m
heapster                10.254.175.15    <none>        80/TCP              27m
kibana-logging          10.254.207.207   <none>        5601/TCP            27m
kube-ui                 10.254.92.166    <none>        80/TCP              27m
monitoring-grafana      10.254.248.89    <none>        80/TCP              27m
monitoring-influxdb     10.254.253.76    <none>        8083/TCP,8086/TCP   27m

@langston-barrett
Copy link
Contributor

@BrianHicks is this ready to merge now that we know it upgrades cleanly?

@BrianHicks
Copy link
Contributor Author

I thought it was, but it looks like Kubelet is killing the logstash and nginx-consul containers on the box when it starts up. This leads to an unhealthy status in Consul (because of the dsitributive check.)

I think it's going to require a few changes to nginx-consul and logstash (specifically to restart them when they fail), which may fall outside the scope of this PR. @ryane do you have a preference for where that should go?

This is needed because Kubelet sends signal 137 (128+9, AKA kill -9) to
the containers on startup. `on-failure` doesn't catch that, so the
services stay dead, despite us wanting them to restart.
@BrianHicks
Copy link
Contributor Author

I added it in 48d0058, but we can revert that commit.

@ryane
Copy link
Contributor

ryane commented Apr 6, 2016

I suppose we could move it to a new PR but I don't have a problem with leaving it in this one...

@BrianHicks
Copy link
Contributor Author

Cool, then it's ready to merge

On Apr 6, 2016, at 5:37 PM, Ryan Eschinger [email protected] wrote:

I suppose we could move it to a new PR but I don't have a problem with leaving it in this one...


You are receiving this because you were mentioned.
Reply to this email directly or view it on GitHub

@langston-barrett langston-barrett merged commit ff4eb85 into master Apr 6, 2016
@langston-barrett langston-barrett deleted the feature/k8s-by-default branch April 6, 2016 22:55
@ryane ryane modified the milestone: 1.1 May 10, 2016
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

5 participants