On 02.04.2025 10:49, Christoph Heiss wrote:
On Fri Mar 28, 2025 at 6:13 PM CET, Gabriel Goller wrote:
[..]
+
+Installation
+~~~~~~~~~~~~
+
+We use the FRR implementations of OpenFabric and OSPF, so first you need to
+install `frr` and the `frr-pythontools` package:
Rather try to avoid first-person, as per our style guide [0].
Could be rephrased as e.g.
The FRR implementations of OpenFabric and OSPF are used, so first the
`frr` and `frr-pythontools` packages must be installed:
[0]
https://pve.proxmox.com/wiki/Technical_Writing_Style_Guide#Avoid_first_person
Oops, yeah my oversight.
+
+----
+apt update
+apt install frr frr-pythontools
+----
+
[..]
+
+[[pvesdn_openfabric_node]]
+On the Node
+^^^^^^^^^^^
+
+Node:: Select the node which will be added to the fabric. Only nodes that
+currently are in the cluster will be shown.
+
+Router-ID:: A unique IPv4 or IPv6 address used to generate the OpenFabric
+Network Entity Title (NET). Each node in the same fabric must have a different
+Router-ID, while a single node must use the same NET address across all fabrics
+(this consistency is automatically managed by {pve}).
+
+NOTE: When using IPv6 addresses, we use the last 3 segments to generate the
+NET. Ensure these segments differ between nodes.
Same here w.r.t. first person ("we use").
Fixed this as well.
+
+Interfaces:: Specify the interfaces used to establish peering connections with
+other OpenFabric nodes. Preferably select interfaces without pre-assigned IP
+addresses, then configure addresses in the IPv4/IPv6 column if needed. A dummy
+"loopback" interface with the router-id is automatically created.
+Interface-specific settings override the global fabric settings.
+
+The following optional parameters can be configured per interface when enabling
+the additional columns:
Perhaps this sentence should be made a bit more prominent, e.g. as
sub-heading? It's easy to read over it and miss it, but it's quite
important piece of information. Especially if your not *that* familiar
with the UI and enablable columns.
There are advanced options, so hiding them by default is good IMO - just
a bit confusing to have big headings with the option names but nowhere
to enable them (directly).
I thought about adding section (even though this is not exactly a
different panel in the UI), e.g.,
On The Interface
^^^^^^^^^^^^^^^^
So we'd have:
OpenFabric
- On the Node
- On the Interface
- On the Fabric
and obviously the same with OSPF.
What do you think?
Thanks for looking through this!
_______________________________________________
pve-devel mailing list
pve-devel@lists.proxmox.com
https://lists.proxmox.com/cgi-bin/mailman/listinfo/pve-devel