Skip to content

Conversation

mateuszpn
Copy link
Contributor

Flamegraphs generation added to CI action and links to flamegraphs (if they exist for given benchmark) added to the dashboard.

Signed-off-by: Mateusz P. Nowak <mateusz.p.nowak@intel.com>
Signed-off-by: Mateusz P. Nowak <mateusz.p.nowak@intel.com>
…ci-flamegraphs

Signed-off-by: Mateusz P. Nowak <mateusz.p.nowak@intel.com>
Signed-off-by: Mateusz P. Nowak <mateusz.p.nowak@intel.com>
@mateuszpn mateuszpn changed the title Update of benchmark dashboard - flamegraphs [CI][Benchmarks] Update of benchmark dashboard - flamegraphs Sep 30, 2025
- name: Install linux-tools package
shell: bash
run: |
sudo apt-get update
Copy link
Contributor

Choose a reason for hiding this comment

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

does this action run inside docker and if so is the image one of the ones we create in this repo? if yes to both i would prefer we do this as part of image creation instead of in an action

Copy link
Contributor Author

@mateuszpn mateuszpn Oct 3, 2025

Choose a reason for hiding this comment

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

@mateuszpn mateuszpn force-pushed the mateuszpn/benchmarks-ci-flamegraphs branch from bf1e85b to 0688ceb Compare October 3, 2025 13:18
…ci-flamegraphs

Signed-off-by: Mateusz P. Nowak <mateusz.p.nowak@intel.com>
@lslusarczyk
Copy link
Contributor

@sarnex , Could you please check if your comments well applied? If so, and there are no more issues, then please give your approval, so we could go on with the merge.

@sarnex sarnex merged commit b0f6da7 into sycl Oct 7, 2025
37 checks passed
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