Skip to content
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.

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

[20822] Refactor Fast DDS Ubuntu CI to include several tests #4860

Merged
merged 26 commits into from
Jun 17, 2024

Conversation

JesusPoderoso
Copy link
Contributor

@JesusPoderoso JesusPoderoso commented May 28, 2024

Description

This PR introduces a "small" refactor in the Fast DDS Ubuntu CI.
It splits current CI into build and tests, and introduces new jobs that uses the already built library to build other products and run their test suite.
That would allow Fast DDS developers to detect possible issues that can affect other products while introducing changes in the Fast DDS library.

graph BT;
    fastdds_build_test --> fastdds_build
    fastdds_python_build --> fastdds_build
    fastdds_shapes_demo_build --> fastdds_build
    discovery_server_build_test --> fastdds_build
    fastdds_python_build_test --> fastdds_python_build
    fastdds_docs_build_test --> fastdds_python_build
Loading

@Mergifyio backport 2.14.x 2.13.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 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.
  • Check CI results: changes do not issue any warning.
  • Check CI results: failing tests are unrelated with the changes.

@JesusPoderoso JesusPoderoso added this to the v3.0.0 milestone May 28, 2024
@JesusPoderoso JesusPoderoso self-assigned this May 28, 2024
@github-actions github-actions bot added ci-pending PR which CI is running labels May 28, 2024
@JesusPoderoso JesusPoderoso requested review from richiprosima and removed request for richiprosima May 28, 2024 09:44
@JesusPoderoso JesusPoderoso marked this pull request as ready for review May 28, 2024 10:52
@JesusPoderoso JesusPoderoso requested review from EduPonz and removed request for richiprosima and EduPonz May 28, 2024 10:52
@elianalf elianalf added the needs-review PR that is ready to be reviewed label May 30, 2024
@JesusPoderoso JesusPoderoso requested review from EduPonz and removed request for EduPonz June 6, 2024 11:02
@JesusPoderoso JesusPoderoso requested review from EduPonz and removed request for EduPonz June 10, 2024 10:06
@JesusPoderoso JesusPoderoso requested review from EduPonz and removed request for EduPonz June 10, 2024 10:29
@JesusPoderoso JesusPoderoso requested review from EduPonz and removed request for EduPonz June 17, 2024 09:25
@JesusPoderoso
Copy link
Contributor Author

  1. Manual run with NO security, NO tests, NO CCache here
  2. Manual run with security, tests and CCache here

@EduPonz
Copy link

EduPonz commented Jun 17, 2024

@Mergifyio backport 2.14.x 2.13.x 2.10.x 2.6.x

Copy link
Contributor

mergify bot commented Jun 17, 2024

backport 2.14.x 2.13.x 2.10.x 2.6.x

✅ Backports have been created

@EduPonz EduPonz merged commit 22338f3 into master Jun 17, 2024
33 of 41 checks passed
@EduPonz EduPonz deleted the ci/python_tests branch June 17, 2024 12:11
mergify bot pushed a commit that referenced this pull request Jun 17, 2024
* Refs #20822: Split current job into different jobs

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

* Refs #20822: Add shapes demo build job

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

* Refs #20822: Add discovery server build test job

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

* Refs #20822: Add python build test job

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

* Refs #20822: Add documentation build test job

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

* Refs #20822: Remove previous documentation workflow

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

* Refs #20822: Avoid building with tests if not running tests

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

* Refs #20822: Try to fix documentation job

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

* Refs #20822: Move conditions to reusable workflows

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

* Refs #20822: Apply internal review suggestions

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

* Refs #20822: Refactor meta files into build and test, and combine them in test steps

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

* Refs #20822: Remove matrix from ubuntu CI

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

* Revert "Refs #20822: Move conditions to reusable workflows"

This reverts commit d4eea0d.

* Refs #20822: Apply rev suggestions

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

* Refs #20822: Add missing discovery server requirements in the job

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

* Refs #20822: Apply rev suggestions (2)

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

* Refs #20822: Fixing windows workflow by merging metas

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

* Refs #20822: Add step with alternative builds

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

* Refs #20822: Use colcon test meta in all jobs, and fix xfail filter

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

* Refs #20822: Remove redundant CMake args, and fix alternative builds

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

* Refs #20822: Apply rev suggestions (3)

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

* Refs #20822: Explicitly compare boolean inputs

Signed-off-by: eduponz <eduardoponz@eprosima.com>

* Refs #20822: Install tools on test steps to avoid losing symlink permissions

Signed-off-by: eduponz <eduardoponz@eprosima.com>

* Refs #20822: Update discovery_server meta file name

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

* Refs #20822: Adjust ternary operators

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

* Refs #20822: Use CCache based on an input parameter

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

---------

Signed-off-by: JesusPoderoso <jesuspoderoso@eprosima.com>
Signed-off-by: eduponz <eduardoponz@eprosima.com>
Co-authored-by: eduponz <eduardoponz@eprosima.com>
(cherry picked from commit 22338f3)

# Conflicts:
#	.github/workflows/config/documentation.meta
#	.github/workflows/config/fastdds_build.meta
#	.github/workflows/documentation-tests.yaml
#	.github/workflows/nightly-ubuntu-ci.yml
#	.github/workflows/reusable-mac-ci.yml
#	.github/workflows/reusable-sanitizers-ci.yml
#	.github/workflows/reusable-ubuntu-ci.yml
#	.github/workflows/reusable-windows-ci.yml
#	.github/workflows/ubuntu-ci.yml
mergify bot pushed a commit that referenced this pull request Jun 17, 2024
* Refs #20822: Split current job into different jobs

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

* Refs #20822: Add shapes demo build job

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

* Refs #20822: Add discovery server build test job

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

* Refs #20822: Add python build test job

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

* Refs #20822: Add documentation build test job

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

* Refs #20822: Remove previous documentation workflow

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

* Refs #20822: Avoid building with tests if not running tests

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

* Refs #20822: Try to fix documentation job

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

* Refs #20822: Move conditions to reusable workflows

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

* Refs #20822: Apply internal review suggestions

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

* Refs #20822: Refactor meta files into build and test, and combine them in test steps

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

* Refs #20822: Remove matrix from ubuntu CI

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

* Revert "Refs #20822: Move conditions to reusable workflows"

This reverts commit d4eea0d.

* Refs #20822: Apply rev suggestions

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

* Refs #20822: Add missing discovery server requirements in the job

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

* Refs #20822: Apply rev suggestions (2)

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

* Refs #20822: Fixing windows workflow by merging metas

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

* Refs #20822: Add step with alternative builds

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

* Refs #20822: Use colcon test meta in all jobs, and fix xfail filter

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

* Refs #20822: Remove redundant CMake args, and fix alternative builds

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

* Refs #20822: Apply rev suggestions (3)

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

* Refs #20822: Explicitly compare boolean inputs

Signed-off-by: eduponz <eduardoponz@eprosima.com>

* Refs #20822: Install tools on test steps to avoid losing symlink permissions

Signed-off-by: eduponz <eduardoponz@eprosima.com>

* Refs #20822: Update discovery_server meta file name

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

* Refs #20822: Adjust ternary operators

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

* Refs #20822: Use CCache based on an input parameter

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

---------

Signed-off-by: JesusPoderoso <jesuspoderoso@eprosima.com>
Signed-off-by: eduponz <eduardoponz@eprosima.com>
Co-authored-by: eduponz <eduardoponz@eprosima.com>
(cherry picked from commit 22338f3)

# Conflicts:
#	.github/workflows/config/documentation.meta
#	.github/workflows/config/fastdds_build.meta
#	.github/workflows/documentation-tests.yaml
#	.github/workflows/nightly-ubuntu-ci.yml
#	.github/workflows/reusable-mac-ci.yml
#	.github/workflows/reusable-sanitizers-ci.yml
#	.github/workflows/reusable-ubuntu-ci.yml
#	.github/workflows/reusable-windows-ci.yml
#	.github/workflows/ubuntu-ci.yml
mergify bot pushed a commit that referenced this pull request Jun 17, 2024
* Refs #20822: Split current job into different jobs

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

* Refs #20822: Add shapes demo build job

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

* Refs #20822: Add discovery server build test job

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

* Refs #20822: Add python build test job

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

* Refs #20822: Add documentation build test job

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

* Refs #20822: Remove previous documentation workflow

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

* Refs #20822: Avoid building with tests if not running tests

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

* Refs #20822: Try to fix documentation job

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

* Refs #20822: Move conditions to reusable workflows

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

* Refs #20822: Apply internal review suggestions

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

* Refs #20822: Refactor meta files into build and test, and combine them in test steps

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

* Refs #20822: Remove matrix from ubuntu CI

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

* Revert "Refs #20822: Move conditions to reusable workflows"

This reverts commit d4eea0d.

* Refs #20822: Apply rev suggestions

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

* Refs #20822: Add missing discovery server requirements in the job

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

* Refs #20822: Apply rev suggestions (2)

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

* Refs #20822: Fixing windows workflow by merging metas

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

* Refs #20822: Add step with alternative builds

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

* Refs #20822: Use colcon test meta in all jobs, and fix xfail filter

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

* Refs #20822: Remove redundant CMake args, and fix alternative builds

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

* Refs #20822: Apply rev suggestions (3)

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

* Refs #20822: Explicitly compare boolean inputs

Signed-off-by: eduponz <eduardoponz@eprosima.com>

* Refs #20822: Install tools on test steps to avoid losing symlink permissions

Signed-off-by: eduponz <eduardoponz@eprosima.com>

* Refs #20822: Update discovery_server meta file name

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

* Refs #20822: Adjust ternary operators

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

* Refs #20822: Use CCache based on an input parameter

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

---------

Signed-off-by: JesusPoderoso <jesuspoderoso@eprosima.com>
Signed-off-by: eduponz <eduardoponz@eprosima.com>
Co-authored-by: eduponz <eduardoponz@eprosima.com>
(cherry picked from commit 22338f3)

# Conflicts:
#	.github/workflows/config/documentation.meta
#	.github/workflows/config/fastdds_build.meta
#	.github/workflows/documentation-tests.yaml
#	.github/workflows/nightly-ubuntu-ci.yml
#	.github/workflows/reusable-mac-ci.yml
#	.github/workflows/reusable-sanitizers-ci.yml
#	.github/workflows/reusable-ubuntu-ci.yml
#	.github/workflows/reusable-windows-ci.yml
#	.github/workflows/ubuntu-ci.yml
mergify bot pushed a commit that referenced this pull request Jun 17, 2024
* Refs #20822: Split current job into different jobs

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

* Refs #20822: Add shapes demo build job

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

* Refs #20822: Add discovery server build test job

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

* Refs #20822: Add python build test job

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

* Refs #20822: Add documentation build test job

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

* Refs #20822: Remove previous documentation workflow

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

* Refs #20822: Avoid building with tests if not running tests

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

* Refs #20822: Try to fix documentation job

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

* Refs #20822: Move conditions to reusable workflows

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

* Refs #20822: Apply internal review suggestions

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

* Refs #20822: Refactor meta files into build and test, and combine them in test steps

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

* Refs #20822: Remove matrix from ubuntu CI

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

* Revert "Refs #20822: Move conditions to reusable workflows"

This reverts commit d4eea0d.

* Refs #20822: Apply rev suggestions

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

* Refs #20822: Add missing discovery server requirements in the job

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

* Refs #20822: Apply rev suggestions (2)

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

* Refs #20822: Fixing windows workflow by merging metas

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

* Refs #20822: Add step with alternative builds

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

* Refs #20822: Use colcon test meta in all jobs, and fix xfail filter

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

* Refs #20822: Remove redundant CMake args, and fix alternative builds

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

* Refs #20822: Apply rev suggestions (3)

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

* Refs #20822: Explicitly compare boolean inputs

Signed-off-by: eduponz <eduardoponz@eprosima.com>

* Refs #20822: Install tools on test steps to avoid losing symlink permissions

Signed-off-by: eduponz <eduardoponz@eprosima.com>

* Refs #20822: Update discovery_server meta file name

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

* Refs #20822: Adjust ternary operators

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

* Refs #20822: Use CCache based on an input parameter

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

---------

Signed-off-by: JesusPoderoso <jesuspoderoso@eprosima.com>
Signed-off-by: eduponz <eduardoponz@eprosima.com>
Co-authored-by: eduponz <eduardoponz@eprosima.com>
(cherry picked from commit 22338f3)

# Conflicts:
#	.github/workflows/config/documentation.meta
#	.github/workflows/config/fastdds_build.meta
#	.github/workflows/documentation-tests.yaml
#	.github/workflows/nightly-ubuntu-ci.yml
#	.github/workflows/reusable-mac-ci.yml
#	.github/workflows/reusable-sanitizers-ci.yml
#	.github/workflows/reusable-ubuntu-ci.yml
#	.github/workflows/reusable-windows-ci.yml
#	.github/workflows/ubuntu-ci.yml
JesusPoderoso added a commit that referenced this pull request Jun 17, 2024
* Refs #20822: Split current job into different jobs

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

* Refs #20822: Add shapes demo build job

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

* Refs #20822: Add discovery server build test job

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

* Refs #20822: Add python build test job

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

* Refs #20822: Add documentation build test job

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

* Refs #20822: Remove previous documentation workflow

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

* Refs #20822: Avoid building with tests if not running tests

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

* Refs #20822: Try to fix documentation job

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

* Refs #20822: Move conditions to reusable workflows

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

* Refs #20822: Apply internal review suggestions

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

* Refs #20822: Refactor meta files into build and test, and combine them in test steps

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

* Refs #20822: Remove matrix from ubuntu CI

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

* Revert "Refs #20822: Move conditions to reusable workflows"

This reverts commit d4eea0d.

* Refs #20822: Apply rev suggestions

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

* Refs #20822: Add missing discovery server requirements in the job

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

* Refs #20822: Apply rev suggestions (2)

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

* Refs #20822: Fixing windows workflow by merging metas

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

* Refs #20822: Add step with alternative builds

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

* Refs #20822: Use colcon test meta in all jobs, and fix xfail filter

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

* Refs #20822: Remove redundant CMake args, and fix alternative builds

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

* Refs #20822: Apply rev suggestions (3)

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

* Refs #20822: Explicitly compare boolean inputs

Signed-off-by: eduponz <eduardoponz@eprosima.com>

* Refs #20822: Install tools on test steps to avoid losing symlink permissions

Signed-off-by: eduponz <eduardoponz@eprosima.com>

* Refs #20822: Update discovery_server meta file name

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

* Refs #20822: Adjust ternary operators

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

* Refs #20822: Use CCache based on an input parameter

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

---------

Signed-off-by: JesusPoderoso <jesuspoderoso@eprosima.com>
Signed-off-by: eduponz <eduardoponz@eprosima.com>
Co-authored-by: eduponz <eduardoponz@eprosima.com>
(cherry picked from commit 22338f3)
JesusPoderoso added a commit that referenced this pull request Jun 17, 2024
* Refs #20822: Split current job into different jobs

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

* Refs #20822: Add shapes demo build job

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

* Refs #20822: Add discovery server build test job

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

* Refs #20822: Add python build test job

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

* Refs #20822: Add documentation build test job

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

* Refs #20822: Remove previous documentation workflow

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

* Refs #20822: Avoid building with tests if not running tests

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

* Refs #20822: Try to fix documentation job

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

* Refs #20822: Move conditions to reusable workflows

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

* Refs #20822: Apply internal review suggestions

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

* Refs #20822: Refactor meta files into build and test, and combine them in test steps

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

* Refs #20822: Remove matrix from ubuntu CI

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

* Revert "Refs #20822: Move conditions to reusable workflows"

This reverts commit d4eea0d.

* Refs #20822: Apply rev suggestions

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

* Refs #20822: Add missing discovery server requirements in the job

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

* Refs #20822: Apply rev suggestions (2)

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

* Refs #20822: Fixing windows workflow by merging metas

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

* Refs #20822: Add step with alternative builds

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

* Refs #20822: Use colcon test meta in all jobs, and fix xfail filter

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

* Refs #20822: Remove redundant CMake args, and fix alternative builds

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

* Refs #20822: Apply rev suggestions (3)

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

* Refs #20822: Explicitly compare boolean inputs

Signed-off-by: eduponz <eduardoponz@eprosima.com>

* Refs #20822: Install tools on test steps to avoid losing symlink permissions

Signed-off-by: eduponz <eduardoponz@eprosima.com>

* Refs #20822: Update discovery_server meta file name

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

* Refs #20822: Adjust ternary operators

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

* Refs #20822: Use CCache based on an input parameter

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

---------

Signed-off-by: JesusPoderoso <jesuspoderoso@eprosima.com>
Signed-off-by: eduponz <eduardoponz@eprosima.com>
Co-authored-by: eduponz <eduardoponz@eprosima.com>
(cherry picked from commit 22338f3)
JesusPoderoso added a commit that referenced this pull request Jun 17, 2024
* Refs #20822: Split current job into different jobs

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

* Refs #20822: Add shapes demo build job

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

* Refs #20822: Add discovery server build test job

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

* Refs #20822: Add python build test job

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

* Refs #20822: Add documentation build test job

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

* Refs #20822: Remove previous documentation workflow

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

* Refs #20822: Avoid building with tests if not running tests

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

* Refs #20822: Try to fix documentation job

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

* Refs #20822: Move conditions to reusable workflows

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

* Refs #20822: Apply internal review suggestions

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

* Refs #20822: Refactor meta files into build and test, and combine them in test steps

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

* Refs #20822: Remove matrix from ubuntu CI

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

* Revert "Refs #20822: Move conditions to reusable workflows"

This reverts commit d4eea0d.

* Refs #20822: Apply rev suggestions

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

* Refs #20822: Add missing discovery server requirements in the job

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

* Refs #20822: Apply rev suggestions (2)

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

* Refs #20822: Fixing windows workflow by merging metas

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

* Refs #20822: Add step with alternative builds

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

* Refs #20822: Use colcon test meta in all jobs, and fix xfail filter

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

* Refs #20822: Remove redundant CMake args, and fix alternative builds

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

* Refs #20822: Apply rev suggestions (3)

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

* Refs #20822: Explicitly compare boolean inputs

Signed-off-by: eduponz <eduardoponz@eprosima.com>

* Refs #20822: Install tools on test steps to avoid losing symlink permissions

Signed-off-by: eduponz <eduardoponz@eprosima.com>

* Refs #20822: Update discovery_server meta file name

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

* Refs #20822: Adjust ternary operators

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

* Refs #20822: Use CCache based on an input parameter

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

---------

Signed-off-by: JesusPoderoso <jesuspoderoso@eprosima.com>
Signed-off-by: eduponz <eduardoponz@eprosima.com>
Co-authored-by: eduponz <eduardoponz@eprosima.com>
(cherry picked from commit 22338f3)
JesusPoderoso added a commit that referenced this pull request Jun 17, 2024
* Refs #20822: Split current job into different jobs

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

* Refs #20822: Add shapes demo build job

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

* Refs #20822: Add discovery server build test job

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

* Refs #20822: Add python build test job

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

* Refs #20822: Add documentation build test job

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

* Refs #20822: Remove previous documentation workflow

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

* Refs #20822: Avoid building with tests if not running tests

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

* Refs #20822: Try to fix documentation job

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

* Refs #20822: Move conditions to reusable workflows

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

* Refs #20822: Apply internal review suggestions

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

* Refs #20822: Refactor meta files into build and test, and combine them in test steps

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

* Refs #20822: Remove matrix from ubuntu CI

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

* Revert "Refs #20822: Move conditions to reusable workflows"

This reverts commit d4eea0d.

* Refs #20822: Apply rev suggestions

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

* Refs #20822: Add missing discovery server requirements in the job

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

* Refs #20822: Apply rev suggestions (2)

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

* Refs #20822: Fixing windows workflow by merging metas

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

* Refs #20822: Add step with alternative builds

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

* Refs #20822: Use colcon test meta in all jobs, and fix xfail filter

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

* Refs #20822: Remove redundant CMake args, and fix alternative builds

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

* Refs #20822: Apply rev suggestions (3)

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

* Refs #20822: Explicitly compare boolean inputs

Signed-off-by: eduponz <eduardoponz@eprosima.com>

* Refs #20822: Install tools on test steps to avoid losing symlink permissions

Signed-off-by: eduponz <eduardoponz@eprosima.com>

* Refs #20822: Update discovery_server meta file name

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

* Refs #20822: Adjust ternary operators

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

* Refs #20822: Use CCache based on an input parameter

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

---------

Signed-off-by: JesusPoderoso <jesuspoderoso@eprosima.com>
Signed-off-by: eduponz <eduardoponz@eprosima.com>
Co-authored-by: eduponz <eduardoponz@eprosima.com>
(cherry picked from commit 22338f3)
JesusPoderoso added a commit that referenced this pull request Jun 17, 2024
* Refs #20822: Split current job into different jobs

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

* Refs #20822: Add shapes demo build job

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

* Refs #20822: Add discovery server build test job

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

* Refs #20822: Add python build test job

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

* Refs #20822: Add documentation build test job

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

* Refs #20822: Remove previous documentation workflow

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

* Refs #20822: Avoid building with tests if not running tests

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

* Refs #20822: Try to fix documentation job

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

* Refs #20822: Move conditions to reusable workflows

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

* Refs #20822: Apply internal review suggestions

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

* Refs #20822: Refactor meta files into build and test, and combine them in test steps

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

* Refs #20822: Remove matrix from ubuntu CI

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

* Revert "Refs #20822: Move conditions to reusable workflows"

This reverts commit d4eea0d.

* Refs #20822: Apply rev suggestions

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

* Refs #20822: Add missing discovery server requirements in the job

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

* Refs #20822: Apply rev suggestions (2)

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

* Refs #20822: Fixing windows workflow by merging metas

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

* Refs #20822: Add step with alternative builds

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

* Refs #20822: Use colcon test meta in all jobs, and fix xfail filter

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

* Refs #20822: Remove redundant CMake args, and fix alternative builds

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

* Refs #20822: Apply rev suggestions (3)

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

* Refs #20822: Explicitly compare boolean inputs

Signed-off-by: eduponz <eduardoponz@eprosima.com>

* Refs #20822: Install tools on test steps to avoid losing symlink permissions

Signed-off-by: eduponz <eduardoponz@eprosima.com>

* Refs #20822: Update discovery_server meta file name

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

* Refs #20822: Adjust ternary operators

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

* Refs #20822: Use CCache based on an input parameter

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

---------

Signed-off-by: JesusPoderoso <jesuspoderoso@eprosima.com>
Signed-off-by: eduponz <eduardoponz@eprosima.com>
Co-authored-by: eduponz <eduardoponz@eprosima.com>
(cherry picked from commit 22338f3)
JesusPoderoso added a commit that referenced this pull request Jun 17, 2024
* Refs #20822: Split current job into different jobs

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

* Refs #20822: Add shapes demo build job

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

* Refs #20822: Add discovery server build test job

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

* Refs #20822: Add python build test job

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

* Refs #20822: Add documentation build test job

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

* Refs #20822: Remove previous documentation workflow

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

* Refs #20822: Avoid building with tests if not running tests

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

* Refs #20822: Try to fix documentation job

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

* Refs #20822: Move conditions to reusable workflows

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

* Refs #20822: Apply internal review suggestions

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

* Refs #20822: Refactor meta files into build and test, and combine them in test steps

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

* Refs #20822: Remove matrix from ubuntu CI

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

* Revert "Refs #20822: Move conditions to reusable workflows"

This reverts commit d4eea0d.

* Refs #20822: Apply rev suggestions

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

* Refs #20822: Add missing discovery server requirements in the job

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

* Refs #20822: Apply rev suggestions (2)

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

* Refs #20822: Fixing windows workflow by merging metas

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

* Refs #20822: Add step with alternative builds

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

* Refs #20822: Use colcon test meta in all jobs, and fix xfail filter

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

* Refs #20822: Remove redundant CMake args, and fix alternative builds

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

* Refs #20822: Apply rev suggestions (3)

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

* Refs #20822: Explicitly compare boolean inputs

Signed-off-by: eduponz <eduardoponz@eprosima.com>

* Refs #20822: Install tools on test steps to avoid losing symlink permissions

Signed-off-by: eduponz <eduardoponz@eprosima.com>

* Refs #20822: Update discovery_server meta file name

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

* Refs #20822: Adjust ternary operators

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

* Refs #20822: Use CCache based on an input parameter

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

---------

Signed-off-by: JesusPoderoso <jesuspoderoso@eprosima.com>
Signed-off-by: eduponz <eduardoponz@eprosima.com>
Co-authored-by: eduponz <eduardoponz@eprosima.com>
(cherry picked from commit 22338f3)
JesusPoderoso added a commit that referenced this pull request Jun 17, 2024
* Refs #20822: Split current job into different jobs

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

* Refs #20822: Add shapes demo build job

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

* Refs #20822: Add discovery server build test job

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

* Refs #20822: Add python build test job

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

* Refs #20822: Add documentation build test job

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

* Refs #20822: Remove previous documentation workflow

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

* Refs #20822: Avoid building with tests if not running tests

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

* Refs #20822: Try to fix documentation job

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

* Refs #20822: Move conditions to reusable workflows

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

* Refs #20822: Apply internal review suggestions

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

* Refs #20822: Refactor meta files into build and test, and combine them in test steps

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

* Refs #20822: Remove matrix from ubuntu CI

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

* Revert "Refs #20822: Move conditions to reusable workflows"

This reverts commit d4eea0d.

* Refs #20822: Apply rev suggestions

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

* Refs #20822: Add missing discovery server requirements in the job

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

* Refs #20822: Apply rev suggestions (2)

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

* Refs #20822: Fixing windows workflow by merging metas

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

* Refs #20822: Add step with alternative builds

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

* Refs #20822: Use colcon test meta in all jobs, and fix xfail filter

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

* Refs #20822: Remove redundant CMake args, and fix alternative builds

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

* Refs #20822: Apply rev suggestions (3)

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

* Refs #20822: Explicitly compare boolean inputs

Signed-off-by: eduponz <eduardoponz@eprosima.com>

* Refs #20822: Install tools on test steps to avoid losing symlink permissions

Signed-off-by: eduponz <eduardoponz@eprosima.com>

* Refs #20822: Update discovery_server meta file name

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

* Refs #20822: Adjust ternary operators

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

* Refs #20822: Use CCache based on an input parameter

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

---------

Signed-off-by: JesusPoderoso <jesuspoderoso@eprosima.com>
Signed-off-by: eduponz <eduardoponz@eprosima.com>
Co-authored-by: eduponz <eduardoponz@eprosima.com>
(cherry picked from commit 22338f3)
EduPonz pushed a commit that referenced this pull request Jun 18, 2024
* Refs #20822: Split current job into different jobs

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

* Refs #20822: Add shapes demo build job

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

* Refs #20822: Add discovery server build test job

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

* Refs #20822: Add python build test job

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

* Refs #20822: Add documentation build test job

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

* Refs #20822: Remove previous documentation workflow

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

* Refs #20822: Avoid building with tests if not running tests

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

* Refs #20822: Try to fix documentation job

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

* Refs #20822: Move conditions to reusable workflows

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

* Refs #20822: Apply internal review suggestions

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

* Refs #20822: Refactor meta files into build and test, and combine them in test steps

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

* Refs #20822: Remove matrix from ubuntu CI

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

* Revert "Refs #20822: Move conditions to reusable workflows"

This reverts commit d4eea0d.

* Refs #20822: Apply rev suggestions

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

* Refs #20822: Add missing discovery server requirements in the job

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

* Refs #20822: Apply rev suggestions (2)

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

* Refs #20822: Fixing windows workflow by merging metas

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

* Refs #20822: Add step with alternative builds

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

* Refs #20822: Use colcon test meta in all jobs, and fix xfail filter

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

* Refs #20822: Remove redundant CMake args, and fix alternative builds

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

* Refs #20822: Apply rev suggestions (3)

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

* Refs #20822: Explicitly compare boolean inputs

Signed-off-by: eduponz <eduardoponz@eprosima.com>

* Refs #20822: Install tools on test steps to avoid losing symlink permissions

Signed-off-by: eduponz <eduardoponz@eprosima.com>

* Refs #20822: Update discovery_server meta file name

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

* Refs #20822: Adjust ternary operators

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

* Refs #20822: Use CCache based on an input parameter

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

---------

Signed-off-by: JesusPoderoso <jesuspoderoso@eprosima.com>
Signed-off-by: eduponz <eduardoponz@eprosima.com>
Co-authored-by: eduponz <eduardoponz@eprosima.com>
(cherry picked from commit 22338f3)

Co-authored-by: Jesús Poderoso <120394830+JesusPoderoso@users.noreply.github.com>
JesusPoderoso added a commit that referenced this pull request Jun 18, 2024
* Refs #20822: Split current job into different jobs

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

* Refs #20822: Add shapes demo build job

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

* Refs #20822: Add discovery server build test job

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

* Refs #20822: Add python build test job

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

* Refs #20822: Add documentation build test job

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

* Refs #20822: Remove previous documentation workflow

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

* Refs #20822: Avoid building with tests if not running tests

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

* Refs #20822: Try to fix documentation job

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

* Refs #20822: Move conditions to reusable workflows

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

* Refs #20822: Apply internal review suggestions

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

* Refs #20822: Refactor meta files into build and test, and combine them in test steps

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

* Refs #20822: Remove matrix from ubuntu CI

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

* Revert "Refs #20822: Move conditions to reusable workflows"

This reverts commit d4eea0d.

* Refs #20822: Apply rev suggestions

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

* Refs #20822: Add missing discovery server requirements in the job

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

* Refs #20822: Apply rev suggestions (2)

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

* Refs #20822: Fixing windows workflow by merging metas

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

* Refs #20822: Add step with alternative builds

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

* Refs #20822: Use colcon test meta in all jobs, and fix xfail filter

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

* Refs #20822: Remove redundant CMake args, and fix alternative builds

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

* Refs #20822: Apply rev suggestions (3)

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

* Refs #20822: Explicitly compare boolean inputs

Signed-off-by: eduponz <eduardoponz@eprosima.com>

* Refs #20822: Install tools on test steps to avoid losing symlink permissions

Signed-off-by: eduponz <eduardoponz@eprosima.com>

* Refs #20822: Update discovery_server meta file name

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

* Refs #20822: Adjust ternary operators

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

* Refs #20822: Use CCache based on an input parameter

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

---------

Signed-off-by: JesusPoderoso <jesuspoderoso@eprosima.com>
Signed-off-by: eduponz <eduardoponz@eprosima.com>
Co-authored-by: eduponz <eduardoponz@eprosima.com>
(cherry picked from commit 22338f3)
JesusPoderoso added a commit that referenced this pull request Jun 18, 2024
* Refs #20822: Split current job into different jobs

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

* Refs #20822: Add shapes demo build job

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

* Refs #20822: Add discovery server build test job

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

* Refs #20822: Add python build test job

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

* Refs #20822: Add documentation build test job

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

* Refs #20822: Remove previous documentation workflow

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

* Refs #20822: Avoid building with tests if not running tests

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

* Refs #20822: Try to fix documentation job

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

* Refs #20822: Move conditions to reusable workflows

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

* Refs #20822: Apply internal review suggestions

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

* Refs #20822: Refactor meta files into build and test, and combine them in test steps

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

* Refs #20822: Remove matrix from ubuntu CI

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

* Revert "Refs #20822: Move conditions to reusable workflows"

This reverts commit d4eea0d.

* Refs #20822: Apply rev suggestions

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

* Refs #20822: Add missing discovery server requirements in the job

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

* Refs #20822: Apply rev suggestions (2)

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

* Refs #20822: Fixing windows workflow by merging metas

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

* Refs #20822: Add step with alternative builds

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

* Refs #20822: Use colcon test meta in all jobs, and fix xfail filter

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

* Refs #20822: Remove redundant CMake args, and fix alternative builds

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

* Refs #20822: Apply rev suggestions (3)

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

* Refs #20822: Explicitly compare boolean inputs

Signed-off-by: eduponz <eduardoponz@eprosima.com>

* Refs #20822: Install tools on test steps to avoid losing symlink permissions

Signed-off-by: eduponz <eduardoponz@eprosima.com>

* Refs #20822: Update discovery_server meta file name

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

* Refs #20822: Adjust ternary operators

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

* Refs #20822: Use CCache based on an input parameter

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

---------

Signed-off-by: JesusPoderoso <jesuspoderoso@eprosima.com>
Signed-off-by: eduponz <eduardoponz@eprosima.com>
Co-authored-by: eduponz <eduardoponz@eprosima.com>
(cherry picked from commit 22338f3)
EduPonz pushed a commit that referenced this pull request Jun 18, 2024
* Refs #20822: Split current job into different jobs

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

* Refs #20822: Add shapes demo build job

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

* Refs #20822: Add discovery server build test job

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

* Refs #20822: Add python build test job

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

* Refs #20822: Add documentation build test job

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

* Refs #20822: Remove previous documentation workflow

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

* Refs #20822: Avoid building with tests if not running tests

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

* Refs #20822: Try to fix documentation job

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

* Refs #20822: Move conditions to reusable workflows

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

* Refs #20822: Apply internal review suggestions

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

* Refs #20822: Refactor meta files into build and test, and combine them in test steps

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

* Refs #20822: Remove matrix from ubuntu CI

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

* Revert "Refs #20822: Move conditions to reusable workflows"

This reverts commit d4eea0d.

* Refs #20822: Apply rev suggestions

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

* Refs #20822: Add missing discovery server requirements in the job

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

* Refs #20822: Apply rev suggestions (2)

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

* Refs #20822: Fixing windows workflow by merging metas

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

* Refs #20822: Add step with alternative builds

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

* Refs #20822: Use colcon test meta in all jobs, and fix xfail filter

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

* Refs #20822: Remove redundant CMake args, and fix alternative builds

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

* Refs #20822: Apply rev suggestions (3)

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

* Refs #20822: Explicitly compare boolean inputs

Signed-off-by: eduponz <eduardoponz@eprosima.com>

* Refs #20822: Install tools on test steps to avoid losing symlink permissions

Signed-off-by: eduponz <eduardoponz@eprosima.com>

* Refs #20822: Update discovery_server meta file name

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

* Refs #20822: Adjust ternary operators

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

* Refs #20822: Use CCache based on an input parameter

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

---------

Signed-off-by: JesusPoderoso <jesuspoderoso@eprosima.com>
Signed-off-by: eduponz <eduardoponz@eprosima.com>
Co-authored-by: eduponz <eduardoponz@eprosima.com>
(cherry picked from commit 22338f3)

Co-authored-by: Jesús Poderoso <120394830+JesusPoderoso@users.noreply.github.com>
EduPonz pushed a commit that referenced this pull request Jun 18, 2024
* Refs #20822: Split current job into different jobs

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

* Refs #20822: Add shapes demo build job

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

* Refs #20822: Add discovery server build test job

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

* Refs #20822: Add python build test job

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

* Refs #20822: Add documentation build test job

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

* Refs #20822: Remove previous documentation workflow

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

* Refs #20822: Avoid building with tests if not running tests

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

* Refs #20822: Try to fix documentation job

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

* Refs #20822: Move conditions to reusable workflows

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

* Refs #20822: Apply internal review suggestions

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

* Refs #20822: Refactor meta files into build and test, and combine them in test steps

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

* Refs #20822: Remove matrix from ubuntu CI

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

* Revert "Refs #20822: Move conditions to reusable workflows"

This reverts commit d4eea0d.

* Refs #20822: Apply rev suggestions

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

* Refs #20822: Add missing discovery server requirements in the job

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

* Refs #20822: Apply rev suggestions (2)

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

* Refs #20822: Fixing windows workflow by merging metas

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

* Refs #20822: Add step with alternative builds

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

* Refs #20822: Use colcon test meta in all jobs, and fix xfail filter

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

* Refs #20822: Remove redundant CMake args, and fix alternative builds

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

* Refs #20822: Apply rev suggestions (3)

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

* Refs #20822: Explicitly compare boolean inputs

Signed-off-by: eduponz <eduardoponz@eprosima.com>

* Refs #20822: Install tools on test steps to avoid losing symlink permissions

Signed-off-by: eduponz <eduardoponz@eprosima.com>

* Refs #20822: Update discovery_server meta file name

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

* Refs #20822: Adjust ternary operators

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

* Refs #20822: Use CCache based on an input parameter

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

---------

Signed-off-by: JesusPoderoso <jesuspoderoso@eprosima.com>
Signed-off-by: eduponz <eduardoponz@eprosima.com>
Co-authored-by: eduponz <eduardoponz@eprosima.com>
(cherry picked from commit 22338f3)

Co-authored-by: Jesús Poderoso <120394830+JesusPoderoso@users.noreply.github.com>
EduPonz pushed a commit that referenced this pull request Jun 19, 2024
* Refactor Fast DDS Ubuntu CI to include several tests (#4860)

* Refs #20822: Split current job into different jobs

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

* Refs #20822: Add shapes demo build job

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

* Refs #20822: Add discovery server build test job

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

* Refs #20822: Add python build test job

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

* Refs #20822: Add documentation build test job

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

* Refs #20822: Remove previous documentation workflow

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

* Refs #20822: Avoid building with tests if not running tests

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

* Refs #20822: Try to fix documentation job

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

* Refs #20822: Move conditions to reusable workflows

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

* Refs #20822: Apply internal review suggestions

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

* Refs #20822: Refactor meta files into build and test, and combine them in test steps

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

* Refs #20822: Remove matrix from ubuntu CI

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

* Revert "Refs #20822: Move conditions to reusable workflows"

This reverts commit d4eea0d.

* Refs #20822: Apply rev suggestions

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

* Refs #20822: Add missing discovery server requirements in the job

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

* Refs #20822: Apply rev suggestions (2)

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

* Refs #20822: Fixing windows workflow by merging metas

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

* Refs #20822: Add step with alternative builds

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

* Refs #20822: Use colcon test meta in all jobs, and fix xfail filter

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

* Refs #20822: Remove redundant CMake args, and fix alternative builds

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

* Refs #20822: Apply rev suggestions (3)

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

* Refs #20822: Explicitly compare boolean inputs

Signed-off-by: eduponz <eduardoponz@eprosima.com>

* Refs #20822: Install tools on test steps to avoid losing symlink permissions

Signed-off-by: eduponz <eduardoponz@eprosima.com>

* Refs #20822: Update discovery_server meta file name

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

* Refs #20822: Adjust ternary operators

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

* Refs #20822: Use CCache based on an input parameter

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

---------

Signed-off-by: JesusPoderoso <jesuspoderoso@eprosima.com>
Signed-off-by: eduponz <eduardoponz@eprosima.com>
Co-authored-by: eduponz <eduardoponz@eprosima.com>
(cherry picked from commit 22338f3)

* Refs #20822: Apply rev suggestion

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

* Refs #20822: Remove python tests and shared libs off build

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

---------

Signed-off-by: JesusPoderoso <jesuspoderoso@eprosima.com>
Co-authored-by: Jesús Poderoso <120394830+JesusPoderoso@users.noreply.github.com>
Co-authored-by: JesusPoderoso <jesuspoderoso@eprosima.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
ci-pending PR which CI is running
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants