-
Notifications
You must be signed in to change notification settings - Fork 2.4k
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
[v4.9] Add a net health recovery service to qemu machines #21276
Conversation
[APPROVALNOTIFIER] This PR is APPROVED This pull-request has been approved by: n1hility 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 |
We're not doing another 4.8 release, so not necessary |
/lgtm |
Sorry got myself confused meant 4.9 |
Validate is angry, you need a quick gofmt and then we can merge |
67e0076
to
6e674b3
Compare
/lgtm |
CI still complaining about gofmt |
@n1hility can you fix the gofmt issue? |
6e674b3
to
ddccab5
Compare
not sure what happened there should be fixed now, i will keep an eye on it |
There is a network stability issue in qemu + virtio, affecting some users after long periods of usage, which can lead to suspended queue delivery. Until the issue is resolved, add a temporary recovery service which restarts networking when host communication becomes inoperable. [NO NEW TESTS NEEDED] Signed-off-by: Jason T. Greene <[email protected]>
ddccab5
to
6c173c0
Compare
/lgtm |
/lgtm |
/unhold |
Backport of #21262