Skip to content

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

Merged
EduPonz merged 26 commits intomasterfrom
ci/python_tests
Jun 17, 2024
Merged

[20822] Refactor Fast DDS Ubuntu CI to include several tests#4860
EduPonz merged 26 commits intomasterfrom
ci/python_tests

Conversation

@JesusPoderoso
Copy link
Copy Markdown
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 EduPonz and richiprosima 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 and others added 21 commits June 17, 2024 11:24
Signed-off-by: JesusPoderoso <jesuspoderoso@eprosima.com>
Signed-off-by: JesusPoderoso <jesuspoderoso@eprosima.com>
Signed-off-by: JesusPoderoso <jesuspoderoso@eprosima.com>
Signed-off-by: JesusPoderoso <jesuspoderoso@eprosima.com>
Signed-off-by: JesusPoderoso <jesuspoderoso@eprosima.com>
…m in test steps

Signed-off-by: JesusPoderoso <jesuspoderoso@eprosima.com>
Signed-off-by: JesusPoderoso <jesuspoderoso@eprosima.com>
Signed-off-by: JesusPoderoso <jesuspoderoso@eprosima.com>
Signed-off-by: JesusPoderoso <jesuspoderoso@eprosima.com>
Signed-off-by: JesusPoderoso <jesuspoderoso@eprosima.com>
Signed-off-by: JesusPoderoso <jesuspoderoso@eprosima.com>
Signed-off-by: JesusPoderoso <jesuspoderoso@eprosima.com>
Signed-off-by: JesusPoderoso <jesuspoderoso@eprosima.com>
Signed-off-by: JesusPoderoso <jesuspoderoso@eprosima.com>
Signed-off-by: JesusPoderoso <jesuspoderoso@eprosima.com>
Signed-off-by: eduponz <eduardoponz@eprosima.com>
…issions

Signed-off-by: eduponz <eduardoponz@eprosima.com>
Signed-off-by: JesusPoderoso <jesuspoderoso@eprosima.com>
Signed-off-by: JesusPoderoso <jesuspoderoso@eprosima.com>
Signed-off-by: JesusPoderoso <jesuspoderoso@eprosima.com>
@JesusPoderoso
Copy link
Copy Markdown
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
Copy Markdown

EduPonz commented Jun 17, 2024

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

@mergify
Copy link
Copy Markdown
Contributor

mergify bot commented Jun 17, 2024

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

✅ Backports have been created

Details

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