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

Improve queries_timeout configuration #19

Merged
merged 1 commit into from
Nov 23, 2023
Merged

Conversation

JEnoch
Copy link
Member

@JEnoch JEnoch commented Nov 23, 2023

Change queries_timeout configuration to allow configuration of timeouts for queries per ROS interface.
Different usage of Zenoh queries are involved:

  • a route for a TRANSIENT_LOCAL Subscriber querying historical publications from a Publisher served by another bridge
  • a route for a Service Client querying a Service Server
  • the 3 routes for an Action Client querying send_goal, cancel_goal and get_result from an Action Server

An example of the new configuration:

      ////
      //// queries_timeout: Timeouts configuration for various Zenoh queries.
      ////                  Each field is optional. If not set, A default value of 5.0 seconds applies.
      ////                  Each value can be either a float in seconds that will apply as a timeout to all queries,
      ////                  either a list of strings with format "<regex>=<float>" where:
      ////                      - "regex" is a regular expression matching an interface name
      ////                      - "float" is the timeout in seconds
      queries_timeout: {
        //// timeouts for TRANSIENT_LOCAL subscriber when querying publishers for historical publications
        transient_local_subscribers: 1.0,
        //// timeouts for Service clients calling a Service server
        services: ["add_two_ints=0.5", ".*=1.0"],
        //// timeouts for Action clients calling an Action server (send_goal, cancel_goal and get_result services)
        actions: {
          send_goal: 1.0,
          cancel_goal: 1.0,
          get_result: [".*long_mission=3600", ".*short_action=10.0"],
        }
      }

@JEnoch JEnoch merged commit 63382f2 into main Nov 23, 2023
6 checks passed
@JEnoch JEnoch deleted the query_timeouts_config branch November 30, 2023 16:06
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.

1 participant