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

Activate Flannel Daemonset deployment #725

Merged
merged 3 commits into from
Jan 3, 2018
Merged

Activate Flannel Daemonset deployment #725

merged 3 commits into from
Jan 3, 2018

Conversation

linki
Copy link
Member

@linki linki commented Dec 4, 2017

Stage two of rolling out flannel as a daemonset, but still with etcd backend (Stage one)

This rolls all nodes. The nodes don't have flannel in the userdata anymore and they can be identified so by checking the node labels (flannel=local). Each new node will instead run an instance of flannel described by the daemonset setup in stage one due to the node-affinity now matching. The dormant flannel daemonset will slowly be rolled-in. At last, the node-affinity is removed from the daemonset.

@linki
Copy link
Member Author

linki commented Dec 13, 2017

Tested and found one issue:

  • two master pods cannot reach each other, have to check whether this is a regression

What happens when kubelet dies and cannot reach api server afterwards

  • kube-proxy will not run as kubelet doesn't know about it
  • iptables rules stay the same which should be the exact behaviour as of now

@linki
Copy link
Member Author

linki commented Dec 13, 2017

two master pods cannot reach each other, have to check whether this is a regression

same behaviour on stups-test so I guess not.

@linki linki force-pushed the etcd-flannel-ds2 branch 2 times, most recently from 4cf7e1c to 4e6f1ca Compare December 20, 2017 14:24
@aermakov-zalando aermakov-zalando force-pushed the etcd-flannel-ds2 branch 2 times, most recently from 553f7db to ce34b49 Compare December 28, 2017 14:29
@linki
Copy link
Member Author

linki commented Jan 2, 2018

This has been running on teapot for two weeks without issues to my knowledge. Can we merge it to dev?

@linki
Copy link
Member Author

linki commented Jan 2, 2018

👍

1 similar comment
@mikkeloscar
Copy link
Contributor

👍

@linki
Copy link
Member Author

linki commented Jan 2, 2018

@mikkeloscar Need a 👍 again, there was a conflict with dev.

@szuecs
Copy link
Member

szuecs commented Jan 3, 2018

retriggered tests

@linki
Copy link
Member Author

linki commented Jan 3, 2018

👍

1 similar comment
@gargravarr
Copy link
Contributor

👍

@linki linki merged commit 74740c0 into dev Jan 3, 2018
@linki linki deleted the etcd-flannel-ds2 branch January 3, 2018 12:50
RomanZavodskikh pushed a commit that referenced this pull request Oct 16, 2024
+ aws: fail on ListCertificatesPages error (#725)
+ Fix staticcheck warning (#726)
+ Bump docker/login-action from 2 to 3 (#717)
+ Bump github.com/zalando/skipper in the all-go-mod-patch-and-minor gro…
+ Bump actions/setup-go from 2 to 5 (#715)
+ Bump docker/metadata-action from 4 to 5 (#716)
+ Bump docker/setup-qemu-action from 2 to 3 (#719)

Signed-off-by: Roman Zavodskikh <roman.zavodskikh@zalando.de>
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.

4 participants