Skip to content

add : get clients,servers info#154

Merged
ahcorde merged 3 commits intoros2:rollingfrom
leeminju531:add-clients-servers-info
Nov 17, 2025
Merged

add : get clients,servers info#154
ahcorde merged 3 commits intoros2:rollingfrom
leeminju531:add-clients-servers-info

Conversation

@leeminju531
Copy link
Copy Markdown
Contributor

@leeminju531 leeminju531 commented Jun 25, 2024

Add get clients, servers info

Refer to ros2/ros2cli#916

@leeminju531 leeminju531 mentioned this pull request Jun 25, 2024
Merged
@leeminju531
Copy link
Copy Markdown
Contributor Author

leeminju531 commented Jun 25, 2024

ref
rmw
rmw_implementation
rmw_fastrtps
rmw_cyclonedds
rmw_connextdds

This was referenced Jun 25, 2024
Merged
Merged
Merged
Merged
Merged
Merged
Merged
@leeminju531 leeminju531 force-pushed the add-clients-servers-info branch from 57bcea1 to 9a8fb37 Compare September 27, 2024 14:49
fgallegosalido
fgallegosalido approved these changes Nov 12, 2024
View reviewed changes
Copy link
Copy Markdown
Collaborator

@fgallegosalido fgallegosalido left a comment

Choose a reason for hiding this comment

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

Looks good to me. Some refactoring could have been done regarding the functions rmw_get_publishers_info_by_topic and rmw_get_subscriptions_info_by_topic, as these new ones are almost identical, but for the moments this looks good.

@leeminju531 leeminju531 force-pushed the add-clients-servers-info branch from 9a8fb37 to 68b96af Compare June 22, 2025 02:29
@ahcorde ahcorde mentioned this pull request Jul 21, 2025
Merged
@ahcorde
Copy link
Copy Markdown
Contributor

ahcorde commented Jul 21, 2025

Pulls: ros2/rmw#371, ros2/rmw_implementation#238, ros2/rcl#1161, ros2/rclcpp#2569, ros2/rclpy#1307, ros2/rmw_zenoh#679, ros2/rmw_fastrtps#771, ros2/rmw_cyclonedds#499, #154, ros2/ros2cli#916
Gist: https://gist.githubusercontent.com/ahcorde/302c3652a6220b1c87e62745e9f9afff/raw/742f65fffc9bfec58f851df6062324b9d1a85a3c/ros2.repos
BUILD args: --packages-above-and-dependencies rmw rmw_implementation rcl rclcpp rclpy ros2cli ros2service
TEST args: --packages-above rmw rmw_implementation rcl rclcpp rclpy ros2cli ros2service
ROS Distro: rolling
Job: ci_launcher
ci_launcher ran: https://ci.ros2.org/job/ci_launcher/16548

  • Linux Build Status
  • Linux-aarch64 Build Status
  • Linux-rhel Build Status
  • Windows Build Status

@fujitatomoya
Copy link
Copy Markdown
Collaborator

fujitatomoya commented Nov 6, 2025

@Mergifyio rebase

@leeminju531 @fujitatomoya
Add rmw_get_service_endpoint_info and update graph cache entity to su…
9087c87 
…pport service type hash

Signed-off-by: Minju, Lee <dlalswn531@naver.com>
@leeminju531 @fujitatomoya
Fix rmw_api_connextdds_count_services
d242434 
Signed-off-by: Minju, Lee <dlalswn531@naver.com>
@mergify
Copy link
Copy Markdown

mergify bot commented Nov 6, 2025

rebase

✅ Branch has been successfully rebased

@fujitatomoya fujitatomoya force-pushed the add-clients-servers-info branch from 68b96af to d242434 Compare November 6, 2025 09:59
@fujitatomoya
Copy link
Copy Markdown
Collaborator

fujitatomoya commented Nov 6, 2025

Pulls: ros2/ros2cli#916, ros2/rmw#371, ros2/rmw_implementation#238, ros2/rmw_fastrtps#771, ros2/rmw_cyclonedds#499, #154, ros2/rcl#1161, ros2/rclcpp#2569, ros2/rclpy#1307, ros2/rmw_dds_common#82, ros2/rmw_zenoh#679
Gist: https://gist.githubusercontent.com/fujitatomoya/01385d70469b48f205a68e21219fd395/raw/b02c84fd17938124e2b045dbf016f260e6209ae0/ros2.repos
BUILD args: --packages-above-and-dependencies rmw_zenoh_cpp rmw_dds_common rclpy rclcpp_lifecycle rclcpp rcl rmw_connextdds rmw_connextdds_common rmw_connextddsmicro rmw_cyclonedds_cpp rmw_fastrtps_cpp rmw_fastrtps_dynamic_cpp rmw_fastrtps_sharedcpp ros2cli ros2service rmw rmw_implementation test_rmw_implementation
TEST args: --packages-above rmw_zenoh_cpp rmw_dds_common rclpy rclcpp_lifecycle rclcpp rcl rmw_connextdds rmw_connextdds_common rmw_connextddsmicro rmw_cyclonedds_cpp rmw_fastrtps_cpp rmw_fastrtps_dynamic_cpp rmw_fastrtps_sharedcpp ros2cli ros2service rmw rmw_implementation test_rmw_implementation
ROS Distro: rolling
Job: ci_launcher
ci_launcher ran: https://ci.ros2.org/job/ci_launcher/17407

  • Linux Build Status
  • Linux-aarch64 Build Status
  • Linux-rhel Build Status
  • Windows Build Status

@fujitatomoya fujitatomoya mentioned this pull request Nov 6, 2025
Merged
@fujitatomoya
Copy link
Copy Markdown
Collaborator

fujitatomoya commented Nov 6, 2025

Pulls: ros2/ros2cli#916, ros2/rmw#371, ros2/rmw_implementation#238, ros2/rmw_fastrtps#771, ros2/rmw_cyclonedds#499, #154, ros2/rcl#1161, ros2/rclcpp#2569, ros2/rclpy#1307, ros2/rmw_dds_common#82, ros2/rmw_zenoh#679
Gist: https://gist.githubusercontent.com/fujitatomoya/3c50929d3b3e884ba6796f6c9f26e92e/raw/b02c84fd17938124e2b045dbf016f260e6209ae0/ros2.repos
BUILD args: --packages-above-and-dependencies rmw_zenoh_cpp rmw_dds_common rclpy rclcpp_lifecycle rclcpp rcl rmw_connextdds rmw_connextdds_common rmw_connextddsmicro rmw_cyclonedds_cpp rmw_fastrtps_cpp rmw_fastrtps_dynamic_cpp rmw_fastrtps_shared_cpp ros2cli ros2service rmw rmw_implementation test_rmw_implementation
TEST args: --packages-above rmw_zenoh_cpp rmw_dds_common rclpy rclcpp_lifecycle rclcpp rcl rmw_connextdds rmw_connextdds_common rmw_connextddsmicro rmw_cyclonedds_cpp rmw_fastrtps_cpp rmw_fastrtps_dynamic_cpp rmw_fastrtps_shared_cpp ros2cli ros2service rmw rmw_implementation test_rmw_implementation
ROS Distro: rolling
Job: ci_launcher
ci_launcher ran: https://ci.ros2.org/job/ci_launcher/17408

  • Linux Build Status
  • Linux-aarch64 Build Status
  • Linux-rhel Build Status
  • Windows Build Status

@leeminju531
Fix test failure
392dbc8 
Signed-off-by: Minju, Lee <dlalswn531@naver.com>
@ahcorde
Copy link
Copy Markdown
Contributor

ahcorde commented Nov 17, 2025

  • Linux Build Status
  • Linux-aarch64 Build Status
  • Linux-rhel Build Status
  • Windows Build Status

@ahcorde ahcorde merged commit 0fb02cf into ros2:rolling Nov 17, 2025
1 check passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@ahcorde ahcorde ahcorde approved these changes

@fgallegosalido fgallegosalido fgallegosalido approved these changes

@fujitatomoya fujitatomoya fujitatomoya approved these changes

Assignees

@fujitatomoya fujitatomoya

Labels

None yet

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

4 participants

@leeminju531 @ahcorde @fujitatomoya @fgallegosalido