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

Remove old and unused Rolling CI jobs. #300

Merged
merged 2 commits into from
Apr 16, 2024
Merged

Conversation

clalancette
Copy link

Neither of these jobs have built in a very long time, and at least the drake one is still targeting focal (which Rolling hasn't supported since early 2022). Remove both of the jobs from our configuration.

This should stop configuring the jobs https://build.ros2.org/view/Rci/job/Rci__clang-tidy_ubuntu_jammy_amd64/ and https://build.ros2.org/view/Rci/job/Rci__drake-ros-underlay_ubuntu_focal_amd64/ (which we may have to remove by hand if we accept this PR).

Note that I'd also like to remove the following jobs:

But there are no config files on the ros2 branch for these. I believe that these were deployed from the https://github.com/ros2/ros_buildfarm_config/tree/nuclearsandwich/ikos-builds/rolling and https://github.com/ros2/ros_buildfarm_config/tree/nuclearsandwich/spaceros-build/rolling jobs.

@cottsay @nuclearsandwich FYI

Neither of these jobs have built in a very long time,
and at least the drake one is still targeting focal
(which Rolling hasn't supported since early 2022).
Remove both of the jobs from our configuration.

Signed-off-by: Chris Lalancette <[email protected]>
Copy link
Member

@cottsay cottsay left a comment

Choose a reason for hiding this comment

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

These need to be removed from the index.yaml as well.

Signed-off-by: Chris Lalancette <[email protected]>
@clalancette
Copy link
Author

These need to be removed from the index.yaml as well.

Ah, good call. Done in 51b309d.

@clalancette clalancette merged commit 2ab473d into ros2 Apr 16, 2024
3 checks passed
@delete-merged-branch delete-merged-branch bot deleted the clalancette/remove-old-jobs branch April 16, 2024 20:11
@nuclearsandwich
Copy link
Member

This should stop configuring the jobs https://build.ros2.org/view/Rci/job/Rci__clang-tidy_ubuntu_jammy_amd64/ and https://build.ros2.org/view/Rci/job/Rci__drake-ros-underlay_ubuntu_focal_amd64/ (which we may have to remove by hand if we accept this PR).

These do indeed need to be removed by hand. Now that this is merged I've done so.

I've also removed the branch-deployed Space ROS related jobs:

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants