TrueCloudLab/frostfs-contract
22
0
Fork 19
Code Issues 14 Pull requests 1 Projects Releases 2 Activity Actions

[#21] Document contract storage schemes #22

Merged
fyrchik merged 1 commit from aarifullin/frostfs-contract:feature/document_contract_storage_schemes into master 2023-05-12 12:46:53 +00:00
Conversation 0 Commits 1 Files changed 11 +114
11 changed files with 114 additions and 0 deletions
Show stats Download patch file Download diff file Expand all files Collapse all files

11
alphabet/doc.go
View file

@ -17,5 +17,16 @@ for each alphabet contract.
# Contract notifications
Alphabet contract does not produce notifications to process.
# Contract storage scheme
| Key | Value | Description |
|--------------------|------------|-------------------------------------------------|
| `netmapScriptHash` | Hash160 | netmap contract hash |
fyrchik marked this conversation as resolved Outdated
fyrchik commented 2023-05-11 08:37:05 +00:00
Outdated
Review
Copy link

Can we use markdown syntax for tables?

| Key | Value | Desc |
|-----|-------|------|
| a   | b     | c    |
| x   | y     | z    |

It is easier to copy in md doc, takes less space and is somewhat widely accepted.

I've searched a bit -- it seems godoc has no support for tables, so markdown is a safe choice.
Also, pre-formatted text should be indented (goimports does this, but we have no pre-commit here).

Can we use markdown syntax for tables? ``` | Key | Value | Desc | |-----|-------|------| | a | b | c | | x | y | z | ``` It is easier to copy in md doc, takes less space and is somewhat widely accepted. I've searched a bit -- it seems `godoc` has no support for tables, so markdown is a safe choice. Also, pre-formatted text should be indented (goimports does this, but we have no pre-commit here).
👍 1
| `proxyScriptHash` | Hash160 | proxy contract hash |
| `name` | string | assigned glagolitic letter |
| `index` | int | the index of deployed alphabet contract |
| `threshold` | int | the total number of deployed alphabet contracts |
*/
package alphabet

8
audit/doc.go
View file

@ -18,5 +18,13 @@ they make a list and get these AuditResultStructures from the audit contract.
# Contract notifications
Audit contract does not produce notifications to process.
# Contract storage scheme
| Key | Value | Description |
|--------------------|------------|-----------------------------------------------------------|
| `netmapScriptHash` | Hash160 | netmap contract hash |
| auditID | ByteArray | serialized DataAuditResult structure |
*/
package audit

9
balance/doc.go
View file

@ -74,5 +74,14 @@ when FrostFS contract has transferred GAS assets back to the user.
type: Hash160
- name: amount
type: Integer
# Contract storage scheme
| Key | Value | Description |
|-----------------------|------------|----------------------------------|
| `netmapScriptHash` | Hash160 | netmap contract hash |
| `containerScriptHash` | Hash160 | container contract hash |
| circulationKey | int | the token circulation key value |
fyrchik marked this conversation as resolved
fyrchik commented 2023-05-12 12:01:11 +00:00
Review
Copy link

Is it variable length?

Is it variable length?
aarifullin commented 2023-05-12 12:04:43 +00:00
Review
Copy link

Right, it's non-const

Right, it's non-`const`
*/
package balance

17
container/doc.go
View file

@ -64,5 +64,22 @@ it in Container contract.
StopEstimation:
- name: epoch
type: Integer
# Contract storage scheme
| Key | Value | Description |
|-----------------------------------------------------------------------------------------------------|
| `netmapScriptHash` | Hash160 | netmap contract hash |
| `balanceScriptHash` | Hash160 | balance contract hash |
| `identityScriptHash` | Hash160 | frostfsID contract hash |
| `nnsContractKey` | Hash160 | nns contract hash |
| `nnsRoot` | string | default value for domain zone |
| `cnr` + epoch + containerID + publicKeyHash[:10] | ByteArray | estimated container size |
| `est` + containerID + publicKeyHash | ByteArray | serialized epochs array |
| `o` + ownerID + containerID | ByteArray | container ID |
| `x` + containerID | ByteArray | serialized container struct |
| `nnsHasAlias` + containerID | string | domain name |
*/
package container

10
frostfs/doc.go
View file

@ -80,5 +80,15 @@ FrostFS network configuration value.
type: ByteArray
- name: value
type: ByteArray
# Contract storage scheme
| Key | Value | Description |
|-----------------------------------------------------------------------------|
| `processingScriptHash` | Hash160 | processing contract hash |
| `candidates` + candidateKey | ByteArray | it flags inner ring candidate |
| `config` + postfix | ByteArray | serialized config data |
*/
package frostfs

9
frostfsid/doc.go
View file

@ -16,5 +16,14 @@ contract.
# Contract notifications
FrostFSID contract does not produce notifications to process.
# Contract storage scheme
| Key | Value | Description |
|-----------------------------|------------|----------------------------------|
| `processingScriptHash` | Hash160 | netmap contract hash |
| `containerScriptHash` | Hash160 | container contract hash |
| `o` + ownerID + publicKey | ByteArray | it flags owner's public key |
*/
package frostfsid

13
netmap/doc.go
View file

@ -29,5 +29,18 @@ in the network by invoking NewEpoch method.
NewEpoch
- name: epoch
type: Integer
# Contract storage scheme
| Key | Value | Description |
|-----------------------------|------------|-----------------------------------|
| `snapshotCount` | int | snapshot count |
| `snapshotEpoch` | int | snapshot epoch |
dstepanov-yadro marked this conversation as resolved Outdated
dstepanov-yadro commented 2023-05-10 11:16:08 +00:00
Outdated
Review
Copy link

' should be replaced with ` ?

`'` should be replaced with ` ?
aarifullin commented 2023-05-10 15:40:19 +00:00
Outdated
Review
Copy link

Thanks!

Thanks!
| `snapshotBlock` | int | snapshot block |
| `snapshot_` + snapshotNum | ByteArray | serialized '[]Node' array |
| `snapshotCurrent` | int | current snapshot |
| `balanceScriptHash` | Hash160 | balance contract hash |
| `containerScriptHash` | Hash160 | container contract hash |
*/
package netmap

17
nns/doc.go Normal file
View file

@ -0,0 +1,17 @@
/*
# Contract storage scheme
fyrchik marked this conversation as resolved Outdated
fyrchik commented 2023-05-11 07:03:24 +00:00
Outdated
Review
Copy link

Let's use a package comment everywhere (like in the netmap contract) -- before the package keyword.

Let's use a package comment everywhere (like in the netmap contract) -- before the `package` keyword.
👍 1
| Key | Value | Description |
|--------------------------------------|------------|-----------------------------------|
| 0x0 | int | total supply of minted domains |
| 0x1 + accountAddr | int | account's balance |
| 0x2 + accountAddr + tokenKey | ByteArray | token ID |
| 0x10 | int | price for domain registration |
| 0x20 | int | set of roots |
| 0x21 + tokenKey | ByteArray | serialized NameState struct |
| 0x22 + tokenKey + Hash160(tokenName) | Hash160 | container contract hash |
*/
package nns

7
processing/doc.go
View file

@ -18,5 +18,12 @@ execution.
# Contract notifications
Processing contract does not produce notifications to process.
# Contract storage scheme
| Key | Value | Description |
|-----------------------------|------------|----------------------------------|
| `frostfsScriptHash` | Hash160 | frostFS contract hash |
*/
package processing

5
proxy/doc.go
View file

@ -17,5 +17,10 @@ verified, Proxy contract pays for the execution.
# Contract notifications
Proxy contract does not produce notifications to process.
# Contract storage scheme
Proxy contract does not use storage
*/
package proxy

8
reputation/doc.go
View file

@ -13,5 +13,13 @@ Inner Ring nodes if data audit succeeds.
# Contract notifications
Reputation contract does not produce notifications to process.
# Contract storage scheme
| Key | Value | Description |
|-----------------------------|------------|-----------------------------------|
| `c` + epoch + peerID | int | peer reputation count |
| `r` + count | ByteArray | serialized DataAuditResult struct |
*/
package reputation