Skip to content

Migrate performance job to GitHub actions #404

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
Merged
EugenioCollado merged 21 commits into from
Sep 29, 2025
Merged

Migrate performance job to GitHub actions #404

EugenioCollado merged 21 commits into from
Sep 29, 2025

Conversation

@4ntn
Copy link
Contributor

@4ntn 4ntn commented Sep 25, 2025 •
edited
Loading

Migrate "Micro-XRCE-DDS-Client Nightly Master Performance".

  • Configured to run on develop on PRs and scheduled weekly.
  • Starts deleting runs after 208 (4 years of weekly runs).
  • Prints plots on job summary.
  • The job will fail when there is a difference in memory usage with the previous run.

@4ntn
Initial approach
f540684 
Signed-off-by: Antón Casas <antoncasas@eprosima.com>
@4ntn 4ntn force-pushed the feature/performance_workflow branch from abaf26e to f540684 Compare September 25, 2025 09:12
4ntn added 3 commits September 25, 2025 11:24
@4ntn
Plot all columns
a951a8b 
Signed-off-by: Antón Casas <antoncasas@eprosima.com>
@4ntn
Add PR trigger for testing
38d0abb 
Signed-off-by: Antón Casas <antoncasas@eprosima.com>
@4ntn
Single file approach
25d67a3 
Signed-off-by: Antón Casas <antoncasas@eprosima.com>
@4ntn 4ntn force-pushed the feature/performance_workflow branch from f139cf6 to 25d67a3 Compare September 25, 2025 12:23
@4ntn
Move python code to file
7d0f704 
Signed-off-by: Antón Casas <antoncasas@eprosima.com>
@4ntn 4ntn force-pushed the feature/performance_workflow branch from 789085e to 7d0f704 Compare September 25, 2025 13:06
4ntn added 2 commits September 25, 2025 15:27
@4ntn
Move into a two jobs paradigm
2db6fb4 
Signed-off-by: Antón Casas <antoncasas@eprosima.com>
@4ntn
Fix results path
75e9b68 
Signed-off-by: Antón Casas <antoncasas@eprosima.com>
@4ntn 4ntn force-pushed the feature/performance_workflow branch from 4b4e56c to 75e9b68 Compare September 25, 2025 13:47
@4ntn
Fix dependencies
94dba70 
Signed-off-by: Antón Casas <antoncasas@eprosima.com>
@4ntn 4ntn force-pushed the feature/performance_workflow branch from d38a86b to 94dba70 Compare September 25, 2025 13:57
4ntn added 4 commits September 25, 2025 16:13
@4ntn
Add all csv
e42df61 
Signed-off-by: Antón Casas <antoncasas@eprosima.com>
@4ntn
Fix details up to save results into memory
6fcaf37 
Signed-off-by: Antón Casas <antoncasas@eprosima.com>
@4ntn
Move plot python code to file
56abadd 
Signed-off-by: Antón Casas <antoncasas@eprosima.com>
@4ntn
Remove python code
9f51c43 
Signed-off-by: Antón Casas <antoncasas@eprosima.com>
@4ntn 4ntn force-pushed the feature/performance_workflow branch from fc9bff9 to 9f51c43 Compare September 26, 2025 06:56
4ntn added 2 commits September 26, 2025 09:58
@4ntn
Add plots to summary
04101e5 
Signed-off-by: Antón Casas <antoncasas@eprosima.com>
@4ntn
Add summary brief
4fd8f7b 
Signed-off-by: Antón Casas <antoncasas@eprosima.com>
@4ntn 4ntn force-pushed the feature/performance_workflow branch 3 times, most recently from 0749938 to d308b68 Compare September 26, 2025 09:38
4ntn added 2 commits September 26, 2025 11:43
@4ntn
Switch to svg
f9ace95 
Signed-off-by: Antón Casas <antoncasas@eprosima.com>
@4ntn
Save images
2add5ee 
Signed-off-by: Antón Casas <antoncasas@eprosima.com>
@4ntn 4ntn force-pushed the feature/performance_workflow branch from d308b68 to 2add5ee Compare September 26, 2025 09:45
@4ntn
Add analysis of last two runs
f5e8030 
Signed-off-by: Antón Casas <antoncasas@eprosima.com>
@4ntn 4ntn force-pushed the feature/performance_workflow branch 2 times, most recently from 2e91cd9 to 35b0b7e Compare September 26, 2025 11:07
@4ntn
[Test] Fake memory difference
b986669 
This reverts commit 35b0b7e.

Signed-off-by: Antón Casas <antoncasas@eprosima.com>
@4ntn 4ntn force-pushed the feature/performance_workflow branch from 35b0b7e to b986669 Compare September 26, 2025 11:15
@4ntn
Revert "[Test] Fake memory difference"
52bbb28 
This reverts commit b986669.

Signed-off-by: Antón Casas <antoncasas@eprosima.com>
@4ntn 4ntn marked this pull request as ready for review September 26, 2025 11:21
4ntn added 2 commits September 26, 2025 13:31
@4ntn
Add summary introduction
fff95d1 
Signed-off-by: Antón Casas <antoncasas@eprosima.com>
@4ntn
Final tweaks
cc90353 
Signed-off-by: Antón Casas <antoncasas@eprosima.com>
EugenioCollado
EugenioCollado approved these changes Sep 29, 2025
View reviewed changes
Copy link
Collaborator

@EugenioCollado EugenioCollado left a comment

Choose a reason for hiding this comment

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

LGTM

@EugenioCollado EugenioCollado merged commit 47faaa6 into develop Sep 29, 2025
7 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@EugenioCollado EugenioCollado EugenioCollado 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.

3 participants

@4ntn @EugenioCollado