|
| 1 | +--- |
| 2 | +title: Network Team Update |
| 3 | +slug: 2025-03-19-network |
| 4 | +authors: [coot] |
| 5 | +tags: [network] |
| 6 | +hide_table_of_contents: false |
| 7 | +--- |
| 8 | + |
| 9 | +## Overview of [sprint 83][sprint-83] |
| 10 | + |
| 11 | +### High-level overview |
| 12 | + |
| 13 | +#### SRV support (Cardano, Mithril) |
| 14 | + |
| 15 | +We merged [ouroboros-network#5018] which adds support for SRV in `ouroboros-network`. |
| 16 | +However, we still need to make a decision whether we require that all Cardano |
| 17 | +SRVs start with `_cardano._tcp`, or we leave it open for the future. The |
| 18 | +former would allow us to use `SRV`s for different applications, e.g. |
| 19 | +`_mithril._tcp` for the mithril diffusion. If that would be the case then we'd |
| 20 | +prepend `_cardano._tcp` to what we find in the SRV field of a registration |
| 21 | +certificate, e.g. prepend `_cardano._tcp` by `cardano-node`'s diffusion, and |
| 22 | +`_mithril._tcp` by the mithril diffusion. The other possibility is to extend |
| 23 | +the registration certificate to include the addresses of mithril nodes. |
| 24 | + |
| 25 | +#### Mithril |
| 26 | + |
| 27 | +We started working on a development plan for Mithril diffusion, |
| 28 | +[ref][mithirl-diffusion] (note that this is a work in progress). |
| 29 | + |
| 30 | +We also have been working on moving network-related code from the |
| 31 | +`cardano-node` to the `cardano-diffusion` library in `ouroboros-network`. In the |
| 32 | +future, this will allow us to share parts of it in the Mithril diffusion [ouroboros-newtork#5082]. |
| 33 | + |
| 34 | +#### Documentation |
| 35 | + |
| 36 | +We improved our CDDL specifications by annotating which codecs require definite |
| 37 | +/ in-definite encoding of lists & maps, see [ouroboros-network#5089]. |
| 38 | + |
| 39 | +#### Mux |
| 40 | + |
| 41 | +Karl Knutsson ([CF]) has been working on performance improvements in the |
| 42 | +multiplexer (`network-mux` library), [ouroboros-network#5093]. |
| 43 | + |
| 44 | +#### Server-side performance considerations |
| 45 | + |
| 46 | +Karl Kntusson ([CF]) noticed a performance bottleneck in the responder |
| 47 | +(server) side of the p2p diffusion layer. We've started working on addressing |
| 48 | +this issue. |
| 49 | + |
| 50 | +#### Tx-Submission logic |
| 51 | + |
| 52 | +We continued reviewing/improving the `tx-submission` PR [ouroboros-network#4887]. |
| 53 | + |
| 54 | +### Pull requests |
| 55 | + |
| 56 | +#### In review |
| 57 | + |
| 58 | +* [**Mux performance improvements**][ouroboros-network#5093] |
| 59 | + |
| 60 | +#### Work in Progress |
| 61 | + |
| 62 | +* [**TX Submission Logic**][ouroboros-network#4887] |
| 63 | +* [**cardano-diffusion sublibrary**][ouroboros-network#5082] |
| 64 | + |
| 65 | +#### Merged |
| 66 | + |
| 67 | +* [**SRV implementation**][ouroboros-network#5018] |
| 68 | +* [**CDDL: comment on definite and indefinite lists/maps**][ouroboros-network#5089] |
| 69 | +* [**LedgerPeers: code cleanup**][ouroboros-network#5089] |
| 70 | +* [**fixed a connection manager test failure**][ouroboros-network#5091] |
| 71 | +* [**Fixed LedgerPeers generators**][ouroboros-network#5092] |
| 72 | + |
| 73 | +[mithril-diffusion]: https://github.com/IntersectMBO/ouroboros-network/wiki/Decentralized-Message-Queue-(DMQ)-Implementation-Overview |
| 74 | + |
| 75 | + |
| 76 | +[sprint-83]: https://github.com/orgs/IntersectMBO/projects/5/views/1?filterQuery=sprint%3A%22Sprint+83%22 |
| 77 | +[CF]: https://cardanofoundation.org/ |
| 78 | + |
| 79 | +[ouroboros-network#4887]: https://github.com/IntersectMBO/ouroboros-network/pull/4887 |
| 80 | +[ouroboros-network#5018]: https://github.com/IntersectMBO/ouroboros-network/pull/5018 |
| 81 | +[ouroboros-network#5082]: https://github.com/IntersectMBO/ouroboros-network/pull/5082 |
| 82 | +[ouroboros-network#5089]: https://github.com/IntersectMBO/ouroboros-network/pull/5089 |
| 83 | +[ouroboros-network#5091]: https://github.com/IntersectMBO/ouroboros-network/pull/5091 |
| 84 | +[ouroboros-network#5092]: https://github.com/IntersectMBO/ouroboros-network/pull/5092 |
| 85 | +[ouroboros-network#5093]: https://github.com/IntersectMBO/ouroboros-network/pull/5093 |
0 commit comments