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

documentation: performance evaluation improvements #13355

Merged
merged 1 commit into from
Sep 30, 2020
Merged

documentation: performance evaluation improvements #13355

merged 1 commit into from
Sep 30, 2020

Conversation

kkourt
Copy link
Contributor

@kkourt kkourt commented Sep 30, 2020 •
edited

Followup of #13297

Updates based on feedback/suggestions:

  • Add multi-stream results
  • Add "lower/higher is better" labels
  • use (lat vs batch) and (tput vs batch) plots for TCP_RR
  • improve text

Signed-off-by: Kornilios Kourtis kornilios@isovalent.com

@kkourt
Documentation: performance evaluation improvements
9bcb85d 
- Add multi-stream results
- Add lower/higher is better labels
- use (lat vs batch) and (tput vs batch) plots for TCP_RR
- improve text

Signed-off-by: Kornilios Kourtis <kornilios@isovalent.com>
@kkourt kkourt added the release-note/misc This PR makes changes that have no direct user impact. label Sep 30, 2020
@kkourt kkourt requested a review from a team as a code owner September 30, 2020 20:59
@maintainer-s-little-helper maintainer-s-little-helper bot added this to Needs backport from master in 1.8.4 Sep 30, 2020
@kkourt
Copy link
Contributor Author

kkourt commented Sep 30, 2020 •
edited

Added the needs-backport/1.8 label. Once this is merged, I will add the label to #13297 as well.

@borkmann borkmann merged commit 2371071 into master Sep 30, 2020
@borkmann borkmann deleted the pr/perf-report-updates branch September 30, 2020 21:14
@joestringer joestringer removed this from Needs backport from master in 1.8.4 Sep 30, 2020
@joestringer joestringer added this to Needs backport from master in 1.8.5 Sep 30, 2020
@kkourt kkourt added the area/documentation Impacts the documentation, including textual changes, sphinx, or other doc generation code. label Oct 1, 2020
@errordeveloper errordeveloper mentioned this pull request Oct 2, 2020
Merged
@tklauser tklauser moved this from Needs backport from master to Backport done to v1.8 in 1.8.5 Oct 13, 2020
@christarazi christarazi mentioned this pull request Oct 28, 2020
Merged
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@borkmann borkmann borkmann approved these changes

Assignees
No one assigned
Labels
area/documentation Impacts the documentation, including textual changes, sphinx, or other doc generation code. release-note/misc This PR makes changes that have no direct user impact.
Projects
No open projects
1.8.5
Backport done to v1.8
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.

None yet
3 participants
@kkourt @borkmann @errordeveloper