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

Bump Kubernetes versions to `1.14.1` #45

Merged
merged 1 commit into from May 8, 2019

Conversation

Projects
None yet
2 participants
@anton-johansson
Copy link
Contributor

commented May 7, 2019

I never upgraded from 1.13.1 to 1.14.0, i went straight to 1.14.1 using the changes of this pull request.

I do want to highlight that it didn't go as smooth as I'd like. Everything went as expected until the end of the playbook. My NGINX ingress controllers all went down for a short period. I have a hard time figuring out why.

I see these events in my NGINX ingress controller pods:

Warning  Unhealthy  7m23s  kubelet, k8s-nginx-11  Readiness probe failed: HTTP probe failed with statuscode: 500

And I see logs like this from the pods:

Received SIGTERM, shutting down
k8s.io/ingress-nginx/internal/ingress/controller/store/store.go:153: watch of *v1.Service ended with: too old resource version: 18603171 (18603183)

I believe the SIGTERM comes from the readiness probe returning 500, but I can't find any indications for why it would return 500. :(

I don't believe this is related to 1.14.1 though, this would probably have happened if I went to 1.14.0 too, so I think we can ignore that when we review this pull request. It would be nice to know what happened though.

@anton-johansson anton-johansson force-pushed the anton-johansson:bump-to-1.14.1 branch from 717b50b to 152fbc5 May 7, 2019

@anton-johansson anton-johansson force-pushed the anton-johansson:bump-to-1.14.1 branch from 152fbc5 to a561c34 May 7, 2019

@amimof

This comment has been minimized.

Copy link
Owner

commented May 8, 2019

Awesome, thanks!

@amimof amimof merged commit 28d40b3 into amimof:master May 8, 2019

1 check passed

continuous-integration/travis-ci/pr The Travis CI build passed
Details
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
You can’t perform that action at this time.