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.

Sign up for GitHub

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

Jump to bottom

query envoy for /contention and show results in CSV. #28

Merged
merged 1 commit into from
Nov 8, 2018
Merged

query envoy for /contention and show results in CSV. #28

merged 1 commit into from
Nov 8, 2018

Conversation

jmarantz
Copy link
Contributor

@jmarantz jmarantz commented Nov 7, 2018

Now that @ambuc added a /contention endpoint, use it in the siege tests to see what contention we experienced under load.

@jmarantz
query enovy for /contention and show results in CSV.
171e108 
Signed-off-by: Joshua Marantz <jmarantz@google.com>
@jmarantz jmarantz requested a review from htuch November 7, 2018 22:27
@jmarantz jmarantz mentioned this pull request Nov 7, 2018
Merged
htuch
htuch approved these changes Nov 8, 2018
View reviewed changes
Copy link
Member

@htuch htuch left a comment

Choose a reason for hiding this comment

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

Rad

@htuch htuch merged commit 63f2284 into envoyproxy:master Nov 8, 2018
@jmarantz jmarantz deleted the add-contention branch November 8, 2018 02:44
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@htuch htuch htuch approved these changes

Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
2 participants
@jmarantz @htuch