[20091] Add macOS Github CI #4100

EduPonz · 2023-12-05T09:32:22Z

Description

This PR adds:

A reusable workflow for macOS CI
A workflow for macOS CI on PR and push events
A nightly workflow for macOS CI

It depends on the following PRs:

[20091] Add actions to support Fast DDS CI on macOS eProsima-CI#44

@Mergifyio backport 2.12.x 2.11.x 2.10.x 2.6.x

Contributor Checklist

Commit messages follow the project guidelines.
N/A: 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: Changes are ABI compatible.
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.
N/A: Applicable backports have been included in the description.

Reviewer Checklist

The PR has a milestone assigned.
Check contributor checklist is correct.
Check CI results: changes do not issue any warning.
Check CI results: failing tests are unrelated with the changes.

EduPonz · 2023-12-08T08:18:12Z

@Mergifyio backport 2.12.x 2.11.x 2.10.x 2.6.x

mergify · 2023-12-08T08:18:19Z

backport 2.12.x 2.11.x 2.10.x 2.6.x

✅ Backports have been created

#4398 [20091] Add macOS Github CI (backport #4100) has been created for branch 2.12.x but encountered conflicts
#4399 [20091] Add macOS Github CI (backport #4100) has been created for branch 2.11.x but encountered conflicts
#4400 [20091] Add macOS Github CI (backport #4100) has been created for branch 2.10.x but encountered conflicts
#4401 [20091] Add macOS Github CI (backport #4100) has been created for branch 2.6.x but encountered conflicts

Signed-off-by: EduPonz <[email protected]>

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

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

Signed-off-by: EduPonz <[email protected]>

JesusPoderoso

LGTM
Please, remember to update all @feature/fastdds_mac_ci_support to @main before merging

* Refs #20091: Add macOS Github CI Signed-off-by: EduPonz <[email protected]> * Refs #20091: Add summary table Signed-off-by: EduPonz <[email protected]> * Refs #20091: Remove commented step to install googletest Signed-off-by: EduPonz <[email protected]> * Refs #20091: Add hosts for Fast DDS testing of DNS features Signed-off-by: EduPonz <[email protected]> * Refs #20091: Fix Prepare colcon workspace step Signed-off-by: EduPonz <[email protected]> * Refs #20091: Disable DNS hosts edit Signed-off-by: EduPonz <[email protected]> * Refs #20091: Use CCache Signed-off-by: EduPonz <[email protected]> * Refs #20091: Add optional flags to junit_summary.py Signed-off-by: EduPonz <[email protected]> * Refs #20091: Use eProsima-CI Signed-off-by: EduPonz <[email protected]> * Refs #20091: Use Fast DDS input branch Signed-off-by: EduPonz <[email protected]> * Refs #20091: Print summary and only show failed list Signed-off-by: EduPonz <[email protected]> * Refs #20091: Do not update brew Signed-off-by: EduPonz <[email protected]> * Refs #20091: Do not upgrade python packages Signed-off-by: EduPonz <[email protected]> * Refs #20091: Try to use python 3.11 Signed-off-by: EduPonz <[email protected]> * Refs #20091: Specify colcon workspace Signed-off-by: EduPonz <[email protected]> * Refs #20091: Set workspace for colcon test Signed-off-by: EduPonz <[email protected]> * Refs #20091: Use PR merge branch for windows and macos gh CI Signed-off-by: EduPonz <[email protected]> * Refs #20091: Adapt to changes in eProsima-CI Signed-off-by: EduPonz <[email protected]> * Refs #20091: Do not build with Wall Signed-off-by: EduPonz <[email protected]> * Refs #20091: Upgrade to macos-13 to get clang 15 Signed-off-by: EduPonz <[email protected]> * Refs #20091: Do not use thirdparty Signed-off-by: EduPonz <[email protected]> * Refs #20091: Avoid enum-constexpr-conversion warning Signed-off-by: EduPonz <[email protected]> * Refs #20091: Correctly exclude DNS tests when necessary Signed-off-by: EduPonz <[email protected]> * Refs #20091: Apply Jesús suggestions Signed-off-by: EduPonz <[email protected]> * Refs #20091: Set eProsima-CI actions to v0 Signed-off-by: EduPonz <[email protected]> --------- Signed-off-by: EduPonz <[email protected]> (cherry picked from commit 8cff0ca) # Conflicts: # .github/workflows/windows-ci.yml

* Refs #20091: Add macOS Github CI Signed-off-by: EduPonz <[email protected]> * Refs #20091: Add summary table Signed-off-by: EduPonz <[email protected]> * Refs #20091: Remove commented step to install googletest Signed-off-by: EduPonz <[email protected]> * Refs #20091: Add hosts for Fast DDS testing of DNS features Signed-off-by: EduPonz <[email protected]> * Refs #20091: Fix Prepare colcon workspace step Signed-off-by: EduPonz <[email protected]> * Refs #20091: Disable DNS hosts edit Signed-off-by: EduPonz <[email protected]> * Refs #20091: Use CCache Signed-off-by: EduPonz <[email protected]> * Refs #20091: Add optional flags to junit_summary.py Signed-off-by: EduPonz <[email protected]> * Refs #20091: Use eProsima-CI Signed-off-by: EduPonz <[email protected]> * Refs #20091: Use Fast DDS input branch Signed-off-by: EduPonz <[email protected]> * Refs #20091: Print summary and only show failed list Signed-off-by: EduPonz <[email protected]> * Refs #20091: Do not update brew Signed-off-by: EduPonz <[email protected]> * Refs #20091: Do not upgrade python packages Signed-off-by: EduPonz <[email protected]> * Refs #20091: Try to use python 3.11 Signed-off-by: EduPonz <[email protected]> * Refs #20091: Specify colcon workspace Signed-off-by: EduPonz <[email protected]> * Refs #20091: Set workspace for colcon test Signed-off-by: EduPonz <[email protected]> * Refs #20091: Use PR merge branch for windows and macos gh CI Signed-off-by: EduPonz <[email protected]> * Refs #20091: Adapt to changes in eProsima-CI Signed-off-by: EduPonz <[email protected]> * Refs #20091: Do not build with Wall Signed-off-by: EduPonz <[email protected]> * Refs #20091: Upgrade to macos-13 to get clang 15 Signed-off-by: EduPonz <[email protected]> * Refs #20091: Do not use thirdparty Signed-off-by: EduPonz <[email protected]> * Refs #20091: Avoid enum-constexpr-conversion warning Signed-off-by: EduPonz <[email protected]> * Refs #20091: Correctly exclude DNS tests when necessary Signed-off-by: EduPonz <[email protected]> * Refs #20091: Apply Jesús suggestions Signed-off-by: EduPonz <[email protected]> * Refs #20091: Set eProsima-CI actions to v0 Signed-off-by: EduPonz <[email protected]> --------- Signed-off-by: EduPonz <[email protected]> (cherry picked from commit 8cff0ca) # Conflicts: # .github/workflows/windows-ci.yml Signed-off-by: EduPonz <[email protected]>

* Refs #20091: Add macOS Github CI Signed-off-by: EduPonz <[email protected]> * Refs #20091: Add summary table Signed-off-by: EduPonz <[email protected]> * Refs #20091: Remove commented step to install googletest Signed-off-by: EduPonz <[email protected]> * Refs #20091: Add hosts for Fast DDS testing of DNS features Signed-off-by: EduPonz <[email protected]> * Refs #20091: Fix Prepare colcon workspace step Signed-off-by: EduPonz <[email protected]> * Refs #20091: Disable DNS hosts edit Signed-off-by: EduPonz <[email protected]> * Refs #20091: Use CCache Signed-off-by: EduPonz <[email protected]> * Refs #20091: Add optional flags to junit_summary.py Signed-off-by: EduPonz <[email protected]> * Refs #20091: Use eProsima-CI Signed-off-by: EduPonz <[email protected]> * Refs #20091: Use Fast DDS input branch Signed-off-by: EduPonz <[email protected]> * Refs #20091: Print summary and only show failed list Signed-off-by: EduPonz <[email protected]> * Refs #20091: Do not update brew Signed-off-by: EduPonz <[email protected]> * Refs #20091: Do not upgrade python packages Signed-off-by: EduPonz <[email protected]> * Refs #20091: Try to use python 3.11 Signed-off-by: EduPonz <[email protected]> * Refs #20091: Specify colcon workspace Signed-off-by: EduPonz <[email protected]> * Refs #20091: Set workspace for colcon test Signed-off-by: EduPonz <[email protected]> * Refs #20091: Use PR merge branch for windows and macos gh CI Signed-off-by: EduPonz <[email protected]> * Refs #20091: Adapt to changes in eProsima-CI Signed-off-by: EduPonz <[email protected]> * Refs #20091: Do not build with Wall Signed-off-by: EduPonz <[email protected]> * Refs #20091: Upgrade to macos-13 to get clang 15 Signed-off-by: EduPonz <[email protected]> * Refs #20091: Do not use thirdparty Signed-off-by: EduPonz <[email protected]> * Refs #20091: Avoid enum-constexpr-conversion warning Signed-off-by: EduPonz <[email protected]> * Refs #20091: Correctly exclude DNS tests when necessary Signed-off-by: EduPonz <[email protected]> * Refs #20091: Apply Jesús suggestions Signed-off-by: EduPonz <[email protected]> * Refs #20091: Set eProsima-CI actions to v0 Signed-off-by: EduPonz <[email protected]> --------- Signed-off-by: EduPonz <[email protected]> (cherry picked from commit 8cff0ca) # Conflicts: # .github/workflows/windows-ci.yml # test/blackbox/CMakeLists.txt Signed-off-by: EduPonz <[email protected]>

* Add macOS Github CI (#4100) * Refs #20091: Add macOS Github CI Signed-off-by: EduPonz <[email protected]> * Refs #20091: Add summary table Signed-off-by: EduPonz <[email protected]> * Refs #20091: Remove commented step to install googletest Signed-off-by: EduPonz <[email protected]> * Refs #20091: Add hosts for Fast DDS testing of DNS features Signed-off-by: EduPonz <[email protected]> * Refs #20091: Fix Prepare colcon workspace step Signed-off-by: EduPonz <[email protected]> * Refs #20091: Disable DNS hosts edit Signed-off-by: EduPonz <[email protected]> * Refs #20091: Use CCache Signed-off-by: EduPonz <[email protected]> * Refs #20091: Add optional flags to junit_summary.py Signed-off-by: EduPonz <[email protected]> * Refs #20091: Use eProsima-CI Signed-off-by: EduPonz <[email protected]> * Refs #20091: Use Fast DDS input branch Signed-off-by: EduPonz <[email protected]> * Refs #20091: Print summary and only show failed list Signed-off-by: EduPonz <[email protected]> * Refs #20091: Do not update brew Signed-off-by: EduPonz <[email protected]> * Refs #20091: Do not upgrade python packages Signed-off-by: EduPonz <[email protected]> * Refs #20091: Try to use python 3.11 Signed-off-by: EduPonz <[email protected]> * Refs #20091: Specify colcon workspace Signed-off-by: EduPonz <[email protected]> * Refs #20091: Set workspace for colcon test Signed-off-by: EduPonz <[email protected]> * Refs #20091: Use PR merge branch for windows and macos gh CI Signed-off-by: EduPonz <[email protected]> * Refs #20091: Adapt to changes in eProsima-CI Signed-off-by: EduPonz <[email protected]> * Refs #20091: Do not build with Wall Signed-off-by: EduPonz <[email protected]> * Refs #20091: Upgrade to macos-13 to get clang 15 Signed-off-by: EduPonz <[email protected]> * Refs #20091: Do not use thirdparty Signed-off-by: EduPonz <[email protected]> * Refs #20091: Avoid enum-constexpr-conversion warning Signed-off-by: EduPonz <[email protected]> * Refs #20091: Correctly exclude DNS tests when necessary Signed-off-by: EduPonz <[email protected]> * Refs #20091: Apply Jesús suggestions Signed-off-by: EduPonz <[email protected]> * Refs #20091: Set eProsima-CI actions to v0 Signed-off-by: EduPonz <[email protected]> --------- Signed-off-by: EduPonz <[email protected]> (cherry picked from commit 8cff0ca) # Conflicts: # .github/workflows/windows-ci.yml * Refs #20091: Fix conflict Signed-off-by: JesusPoderoso <[email protected]> * Refs #20091: Set default branch as '2.12.x' Signed-off-by: JesusPoderoso <[email protected]> * Refs #20091: Adjust for 2.12.x Signed-off-by: EduPonz <[email protected]> --------- Signed-off-by: JesusPoderoso <[email protected]> Signed-off-by: EduPonz <[email protected]> Co-authored-by: Eduardo Ponz Segrelles <[email protected]> Co-authored-by: JesusPoderoso <[email protected]>

* Add macOS Github CI (#4100) * Refs #20091: Add macOS Github CI Signed-off-by: EduPonz <[email protected]> * Refs #20091: Add summary table Signed-off-by: EduPonz <[email protected]> * Refs #20091: Remove commented step to install googletest Signed-off-by: EduPonz <[email protected]> * Refs #20091: Add hosts for Fast DDS testing of DNS features Signed-off-by: EduPonz <[email protected]> * Refs #20091: Fix Prepare colcon workspace step Signed-off-by: EduPonz <[email protected]> * Refs #20091: Disable DNS hosts edit Signed-off-by: EduPonz <[email protected]> * Refs #20091: Use CCache Signed-off-by: EduPonz <[email protected]> * Refs #20091: Add optional flags to junit_summary.py Signed-off-by: EduPonz <[email protected]> * Refs #20091: Use eProsima-CI Signed-off-by: EduPonz <[email protected]> * Refs #20091: Use Fast DDS input branch Signed-off-by: EduPonz <[email protected]> * Refs #20091: Print summary and only show failed list Signed-off-by: EduPonz <[email protected]> * Refs #20091: Do not update brew Signed-off-by: EduPonz <[email protected]> * Refs #20091: Do not upgrade python packages Signed-off-by: EduPonz <[email protected]> * Refs #20091: Try to use python 3.11 Signed-off-by: EduPonz <[email protected]> * Refs #20091: Specify colcon workspace Signed-off-by: EduPonz <[email protected]> * Refs #20091: Set workspace for colcon test Signed-off-by: EduPonz <[email protected]> * Refs #20091: Use PR merge branch for windows and macos gh CI Signed-off-by: EduPonz <[email protected]> * Refs #20091: Adapt to changes in eProsima-CI Signed-off-by: EduPonz <[email protected]> * Refs #20091: Do not build with Wall Signed-off-by: EduPonz <[email protected]> * Refs #20091: Upgrade to macos-13 to get clang 15 Signed-off-by: EduPonz <[email protected]> * Refs #20091: Do not use thirdparty Signed-off-by: EduPonz <[email protected]> * Refs #20091: Avoid enum-constexpr-conversion warning Signed-off-by: EduPonz <[email protected]> * Refs #20091: Correctly exclude DNS tests when necessary Signed-off-by: EduPonz <[email protected]> * Refs #20091: Apply Jesús suggestions Signed-off-by: EduPonz <[email protected]> * Refs #20091: Set eProsima-CI actions to v0 Signed-off-by: EduPonz <[email protected]> --------- Signed-off-by: EduPonz <[email protected]> (cherry picked from commit 8cff0ca) # Conflicts: # .github/workflows/windows-ci.yml # test/blackbox/CMakeLists.txt Signed-off-by: EduPonz <[email protected]> * Refs #20091: Fix conflicts and adjust for 2.6.x Signed-off-by: EduPonz <[email protected]> * Refs #20091: Apply suggestions Signed-off-by: EduPonz <[email protected]> --------- Signed-off-by: EduPonz <[email protected]> Co-authored-by: Eduardo Ponz Segrelles <[email protected]>

* Add macOS Github CI (#4100) * Refs #20091: Add macOS Github CI Signed-off-by: EduPonz <[email protected]> * Refs #20091: Add summary table Signed-off-by: EduPonz <[email protected]> * Refs #20091: Remove commented step to install googletest Signed-off-by: EduPonz <[email protected]> * Refs #20091: Add hosts for Fast DDS testing of DNS features Signed-off-by: EduPonz <[email protected]> * Refs #20091: Fix Prepare colcon workspace step Signed-off-by: EduPonz <[email protected]> * Refs #20091: Disable DNS hosts edit Signed-off-by: EduPonz <[email protected]> * Refs #20091: Use CCache Signed-off-by: EduPonz <[email protected]> * Refs #20091: Add optional flags to junit_summary.py Signed-off-by: EduPonz <[email protected]> * Refs #20091: Use eProsima-CI Signed-off-by: EduPonz <[email protected]> * Refs #20091: Use Fast DDS input branch Signed-off-by: EduPonz <[email protected]> * Refs #20091: Print summary and only show failed list Signed-off-by: EduPonz <[email protected]> * Refs #20091: Do not update brew Signed-off-by: EduPonz <[email protected]> * Refs #20091: Do not upgrade python packages Signed-off-by: EduPonz <[email protected]> * Refs #20091: Try to use python 3.11 Signed-off-by: EduPonz <[email protected]> * Refs #20091: Specify colcon workspace Signed-off-by: EduPonz <[email protected]> * Refs #20091: Set workspace for colcon test Signed-off-by: EduPonz <[email protected]> * Refs #20091: Use PR merge branch for windows and macos gh CI Signed-off-by: EduPonz <[email protected]> * Refs #20091: Adapt to changes in eProsima-CI Signed-off-by: EduPonz <[email protected]> * Refs #20091: Do not build with Wall Signed-off-by: EduPonz <[email protected]> * Refs #20091: Upgrade to macos-13 to get clang 15 Signed-off-by: EduPonz <[email protected]> * Refs #20091: Do not use thirdparty Signed-off-by: EduPonz <[email protected]> * Refs #20091: Avoid enum-constexpr-conversion warning Signed-off-by: EduPonz <[email protected]> * Refs #20091: Correctly exclude DNS tests when necessary Signed-off-by: EduPonz <[email protected]> * Refs #20091: Apply Jesús suggestions Signed-off-by: EduPonz <[email protected]> * Refs #20091: Set eProsima-CI actions to v0 Signed-off-by: EduPonz <[email protected]> --------- Signed-off-by: EduPonz <[email protected]> (cherry picked from commit 8cff0ca) # Conflicts: # .github/workflows/windows-ci.yml Signed-off-by: EduPonz <[email protected]> * Refs #20091: Fix conflicts and adjust for 2.10.x Signed-off-by: EduPonz <[email protected]> * Refs #20091: Apply suggestions Signed-off-by: EduPonz <[email protected]> --------- Signed-off-by: EduPonz <[email protected]> Co-authored-by: Eduardo Ponz Segrelles <[email protected]>

EduPonz changed the title ~~[20091: Add macOS Github CI~~ [20091] Add macOS Github CI Dec 5, 2023

EduPonz added this to the v2.13.0 milestone Dec 5, 2023

EduPonz force-pushed the ci/gh_mac_ci branch 2 times, most recently from 0de0ea4 to 8ddeb87 Compare December 6, 2023 07:45

EduPonz closed this Dec 6, 2023

EduPonz reopened this Dec 6, 2023

EduPonz force-pushed the ci/gh_mac_ci branch from 97a5fe3 to 1a5f020 Compare December 6, 2023 08:51

This was referenced Dec 6, 2023

[20091] Add actions to support Fast DDS CI on macOS eProsima/eProsima-CI#44

Merged

[17513] Add nightly Ubuntu Github CI #4109

Merged

EduPonz modified the milestones: v2.13.0, v2.13.1 Dec 19, 2023

MiguelCompany modified the milestones: v2.13.1, v2.13.2 Dec 22, 2023

EduPonz modified the milestones: v2.13.2, v2.13.3 Feb 1, 2024

EduPonz added 13 commits February 16, 2024 09:42

Refs #20091: Add macOS Github CI

f921502

Signed-off-by: EduPonz <[email protected]>

Refs #20091: Add summary table

c292eba

Signed-off-by: EduPonz <[email protected]>

Refs #20091: Remove commented step to install googletest

ac18bab

Signed-off-by: EduPonz <[email protected]>

Refs #20091: Add hosts for Fast DDS testing of DNS features

412ca02

Signed-off-by: EduPonz <[email protected]>

Refs #20091: Fix Prepare colcon workspace step

c0e9604

Signed-off-by: EduPonz <[email protected]>

Refs #20091: Disable DNS hosts edit

455efcc

Signed-off-by: EduPonz <[email protected]>

Refs #20091: Use CCache

db769c4

Signed-off-by: EduPonz <[email protected]>

Refs #20091: Add optional flags to junit_summary.py

7fbc075

Signed-off-by: EduPonz <[email protected]>

Refs #20091: Use eProsima-CI

ebda49d

Signed-off-by: EduPonz <[email protected]>

Refs #20091: Use Fast DDS input branch

36056ca

Signed-off-by: EduPonz <[email protected]>

Refs #20091: Print summary and only show failed list

eaac27a

Signed-off-by: EduPonz <[email protected]>

Refs #20091: Do not update brew

3964ee6

Signed-off-by: EduPonz <[email protected]>

Refs #20091: Do not upgrade python packages

4c70195

Signed-off-by: EduPonz <[email protected]>

EduPonz force-pushed the ci/gh_mac_ci branch 2 times, most recently from 70bef81 to 9da466e Compare February 17, 2024 08:28

Refs #20091: Avoid enum-constexpr-conversion warning

93aa937

Signed-off-by: EduPonz <[email protected]>

EduPonz force-pushed the ci/gh_mac_ci branch from 9da466e to 93aa937 Compare February 17, 2024 08:38

Refs #20091: Correctly exclude DNS tests when necessary

3ecdb0c

Signed-off-by: EduPonz <[email protected]>

JesusPoderoso reviewed Feb 19, 2024

View reviewed changes

.github/workflows/reusable-windows-ci.yml Show resolved Hide resolved

.github/workflows/reusable-mac-ci.yml Outdated Show resolved Hide resolved

.github/workflows/reusable-mac-ci.yml Outdated Show resolved Hide resolved

Refs #20091: Apply Jesús suggestions

833debe

Signed-off-by: EduPonz <[email protected]>

EduPonz removed the needs-review PR that is ready to be reviewed label Feb 19, 2024

Refs #20091: Set eProsima-CI actions to v0

0b333e8

Signed-off-by: EduPonz <[email protected]>

JesusPoderoso previously approved these changes Feb 20, 2024

View reviewed changes

EduPonz dismissed JesusPoderoso’s stale review via 0b333e8 February 20, 2024 07:01

JesusPoderoso approved these changes Feb 20, 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 Feb 20, 2024

EduPonz merged commit 8cff0ca into master Feb 20, 2024
7 of 9 checks passed

EduPonz deleted the ci/gh_mac_ci branch February 20, 2024 07:17

mergify bot mentioned this pull request Feb 20, 2024

[20091] Add macOS Github CI (backport #4100) #4398

Merged

mergify bot mentioned this pull request Feb 20, 2024

[20091] Add macOS Github CI (backport #4100) #4399

Closed

mergify bot mentioned this pull request Feb 20, 2024

[20091] Add macOS Github CI (backport #4100) #4400

Merged

mergify bot mentioned this pull request Feb 20, 2024

[20091] Add macOS Github CI (backport #4100) #4401

Merged

Provide feedback

Saved searches

Use saved searches to filter your results more quickly

[20091] Add macOS Github CI #4100

[20091] Add macOS Github CI #4100

EduPonz commented Dec 5, 2023 •

edited by JesusPoderoso

Loading

EduPonz commented Dec 8, 2023

mergify bot commented Dec 8, 2023 •

edited

Loading

JesusPoderoso left a comment

[20091] Add macOS Github CI #4100

[20091] Add macOS Github CI #4100

Conversation

EduPonz commented Dec 5, 2023 • edited by JesusPoderoso Loading

Description

Contributor Checklist

Reviewer Checklist

EduPonz commented Dec 8, 2023

mergify bot commented Dec 8, 2023 • edited Loading

✅ Backports have been created

JesusPoderoso left a comment

Choose a reason for hiding this comment

EduPonz commented Dec 5, 2023 •

edited by JesusPoderoso

Loading

mergify bot commented Dec 8, 2023 •

edited

Loading