Skip to content

Fix weekly CI - #58

Merged
David-LP99 merged 1 commit into
kiltedfrom
kilted_fix_weekly_ci
Sep 8, 2026
Merged

Fix weekly CI#58
David-LP99 merged 1 commit into
kiltedfrom
kilted_fix_weekly_ci

Conversation

@David-LP99

@David-LP99 David-LP99 commented Sep 8, 2026

Copy link
Copy Markdown

Description

This PR aims to fix the weekly CI, since it was cloning the repository of the default branch (the one that triggers it) instead of the correct one for each branch.

Main changes

  • Dropped branch: ${{ github.ref }} in the per-distro CIs, it is not needed, the branch reference is taken automatically
  • Dropped actions/checkout@v5 in reusable-ci.yml, it does nothing, since ros-tooling/action-ros-ci already imports the repository
  • Added ref: ${{ inputs.branch }} to ros-tooling/action-ros-ci so that it takes the correct branch in the weekly CI

@David-LP99
Fix weekly CI
d9832fb 
Signed-off-by: David Laseca Perez <davidlaseca@eprosima.com>
@mergify

mergify Bot commented Sep 8, 2026

Copy link
Copy Markdown

Tick the box to add this pull request to the merge queue (same as @mergifyio queue).

  • Queue this pull request

@David-LP99

David-LP99 commented Sep 8, 2026

Copy link
Copy Markdown
Author

I ran it manually from the PR branch and it worked:
https://github.com/micro-ROS/rcutils/actions/runs/34218393214

EugenioCollado
EugenioCollado approved these changes Sep 8, 2026
View reviewed changes

@EugenioCollado EugenioCollado left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM

@David-LP99
David-LP99 merged commit d7b5088 into kilted Sep 8, 2026
8 checks passed
@David-LP99
David-LP99 deleted the kilted_fix_weekly_ci branch September 8, 2026 14:20
@David-LP99

David-LP99 commented Sep 8, 2026

Copy link
Copy Markdown
Author

@mergify backport rolling lyrical jazzy humble

@mergify

mergify Bot commented Sep 8, 2026 •
edited
Loading

Copy link
Copy Markdown

backport rolling lyrical jazzy humble

✅ Backports have been created

Details

This was referenced Sep 8, 2026
Merged
Merged
Merged
Merged
David-LP99 added a commit that referenced this pull request Sep 8, 2026
@David-LP99 @pablogs9
@jamoralp @bjsowa @4ntn @Carlosespicur @jimmy-mcelwain
micro-ROS Kilted patch
ac7a8e0 
* micro-ROS changes over dashing

* Added and updated security directory (#1)

* RCUTILS_NO_FILESYSTEM and RCUTILS_AVOID_DYNAMIC_ALLOCATION (#2)
  - No filesystem options; default allocators write access
  - Avoid dynamic allocation and no filesystem on error handling
  - Error handling template; new allocator approach

* Add test_security_directory test from rcl (#3)

* Zephyr fixes (#4)

* CMake refactor (#5)

* Update approach (#6)
  - Remove target_compile_definitions and refactor flags install
  - Added RCUTILS_NO_FILESYSTEM on new functions

* Updates 17092020

* Fix atomics 64bits (#9)
  - Add hashing and lock pool

* Updates 09102020

* Release micro-ROS Foxy (#8)
  - Update filesystem; adjust logger level; cleaning

* Remove build warning (#10) - avoid not used warnings

* Reduce error handling static size (#14) (#15)

* Revert "Revert "Install headers to include/${PROJECT_NAME} (ros2#351)""

* Fix atomic 64 b description (#17) (#18)

* Add fork checker for humble

* Don't link dl library when not needed (#28)

* Revert "Set hints to find the python version we actually want. (ros2#451)"

* Fix struct tm init

* Add PRId64

* Don't export dl library when not used (backport #33) (#35)

* Fix test_error_handling_helpers target_link_libraries (#37)

* Update CI (#43)
  - Add CI and nightly for each distro; update git version; skip tests
  - Change nightly to weekly; change master to rolling in fork checker

* Add RCUTILS_NO_PROCESS_SUPPORT option to build without fork/exec/wait (#50)

* Update CI (backport #51) (#52)

* Fix weekly CI (#58)

Dropped during the rebase as superseded by upstream:
* Don't export dl library when not used (backport #33) (#35)

Co-authored-by: Pablo Garrido <pablogs9@gmail.com>
Co-authored-by: Jose Antonio Moral <joseantoniomoralparras@gmail.com>
Co-authored-by: Blazej Sowa <bsowa123@gmail.com>
Co-authored-by: Anton Casas <antoncasas@eprosima.com>
Co-authored-by: Carlos Espinoza Curto <148376273+Carlosespicur@users.noreply.github.com>
Co-authored-by: Jimmy McElwain <jimmy.mcelwain@motoman.com>
Signed-off-by: David Laseca <davidlaseca@eprosima.com>
David-LP99 added a commit that referenced this pull request Sep 8, 2026
@mergify @David-LP99
Fix weekly CI (backport #58) (#59)
253e8e7 
(cherry picked from commit d7b5088)

Signed-off-by: David Laseca Perez <davidlaseca@eprosima.com>
Co-authored-by: David Laseca Perez <davidlaseca@eprosima.com>
David-LP99 added a commit that referenced this pull request Sep 8, 2026
@mergify @David-LP99
Fix weekly CI (backport #58) (#60)
146e4b8 
(cherry picked from commit d7b5088)

Signed-off-by: David Laseca Perez <davidlaseca@eprosima.com>
Co-authored-by: David Laseca Perez <davidlaseca@eprosima.com>
David-LP99 added a commit that referenced this pull request Sep 8, 2026
@mergify @David-LP99
Fix weekly CI (backport #58) (#61)
88ab24c 
(cherry picked from commit d7b5088)

Signed-off-by: David Laseca Perez <davidlaseca@eprosima.com>
Co-authored-by: David Laseca Perez <davidlaseca@eprosima.com>
David-LP99 added a commit that referenced this pull request Sep 8, 2026
@mergify @David-LP99
Fix weekly CI (backport #58) (#62)
0c1de5b 
(cherry picked from commit d7b5088)

Signed-off-by: David Laseca Perez <davidlaseca@eprosima.com>
Co-authored-by: David Laseca Perez <davidlaseca@eprosima.com>
David-LP99 added a commit that referenced this pull request Sep 8, 2026
@David-LP99 @pablogs9
@jamoralp @bjsowa @4ntn @EugenioCollado @Carlosespicur @jimmy-mcelwain
micro-ROS Lyrical patch
73d448e 
* micro-ROS changes over dashing

* Added and updated security directory (#1)

* RCUTILS_NO_FILESYSTEM and RCUTILS_AVOID_DYNAMIC_ALLOCATION (#2)
  - No filesystem options; default allocators write access
  - Avoid dynamic allocation and no filesystem on error handling
  - Error handling template; new allocator approach

* Add test_security_directory test from rcl (#3)

* Zephyr fixes (#4)

* CMake refactor (#5)

* Update approach (#6)
  - Remove target_compile_definitions and refactor flags install
  - Added RCUTILS_NO_FILESYSTEM on new functions

* Updates 17092020

* Fix atomics 64bits (#9)
  - Add hashing and lock pool

* Updates 09102020

* Release micro-ROS Foxy (#8)
  - Update filesystem; adjust logger level; cleaning

* Remove build warning (#10) - avoid not used warnings

* Reduce error handling static size (#14) (#15)

* Revert "Revert "Install headers to include/${PROJECT_NAME} (ros2#351)""

* Fix atomic 64 b description (#17) (#18)

* Add fork checker for humble

* Don't link dl library when not needed (#28)

* Revert "Set hints to find the python version we actually want. (ros2#451)"

* Fix struct tm init

* Add PRId64

* Don't export dl library when not used (#33) (#34)

* Remove unnecessary folder (#42)

* Update CI (backport #43) (#46)
  - Add CI and nightly for each distro; update git version; skip tests
  - Change nightly to weekly; change master to rolling in fork checker

* Add RCUTILS_NO_PROCESS_SUPPORT option to build without fork/exec/wait (#50)

* Update CI (#51)

* Fix thread issues added upstream (backport #56) (#57)

* Fix weekly CI (backport #58) (#60)

Dropped during the rebase as superseded by upstream:
* Don't export dl library when not used (#33) (#34)

Co-authored-by: Pablo Garrido <pablogs9@gmail.com>
Co-authored-by: Jose Antonio Moral <joseantoniomoralparras@gmail.com>
Co-authored-by: Blazej Sowa <bsowa123@gmail.com>
Co-authored-by: Anton Casas <antoncasas@eprosima.com>
Co-authored-by: Eugenio Collado <eugeniocollado@eprosima.com>
Co-authored-by: Carlos Espinoza Curto <148376273+Carlosespicur@users.noreply.github.com>
Co-authored-by: Jimmy McElwain <jimmy.mcelwain@motoman.com>
Signed-off-by: David Laseca <davidlaseca@eprosima.com>
David-LP99 added a commit that referenced this pull request Sep 8, 2026
@David-LP99 @pablogs9
@jamoralp @bjsowa @4ntn @Carlosespicur
micro-ROS Jazzy patch
0a9367e 
* micro-ROS changes over dashing

* Added and updated security directory (#1)

* RCUTILS_NO_FILESYSTEM and RCUTILS_AVOID_DYNAMIC_ALLOCATION (#2)
  - No filesystem options; default allocators write access
  - Avoid dynamic allocation and no filesystem on error handling
  - Error handling template; new allocator approach

* Add test_security_directory test from rcl (#3)

* Zephyr fixes (#4)

* CMake refactor (#5)

* Update approach (#6)
  - Remove target_compile_definitions and refactor flags install
  - Added RCUTILS_NO_FILESYSTEM on new functions

* Updates 17092020

* Fix atomics 64bits (#9)
  - Add hashing and lock pool

* Updates 09102020

* Release micro-ROS Foxy (#8)
  - Update filesystem; adjust logger level; cleaning

* Remove build warning (#10) - avoid not used warnings

* Reduce error handling static size (#14) (#15)

* Revert "Revert "Install headers to include/${PROJECT_NAME} (ros2#351)""

* Fix atomic 64 b description (#17) (#18)

* Add fork checker for humble

* Don't link dl library when not needed (#28)

* Revert "Set hints to find the python version we actually want. (ros2#451)"

* Fix struct tm init

* Add PRId64

* Don't export dl library when not used (#33)

* Fix test_error_handling_helpers target_link_libraries (#37) (#39)

* Update CI (backport #43) (#45)
  - Add CI and nightly for each distro; update git version; skip tests
  - Change nightly to weekly; change master to rolling in fork checker

* Update CI (backport #51) (#53)

* Fix weekly CI (backport #58) (#61)

Co-authored-by: Pablo Garrido <pablogs9@gmail.com>
Co-authored-by: Jose Antonio Moral <joseantoniomoralparras@gmail.com>
Co-authored-by: Blazej Sowa <bsowa123@gmail.com>
Co-authored-by: Anton Casas <antoncasas@eprosima.com>
Co-authored-by: Carlos Espinoza Curto <148376273+Carlosespicur@users.noreply.github.com>
Signed-off-by: David Laseca <davidlaseca@eprosima.com>
David-LP99 added a commit that referenced this pull request Sep 8, 2026
@David-LP99 @pablogs9
@jamoralp @bjsowa @4ntn @EugenioCollado @Carlosespicur @jimmy-mcelwain
micro-ROS Rolling patch
f58160b 
* micro-ROS changes over dashing

* Added and updated security directory (#1)

* RCUTILS_NO_FILESYSTEM and RCUTILS_AVOID_DYNAMIC_ALLOCATION (#2)
  - No filesystem options; default allocators write access
  - Avoid dynamic allocation and no filesystem on error handling
  - Error handling template; new allocator approach

* Add test_security_directory test from rcl (#3)

* Zephyr fixes (#4)

* CMake refactor (#5)

* Update approach (#6)
  - Remove target_compile_definitions and refactor flags install
  - Added RCUTILS_NO_FILESYSTEM on new functions

* Updates 17092020

* Fix atomics 64bits (#9)
  - Add hashing and lock pool

* Updates 09102020

* Release micro-ROS Foxy (#8)
  - Update filesystem; adjust logger level; cleaning

* Remove build warning (#10) - avoid not used warnings

* Reduce error handling static size (#14) (#15)

* Revert "Revert "Install headers to include/${PROJECT_NAME} (ros2#351)""

* Fix atomic 64 b description (#17) (#18)

* Add fork checker for humble

* Don't link dl library when not needed (#28)

* Revert "Set hints to find the python version we actually want. (ros2#451)"

* Fix struct tm init

* Add PRId64

* Don't export dl library when not used (#33) (#34)

* Remove unnecessary folder (#42)

* Update CI (backport #43) (#46)
  - Add CI and nightly for each distro; update git version; skip tests
  - Change nightly to weekly; change master to rolling in fork checker

* Add RCUTILS_NO_PROCESS_SUPPORT option to build without fork/exec/wait (#50)

* Update CI (backport #51) (#54)

* Fix thread issues from upstream (#56)

* Fix weekly CI (backport #58) (#59)

Dropped during the rebase as superseded by upstream:
* Don't export dl library when not used (#33) (#34)

Co-authored-by: Pablo Garrido <pablogs9@gmail.com>
Co-authored-by: Jose Antonio Moral <joseantoniomoralparras@gmail.com>
Co-authored-by: Blazej Sowa <bsowa123@gmail.com>
Co-authored-by: Anton Casas <antoncasas@eprosima.com>
Co-authored-by: Eugenio Collado <eugeniocollado@eprosima.com>
Co-authored-by: Carlos Espinoza Curto <148376273+Carlosespicur@users.noreply.github.com>
Co-authored-by: Jimmy McElwain <jimmy.mcelwain@motoman.com>
Signed-off-by: David Laseca <davidlaseca@eprosima.com>
David-LP99 added a commit that referenced this pull request Sep 8, 2026
@David-LP99 @pablogs9
@jamoralp @bjsowa @4ntn @EugenioCollado @Carlosespicur
micro-ROS Humble patch
61ab3ff 
* micro-ROS changes over dashing

* Added and updated security directory (#1)

* RCUTILS_NO_FILESYSTEM and RCUTILS_AVOID_DYNAMIC_ALLOCATION (#2)
  - No filesystem options; default allocators write access
  - Avoid dynamic allocation and no filesystem on error handling
  - Error handling template; new allocator approach

* Add test_security_directory test from rcl (#3)

* Zephyr fixes (#4)

* CMake refactor (#5)

* Update approach (#6)
  - Remove target_compile_definitions and refactor flags install
  - Added RCUTILS_NO_FILESYSTEM on new functions

* Updates 17092020

* Fix atomics 64bits (#9)
  - Add hashing and lock pool

* Updates 09102020

* Release micro-ROS Foxy (#8)
  - Update filesystem; adjust logger level; cleaning

* Remove build warning (#10) - avoid not used warnings

* Reduce error handling static size (#14) (#15)

* Revert "Revert "Install headers to include/${PROJECT_NAME} (ros2#351)""

* Fix atomic 64 b description (#17) (#18)

* Add fork checker for humble

* Zephyr POSIX time support, cherry-picked from upstream since it was never
  backported to the ros2/rcutils humble branch
  - time_unix: add zephyr posix time (ros2#368)
  - time_unix: uses ZEPHYR_VERSION_CODE instead of KERNELVERSION (ros2#390)

* Don't link dl library when not needed (#28) (#30)

* Don't export dl library when not used (#33) (#36)

* Fix test_error_handling_helpers target_link_libraries (#37) (#38)

* Fix tm struct init (#41)

* Update CI (backport #43) (#47)
  - Add CI and nightly for each distro; update git version; skip tests
  - Change nightly to weekly; change master to rolling in fork checker

* Update CI (backport #51) (#55)

* Fix weekly CI (backport #58) (#62)

Co-authored-by: Pablo Garrido <pablogs9@gmail.com>
Co-authored-by: Jose Antonio Moral <joseantoniomoralparras@gmail.com>
Co-authored-by: Blazej Sowa <bsowa123@gmail.com>
Co-authored-by: Anton Casas <antoncasas@eprosima.com>
Co-authored-by: Eugenio Collado <eugeniocollado@eprosima.com>
Co-authored-by: Carlos Espinoza Curto <148376273+Carlosespicur@users.noreply.github.com>
Signed-off-by: David Laseca <davidlaseca@eprosima.com>
David-LP99 added a commit that referenced this pull request Sep 8, 2026
@David-LP99 @pablogs9
@jamoralp @bjsowa @4ntn @EugenioCollado @Carlosespicur
micro-ROS Humble patch
7cf6c18 
* micro-ROS changes over dashing

* Added and updated security directory (#1)

* RCUTILS_NO_FILESYSTEM and RCUTILS_AVOID_DYNAMIC_ALLOCATION (#2)
  - No filesystem options; default allocators write access
  - Avoid dynamic allocation and no filesystem on error handling
  - Error handling template; new allocator approach

* Add test_security_directory test from rcl (#3)

* Zephyr fixes (#4)

* CMake refactor (#5)

* Update approach (#6)
  - Remove target_compile_definitions and refactor flags install
  - Added RCUTILS_NO_FILESYSTEM on new functions

* Updates 17092020

* Fix atomics 64bits (#9)
  - Add hashing and lock pool

* Updates 09102020

* Release micro-ROS Foxy (#8)
  - Update filesystem; adjust logger level; cleaning

* Remove build warning (#10) - avoid not used warnings

* Reduce error handling static size (#14) (#15)

* Revert "Revert "Install headers to include/${PROJECT_NAME} (ros2#351)""

* Fix atomic 64 b description (#17) (#18)

* Add fork checker for humble

* Zephyr POSIX time support, cherry-picked from upstream since it was never
  backported to the ros2/rcutils humble branch
  - time_unix: add zephyr posix time (ros2#368)
  - time_unix: uses ZEPHYR_VERSION_CODE instead of KERNELVERSION (ros2#390)

* Don't link dl library when not needed (#28) (#30)

* Don't export dl library when not used (#33) (#36)

* Fix test_error_handling_helpers target_link_libraries (#37) (#38)

* Fix tm struct init (#41)

* Update CI (backport #43) (#47)
  - Add CI and nightly for each distro; update git version; skip tests
  - Change nightly to weekly; change master to rolling in fork checker

* Update CI (backport #51) (#55)

* Fix weekly CI (backport #58) (#62)

Co-authored-by: Pablo Garrido <pablogs9@gmail.com>
Co-authored-by: Jose Antonio Moral <joseantoniomoralparras@gmail.com>
Co-authored-by: Blazej Sowa <bsowa123@gmail.com>
Co-authored-by: Anton Casas <antoncasas@eprosima.com>
Co-authored-by: Eugenio Collado <eugeniocollado@eprosima.com>
Co-authored-by: Carlos Espinoza Curto <148376273+Carlosespicur@users.noreply.github.com>
Signed-off-by: David Laseca <davidlaseca@eprosima.com>
David-LP99 added a commit that referenced this pull request Sep 9, 2026
@David-LP99 @pablogs9
@jamoralp @bjsowa @4ntn @Carlosespicur @jimmy-mcelwain
micro-ROS Kilted patch
f40bb29 
* micro-ROS changes over dashing

* Added and updated security directory (#1)

* RCUTILS_NO_FILESYSTEM and RCUTILS_AVOID_DYNAMIC_ALLOCATION (#2)
  - No filesystem options; default allocators write access
  - Avoid dynamic allocation and no filesystem on error handling
  - Error handling template; new allocator approach

* Add test_security_directory test from rcl (#3)

* Zephyr fixes (#4)

* CMake refactor (#5)

* Update approach (#6)
  - Remove target_compile_definitions and refactor flags install
  - Added RCUTILS_NO_FILESYSTEM on new functions

* Updates 17092020

* Fix atomics 64bits (#9)
  - Add hashing and lock pool

* Updates 09102020

* Release micro-ROS Foxy (#8)
  - Update filesystem; adjust logger level; cleaning

* Remove build warning (#10) - avoid not used warnings

* Reduce error handling static size (#14) (#15)

* Revert "Revert "Install headers to include/${PROJECT_NAME} (ros2#351)""

* Fix atomic 64 b description (#17) (#18)

* Add fork checker for humble

* Don't link dl library when not needed (#28)

* Revert "Set hints to find the python version we actually want. (ros2#451)"

* Fix struct tm init

* Add PRId64

* Don't export dl library when not used (backport #33) (#35)

* Fix test_error_handling_helpers target_link_libraries (#37)

* Update CI (#43)
  - Add CI and nightly for each distro; update git version; skip tests
  - Change nightly to weekly; change master to rolling in fork checker

* Add RCUTILS_NO_PROCESS_SUPPORT option to build without fork/exec/wait (#50)

* Update CI (backport #51) (#52)

* Fix weekly CI (#58)

Dropped during the rebase as superseded by upstream:
* Don't export dl library when not used (backport #33) (#35)

Co-authored-by: Pablo Garrido <pablogs9@gmail.com>
Co-authored-by: Jose Antonio Moral <joseantoniomoralparras@gmail.com>
Co-authored-by: Blazej Sowa <bsowa123@gmail.com>
Co-authored-by: Anton Casas <antoncasas@eprosima.com>
Co-authored-by: Carlos Espinoza Curto <148376273+Carlosespicur@users.noreply.github.com>
Co-authored-by: Jimmy McElwain <jimmy.mcelwain@motoman.com>
Signed-off-by: David Laseca <davidlaseca@eprosima.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@EugenioCollado EugenioCollado EugenioCollado approved these changes

Assignees

No one assigned

Labels

None yet

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

2 participants

@David-LP99 @EugenioCollado