Skip to content

[23253] Split windows workflow in different test suites#5849

Merged
MiguelCompany merged 6 commits intomasterfrom
ci/23253
Jun 17, 2025
Merged

[23253] Split windows workflow in different test suites#5849
MiguelCompany merged 6 commits intomasterfrom
ci/23253

Conversation

@Mario-DL
Copy link
Copy Markdown
Contributor

@Mario-DL Mario-DL commented Jun 3, 2025

Description

This PR splits the windows reusable workflow in building and testing. In turn, testing was divided in three jobs:

  • blackbox: including BlackboxTests|ParticipantTests|SecureDiscoverServer|SimpleCommunication|system.
  • examples: including example_tests
  • unittests-I: including DDSSQLFilterValue. This test suite runs in parallel
  • unittests-II: excluding the rest of the former ones

Mind that the fastdds_build step does not build with -DFASTDDS_EXAMPLE_TESTS , instead it build with with DFASTDDS_COMPILE_EXAMPLES since generating the docker image for testing takes too long and it has to be regenerated again in the fastdds_test proper job again.

@Mergifyio backport 2.14.x

Contributor Checklist

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

Reviewer Checklist

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

@Mario-DL Mario-DL added this to the v3.2.3 milestone Jun 3, 2025
@Mario-DL Mario-DL requested review from richiprosima and removed request for richiprosima June 3, 2025 14:56
@github-actions github-actions bot added the ci-pending PR which CI is running label Jun 3, 2025
@Mario-DL Mario-DL requested review from richiprosima and removed request for richiprosima June 3, 2025 15:01
@Mario-DL Mario-DL modified the milestones: v3.2.3, v3.3.0 Jun 4, 2025
@rsanchez15 rsanchez15 self-requested a review June 10, 2025 05:54
Mario-DL added 3 commits June 11, 2025 09:01
Signed-off-by: Mario Dominguez <mariodominguez@eprosima.com>
Signed-off-by: Mario Dominguez <mariodominguez@eprosima.com>
Signed-off-by: Mario Dominguez <mariodominguez@eprosima.com>
@Mario-DL Mario-DL requested review from rsanchez15 and removed request for rsanchez15 June 11, 2025 07:07
@Mario-DL
Copy link
Copy Markdown
Contributor Author

Mario-DL commented Jun 11, 2025

For testing the nightly i did another branch ci/windows/nightly-test changing the uses: eProsima/Fast-DDS/.github/workflows/reusable-windows-ci.yml@master to @this_branch

Signed-off-by: Mario Dominguez <mariodominguez@eprosima.com>
@Mario-DL Mario-DL requested review from rsanchez15 and removed request for rsanchez15 June 16, 2025 07:48
Signed-off-by: Mario Dominguez <mariodominguez@eprosima.com>
@Mario-DL Mario-DL requested review from rsanchez15 and removed request for rsanchez15 June 17, 2025 06:12
Signed-off-by: Mario Dominguez <mariodominguez@eprosima.com>
@Mario-DL Mario-DL requested review from rsanchez15 and removed request for rsanchez15 June 17, 2025 06:32
@Mario-DL
Copy link
Copy Markdown
Contributor Author

@MiguelCompany MiguelCompany merged commit 74d3677 into master Jun 17, 2025
8 of 21 checks passed
@MiguelCompany MiguelCompany deleted the ci/23253 branch June 17, 2025 10:43
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.

2 participants