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

Decrease TCP retries in case of VIP failover #168

Merged
merged 1 commit into from Mar 1, 2021

Conversation

noonedeadpunk
Copy link
Contributor

In case of VIP failover some connections (like mysql) can stuck
in retrying to connect and detect a dead connection. We
should probably make this failover to be detected faster then the
default value as suggested in [1]

[1] https://access.redhat.com/solutions/726753

Related-Bug: https://bugs.launchpad.net/kolla-ansible/+bug/1917068

@noonedeadpunk
Copy link
Contributor Author

Um, where is CI?

@evrardjp
Copy link
Owner

evrardjp commented Mar 1, 2021

That's weird. It's supposed to be github actions.

@evrardjp
Copy link
Owner

evrardjp commented Mar 1, 2021

FYI that solution is unverified :)

I can see why we want to reduce it, but 3 is really agressive. Should we move down to 8? 6? I think that below is very agressive. We might want to introduce a variable for the brave people out there :)

@evrardjp
Copy link
Owner

evrardjp commented Mar 1, 2021

This is on hold until #169 is confirmed working (fixing CI).

@evrardjp
Copy link
Owner

evrardjp commented Mar 1, 2021

In case of VIP failover some connections (like mysql) can stuck
in retrying to connect and detect a dead connection. We
should probably make this failover to be detected faster then the
default value as suggested in [1]

[1] https://access.redhat.com/solutions/726753

Related-Bug: https://bugs.launchpad.net/kolla-ansible/+bug/1917068
@evrardjp
Copy link
Owner

evrardjp commented Mar 1, 2021

I think it's fair, and I will merge this.

@evrardjp evrardjp merged commit 727cb94 into evrardjp:master Mar 1, 2021
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.

None yet

2 participants