ros2 / ros2/ci

Add ignore zenoh to ci

Open Beginner friendly
#889 0 comments 0 reactions 0 assignees View on GitHub

Nobody has claimed this yet.

enhancement
Dominant language
Python
Stars
55
Forks
39
Avg merge
1d 13h
Merged PRs (30d)
7

Description

Description

It is possible to exclude the other standard rmws from the ci build but it is not possible to do this for zenoh it seems

Basically a checkbox for CI_USE_ZENOH that is in default on but can also be removed

Motivation

It is important that if there are issues with the current builds of zenoh that it does not restrict buildfarmers to test out the rest of the code base.

Design / Implementation Considerations

There are a couple of packages relying on rmw_zenoh_cpp so this will be needed to be added to main

        if 'rmw_zenoh_cpp' in args.ignore_rmw:
            blacklisted_package_names += [
                'zenoh_cpp_vendor',
                'rmw_zenoh_cpp',
                'test_rmw_zenoh_cpp',
                'zenoh_security_tools',
            ]
Additional Information

No response

Contributor guide

No contributing guide indexed for this repository

First steps

  1. Read the whole issue, then the project's contributing guide.
  2. Comment on the issue to say you are picking it up — it saves two people doing the same work.
  3. Fork the repository and make your change on a branch.
  4. Open a pull request that references the issue number.

Research direction

Start in the CI entry point that handles args.ignore_rmw and compare how other standard RMW implementations are excluded. Add zenoh to the existing ignore path, including zenoh_cpp_vendor, rmw_zenoh_cpp, test_rmw_zenoh_cpp, and zenoh_security_tools, while keeping it enabled by default. Done means CI_USE_ZENOH can be disabled without restricting builds of the remaining code.

Written by the indexing model from the issue text.

Assessment

Tech stack
python
Domain
ci-cd
Issue type
Feature
Difficulty
2/5
Estimated time
1-3 hours
Activity status
Quiet
Clarity
Mostly clear
Newbie friendliness
68/100

Get new issues in your inbox

A short digest of beginner-friendly GitHub issues.