-
Notifications
You must be signed in to change notification settings - Fork 49
Implement kube addon manager #688
Implement kube addon manager #688
Conversation
67322d7
to
3e185eb
Compare
/assign @simonswine |
There seems to be a race between the masters (so somehow the election of kube-addon-managers and/or cluster-manager is not stable.
This is from a cluster with three master nodes:
diff between the two RSs is:
/assign @JoshVanL |
/test puppet-tarmak-acceptance-centos v1.11 |
…rmak Signed-off-by: JoshVanL <vleeuwenjoshua@gmail.com>
Signed-off-by: JoshVanL <vleeuwenjoshua@gmail.com>
Signed-off-by: JoshVanL <vleeuwenjoshua@gmail.com>
Signed-off-by: JoshVanL <vleeuwenjoshua@gmail.com>
235d09b
to
91d147c
Compare
e84410b
to
845caeb
Compare
Signed-off-by: JoshVanL <vleeuwenjoshua@gmail.com>
845caeb
to
4698a8b
Compare
/unassign |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Just some minor bits now. The question is how sure are we that we did not forget the labe anywhere
/assign @JoshVanL
/unassign
Service[$service_apiserver], | ||
] | ||
|
||
$command = "/bin/bash -c \"while true; do if [[ \$(curl -k -w '%{http_code}' -s -o /dev/null ${protocol}://localhost:${server_port}/healthz) == 200 ]]; then break; fi; done; kubectl apply -f '${apply_file}' || rm -f '${apply_file})'\"" |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Maybe we add a sleep 2 or something, as this might be quite a few connections on bulk.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Yep, good idea. I did at one point during testing but must have removed it...
@@ -25,6 +27,8 @@ apiVersion: rbac.authorization.k8s.io/v1beta1 | |||
kind: ClusterRoleBinding | |||
metadata: | |||
name: calico-node | |||
labels: | |||
addonmanager.kubernetes.io/mode: EnsureExists |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Just a bit below: ServiceAccount is missing the label addonmanager.kubernetes.io/mode
d6ffd73
to
0cf167e
Compare
/unassign |
Signed-off-by: JoshVanL <vleeuwenjoshua@gmail.com>
0cf167e
to
d42f35b
Compare
/test puppet-tarmak-acceptance-ubuntu v1.11 |
Signed-off-by: JoshVanL <vleeuwenjoshua@gmail.com>
/test puppet-tarmak-acceptance-centos v1.11 |
/test puppet-tarmak-acceptance-ubuntu v1.11 |
Good work /approve |
[APPROVALNOTIFIER] This PR is APPROVED This pull-request has been approved by: JoshVanL, simonswine The full list of commands accepted by this bot can be found here. The pull request process is described here
Needs approval from an approver in each of these files:
Approvers can indicate their approval by writing |
/test puppet-fluent_bit-acceptance |
/test puppet-tarmak-acceptance-centos v1.13 |
/test puppet-tarmak-acceptance-ubuntu v1.11 |
Uses kube addon-manager to manage the life cycle of resources created by tarmak.
fixes #665