Skip to content

Commit deba3fa

Browse files
committed
mutex on stats
1 parent 311d726 commit deba3fa

11 files changed

+44
-0
lines changed

app/controllers/health_controller.go

Lines changed: 4 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -27,6 +27,7 @@ type HealthController struct {
2727

2828
//HealthStats is holds the latest heartbeats
2929
type HealthStats struct {
30+
sync.Mutex
3031
Healthy bool
3132
MetricsControllerAlive time.Time
3233
NetworkPolicyControllerAlive time.Time
@@ -69,6 +70,9 @@ func (hc *HealthController) Handler(w http.ResponseWriter, req *http.Request) {
6970
func (hc *HealthController) HandleHeartbeat(beat *ControllerHeartbeat) {
7071
glog.V(3).Infof("Received heartbeat from %s", beat.Component)
7172

73+
hc.Status.Lock()
74+
defer hc.Status.Unlock()
75+
7276
switch component := beat.Component; component {
7377
case "NSC":
7478
hc.Status.NetworkServicesControllerAlive = time.Now()

daemonset/generic-kuberouter-all-features-advertise-routes.yaml

Lines changed: 4 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -47,6 +47,10 @@ metadata:
4747
name: kube-router
4848
namespace: kube-system
4949
spec:
50+
updateStrategy:
51+
type: RollingUpdate
52+
rollingUpdate:
53+
maxUnavailable: 2
5054
template:
5155
metadata:
5256
labels:

daemonset/generic-kuberouter-all-features.yaml

Lines changed: 4 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -47,6 +47,10 @@ metadata:
4747
name: kube-router
4848
namespace: kube-system
4949
spec:
50+
updateStrategy:
51+
type: RollingUpdate
52+
rollingUpdate:
53+
maxUnavailable: 2
5054
template:
5155
metadata:
5256
labels:

daemonset/generic-kuberouter.yaml

Lines changed: 4 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -28,6 +28,10 @@ metadata:
2828
name: kube-router
2929
namespace: kube-system
3030
spec:
31+
updateStrategy:
32+
type: RollingUpdate
33+
rollingUpdate:
34+
maxUnavailable: 2
3135
template:
3236
metadata:
3337
labels:

daemonset/kube-router-all-service-daemonset-advertise-routes.yaml

Lines changed: 4 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -26,6 +26,10 @@ metadata:
2626
labels:
2727
k8s-app: kube-router
2828
spec:
29+
updateStrategy:
30+
type: RollingUpdate
31+
rollingUpdate:
32+
maxUnavailable: 2
2933
template:
3034
metadata:
3135
labels:

daemonset/kube-router-all-service-daemonset.yaml

Lines changed: 4 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -26,6 +26,10 @@ metadata:
2626
labels:
2727
k8s-app: kube-router
2828
spec:
29+
updateStrategy:
30+
type: RollingUpdate
31+
rollingUpdate:
32+
maxUnavailable: 2
2933
template:
3034
metadata:
3135
labels:

daemonset/kube-router-firewall-daemonset.yaml

Lines changed: 4 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -26,6 +26,10 @@ metadata:
2626
labels:
2727
k8s-app: kube-router
2828
spec:
29+
updateStrategy:
30+
type: RollingUpdate
31+
rollingUpdate:
32+
maxUnavailable: 2
2933
template:
3034
metadata:
3135
labels:

daemonset/kube-router-proxy-daemonset.yaml

Lines changed: 4 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -26,6 +26,10 @@ metadata:
2626
labels:
2727
k8s-app: kube-router
2828
spec:
29+
updateStrategy:
30+
type: RollingUpdate
31+
rollingUpdate:
32+
maxUnavailable: 2
2933
template:
3034
metadata:
3135
labels:

daemonset/kubeadm-kuberouter-all-features-dsr.yaml

Lines changed: 4 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -27,6 +27,10 @@ metadata:
2727
name: kube-router
2828
namespace: kube-system
2929
spec:
30+
updateStrategy:
31+
type: RollingUpdate
32+
rollingUpdate:
33+
maxUnavailable: 2
3034
template:
3135
metadata:
3236
labels:

daemonset/kubeadm-kuberouter-all-features.yaml

Lines changed: 4 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -27,6 +27,10 @@ metadata:
2727
name: kube-router
2828
namespace: kube-system
2929
spec:
30+
updateStrategy:
31+
type: RollingUpdate
32+
rollingUpdate:
33+
maxUnavailable: 2
3034
template:
3135
metadata:
3236
labels:

0 commit comments

Comments
 (0)