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

[BACKPORT] set vreplication net read and net write timeout session vars to high values #14203 #129

Closed
wants to merge 1 commit into from

Conversation

pjambet
Copy link

@pjambet pjambet commented Nov 8, 2023

Description

Attempt at backporting vitessio#14203

Changes made:

Related Issue(s)

This would hopefully help with https://github.com/Shopify/vitess-project/issues/577

Checklist

  • "Backport to:" labels have been added if this change should be back-ported
  • Tests were added or are not required
  • Did the new or modified tests pass consistently locally and on the CI
  • Documentation was added or is not required

Deployment Notes

@pjambet pjambet force-pushed the pj/backport-14203 branch 2 times, most recently from 67e4e6c to ca390e9 Compare November 8, 2023 20:52
…values (vitessio#14203)

Signed-off-by: Olga Shestopalova <[email protected]>
Co-authored-by: Olga Shestopalova <[email protected]>
@pjambet pjambet changed the title set vreplication net read and net write timeout session vars to high … [BACKPORT] set vreplication net read and net write timeout session vars to high values vitessio#14203 Nov 8, 2023
@pjambet pjambet changed the title [BACKPORT] set vreplication net read and net write timeout session vars to high values vitessio#14203 [BACKPORT] set vreplication net read and net write timeout session vars to high values [vitessio#14203](https://github.com/vitessio/vitess/pull/14203) Nov 8, 2023
@pjambet pjambet changed the title [BACKPORT] set vreplication net read and net write timeout session vars to high values [vitessio#14203](https://github.com/vitessio/vitess/pull/14203) [BACKPORT] set vreplication net read and net write timeout session vars to high values #14203 Nov 8, 2023
@pjambet pjambet marked this pull request as ready for review November 8, 2023 22:23
@pjambet pjambet requested review from brendar and austenLacy November 8, 2023 22:23
@austenLacy
Copy link

We tested the image in staging, but it's difficult to reproduce the error so we haven't been able to test if the fix actually improves reliability of migrations without releasing to the wild.

Copy link

This PR is being marked as stale because it has been open for 30 days with no activity. To rectify, you may do any of the following:

  • Push additional commits to the associated branch.
  • Remove the stale label.
  • Add a comment indicating why it is not stale.

If no action is taken within 7 days, this PR will be closed.

@github-actions github-actions bot added the Stale label Dec 16, 2023
Copy link

This PR was closed because it has been stale for 7 days with no activity.

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

Successfully merging this pull request may close these issues.

3 participants