Skip to content

[installer] Set latency and jitter values for Toxiproxy mysql proxy #14932

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

Merged
merged 1 commit into from
Nov 28, 2022

Conversation

andrew-farries
Copy link
Contributor

@andrew-farries andrew-farries commented Nov 24, 2022

Description

Set these to more realistic values for simulating a connection to a US west coast database.

Related Issue(s)

Part of #9198

Closes #14958

How to test

Release Notes

NONE

Documentation

Werft options:

  • /werft with-local-preview
    If enabled this will build install/preview
  • /werft with-preview
  • /werft with-large-vm
  • /werft with-integration-tests=all
    Valid options are all, workspace, webapp, ide, jetbrains, vscode, ssh

@andrew-farries andrew-farries requested a review from a team November 24, 2022 16:35
@github-actions github-actions bot added the team: webapp Issue belongs to the WebApp team label Nov 24, 2022
@easyCZ
Copy link
Member

easyCZ commented Nov 24, 2022

How's the value of 250 determined? I'd have expected somewhere around 40-80ms to be the extra round-trip latency from EU to US or vice versa

Set these to more realistic values for simulating a connection to a US
west coast database.
@andrew-farries
Copy link
Contributor Author

I've adjusted the values and added a comment as to where they come from.

Copy link
Member

@geropl geropl left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

✔️

@roboquat roboquat merged commit 97a64c9 into main Nov 28, 2022
@roboquat roboquat deleted the af/set-latency-jitter branch November 28, 2022 07:46
@roboquat roboquat added deployed: webapp Meta team change is running in production deployed Change is completely running in production labels Nov 28, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
deployed: webapp Meta team change is running in production deployed Change is completely running in production release-note-none size/XS team: webapp Issue belongs to the WebApp team
Projects
None yet
Development

Successfully merging this pull request may close these issues.

[db-sync] Configure the correct values for the latency toxic to simulate a connection to a US DB.
4 participants