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

Default config does not listen on tcp/0.0.0.0:7447 by default #243

Merged
merged 1 commit into from
Sep 12, 2024

Conversation

tomkimsour
Copy link
Contributor

When passing the default config file as a parameter, the endpoints are set as an empty list. c.f. endpoints: Unique([]),

2024-09-04T15:27:20.655073Z  INFO main ThreadId(01) zenoh_bridge_ros2dds: zenoh-bridge-ros2dds v1.0.0-beta.1
2024-09-04T15:27:20.655399Z  INFO main ThreadId(01) zenoh_bridge_ros2dds: Zenoh Config { id: d663c01664fae7878345e99d59b0fb5f, metadata: Null, mode: Some(Router), connect: ConnectConfig { timeout_ms: None, endpoints: Unique([]), exit_on_failure: None, retry: None }, listen: ListenConfig { timeout_ms: None, endpoints: Unique([]), exit_on_failure: None, retry: None }, scouting: ScoutingConf { timeout: None, delay: None, multicast: ScoutingMulticastConf { enabled: None, address: None, interface: None, ttl: None, autoconnect: None, listen: None }, gossip: GossipConf { enabled: None, multihop: None, autoconnect: None } }, timestamping: TimestampingConf { enabled: Some(Unique(true)), drop_future_timestamp: None }, queries_default_timeout: None, routing: RoutingConf { router: RouterRoutingConf { peers_failover_brokering: None }, peer: PeerRoutingConf { mode: None } }, aggregation: AggregationConf { subscribers: [], publishers: [] }, transport: TransportConf { unicast: TransportUnicastConf { accept_timeout: 10000, accept_pending: 100, max_sessions: 1000, max_links: 1, lowlatency: false, qos: QoSUnicastConf { enabled: true }, compression: CompressionUnicastConf { enabled: false } }, multicast: TransportMulticastConf { join_interval: Some(2500), max_sessions: Some(1000), qos: QoSMulticastConf { enabled: false }, compression: CompressionMulticastConf { enabled: false } }, link: TransportLinkConf { protocols: None, tx: LinkTxConf { sequence_number_resolution: U32, lease: 10000, keep_alive: 4, batch_size: 65535, queue: QueueConf { size: QueueSizeConf { control: 1, real_time: 1, interactive_high: 1, interactive_low: 1, data_high: 2, data: 4, data_low: 2, background: 1 }, congestion_control: CongestionControlConf { wait_before_drop: 1000 }, batching: BatchingConf { enabled: true, time_limit: 1 } }, threads: 4 }, rx: LinkRxConf { buffer_size: 65535, max_message_size: 1073741824 }, tls: TLSConf { root_ca_certificate: None, server_private_key: None, server_certificate: None, client_auth: None, client_private_key: None, client_certificate: None, server_name_verification: None, root_ca_certificate_base64: None, server_private_key_base64: None, server_certificate_base64: None, client_private_key_base64: None, client_certificate_base64: None }, unixpipe: UnixPipeConf { file_access_mask: None } }, shared_memory: ShmConf { enabled: true }, auth: AuthConf { usrpwd: UsrPwdConf { user: None, password: None, dictionary_file: None }, pubkey: PubKeyConf { public_key_pem: None, private_key_pem: None, public_key_file: None, private_key_file: None, key_size: None, known_keys_file: None } } }, adminspace: AdminSpaceConf { enabled: true, permissions: PermissionsConf { read: true, write: false } }, downsampling: [], access_control: AclConfig { enabled: false, default_permission: Deny, rules: None, subjects: None, policies: None }, plugins_loading: PluginsLoading { enabled: true, search_dirs: LibSearchDirs([Spec(LibSearchSpec { kind: CurrentExeParent, value: None }), Path("."), Path("~/.zenoh/lib"), Path("/opt/homebrew/lib"), Path("/usr/local/lib"), Path("/usr/lib")]) }, plugins: Object {"ros2dds": Object {"domain": Number(59), "ros_localhost_only": Bool(true)}} }

Default value is not applied if given an empty array
Copy link

github-actions bot commented Sep 4, 2024

PR missing one of the required labels: {'enhancement', 'internal', 'dependencies', 'bug', 'new feature', 'documentation', 'breaking-change'}

@tomkimsour tomkimsour changed the title Default config does not listen by default on tcp/0.0.0.0:7447 Default config does not listen on tcp/0.0.0.0:7447 by default Sep 4, 2024
@JEnoch JEnoch added the documentation Improvements or additions to documentation label Sep 11, 2024
@JEnoch
Copy link
Member

JEnoch commented Sep 11, 2024

Thanks for spotting and fixing this.
However we can't approve it and merge it until you sign the Eclipse Contribution Agreement. You need to create an Eclipse account for this. And please make sure you use the same email address you use on GitHub.

@tomkimsour
Copy link
Contributor Author

Done

@JEnoch JEnoch enabled auto-merge (squash) September 12, 2024 10:07
@JEnoch JEnoch merged commit 6de1ee2 into eclipse-zenoh:main Sep 12, 2024
6 of 7 checks passed
@tomkimsour tomkimsour deleted the patch-1 branch September 12, 2024 12:11
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
documentation Improvements or additions to documentation
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants