Skip to content

Remove stress test framework.#168

Closed
PiotrSikora wants to merge 1 commit intoistio:release-1.5from
PiotrSikora:remove-stress-test-1.5
Closed

Remove stress test framework.#168
PiotrSikora wants to merge 1 commit intoistio:release-1.5from
PiotrSikora:remove-stress-test-1.5

Conversation

@PiotrSikora
Copy link
Copy Markdown

Signed-off-by: Piotr Sikora piotrsikora@google.com

Signed-off-by: Piotr Sikora <piotrsikora@google.com>
@gargnupur
Copy link
Copy Markdown

/lgtm

@jplevyak
Copy link
Copy Markdown

This is blocking other PRs, so let's get it in ASAP

Copy link
Copy Markdown

@jplevyak jplevyak left a comment

Choose a reason for hiding this comment

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

You could also just set the test to manual. That would be a minimal change.

@PiotrSikora
Copy link
Copy Markdown
Author

/retest

@PiotrSikora
Copy link
Copy Markdown
Author

I'm going to abandon this change, since it looks that the stress test actually found a real issue, and we don't have the "messy sync with upstream" issue in the release branch.

Miss-you pushed a commit to Miss-you/envoy that referenced this pull request Nov 17, 2020
zh-translation: /intro/arch_overview/upstream/load_balancing/load_balancing.rst
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

5 participants