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.

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

Syscall Latency Test #1640

Closed
wants to merge 34 commits into from
Closed

Syscall Latency Test #1640

wants to merge 34 commits into from

Conversation

Stringy
Copy link
Collaborator

@Stringy Stringy commented Apr 22, 2024

Description

This PR adds a new benchmark test that will analyse the syscall latency of a system with and without collector running. It does this by running a program which repeatedly opens, reads, and closes a file. Those syscalls were chosen in order to capture syscalls that Collector is both interested in (close) and not interested in (read)

Having captured the logs, the benchmark will process the data into json, storing a basic analysis (p90, median, mean) alongside the raw data.

Checklist

  • Investigated and inspected CI test results
  • Updated documentation accordingly

Automated testing
- [ ] Added unit tests

  • Added integration tests
    - [ ] Added regression tests

@Stringy Stringy requested a review from a team as a code owner April 22, 2024 16:12
@Stringy Stringy force-pushed the giles/syscall-latency-test branch from 2ba40b6 to 659e0c3 Compare April 29, 2024 11:14
erthalion and others added 28 commits May 3, 2024 13:35
Currently benchmark CI step is disabled. Turn it on with following
changes:

* Use berserker instread of hackbench
* Use regular ttest instead of IQR
* Compare CPU/Memory usage instead of hackbench time
Adjust reporting for new format, plus add reset command for cleaning up
the baseline if needed.
Add systemtap into the builder to allow defining USDT, and add one to
capture events when a new process was discovered.
USDT support requires systemtap, but Konflux setup doesn't use the
builder image. Add the dependency directly.
@Stringy Stringy force-pushed the giles/syscall-latency-test branch from 1622475 to 65cf135 Compare May 8, 2024 10:44
@Stringy Stringy changed the base branch from master to feature/berserker May 8, 2024 10:44
@Stringy Stringy changed the base branch from feature/berserker to master May 8, 2024 10:44
@Stringy Stringy changed the base branch from master to feature/berserker May 8, 2024 10:45
@erthalion erthalion force-pushed the feature/berserker branch from 98b646d to 39a9c89 Compare May 8, 2024 16:03
@erthalion erthalion force-pushed the feature/berserker branch from 2f7cfee to 683f413 Compare May 21, 2024 12:44
Base automatically changed from feature/berserker to master May 22, 2024 13:04
@Stringy Stringy closed this Sep 2, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants