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

Blue lahar patch 1 #17

Open
wants to merge 3 commits into
base: master
Choose a base branch
from

Conversation

ZaatarHerb
Copy link

No description provided.

1/ "is comprised of" is incorrect; should be "comprises" or "is composed of"
2/ "three" not "3" - the rule is write out numerals up to nine
3/ Nodes service these Relays by submitting them ... and sending the "responses" - "responses" reads better than "response" as the subject, "relays," is plural
4/ The mechanism the Network uses to regulate the interactions between Applications and Nodes *is* [not "are" as the subject, "mechanism." is singular] 
5/ Sessions - a data structure - are generated... [using the dashes avoids having to write "Sessions are a data structure that *is* generated..." which sounds awkward ["are" generated is incorrect as  the subject is “structure"]
6/ changed "which" to "that" to avoid using which twice in the same sentence, and deleted "will" as it's implied
7/ deleted a few unnecessary commas
@height
Copy link

height bot commented Feb 19, 2022

Link Height tasks by mentioning a task ID in the pull request title or commit messages, or description and comments with the keyword link (e.g. "Link T-123").

💡Tip: You can also use "Close T-X" to automatically close a task when the pull request is merged.


An Application submits Relays, or API requests meant to be routed to any public database node. Nodes service these Relays, by submitting them to the public databases they are meant for, and sending the response (if any) back to the Application. The Network Layer is comprised of all the rules, protocols and finality storage that serve as the backbone of the interactions between Applications and Nodes, including (but not limited to), configuration, record tracking, governance and economic policy. 
An Application submits Relays, or API requests meant to be routed to any public database node. Nodes service these Relays by submitting them to the public databases they are meant for, and sending the responses (if any) back to the Application. The Network Layer comprises all the rules, protocols and finality storage that serve as the backbone of the interactions between Applications and Nodes, including (but not limited to) configuration, record tracking, governance and economic policy. 
Copy link
Member

Choose a reason for hiding this comment

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

The Pocket Network (composed of the utility, network, persistence and consensus layers) comprises...

@@ -42,11 +42,40 @@ If you're determined to learn by doing, we recommend using [Node Pilot](https://

### Hardware

**Hardware Requirements:** 4 CPU’s (or vCPU’s) | 8 GB RAM | 200GB Disk
**Hardware Requirements:** 4 CPUs (or vCPUs) | 8 GB RAM | 200GB Disk
Copy link
Member

Choose a reason for hiding this comment

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

Just to be safe: 8 vCPUs | 16 GB RAM | 300 GB Disk

This will require an update to some sentences below.


Plan hardware for 4 vCPUs per Pocket node, but when creating the node allocate 16. This will allow nodes extra resources to close app large sessions.

Since the Pocket node is a continuously running blockchain, new data is always being stored on it. As 200GB might suffice only for about six months, consider having much more storage than minimally required.
Copy link
Member

Choose a reason for hiding this comment

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

Since the Pocket node is a continuously running blockchain, new data is always being stored on it. -> Since the Pocket node is a continuously synching the blockchain, new data is always being written to disk.


**NOTE**: You'll also need to run the full nodes of other blockchains, which may have their own hardware requirements that exceed Pocket's.

The blockchains, i.e., relay chains, you wish to support will determine the vCPU, RAM and disk space you’ll need for the server hosting them. A guide is available [here]: (https://docs.decentralizedauthority.com/hardware-recommendations)
Copy link
Member

Choose a reason for hiding this comment

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

A guide is available here.


If you plan to run the relay chains on the same machine as your Pocket node, make sure you have enough resources to support them.

TIP: Take the minimum required disk space you calculated for your blockchain node and triple it. This will accommodate potential rapid blockchain growth. As chains and validators tend to get hungrier, e.g., due to unforeseen bugs or natural growth, additional vCPU and RAM may be required later. {: .alert .alert-info}
Copy link
Member

Choose a reason for hiding this comment

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

NIT: Make the TIPS bolded or italicize the whole sentence so it stands out a bit.

Ditto for the TIPs below.

Copy link
Member

Choose a reason for hiding this comment

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

I don't think {: .alert .alert-info} works on github markdown.


For VPS or Dedicated Servers, you may not have your choice of hardware. So select the closest to the required specifications. You will want as dedicated a server as possible without "noisy neighbours" or shared resources to achieve maximum performance.

**HOST**: Pick a reliable host or data centre. Remember: if they go down - you go down.
Copy link
Member

Choose a reason for hiding this comment

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

Remember: if they go down - you go down. It makes sense, but I'm just not a fan of the wording.

Maybe, "Please note that if validator may get jailed if your host provider is down for a prolonged period of time".


**HOST**: Pick a reliable host or data centre. Remember: if they go down - you go down.

**REGION**: Host as close as possible to one of the Pocket network portals. For a list of portal locations see: [insert info or link here]
Copy link
Member

Choose a reason for hiding this comment

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

TODO: link is still missing.


**REGION**: Host as close as possible to one of the Pocket network portals. For a list of portal locations see: [insert info or link here]

**Pocket Nodes per Relay Chain**: A single relay-chain can be connected to multiple Pocket nodes. The maximum number depends on the hardware and bandwidth limitations of the chain node and the number of RPC requests it will receive at a time. How to check if you’ve reached your limit: trial and error.
Copy link
Member

Choose a reason for hiding this comment

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

How to check if you’ve reached your limit: trial and error.

Optimizing your rewards will come with empirical experience over time; i.e. trial and error.


NOTE: Addresses below in params are random from the explorer, but you should get a hash in the result that represents some value other than 0x0.

Etherum full node:
Copy link
Member

Choose a reason for hiding this comment

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

Consider making a bulleted list

NOTE: Addresses below in params are random from the explorer, but you should get a hash in the result that represents some value other than 0x0.

Etherum full node:
curl -X POST --data '{"relay_network_id":"0021","payload":{"data":"{\"jsonrpc\":\"2.0\",\"method\":\"eth_getBalance\",\"params\":[\"0xD45C4950741A023a6Bd426EB11178AeF943728a0\", \"latest\"],\"id\":1}","method":"POST","path":"","headers":{}}}' localhost:8082/v1/client/sim
Copy link
Member

Choose a reason for hiding this comment

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

Commands should be wrapped in triple quotes(```): ...

@Olshansk
Copy link
Member

@BlueLahar Do you still plan to respond to the comments?

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.

3 participants