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

Poldercats should support static topology #21

Closed
mark-stopka opened this issue Mar 1, 2020 · 3 comments
Closed

Poldercats should support static topology #21

mark-stopka opened this issue Mar 1, 2020 · 3 comments

Comments

@mark-stopka
Copy link

If Poldercast would be the future pub/sub for Cardano next-gen as indicated by CH in this AMA 03-Jan-2020, logs of issues should be made that are known for the current Poldercast implementation.

It's known from the Cardano Shelley ITNv1 running Jormungandr, operators would like the ability to add and keep their operated nodes in the topology, however Jormungandr, nor does Poldercast library enable static topology layers.

It was a justification given in the #16 PolderCast Layers Not Communicating that the deviation from the paper architecture and individual layers not communication was to enable layer flexibility such as adding custom topology layers.

Static topology layer

Static topology layer should be added as the next topology, pub/sub nodes should be able to add static network topology that is to have some functionality that indicate the nodes within static topology are more trusted then the otherwise discovered peers.

Effectively this should enable Jormungandr node operators to add their relay nodes to their leader node and keep them there. This is basically a dependency for input-output-hk/jormungandr/#883.

@mark-stopka
Copy link
Author

Linking relevant issue #18 Document the implementation decisions differences from the paper

@NicolasDP
Copy link
Contributor

So this is an IOHK/Jormungandr specific issue. Anyone can write a Layer to implement such a feature and it would be better to live in within that product specifically. I.e. there is not much that needs to be done in this repo in order to have such a feature, this is something that can be done directly on the client side by implementing a Layer.

@NicolasDP
Copy link
Contributor

adding the links to the relevant part:

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

No branches or pull requests

2 participants