rosotacom-dev 2.5.dev65__tar.gz → 2.5.dev66__tar.gz

{rosotacom_dev-2.5.dev65 → rosotacom_dev-2.5.dev66}/PKG-INFO

@@ -1,6 +1,6 @@
 Metadata-Version: 2.4
 Name: rosotacom-dev
-Version: 2.5.dev65
+Version: 2.5.dev66
 Summary: Host CLI for ROS OTA communication sessions.
 Author: develNor
 License-Expression: BSD-3-Clause

{rosotacom_dev-2.5.dev65 → rosotacom_dev-2.5.dev66}/docs/findings/README.md

@@ -32,6 +32,7 @@ living verification hook.
 - [A bridge that learns types from the graph cannot use a transport that does not carry the graph](graph-derived-types-block-a-non-graph-transport.md) - over `zenoh_ros2dds` the payload topic never arrived in either run: the bridge routes a topic once a local reader exists and the reader waited for the bridge, leaving only `Pending=1` in a log.
 - [A Fast DDS datagram cap stops every message larger than it](fastdds-datagram-cap-drops-large-samples.md) - copying CycloneDDS's 1200 B fragment size into the Fast DDS OTA profile makes every sample above the cap never arrive, at 1200 B and at 8192 B, sync and async — while the 84 B heartbeat keeps flowing at 10 Hz.
 - [A fragment sized exactly like the datagram cap stops every fragmented sample](cyclone-fragment-equal-to-max-drops-fragmented-samples.md) - CycloneDDS 11.0.1 delivers nothing above one fragment when `FragmentSize` equals `MaxMessageSize`, where 0.10.5 delivers everything on the identical profile; a 40 B string still crosses, and discovery, endpoint announcement and proxy-writer matching all succeed while no sample arrives.
+- [Two CycloneDDS configurations on one domain stop interoperating](one-domain-two-cyclone-configs-stop-interoperating.md) - on 11.0.1 a publisher and a subscriber on the same host and domain deliver nothing when one carries the local profile and the other the OTA profile, where 0.10.5 delivers in all four combinations; discovery, endpoint announcement and matching all succeed while no sample moves.
 - [A lost keyframe silences exactly the next gop_size−2 deltas](keyframe-loss-kills-the-next-deltas.md) - libavcodec emits nothing for them, the last delta decodes with artifacts, the next keyframe restores clean decode; a lost delta never blocks decode — so the display hole is gop_size × frame period, deterministically.
 - [Any transport element in a Fast DDS profile doubles every sample on the wire](a-transport-element-doubles-every-fastdds-sample.md) - 366 datagrams for 180 samples against 183 without it; `<builtinTransports>DEFAULT` is a no-op selection and doubles too, so the trigger is the element's presence — removing it takes the shipped OTA profile from 2.10x the payload to 1.04x at identical delivery.
 - [Lifespan is enforced by one DDS reader and not the other](lifespan-is-enforced-by-one-reader-and-not-the-other.md) - under the same 20% oversubscription both stacks queue alike (472 and 433 delivered at ~1.1-1.2 s); with `lifespan: 0.7` CycloneDDS is unchanged and delivers 9.1% inside the bound, while Fast DDS drops to 4 delivered and never exceeds 570 ms — the policy decides shown-vs-suppressed, not how long the wait is.

rosotacom_dev-2.5.dev66/docs/findings/one-domain-two-cyclone-configs-stop-interoperating.md

@@ -0,0 +1,94 @@
+# Two CycloneDDS Configurations on One Domain Stop Interoperating
+
+## Claim
+
+On CycloneDDS 11.0.1 two participants on the **same host and the same domain**
+exchange no data when they are given different configurations. On 0.10.5 they
+do. Discovery is unaffected in the failing combination — the topic appears in
+`ros2 topic list`, both sides announce their endpoints, the reader builds its
+proxy writers — and no sample is delivered.
+
+This is what a session relies on without saying so. A peer renders two whole
+configurations, a local one and an OTA one, and exports `CYCLONEDDS_URI` twice,
+so the second replaces the first: processes started after it view the *local*
+domain through the *OTA* profile while the relay and the application containers
+view it through the local one. Under 0.10.5 that mixture worked. Under 11.0.1
+the bridge subscribes on the local domain, receives nothing, forwards nothing
+into the OTA domain, and the far peer receives nothing — with every stage
+reporting itself healthy.
+
+## Setup
+
+- Host pair / topology: for the effect itself, one host is enough — two
+  containers from the same communication image, `--network host`, one ROS
+  domain, one publisher and one subscriber. The session it was found in runs
+  across two hosts joined by a point-to-point VPN.
+- Session: none for the isolated measurement; a `std_msgs/msg/String` at 2 Hz
+  and `ros2 topic echo … --once`. The two configurations are the ones a session
+  renders: `cyclonedds_local_participants.xml` and `cyclonedds_tuned.xml`.
+- rosotacom SHA: measured on the packaged 2.5.dev61 images; re-runs record
+  theirs.
+- Profile: none — no impairment; the effect needs none.
+- Seed policy: deterministic (no netem, no random load), `n=1` per cell; the
+  effect is present/absent, not a distribution.
+- Stacks: `ros-kilted-cyclonedds` 0.10.5-2noble against
+  `ros-lyrical-cyclonedds` 11.0.1-4resolute.
+
+## Evidence
+
+| publisher / subscriber | 0.10.5 | 11.0.1 |
+|---|---|---|
+| local / local | 12 B | 12 B |
+| local / **ota** | 12 B | **0** |
+| **ota** / local | 12 B | **0** |
+| ota / ota | 12 B | 12 B |
+
+Inside a failing session, one topic the relay publishes on the local domain:
+
+```
+/com/out/<peer>/can/twist/max10hz/ota_stamped, domain 47
+  subscriber with local_dds.xml : 512 B received
+  subscriber with ota_dds.xml   :   0 B
+domain 47 seen through local_dds.xml : 103 topics
+domain 47 seen through ota_dds.xml   :  29 topics
+```
+
+Ruled out on the way to it, each measured with the real images on the same
+hosts: cross-host delivery, same-host delivery under one profile,
+`com_msgs/msg/OtaStamped`, the container's IPC namespace, `domain_bridge`
+itself (minimal config, the session's full 27-topic config, with and without the
+OTA profile, with and without `ROS_DOMAIN_ID` set), participant count (0, 6 and
+12 extra participants), and adding the local profile's
+`ParticipantIndex`/`MaxAutoParticipantIndex` to the OTA profile.
+
+Verification: automated:
+`pytest tests/unit/test_ota_xml.py::test_shipped_scoped_template_carries_both_domains_and_only_ota_restrictions`
+holds the shape of the configuration that avoids the mixture — OTA restrictions
+scoped to the OTA domain, the local domain left with default discovery. Manual,
+for the effect itself: two containers on one host and one domain, one publisher
+and one subscriber, each given one of the two rendered profiles, all four
+combinations.
+
+## Status
+
+confirmed, 2026-08-20.
+
+Addressed by `cyclonedds_scoped.xml.template`, which carries both domains in one
+file with each section scoped by `Domain Id`, so one configuration can be shared
+by every process on the host. It is an additional template: the two-file
+arrangement is unchanged, and a session that does not name the new one behaves
+exactly as before.
+
+## Publication notes
+
+The reportable part is not "a DDS version changed". It is that a configuration
+which was *never* uniform became load-bearing without anybody choosing it: the
+mixture had been there all along and was harmless, so nothing recorded it as a
+property of the deployment. A measurement stack should state, beside the
+middleware version, whether every process on a domain was configured
+identically — because on one side of that version boundary the answer does not
+matter and on the other it decides whether any data moves.
+
+Second, it is another instance of the same trap as the datagram cap: a link that
+passes discovery, endpoint matching and a heartbeat check while carrying
+nothing. Neither check tests what the payload uses.

rosotacom_dev-2.5.dev66/src/rosotacom/resources/ws/ota_configs/cyclonedds_scoped.xml.template

@@ -0,0 +1,87 @@
+<?xml version="1.0" encoding="UTF-8"?>
+<!--
+One CycloneDDS configuration for both of a peer's domains, each section scoped
+to the domain it belongs to.
+
+Set it as `shared.rmw.ota.cyclone.config` and leave the local one unset: this
+file already carries the local domain's section, so nothing has to overwrite
+anything.
+
+Why it exists. The two-file arrangement renders a local profile and an OTA
+profile and exports CYCLONEDDS_URI twice, so the second replaces the first and
+every process started after it views the *local* domain through the *OTA*
+profile. On CycloneDDS 0.10.5 that was survivable: participants configured
+differently on one host and one domain still exchanged data. On 11.0.1 they do
+not — they discover each other, announce their endpoints, match, and deliver
+nothing. Measured on one host, one domain, a publisher and a subscriber:
+
+  publisher / subscriber     0.10.5     11.0.1
+  local     / local          12 B       12 B
+  local     / ota            12 B       0
+  ota       / local          12 B       0
+  ota       / ota            12 B       12 B
+
+A session whose relay publishes under one profile and whose bridge subscribes
+under the other therefore hands nothing from the local domain into the OTA
+domain, and the link carries nothing while every stage looks healthy
+(docs/findings/one-domain-two-cyclone-configs-stop-interoperating.md).
+
+Scoping fixes the cause rather than the symptom: the OTA settings apply to the
+OTA domain, the local domain keeps default discovery with room for many
+participants, and one file can be shared by every process on the host.
+
+The two-file arrangement stays exactly as it was — this is an additional
+template, not a replacement, and a session that does not name it is unchanged.
+-->
+<CycloneDDS xmlns="https://cdds.io/config">
+  <!--
+  The OTA domain: interface-pinned, unicast-only, with the datagram caps the
+  provisioned link is sized for. Identical to cyclonedds_tuned.xml except that
+  it says which domain it is for.
+  -->
+  <Domain Id="#ota_domain">
+    <General>
+      <AllowMulticast>false</AllowMulticast>
+      <Interfaces>
+        <NetworkInterface address="#host_ip"/>
+      </Interfaces>
+      <MulticastRecvNetworkInterfaceAddresses>none</MulticastRecvNetworkInterfaceAddresses>
+      <!-- Strictly below MaxMessageSize; see cyclonedds_tuned.xml for what
+           making them equal cost. -->
+      <FragmentSize>1024B</FragmentSize>
+      <MaxMessageSize>1200B</MaxMessageSize>
+      <MaxRexmitMessageSize>1200B</MaxRexmitMessageSize>
+    </General>
+
+    <Discovery>
+      <Peers>
+        <!--peer-block--><Peer Address="#peer"/><!--/peer-block-->
+      </Peers>
+      <EnableTopicDiscoveryEndpoints>false</EnableTopicDiscoveryEndpoints>
+      <LeaseDuration>60s</LeaseDuration>
+      <SPDPInterval>#spdp_interval</SPDPInterval>
+    </Discovery>
+
+    <Internal>
+      <SquashParticipants>true</SquashParticipants>
+    </Internal>
+
+    <Tracing>
+      <Verbosity>config</Verbosity>
+      <OutputFile>/tmp/cyclonedds.log</OutputFile>
+    </Tracing>
+  </Domain>
+
+  <!--
+  The local domain: default discovery, more participants. It pins no interface,
+  disables no multicast and sets no peer list, because intra-host discovery has
+  to keep working — the same reasoning as cyclonedds_local_participants.xml,
+  including why 119 and not higher.
+  -->
+  <Domain Id="#local_domain">
+    <Discovery>
+      <ParticipantIndex>auto</ParticipantIndex>
+      <MaxAutoParticipantIndex>119</MaxAutoParticipantIndex>
+    </Discovery>
+  </Domain>
+</CycloneDDS>

{rosotacom_dev-2.5.dev65 → rosotacom_dev-2.5.dev66}/src/rosotacom/resources/ws/ota_configs/get_ota_xml.py

@@ -15,8 +15,20 @@ import re
 import sys
 import tempfile
 
-
-KNOWN_PLACEHOLDERS = {"#host_ip", "#peer", "#easy_mode_ip", "#spdp_interval"}
+KNOWN_PLACEHOLDERS = {
+    "#host_ip",
+    "#peer",
+    "#easy_mode_ip",
+    "#spdp_interval",
+    #: A template that scopes its settings to one ROS domain resolves these.
+    #: Cyclone applies a <Domain> section to the domain its Id names, so a
+    #: single configuration can hold the OTA settings for the OTA domain and
+    #: leave the local domain alone — which is what lets every process on a
+    #: host share one configuration instead of half of them carrying the OTA
+    #: profile and half the local one.
+    "#local_domain",
+    "#ota_domain",
+}
 PLACEHOLDER_PATTERN = re.compile(r"#[A-Za-z_][A-Za-z0-9_]*")
 DEFAULT_SPDP_INTERVAL = "30s"
 
@@ -48,6 +60,19 @@ def _resolved_address(value: str, label: str) -> str:
     return address
 
 
+def _resolved_domain(value: str, label: str) -> str:
+    """A ROS domain id, validated rather than pasted into the XML.
+
+    An unusable Id is worse than a missing one: Cyclone keeps a <Domain>
+    section it cannot match to any domain, so the settings vanish without a
+    word and the link comes up on defaults.
+    """
+    domain = str(value).strip()
+    if not domain.isdigit() or not 0 <= int(domain) <= 232:
+        raise ValueError(f"{label} must be a ROS domain id between 0 and 232, got {value!r}.")
+    return str(int(domain))
+
+
 def _resolved_spdp_interval(value: str) -> str:
     interval = value.strip().lower()
     if not interval:
@@ -99,9 +124,7 @@ def _expand_peer_block(content: str, peer_ips: list) -> str:
 
     has_markers = PEER_BLOCK_OPEN in content
     if has_markers and (PEER_BLOCK_CLOSE not in content):
-        raise RuntimeError(
-            f"Template has '{PEER_BLOCK_OPEN}' without matching '{PEER_BLOCK_CLOSE}'."
-        )
+        raise RuntimeError(f"Template has '{PEER_BLOCK_OPEN}' without matching '{PEER_BLOCK_CLOSE}'.")
 
     if len(peer_ips) == 1 and not has_markers:
         return content.replace("#peer", peer_ips[0])
@@ -116,9 +139,7 @@ def _expand_peer_block(content: str, peer_ips: list) -> str:
 
     matches = list(PEER_BLOCK_PATTERN.finditer(content))
     if len(matches) != 1:
-        raise RuntimeError(
-            f"Template must contain exactly one peer-block region; found {len(matches)}."
-        )
+        raise RuntimeError(f"Template must contain exactly one peer-block region; found {len(matches)}.")
     m = matches[0]
     block = m.group(1)
     if "#peer" not in block:
@@ -128,11 +149,11 @@ def _expand_peer_block(content: str, peer_ips: list) -> str:
 
     # Preserve the indentation of the marker line so duplicated blocks align.
     line_start = content.rfind("\n", 0, m.start()) + 1
-    indent = content[line_start:m.start()]
+    indent = content[line_start : m.start()]
     sep = ("\n" + indent) if not indent.strip() else ""
 
     rendered = sep.join(block.replace("#peer", ip) for ip in peer_ips)
-    return content[:m.start()] + rendered + content[m.end():]
+    return content[: m.start()] + rendered + content[m.end() :]
 
 
 def main(
@@ -141,12 +162,14 @@ def main(
     peer: str = None,
     easy_mode_ip: str = None,
     spdp_interval: str = DEFAULT_SPDP_INTERVAL,
+    local_domain: str = None,
+    ota_domain: str = None,
 ) -> str:
     path = _template_path(config)
     if not os.path.exists(path):
         raise FileNotFoundError(f"Template not found: {path}")
 
-    with open(path, "r", encoding="utf-8") as f:
+    with open(path, encoding="utf-8") as f:
         content = f.read()
 
     found = set(PLACEHOLDER_PATTERN.findall(content))
@@ -159,9 +182,7 @@ def main(
 
     if "#host_ip" in found:
         if not host_ip:
-            raise RuntimeError(
-                f"Template '{os.path.basename(path)}' uses #host_ip but --host-ip was not provided."
-            )
+            raise RuntimeError(f"Template '{os.path.basename(path)}' uses #host_ip but --host-ip was not provided.")
         content = content.replace("#host_ip", _resolved_address(host_ip, "Host IP"))
 
     if "#easy_mode_ip" in found:
@@ -169,45 +190,60 @@ def main(
             raise RuntimeError(
                 f"Template '{os.path.basename(path)}' uses #easy_mode_ip but --easy-mode-ip was not provided."
             )
-        content = content.replace(
-            "#easy_mode_ip", _resolved_address(easy_mode_ip, "Easy Mode IP")
-        )
+        content = content.replace("#easy_mode_ip", _resolved_address(easy_mode_ip, "Easy Mode IP"))
 
     if "#spdp_interval" in found:
         content = content.replace("#spdp_interval", _resolved_spdp_interval(spdp_interval))
 
+    for placeholder, value, option in (
+        ("#local_domain", local_domain, "--local-domain"),
+        ("#ota_domain", ota_domain, "--ota-domain"),
+    ):
+        if placeholder in found:
+            if value is None or not str(value).strip():
+                raise RuntimeError(
+                    f"Template '{os.path.basename(path)}' uses {placeholder} but {option} was not provided."
+                )
+            content = content.replace(placeholder, _resolved_domain(value, placeholder))
+
     if "#peer" in found:
         if not peer:
-            raise RuntimeError(
-                f"Template '{os.path.basename(path)}' uses #peer but --peer was not provided."
-            )
+            raise RuntimeError(f"Template '{os.path.basename(path)}' uses #peer but --peer was not provided.")
         content = _expand_peer_block(content, _resolve_peer_ips(peer))
 
     return content
 
 
 if __name__ == "__main__":
-    parser = argparse.ArgumentParser(
-        description="Resolve placeholders in an OTA DDS XML template."
-    )
+    parser = argparse.ArgumentParser(description="Resolve placeholders in an OTA DDS XML template.")
     parser.add_argument(
-        "-c", "--config", required=True,
+        "-c",
+        "--config",
+        required=True,
         help="Template name (e.g. fastdds_v1.xml, cyclonedds.xml, fastdds_easy_mode.xml).",
     )
     parser.add_argument(
-        "-i", "--host-ip", dest="host_ip",
+        "-i",
+        "--host-ip",
+        dest="host_ip",
         help="Resolved local host address (#host_ip).",
     )
     parser.add_argument(
-        "-p", "--peer", dest="peer",
+        "-p",
+        "--peer",
+        dest="peer",
         help="Resolved peer address, or comma-separated addresses (#peer).",
     )
     parser.add_argument(
-        "-e", "--easy-mode-ip", dest="easy_mode_ip",
+        "-e",
+        "--easy-mode-ip",
+        dest="easy_mode_ip",
         help="Resolved Fast DDS Easy Mode address (#easy_mode_ip).",
     )
     parser.add_argument(
-        "-o", "--output", dest="output",
+        "-o",
+        "--output",
+        dest="output",
         help=(
             "Write the rendered XML to this path, atomically. Prefer this over a shell "
             "redirect: `> file` truncates the target before this script runs, so a "
@@ -216,6 +252,16 @@ if __name__ == "__main__":
             "comes up carrying nothing."
         ),
     )
+    parser.add_argument(
+        "--local-domain",
+        dest="local_domain",
+        help="Local ROS domain id, for a template that scopes a section to it (#local_domain).",
+    )
+    parser.add_argument(
+        "--ota-domain",
+        dest="ota_domain",
+        help="OTA ROS domain id, for a template that scopes a section to it (#ota_domain).",
+    )
     parser.add_argument(
         "--spdp-interval",
         dest="spdp_interval",
@@ -224,8 +270,7 @@ if __name__ == "__main__":
     )
     args = parser.parse_args()
     output = args.output
-    rendered = main(**{k: v for k, v in vars(args).items()
-                       if v is not None and k != "output"})
+    rendered = main(**{k: v for k, v in vars(args).items() if v is not None and k != "output"})
     if not output:
         print(rendered)
         raise SystemExit(0)

{rosotacom_dev-2.5.dev65 → rosotacom_dev-2.5.dev66}/src/rosotacom/resources/ws/session/content/base/session_plugin_base.yaml

@@ -292,7 +292,7 @@ common:
   before_commands:
     - if [ '${local_domain_id}' != "None" ] && [ -n '${local_domain_id}' ]; then export ROS_DOMAIN_ID='${local_domain_id}'; fi
     - if [ '${rmw_local}' != "None" ]; then case ${rmw_local} in cyclone) export RMW_IMPLEMENTATION=rmw_cyclonedds_cpp;; fastdds) export RMW_IMPLEMENTATION=rmw_fastrtps_cpp; export FASTDDS_BUILTIN_TRANSPORTS=${FASTDDS_BUILTIN_TRANSPORTS:-UDPv4};; zenoh) export RMW_IMPLEMENTATION=rmw_zenoh_cpp;; *) export RMW_IMPLEMENTATION='${rmw_local}';; esac; fi
-    - if [ '${local_config_template}' != "None" ] && [ -n '${local_config_template}' ]; then local_args=(--config "${local_config_template}" --host-ip "${ip_local}" --peer "${ip_remote}"); [ '${local_easy_mode_ip}' != "None" ] && [ -n '${local_easy_mode_ip}' ] && local_args+=(--easy-mode-ip "${local_easy_mode_ip}"); [ '${local_spdp_interval}' != "None" ] && [ -n '${local_spdp_interval}' ] && local_args+=(--spdp-interval "${local_spdp_interval}"); python3 /ws/ota_configs/get_ota_xml.py "${local_args[@]}" --output "${local_config_file}" || { echo "FATAL - could not render the local DDS profile ${local_config_template}; refusing to start on defaults." >&2; exit 1; }; case ${rmw_local} in cyclone) export CYCLONEDDS_URI="file://${local_config_file}";; fastdds) export FASTDDS_DEFAULT_PROFILES_FILE="${local_config_file}"; export RMW_FASTRTPS_USE_QOS_FROM_XML=1;; esac; fi
+    - if [ '${local_config_template}' != "None" ] && [ -n '${local_config_template}' ]; then local_args=(--config "${local_config_template}" --host-ip "${ip_local}" --peer "${ip_remote}"); [ '${local_domain_id}' != "None" ] && [ -n '${local_domain_id}' ] && local_args+=(--local-domain "${local_domain_id}"); [ '${ota_domain_id}' != "None" ] && [ -n '${ota_domain_id}' ] && local_args+=(--ota-domain "${ota_domain_id}"); [ '${local_easy_mode_ip}' != "None" ] && [ -n '${local_easy_mode_ip}' ] && local_args+=(--easy-mode-ip "${local_easy_mode_ip}"); [ '${local_spdp_interval}' != "None" ] && [ -n '${local_spdp_interval}' ] && local_args+=(--spdp-interval "${local_spdp_interval}"); python3 /ws/ota_configs/get_ota_xml.py "${local_args[@]}" --output "${local_config_file}" || { echo "FATAL - could not render the local DDS profile ${local_config_template}; refusing to start on defaults." >&2; exit 1; }; case ${rmw_local} in cyclone) export CYCLONEDDS_URI="file://${local_config_file}";; fastdds) export FASTDDS_DEFAULT_PROFILES_FILE="${local_config_file}"; export RMW_FASTRTPS_USE_QOS_FROM_XML=1;; esac; fi
     - if [ '${before_command}' != "None" ]; then ${before_command}; fi
 
 windows:
@@ -338,7 +338,7 @@ windows:
     if: use_domain_bridge
     splits:
       - commands:
-        - if [ '${ota_config_template}' != "None" ] && [ -n '${ota_config_template}' ]; then ota_args=(--config "${ota_config_template}" --host-ip "${ip_local}" --peer "${ip_remote}"); [ '${ota_easy_mode_ip}' != "None" ] && [ -n '${ota_easy_mode_ip}' ] && ota_args+=(--easy-mode-ip "${ota_easy_mode_ip}"); [ '${ota_spdp_interval}' != "None" ] && [ -n '${ota_spdp_interval}' ] && ota_args+=(--spdp-interval "${ota_spdp_interval}"); python3 /ws/ota_configs/get_ota_xml.py "${ota_args[@]}" --output "${ota_config_file}" || { echo "FATAL - could not render the OTA DDS profile ${ota_config_template}; refusing to start on defaults." >&2; exit 1; }; case ${rmw_ota} in cyclone) export CYCLONEDDS_URI="file://${ota_config_file}";; fastdds) export FASTDDS_DEFAULT_PROFILES_FILE="${ota_config_file}"; export RMW_FASTRTPS_USE_QOS_FROM_XML=1;; esac; fi
+        - if [ '${ota_config_template}' != "None" ] && [ -n '${ota_config_template}' ]; then ota_args=(--config "${ota_config_template}" --host-ip "${ip_local}" --peer "${ip_remote}"); [ '${local_domain_id}' != "None" ] && [ -n '${local_domain_id}' ] && ota_args+=(--local-domain "${local_domain_id}"); [ '${ota_domain_id}' != "None" ] && [ -n '${ota_domain_id}' ] && ota_args+=(--ota-domain "${ota_domain_id}"); [ '${ota_easy_mode_ip}' != "None" ] && [ -n '${ota_easy_mode_ip}' ] && ota_args+=(--easy-mode-ip "${ota_easy_mode_ip}"); [ '${ota_spdp_interval}' != "None" ] && [ -n '${ota_spdp_interval}' ] && ota_args+=(--spdp-interval "${ota_spdp_interval}"); python3 /ws/ota_configs/get_ota_xml.py "${ota_args[@]}" --output "${ota_config_file}" || { echo "FATAL - could not render the OTA DDS profile ${ota_config_template}; refusing to start on defaults." >&2; exit 1; }; case ${rmw_ota} in cyclone) export CYCLONEDDS_URI="file://${ota_config_file}";; fastdds) export FASTDDS_DEFAULT_PROFILES_FILE="${ota_config_file}"; export RMW_FASTRTPS_USE_QOS_FROM_XML=1;; esac; fi
         - ros2 run domain_bridge domain_bridge --wait-for-publisher false ${domain_bridge_config_file}
   - name: IN
     if: in
@@ -346,7 +346,7 @@ windows:
       - commands:
         - if [ '${ota_domain_id}' != "None" ] && [ -n '${ota_domain_id}' ]; then export ROS_DOMAIN_ID='${ota_domain_id}'; fi
         - case ${rmw_ota} in cyclone) export RMW_IMPLEMENTATION=rmw_cyclonedds_cpp;; fastdds) export RMW_IMPLEMENTATION=rmw_fastrtps_cpp; export FASTDDS_BUILTIN_TRANSPORTS=${FASTDDS_BUILTIN_TRANSPORTS:-UDPv4};; zenoh) export RMW_IMPLEMENTATION=rmw_zenoh_cpp;; esac
-        - if [ '${ota_config_template}' != "None" ] && [ -n '${ota_config_template}' ]; then ota_args=(--config "${ota_config_template}" --host-ip "${ip_local}" --peer "${ip_remote}"); [ '${ota_easy_mode_ip}' != "None" ] && [ -n '${ota_easy_mode_ip}' ] && ota_args+=(--easy-mode-ip "${ota_easy_mode_ip}"); [ '${ota_spdp_interval}' != "None" ] && [ -n '${ota_spdp_interval}' ] && ota_args+=(--spdp-interval "${ota_spdp_interval}"); python3 /ws/ota_configs/get_ota_xml.py "${ota_args[@]}" --output "${ota_config_file}" || { echo "FATAL - could not render the OTA DDS profile ${ota_config_template}; refusing to start on defaults." >&2; exit 1; }; case ${rmw_ota} in cyclone) export CYCLONEDDS_URI="file://${ota_config_file}";; fastdds) export FASTDDS_DEFAULT_PROFILES_FILE="${ota_config_file}"; export RMW_FASTRTPS_USE_QOS_FROM_XML=1;; esac; fi
+        - if [ '${ota_config_template}' != "None" ] && [ -n '${ota_config_template}' ]; then ota_args=(--config "${ota_config_template}" --host-ip "${ip_local}" --peer "${ip_remote}"); [ '${local_domain_id}' != "None" ] && [ -n '${local_domain_id}' ] && ota_args+=(--local-domain "${local_domain_id}"); [ '${ota_domain_id}' != "None" ] && [ -n '${ota_domain_id}' ] && ota_args+=(--ota-domain "${ota_domain_id}"); [ '${ota_easy_mode_ip}' != "None" ] && [ -n '${ota_easy_mode_ip}' ] && ota_args+=(--easy-mode-ip "${ota_easy_mode_ip}"); [ '${ota_spdp_interval}' != "None" ] && [ -n '${ota_spdp_interval}' ] && ota_args+=(--spdp-interval "${ota_spdp_interval}"); python3 /ws/ota_configs/get_ota_xml.py "${ota_args[@]}" --output "${ota_config_file}" || { echo "FATAL - could not render the OTA DDS profile ${ota_config_template}; refusing to start on defaults." >&2; exit 1; }; case ${rmw_ota} in cyclone) export CYCLONEDDS_URI="file://${ota_config_file}";; fastdds) export FASTDDS_DEFAULT_PROFILES_FILE="${ota_config_file}"; export RMW_FASTRTPS_USE_QOS_FROM_XML=1;; esac; fi
         # For zenoh_ros2dds the OTA domain is loopback-only (zenoh is the inter-host
         # transport), so confine discovery to localhost instead of statically peering
         # the remote host — otherwise the OTA-domain DDS double-delivers alongside zenoh.
@@ -373,7 +373,7 @@ windows:
       - commands:
         - if [ '${ota_domain_id}' != "None" ] && [ -n '${ota_domain_id}' ]; then export ROS_DOMAIN_ID='${ota_domain_id}'; fi
         - case ${rmw_ota} in cyclone) export RMW_IMPLEMENTATION=rmw_cyclonedds_cpp;; fastdds) export RMW_IMPLEMENTATION=rmw_fastrtps_cpp; export FASTDDS_BUILTIN_TRANSPORTS=${FASTDDS_BUILTIN_TRANSPORTS:-UDPv4};; zenoh) export RMW_IMPLEMENTATION=rmw_zenoh_cpp;; esac
-        - if [ '${ota_config_template}' != "None" ] && [ -n '${ota_config_template}' ]; then ota_args=(--config "${ota_config_template}" --host-ip "${ip_local}" --peer "${ip_remote}"); [ '${ota_easy_mode_ip}' != "None" ] && [ -n '${ota_easy_mode_ip}' ] && ota_args+=(--easy-mode-ip "${ota_easy_mode_ip}"); [ '${ota_spdp_interval}' != "None" ] && [ -n '${ota_spdp_interval}' ] && ota_args+=(--spdp-interval "${ota_spdp_interval}"); python3 /ws/ota_configs/get_ota_xml.py "${ota_args[@]}" --output "${ota_config_file}" || { echo "FATAL - could not render the OTA DDS profile ${ota_config_template}; refusing to start on defaults." >&2; exit 1; }; case ${rmw_ota} in cyclone) export CYCLONEDDS_URI="file://${ota_config_file}";; fastdds) export FASTDDS_DEFAULT_PROFILES_FILE="${ota_config_file}"; export RMW_FASTRTPS_USE_QOS_FROM_XML=1;; esac; fi
+        - if [ '${ota_config_template}' != "None" ] && [ -n '${ota_config_template}' ]; then ota_args=(--config "${ota_config_template}" --host-ip "${ip_local}" --peer "${ip_remote}"); [ '${local_domain_id}' != "None" ] && [ -n '${local_domain_id}' ] && ota_args+=(--local-domain "${local_domain_id}"); [ '${ota_domain_id}' != "None" ] && [ -n '${ota_domain_id}' ] && ota_args+=(--ota-domain "${ota_domain_id}"); [ '${ota_easy_mode_ip}' != "None" ] && [ -n '${ota_easy_mode_ip}' ] && ota_args+=(--easy-mode-ip "${ota_easy_mode_ip}"); [ '${ota_spdp_interval}' != "None" ] && [ -n '${ota_spdp_interval}' ] && ota_args+=(--spdp-interval "${ota_spdp_interval}"); python3 /ws/ota_configs/get_ota_xml.py "${ota_args[@]}" --output "${ota_config_file}" || { echo "FATAL - could not render the OTA DDS profile ${ota_config_template}; refusing to start on defaults." >&2; exit 1; }; case ${rmw_ota} in cyclone) export CYCLONEDDS_URI="file://${ota_config_file}";; fastdds) export FASTDDS_DEFAULT_PROFILES_FILE="${ota_config_file}"; export RMW_FASTRTPS_USE_QOS_FROM_XML=1;; esac; fi
         # For zenoh_ros2dds the OTA domain is loopback-only (zenoh is the inter-host
         # transport), so confine discovery to localhost instead of statically peering
         # the remote host — otherwise the OTA-domain DDS double-delivers alongside zenoh.

{rosotacom_dev-2.5.dev65 → rosotacom_dev-2.5.dev66}/src/rosotacom_dev.egg-info/PKG-INFO

@@ -1,6 +1,6 @@
 Metadata-Version: 2.4
 Name: rosotacom-dev
-Version: 2.5.dev65
+Version: 2.5.dev66
 Summary: Host CLI for ROS OTA communication sessions.
 Author: develNor
 License-Expression: BSD-3-Clause

{rosotacom_dev-2.5.dev65 → rosotacom_dev-2.5.dev66}/src/rosotacom_dev.egg-info/SOURCES.txt

@@ -72,6 +72,7 @@ docs/findings/jitter-loss-bandwidth-latency.md
 docs/findings/keyframe-loss-kills-the-next-deltas.md
 docs/findings/lifespan-is-enforced-by-one-reader-and-not-the-other.md
 docs/findings/lighter-message-loss.md
+docs/findings/one-domain-two-cyclone-configs-stop-interoperating.md
 docs/findings/oversubscription-queues-not-losses.md
 docs/findings/reorder-becomes-reader-loss.md
 docs/findings/sender-rows-make-a-dead-link-look-loss-free.md
@@ -212,6 +213,7 @@ src/rosotacom/resources/ws/.vscode/settings.json
 src/rosotacom/resources/ws/ota_configs/create_zenoh_json5.py
 src/rosotacom/resources/ws/ota_configs/cyclonedds_local_participants.xml.template
 src/rosotacom/resources/ws/ota_configs/cyclonedds_minimal.xml.template
+src/rosotacom/resources/ws/ota_configs/cyclonedds_scoped.xml.template
 src/rosotacom/resources/ws/ota_configs/cyclonedds_tuned.xml.template
 src/rosotacom/resources/ws/ota_configs/fastdds_easy_mode.xml.template
 src/rosotacom/resources/ws/ota_configs/fastdds_local.xml.template

{rosotacom_dev-2.5.dev65 → rosotacom_dev-2.5.dev66}/src/rosotacom_dev.egg-info/scm_file_list.json

@@ -57,6 +57,7 @@
     "docs/findings/keyframe-loss-kills-the-next-deltas.md",
     "docs/findings/lifespan-is-enforced-by-one-reader-and-not-the-other.md",
     "docs/findings/lighter-message-loss.md",
+    "docs/findings/one-domain-two-cyclone-configs-stop-interoperating.md",
     "docs/findings/oversubscription-queues-not-losses.md",
     "docs/findings/reorder-becomes-reader-loss.md",
     "docs/findings/repro/decode_chain_repro.py",
@@ -215,6 +216,7 @@
     "src/rosotacom/resources/ws/ota_configs/create_zenoh_json5.py",
     "src/rosotacom/resources/ws/ota_configs/cyclonedds_local_participants.xml.template",
     "src/rosotacom/resources/ws/ota_configs/cyclonedds_minimal.xml.template",
+    "src/rosotacom/resources/ws/ota_configs/cyclonedds_scoped.xml.template",
     "src/rosotacom/resources/ws/ota_configs/cyclonedds_tuned.xml.template",
     "src/rosotacom/resources/ws/ota_configs/fastdds_easy_mode.xml.template",
     "src/rosotacom/resources/ws/ota_configs/fastdds_local.xml.template",

{rosotacom_dev-2.5.dev65 → rosotacom_dev-2.5.dev66}/src/rosotacom_dev.egg-info/scm_version.json

@@ -1,7 +1,7 @@
 {
   "tag": "2.4",
-  "distance": 65,
-  "node": "g10a4e383f32456a7740c259e780111c21ea071de",
+  "distance": 66,
+  "node": "g77d4b7e2148ba0d437581a7481f1a96b67a3a6a9",
   "dirty": false,
   "branch": "HEAD",
   "node_date": "2026-08-20"

rosotacom_dev-2.5.dev66/tests/unit/test_ota_xml.py

@@ -0,0 +1,134 @@
+from __future__ import annotations
+
+import importlib.util
+from pathlib import Path
+from types import ModuleType
+
+import pytest
+
+
+def _module() -> ModuleType:
+    path = (
+        Path(__file__).resolve().parents[2]
+        / "src"
+        / "rosotacom"
+        / "resources"
+        / "ws"
+        / "ota_configs"
+        / "get_ota_xml.py"
+    )
+    spec = importlib.util.spec_from_file_location("test_get_ota_xml", path)
+    assert spec is not None and spec.loader is not None
+    module = importlib.util.module_from_spec(spec)
+    spec.loader.exec_module(module)
+    return module
+
+
+def test_ota_xml_uses_resolved_literal_addresses(tmp_path: Path, monkeypatch: pytest.MonkeyPatch) -> None:
+    module = _module()
+    template = tmp_path / "test.xml.template"
+    template.write_text("<host>#host_ip</host><peer>#peer</peer><easy>#easy_mode_ip</easy>\n", encoding="utf-8")
+    monkeypatch.setattr(module, "_template_path", lambda _name: str(template))
+
+    rendered = module.main(
+        config="test.xml",
+        host_ip="10.0.0.10",
+        peer="10.0.0.11",
+        easy_mode_ip="10.0.0.10",
+    )
+
+    assert rendered == "<host>10.0.0.10</host><peer>10.0.0.11</peer><easy>10.0.0.10</easy>\n"
+
+
+def test_ota_xml_renders_default_and_overridden_spdp_interval(tmp_path: Path, monkeypatch: pytest.MonkeyPatch) -> None:
+    module = _module()
+    template = tmp_path / "test.xml.template"
+    template.write_text("<SPDPInterval>#spdp_interval</SPDPInterval>\n", encoding="utf-8")
+    monkeypatch.setattr(module, "_template_path", lambda _name: str(template))
+
+    assert module.main(config="test.xml") == "<SPDPInterval>30s</SPDPInterval>\n"
+    assert module.main(config="test.xml", spdp_interval="150s") == "<SPDPInterval>150s</SPDPInterval>\n"
+
+
+def test_ota_xml_rejects_empty_resolved_address(tmp_path: Path, monkeypatch: pytest.MonkeyPatch) -> None:
+    module = _module()
+    template = tmp_path / "test.xml.template"
+    template.write_text("<host>#host_ip</host>\n", encoding="utf-8")
+    monkeypatch.setattr(module, "_template_path", lambda _name: str(template))
+
+    with pytest.raises(ValueError, match="non-empty resolved address"):
+        module.main(config="test.xml", host_ip=" ")
+
+
+def test_ota_xml_rejects_invalid_spdp_interval(tmp_path: Path, monkeypatch: pytest.MonkeyPatch) -> None:
+    module = _module()
+    template = tmp_path / "test.xml.template"
+    template.write_text("<SPDPInterval>#spdp_interval</SPDPInterval>\n", encoding="utf-8")
+    monkeypatch.setattr(module, "_template_path", lambda _name: str(template))
+
+    with pytest.raises(ValueError, match="seconds or milliseconds"):
+        module.main(config="test.xml", spdp_interval="2min")
+
+
+def test_ota_xml_scopes_sections_to_their_domains(tmp_path: Path, monkeypatch: pytest.MonkeyPatch) -> None:
+    module = _module()
+    template = tmp_path / "test.xml.template"
+    template.write_text('<Domain Id="#ota_domain"/><Domain Id="#local_domain"/>\n', encoding="utf-8")
+    monkeypatch.setattr(module, "_template_path", lambda _name: str(template))
+
+    rendered = module.main(config="test.xml", local_domain="47", ota_domain="48")
+
+    assert rendered == '<Domain Id="48"/><Domain Id="47"/>\n'
+
+
+def test_ota_xml_refuses_a_domain_it_cannot_use(tmp_path: Path, monkeypatch: pytest.MonkeyPatch) -> None:
+    """A section Cyclone matches to no domain is silently ignored, so refuse first."""
+    module = _module()
+    template = tmp_path / "test.xml.template"
+    template.write_text('<Domain Id="#ota_domain"/>\n', encoding="utf-8")
+    monkeypatch.setattr(module, "_template_path", lambda _name: str(template))
+
+    for bad in ("233", "-1", "any", "", "4 7"):
+        with pytest.raises((ValueError, RuntimeError)):
+            module.main(config="test.xml", ota_domain=bad)
+
+
+def test_ota_xml_names_the_option_a_scoped_template_needs(tmp_path: Path, monkeypatch: pytest.MonkeyPatch) -> None:
+    module = _module()
+    template = tmp_path / "test.xml.template"
+    template.write_text('<Domain Id="#local_domain"/>\n', encoding="utf-8")
+    monkeypatch.setattr(module, "_template_path", lambda _name: str(template))
+
+    with pytest.raises(RuntimeError) as excinfo:
+        module.main(config="test.xml")
+
+    assert "--local-domain" in str(excinfo.value)
+
+
+def test_shipped_scoped_template_carries_both_domains_and_only_ota_restrictions() -> None:
+    """The point of the file: OTA settings on the OTA domain, local domain left alone.
+
+    A section that pinned an interface or disabled multicast on the *local*
+    domain would recreate the failure it exists to avoid.
+    """
+    module = _module()
+    rendered = module.main(
+        config="cyclonedds_scoped.xml",
+        host_ip="10.0.0.10",
+        peer="10.0.0.11",
+        local_domain="47",
+        ota_domain="48",
+    )
+
+    assert '<Domain Id="48">' in rendered
+    assert '<Domain Id="47">' in rendered
+
+    ota_section = rendered.split('<Domain Id="48">', 1)[1].split("</Domain>", 1)[0]
+    local_section = rendered.split('<Domain Id="47">', 1)[1].split("</Domain>", 1)[0]
+
+    assert "<AllowMulticast>false</AllowMulticast>" in ota_section
+    assert "10.0.0.10" in ota_section and "10.0.0.11" in ota_section
+
+    for restriction in ("AllowMulticast", "NetworkInterface", "Peer ", "MaxMessageSize"):
+        assert restriction not in local_section, restriction
+    assert "MaxAutoParticipantIndex" in local_section