Skip to content

Conversation

jtakalai
Copy link
Contributor

No description provided.

@linear
Copy link

linear bot commented Mar 27, 2023

ETH-474 BrokerPool additions for miner subsystems

Parent ticket for Tatum smart contract features needed/used by miner plugin in Broker.

ETH-475 BrokerPool heartbeat method

The nodes declared via ETH-472 should be able to announce connectivity-related metadata through a function on the smart contract. The information is intended for off-chain use and doesn't need to be stored in the contract. The contract is therefore (ab)used as a sort of shared communication channel to facilitate, among other things, discovery of entry points to the Streamr Network.

  • Add function heartbeat(jsonData) which is callable by the set of nodes declared via ETH-472 or the owner of the BrokerPool (the 'broker')
  • The jsonData argument is similar in spirit to the metadata stored in StreamRegistry and ProjectRegistry - a freeform blob intended to be interpreted as a string off-chain. I didn't specify the solidity type here as I'm not sure what's appropriate (a string type or just a byte array).
  • The only thing the heartbeat method should do is fire a Heartbeat(nodeAddress, jsonData) event.

@jtakalai jtakalai requested a review from samt1803 March 27, 2023 10:27
Copy link
Contributor

@samt1803 samt1803 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

👍

@samt1803 samt1803 merged commit 435146c into master Mar 28, 2023
@samt1803 samt1803 deleted the ETH-474-heartbeat branch March 28, 2023 07:13
samt1803 added a commit that referenced this pull request Mar 28, 2023
…_bounty

* origin/master:
  ETH-401 Consolidate all subgraphs into one (#497)
  feat: heartbeat (#502)

# Conflicts:
#	packages/network-subgraphs/README.md
#	packages/network-subgraphs/schema.graphql
#	packages/network-subgraphs/start.sh
#	packages/network-subgraphs/subgraph.yaml
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

Successfully merging this pull request may close these issues.

2 participants