[21267] Add Ubuntu weekly ci #5038

Mario-DL · 2024-07-08T15:56:00Z

Description

This PR addresses the following points related to the github CI:

Change windows reusable ci to use new merge meta files eProsima CI action
Add weekly ubuntu CI to test 2.x branches with CDR 1.1.x
move the if (!skip-ci) inside the reusable workflows (as done in other repos with the run-build input), but keep the if (!conflicts) in the PR workflow so the GitHub CI requirement is not satisfied if skipped by conflicts.
Refactor ubuntu ci - python build job:
build packages-select fastdds_python package

@Mergifyio backport 2.14.x 2.10.x

Contributor Checklist

Commit messages follow the project guidelines.
The code follows the style guidelines of this project.
N/A Tests that thoroughly check the new feature have been added/Regression tests checking the bug and its fix have been added; the added tests pass locally
N/A Any new/modified methods have been properly documented using Doxygen.
N/A Any new configuration API has an equivalent XML API (with the corresponding XSD extension)
N/A Changes are backport compatible: they do NOT break ABI nor change library core behavior.
N/A Changes are API compatible.
N/A New feature has been added to the versions.md file (if applicable).
N/A New feature has been documented/Current behavior is correctly described in the documentation.
Applicable backports have been included in the description.

Reviewer Checklist

The PR has a milestone assigned.
The title and description correctly express the PR's purpose.
Check contributor checklist is correct.
N/A If this is a critical bug fix, backports to the critical-only supported branches have been requested.
Check CI results: changes do not issue any warning.
Check CI results: failing tests are unrelated with the changes.

Signed-off-by: Mario Dominguez <mariodominguez@eprosima.com>

…Prosima CI action Signed-off-by: Mario Dominguez <mariodominguez@eprosima.com>

Signed-off-by: Mario Dominguez <mariodominguez@eprosima.com>

.github/workflows/reusable-ubuntu-ci.yml

.github/workflows/weekly-ubuntu-ci.yml

.github/workflows/reusable-ubuntu-ci.yml

Signed-off-by: Mario Dominguez <mariodominguez@eprosima.com>

JesusPoderoso

LGTM with green CI

JesusPoderoso · 2024-07-10T13:26:33Z

@Mergifyio backport 2.14.x 2.10.x

mergify · 2024-07-10T13:27:05Z

backport 2.14.x 2.10.x

✅ Backports have been created

#5054 [21267] Add Ubuntu weekly ci (backport #5038) has been created for branch 2.14.x but encountered conflicts
#5055 [21267] Add Ubuntu weekly ci (backport #5038) has been created for branch 2.10.x but encountered conflicts

* Refs #21267: Add weekly ubuntu CI to test 2.x branches with CDR 1.1.x Signed-off-by: Mario Dominguez <mariodominguez@eprosima.com> * Refs #21267: Change windows reusable ci to use new merge meta files eProsima CI action Signed-off-by: Mario Dominguez <mariodominguez@eprosima.com> * Refs #21267: move the !(skip_ci) inside the reusable workflows Signed-off-by: Mario Dominguez <mariodominguez@eprosima.com> * Refs #21267: compile fastdds_python package only Signed-off-by: Mario Dominguez <mariodominguez@eprosima.com> * Refs #21267: Apply Jesus suggestions Signed-off-by: Mario Dominguez <mariodominguez@eprosima.com> --------- Signed-off-by: Mario Dominguez <mariodominguez@eprosima.com> (cherry picked from commit e6773fb) # Conflicts: # .github/workflows/nightly-ubuntu-ci.yml # .github/workflows/reusable-ubuntu-ci.yml # .github/workflows/reusable-windows-ci.yml # .github/workflows/weekly-ubuntu-ci.yml

* Refs #21267: Add weekly ubuntu CI to test 2.x branches with CDR 1.1.x Signed-off-by: Mario Dominguez <mariodominguez@eprosima.com> * Refs #21267: Change windows reusable ci to use new merge meta files eProsima CI action Signed-off-by: Mario Dominguez <mariodominguez@eprosima.com> * Refs #21267: move the !(skip_ci) inside the reusable workflows Signed-off-by: Mario Dominguez <mariodominguez@eprosima.com> * Refs #21267: compile fastdds_python package only Signed-off-by: Mario Dominguez <mariodominguez@eprosima.com> * Refs #21267: Apply Jesus suggestions Signed-off-by: Mario Dominguez <mariodominguez@eprosima.com> --------- Signed-off-by: Mario Dominguez <mariodominguez@eprosima.com> (cherry picked from commit e6773fb)

* Refs #21267: Add weekly ubuntu CI to test 2.x branches with CDR 1.1.x Signed-off-by: Mario Dominguez <mariodominguez@eprosima.com> * Refs #21267: Change windows reusable ci to use new merge meta files eProsima CI action Signed-off-by: Mario Dominguez <mariodominguez@eprosima.com> * Refs #21267: move the !(skip_ci) inside the reusable workflows Signed-off-by: Mario Dominguez <mariodominguez@eprosima.com> * Refs #21267: compile fastdds_python package only Signed-off-by: Mario Dominguez <mariodominguez@eprosima.com> * Refs #21267: Apply Jesus suggestions Signed-off-by: Mario Dominguez <mariodominguez@eprosima.com> --------- Signed-off-by: Mario Dominguez <mariodominguez@eprosima.com> (cherry picked from commit e6773fb) Co-authored-by: Mario Domínguez López <116071334+Mario-DL@users.noreply.github.com>

* Refs #21267: Add weekly ubuntu CI to test 2.x branches with CDR 1.1.x Signed-off-by: Mario Dominguez <mariodominguez@eprosima.com> * Refs #21267: Change windows reusable ci to use new merge meta files eProsima CI action Signed-off-by: Mario Dominguez <mariodominguez@eprosima.com> * Refs #21267: move the !(skip_ci) inside the reusable workflows Signed-off-by: Mario Dominguez <mariodominguez@eprosima.com> * Refs #21267: compile fastdds_python package only Signed-off-by: Mario Dominguez <mariodominguez@eprosima.com> * Refs #21267: Apply Jesus suggestions Signed-off-by: Mario Dominguez <mariodominguez@eprosima.com> --------- Signed-off-by: Mario Dominguez <mariodominguez@eprosima.com> (cherry picked from commit e6773fb) Co-authored-by: Mario Domínguez López <116071334+Mario-DL@users.noreply.github.com> (cherry picked from commit 064ce70)

* Add Ubuntu weekly ci (#5038) (#5054) * Refs #21267: Add weekly ubuntu CI to test 2.x branches with CDR 1.1.x Signed-off-by: Mario Dominguez <mariodominguez@eprosima.com> * Refs #21267: Change windows reusable ci to use new merge meta files eProsima CI action Signed-off-by: Mario Dominguez <mariodominguez@eprosima.com> * Refs #21267: move the !(skip_ci) inside the reusable workflows Signed-off-by: Mario Dominguez <mariodominguez@eprosima.com> * Refs #21267: compile fastdds_python package only Signed-off-by: Mario Dominguez <mariodominguez@eprosima.com> * Refs #21267: Apply Jesus suggestions Signed-off-by: Mario Dominguez <mariodominguez@eprosima.com> --------- Signed-off-by: Mario Dominguez <mariodominguez@eprosima.com> (cherry picked from commit e6773fb) Co-authored-by: Mario Domínguez López <116071334+Mario-DL@users.noreply.github.com> (cherry picked from commit 064ce70) * Refs #21267: Add missing parameters in the workflow Signed-off-by: JesusPoderoso <jesuspoderoso@eprosima.com> --------- Signed-off-by: JesusPoderoso <jesuspoderoso@eprosima.com> Co-authored-by: mergify[bot] <37929162+mergify[bot]@users.noreply.github.com> Co-authored-by: JesusPoderoso <jesuspoderoso@eprosima.com>

Mario-DL added this to the v3.0.0 milestone Jul 8, 2024

Mario-DL changed the title ~~[21267] Update Ubuntu weekly ci~~ [21267] Add Ubuntu weekly ci Jul 8, 2024

Mario-DL added 4 commits July 8, 2024 18:06

Refs #21267: Add weekly ubuntu CI to test 2.x branches with CDR 1.1.x

009d83a

Signed-off-by: Mario Dominguez <mariodominguez@eprosima.com>

Refs #21267: Change windows reusable ci to use new merge meta files e…

5681180

…Prosima CI action Signed-off-by: Mario Dominguez <mariodominguez@eprosima.com>

Refs #21267: move the !(skip_ci) inside the reusable workflows

53f41ae

Signed-off-by: Mario Dominguez <mariodominguez@eprosima.com>

Refs #21267: compile fastdds_python package only

ff766ca

Signed-off-by: Mario Dominguez <mariodominguez@eprosima.com>

Mario-DL force-pushed the feature/21267 branch from 7358298 to ff766ca Compare July 8, 2024 16:06

Mario-DL requested a review from JesusPoderoso July 9, 2024 06:35

github-actions bot added the ci-pending PR which CI is running label Jul 9, 2024

JesusPoderoso suggested changes Jul 9, 2024

View reviewed changes

Refs #21267: Apply Jesus suggestions

e396713

Signed-off-by: Mario Dominguez <mariodominguez@eprosima.com>

Mario-DL requested review from JesusPoderoso and removed request for JesusPoderoso July 9, 2024 08:36

JesusPoderoso approved these changes Jul 9, 2024

View reviewed changes

JesusPoderoso added ready-to-merge Ready to be merged. CI and changes have been reviewed and approved. and removed ci-pending PR which CI is running labels Jul 10, 2024

EduPonz merged commit e6773fb into master Jul 11, 2024
16 of 17 checks passed

EduPonz deleted the feature/21267 branch July 11, 2024 08:06

This was referenced Jul 11, 2024

[21267] Add Ubuntu weekly ci (backport #5038) #5054

Merged

[21267] Add Ubuntu weekly ci (backport #5038) #5055

Merged

mergify bot mentioned this pull request Jul 15, 2024

[21267] Add Ubuntu weekly ci (backport #5038) (backport #5054) #5068

Merged

8 tasks

Provide feedback

Saved searches

Use saved searches to filter your results more quickly

[21267] Add Ubuntu weekly ci #5038

[21267] Add Ubuntu weekly ci #5038

Mario-DL commented Jul 8, 2024 •

edited by JesusPoderoso

Loading

JesusPoderoso left a comment

JesusPoderoso commented Jul 10, 2024

mergify bot commented Jul 10, 2024 •

edited

Loading

[21267] Add Ubuntu weekly ci #5038

[21267] Add Ubuntu weekly ci #5038

Conversation

Mario-DL commented Jul 8, 2024 • edited by JesusPoderoso Loading

Description

Contributor Checklist

Reviewer Checklist

JesusPoderoso left a comment

Choose a reason for hiding this comment

JesusPoderoso commented Jul 10, 2024

mergify bot commented Jul 10, 2024 • edited Loading

✅ Backports have been created

Mario-DL commented Jul 8, 2024 •

edited by JesusPoderoso

Loading

mergify bot commented Jul 10, 2024 •

edited

Loading