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

haproxy-ingress locks up attempting leader election #47

Closed
zapman449 opened this issue Sep 12, 2017 · 15 comments
Closed

haproxy-ingress locks up attempting leader election #47

zapman449 opened this issue Sep 12, 2017 · 15 comments
Labels
Milestone

Comments

@zapman449
Copy link

zapman449 commented Sep 12, 2017

E0912 17:17:48.511187       6 leaderelection.go:258] Failed to update lock: Operation cannot be fulfilled on configmaps "ingress-controller-leader-haproxy": the object has been modified; please apply your changes to the latest version and try again

haproxy-ingress version haproxy-ingress:v0.4-snapshot.4
Kubernetes version 1.5.5

Works great, other than this lock up. After that happens, no new changes in the cluster get registered.

@jcmoraisjr
Copy link
Owner

Couldn't reproduce this (yet). How many instances do you have running at the same time? Any other detail of your deployment which worth to mention?

@zapman449
Copy link
Author

2 worker nodes, 3 masters built by kops.

It MIGHT have been triggered by a lurking nginx-ingress we had (thought we) deleted. We haven't replicated it since actually deleting it. If it survives the night, I think we can close the ticket.

@Scukerman
Copy link
Contributor

I have the same messages in log only if I run controller as a DaemonSet.

@zapman449
Copy link
Author

we are running as a daemonset...

@zapman449
Copy link
Author

And we locked up overnight

@Scukerman
Copy link
Contributor

Scukerman commented Sep 13, 2017

kubernetes/ingress-nginx#1297

So it seems like Ingress update is required.
Merged PR: kubernetes/ingress-nginx#1338

@zapman449
Copy link
Author

@Scukerman Dumb question: What is the relationship between kubernetes/ingress/haproxy and jcmoraisjr/haproxy-ingress?

@Scukerman
Copy link
Contributor

Scukerman commented Sep 13, 2017

It is the same project (repo is here, docs are there).

Check the README.md and you'll see that some of the links point to the Ingress repo docs.

Here is a list of available and known Ingress Controllers:
https://github.com/kubernetes/ingress/blob/master/docs/catalog.md

nginx and gce controllers are not in the list because they are official.

@mbreitung
Copy link

I can confirm that kubernetes/ingress-nginx#1338 fixes this issue.

@jcmoraisjr jcmoraisjr added this to the v0.4 milestone Sep 27, 2017
@jcmoraisjr
Copy link
Owner

Sorry the long delay, vacation here.

@zapman449 b66c7f9 should fix this. I'll provide a new image in the following days. Let us know if you still have any problem.

@Scukerman and @mbreitung thanks for sharing your findings, this way fix HAProxy Ingress is a lot easier.

@jcmoraisjr
Copy link
Owner

@zapman449 please report back - commenting and/or closing the issue - if the problem persists. Fyi the canary tag was just updated with this fix.

@zapman449
Copy link
Author

Building from master allowed us to move past this problem. Thank you.

@zapman449
Copy link
Author

Out of curiosity, when will a snapshot release be forthcoming?

@Scukerman
Copy link
Contributor

@zapman449 docker pull quay.io/jcmoraisjr/haproxy-ingress:v0.4-beta.1
Doesn't it work yet?

@zapman449
Copy link
Author

It does. I missed the shift from snapshot to beta

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

No branches or pull requests

4 participants