[21228] Improve CI workflows in main, and branching model refactor by JesusPoderoso · Pull Request #149 · eProsima/Fast-DDS-python

JesusPoderoso · 2024-06-25T11:09:42Z

Description

This PR improves the CI workflows, making them as similar as possible to the Fast DDS workflows.
It also introduces a braching model refactor, similiar as the performed in Fast DDS repository.

Backports to the supported branches will be manually opened:

Contributor Checklist

Commit messages follow the project guidelines.
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 Changes are API compatible.
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.
Check CI results: changes do not issue any warning.
Check CI results: failing tests are unrelated with the changes.

Signed-off-by: JesusPoderoso <jesuspoderoso@eprosima.com>

…quired CI Signed-off-by: JesusPoderoso <jesuspoderoso@eprosima.com>

Signed-off-by: JesusPoderoso <jesuspoderoso@eprosima.com>

EduPonz

NIT

Signed-off-by: JesusPoderoso <jesuspoderoso@eprosima.com>

) * Refs #21228: Refactor Ubuntu CI Signed-off-by: JesusPoderoso <jesuspoderoso@eprosima.com> * Refs #21228: Refactor Windows CI Signed-off-by: JesusPoderoso <jesuspoderoso@eprosima.com> * Refs #21228: Remove previous workflow Signed-off-by: JesusPoderoso <jesuspoderoso@eprosima.com> * Refs #21228: Add RELEASE_SUPPORT.md with Fast DDS references Signed-off-by: JesusPoderoso <jesuspoderoso@eprosima.com> * Refs #21228: Fix windows CI removing win2017 Signed-off-by: JesusPoderoso <jesuspoderoso@eprosima.com> * Refs #21228: Apply rev suggestions in main PR Signed-off-by: JesusPoderoso <jesuspoderoso@eprosima.com> * Refs #21228: Avoid building dependencies again Signed-off-by: JesusPoderoso <jesuspoderoso@eprosima.com> * Refs #21228: Apply rev suggestions Signed-off-by: JesusPoderoso <jesuspoderoso@eprosima.com> * Refs #21228: Apply rev suggestions Signed-off-by: JesusPoderoso <jesuspoderoso@eprosima.com> --------- Signed-off-by: JesusPoderoso <jesuspoderoso@eprosima.com>

) * Refs #21228: Refactor Ubuntu CI Signed-off-by: JesusPoderoso <jesuspoderoso@eprosima.com> * Refs #21228: Refactor Windows CI Signed-off-by: JesusPoderoso <jesuspoderoso@eprosima.com> * Refs #21228: Remove previous workflow Signed-off-by: JesusPoderoso <jesuspoderoso@eprosima.com> * Refs #21228: Add RELEASE_SUPPORT.md with Fast DDS references Signed-off-by: JesusPoderoso <jesuspoderoso@eprosima.com> * Refs #21228: Avoid running tests in 1.0.x Signed-off-by: JesusPoderoso <jesuspoderoso@eprosima.com> * Refs #21228: Fix windows CI removing win2017 Signed-off-by: JesusPoderoso <jesuspoderoso@eprosima.com> * Refs #21228: Deactivate Windows build temporary Signed-off-by: JesusPoderoso <jesuspoderoso@eprosima.com> * Refs #21228: Apply rev suggestions in main PR Signed-off-by: JesusPoderoso <jesuspoderoso@eprosima.com> * Refs #21228: Avoid building dependencies again Signed-off-by: JesusPoderoso <jesuspoderoso@eprosima.com> * Refs #21228: Apply rev suggestions Signed-off-by: JesusPoderoso <jesuspoderoso@eprosima.com> * Refs #21228: Apply rev suggestions Signed-off-by: JesusPoderoso <jesuspoderoso@eprosima.com> --------- Signed-off-by: JesusPoderoso <jesuspoderoso@eprosima.com>

JesusPoderoso added this to the v2.0.0 milestone Jun 25, 2024

JesusPoderoso self-assigned this Jun 25, 2024

JesusPoderoso added the skip-ci Automatically pass CI label Jun 25, 2024

JesusPoderoso requested a review from EduPonz June 25, 2024 11:10

JesusPoderoso force-pushed the feature/21228 branch from c20823c to 44fbab6 Compare June 25, 2024 12:15

JesusPoderoso requested review from EduPonz and removed request for EduPonz June 25, 2024 12:15

JesusPoderoso added no-test Skip CI tests if PR marked with this label and removed skip-ci Automatically pass CI labels Jun 25, 2024

JesusPoderoso requested review from EduPonz and removed request for EduPonz June 25, 2024 13:23

github-actions Bot added the ci-pending PR which CI is running label Jun 25, 2024

JesusPoderoso force-pushed the feature/21228 branch from 44fbab6 to 850cfc7 Compare June 25, 2024 13:28

JesusPoderoso requested review from EduPonz and removed request for EduPonz June 25, 2024 13:28

JesusPoderoso mentioned this pull request Jun 25, 2024

[21228] Improve CI workflows in 1.4.x, and branching model refactor (backport #149) #150

Merged

6 tasks

JesusPoderoso force-pushed the feature/21228 branch from 850cfc7 to 63cdfb3 Compare June 25, 2024 14:03

JesusPoderoso requested review from EduPonz and removed request for EduPonz June 25, 2024 14:03

This was referenced Jun 25, 2024

[21228] Improve CI workflows in 1.2.x, and branching model refactor (backport #149) #151

Merged

[21228] Improve CI workflows in 1.0.x, and branching model refactor (backport #149) #152

Merged

JesusPoderoso removed the no-test Skip CI tests if PR marked with this label label Jun 25, 2024

JesusPoderoso requested review from EduPonz and removed request for EduPonz June 25, 2024 14:43

JesusPoderoso force-pushed the feature/21228 branch from 63cdfb3 to 73562d4 Compare June 25, 2024 15:12

JesusPoderoso requested review from EduPonz and removed request for EduPonz June 25, 2024 15:12

JesusPoderoso force-pushed the feature/21228 branch from 73562d4 to c45099c Compare June 26, 2024 06:05

JesusPoderoso requested review from richiprosima and removed request for EduPonz June 26, 2024 06:05

JesusPoderoso added 12 commits July 4, 2024 11:40

Refs #21228: Deactivate Windows build temporary on 1.0.x

e9d9759

Signed-off-by: JesusPoderoso <jesuspoderoso@eprosima.com>

Refs #21228: Move build documentation in tests steps only in Ubuntu CI

2cc0571

Signed-off-by: JesusPoderoso <jesuspoderoso@eprosima.com>

Refs #21228: Apply rev suggestions

7c66a53

Signed-off-by: JesusPoderoso <jesuspoderoso@eprosima.com>

Refs #21228: Fix test.meta issue in Ubuntu vs Windows

342e6f9

Signed-off-by: JesusPoderoso <jesuspoderoso@eprosima.com>

Refs #21228: Remove test.repos

0eabeb7

Signed-off-by: JesusPoderoso <jesuspoderoso@eprosima.com>

Refs #21228: Apply last rev suggestion

06ff62b

Signed-off-by: JesusPoderoso <jesuspoderoso@eprosima.com>

Refs #21228: Add if conflicts check in PR trigger to avoid passing re…

236083c

…quired CI Signed-off-by: JesusPoderoso <jesuspoderoso@eprosima.com>

Refs #21228: Update eProsima-CI action references

fde37ad

Signed-off-by: JesusPoderoso <jesuspoderoso@eprosima.com>

Refs #21228: Avoid building dependencies again

283ae4e

Signed-off-by: JesusPoderoso <jesuspoderoso@eprosima.com>

Refs #21228: Apply rev suggestions

fcd7454

Signed-off-by: JesusPoderoso <jesuspoderoso@eprosima.com>

Refs #21228: Simplify nightly removing unnecessary matrix options

557a5d8

Signed-off-by: JesusPoderoso <jesuspoderoso@eprosima.com>

Refs #21228: Move 2.6.x related jobs to weekly workflow

91e1d95

Signed-off-by: JesusPoderoso <jesuspoderoso@eprosima.com>

JesusPoderoso force-pushed the feature/21228 branch from 7a4dd6f to 91e1d95 Compare July 4, 2024 09:44

Refs #21228: Fix weekly cron expression

85f7216

Signed-off-by: JesusPoderoso <jesuspoderoso@eprosima.com>

EduPonz suggested changes Jul 9, 2024

View reviewed changes

Comment thread .github/workflows/nightly-windows-ci.yml Outdated

Comment thread .github/workflows/nightly-windows-ci.yml Outdated

Comment thread .github/workflows/nightly-windows-ci.yml Outdated

Comment thread .github/workflows/weekly-windows-ci.yml Outdated

Refs #21228: Apply rev suggestions

7dd2d1a

Signed-off-by: JesusPoderoso <jesuspoderoso@eprosima.com>

EduPonz approved these changes Jul 9, 2024

View reviewed changes

EduPonz self-requested a review July 10, 2024 14:06

EduPonz previously approved these changes Jul 10, 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

JesusPoderoso dismissed EduPonz’s stale review via d67ea98 July 11, 2024 13:12

JesusPoderoso force-pushed the feature/21228 branch from d67ea98 to 4dc0fac Compare July 11, 2024 14:06

Refs #21228: Apply rev suggestions

a80e8ae

Signed-off-by: JesusPoderoso <jesuspoderoso@eprosima.com>

JesusPoderoso force-pushed the feature/21228 branch from 4dc0fac to a80e8ae Compare July 11, 2024 14:07

EduPonz approved these changes Jul 11, 2024

View reviewed changes

EduPonz merged commit f9575d2 into main Jul 11, 2024

EduPonz deleted the feature/21228 branch July 11, 2024 14:09

Provide feedback

Saved searches

Use saved searches to filter your results more quickly

Uh oh!

[21228] Improve CI workflows in main, and branching model refactor#149

[21228] Improve CI workflows in main, and branching model refactor#149
EduPonz merged 21 commits into
mainfrom
feature/21228

JesusPoderoso commented Jun 25, 2024 •

edited

Loading

Uh oh!

EduPonz left a comment

Uh oh!

Uh oh!

Uh oh!

Uh oh!

Uh oh!

Reviewers

Assignees

Labels

Projects

Milestone

Development

Uh oh!

2 participants

Uh oh!

Conversation

JesusPoderoso commented Jun 25, 2024 • edited Loading Uh oh! There was an error while loading. Please reload this page.

Uh oh!

Description

Contributor Checklist

Reviewer Checklist

Uh oh!

EduPonz left a comment

Choose a reason for hiding this comment

Uh oh!

Uh oh!

Uh oh!

Uh oh!

Uh oh!

Reviewers

Assignees

Labels

Projects

Milestone

Development

Uh oh!

2 participants

JesusPoderoso commented Jun 25, 2024 •

edited

Loading