Skip to content

[installer] Run the toxic-config component as a sidecar in the Toxiproxy pod #14487

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 3 commits into from
Nov 7, 2022

Conversation

andrew-farries
Copy link
Contributor

Description

As part of #9198 we will run a Toxiproxy instance in the application cluster to allow us to simulate a high-latency connection to the Gitpod database.

#14473 added a new component, toxic-config, intended to run as a sidecar container in a Kubernetes pod alongside Toxiproxy, that configures a given proxy with a latency toxic.

This PR makes that new component run as a sidecar in the Toxiproxy component to ensure that the mysql component is configured with latency.

Related Issue(s)

Part of #9198

How to test

  1. Port forward to Toxiproxy:
kubectl port-forward svc/toxiproxy 8474:8474
  1. Verify that the sidecar container successfully configured the mysql proxy:
kubcectl logs deploy/toxiproxy -c toxic-config
  1. Verify that the mysql proxy has been configured with a latency toxic:
docker run --rm --net=host --entrypoint="/toxiproxy-cli" -it ghcr.io/shopify/toxiproxy inspect mysql

Release Notes

NONE

Documentation

Werft options:

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

Andrew Farries added 3 commits November 7, 2022 12:51
Run the `toxic-config` component as sidecar container alongside
Toxiproxy to configure the Toxiproxy instance with a latency toxic.
@werft-gitpod-dev-com
Copy link

started the job as gitpod-build-af-run-toxic-config-sidecar.17 because the annotations in the pull request description changed
(with .werft/ from main)

@roboquat roboquat added the size/S label Nov 7, 2022
@github-actions github-actions bot added team: IDE team: webapp Issue belongs to the WebApp team labels Nov 7, 2022
@roboquat roboquat merged commit f08d3a8 into af/add-toxic-config-component Nov 7, 2022
@roboquat roboquat deleted the af/run-toxic-config-sidecar branch November 7, 2022 17:00
@andrew-farries andrew-farries restored the af/run-toxic-config-sidecar branch November 7, 2022 17:18
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