Skip to content

Migrate performance job to GitHub actions - #404

Merged
EugenioCollado merged 21 commits into
developfrom
feature/performance_workflow
Sep 29, 2025
Merged

Migrate performance job to GitHub actions#404
EugenioCollado merged 21 commits into
developfrom
feature/performance_workflow

Conversation

@4ntn

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

Copy link
Copy Markdown
Contributor

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

@EugenioCollado EugenioCollado left a comment

Copy link
Copy Markdown
Collaborator

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.

2 participants

@4ntn @EugenioCollado