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

ADR 0004: Runtime Governance #3284

Merged
merged 1 commit into from
Oct 22, 2020
Merged

ADR 0004: Runtime Governance #3284

merged 1 commit into from
Oct 22, 2020

Conversation

kostko
Copy link
Member

@kostko kostko commented Sep 17, 2020

See #2515

@kostko kostko added the c:docs/adr Category: documentation/ADR label Sep 17, 2020
@kostko kostko force-pushed the kostko/docs/adr-rt-governance branch 5 times, most recently from 06e56f4 to b64da28 Compare September 17, 2020 10:03
@kostko kostko force-pushed the kostko/docs/adr-rt-token-transfer branch from 37b1a96 to ed6039d Compare September 17, 2020 10:04
@kostko kostko force-pushed the kostko/docs/adr-rt-governance branch from b64da28 to a257cc1 Compare September 17, 2020 10:04
docs/adr/0004-runtime-governance.md Outdated Show resolved Hide resolved
docs/adr/0004-runtime-governance.md Outdated Show resolved Hide resolved

Registering a runtime may require sufficient stake in either the owning entity's
(when entity governance is used) or the runtime's (when runtime governance is
used) escrow account.
Copy link
Contributor

Choose a reason for hiding this comment

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

what is the workflow for creating a runtime with runtime governance? do you create it that way, and it's immediately suspended and then you do something? or maybe you create it under entity governance, and you transfer some tokens and then you update it to runtime governance?

Copy link
Member Author

@kostko kostko Sep 22, 2020

Choose a reason for hiding this comment

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

You can escrow tokens to the runtime account before registering the runtime itself. And then you can register it with runtime governance directly.

docs/adr/0004-runtime-governance.md Show resolved Hide resolved

Additionally it also introduces an additional safeguard for (all) runtime
descriptor updates -- a configurable delay period after which any modifications
to a runtime descriptor take effect instead of them taking effect immediately.
Copy link
Contributor

Choose a reason for hiding this comment

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

what does this safeguard against?

Copy link
Member Author

Choose a reason for hiding this comment

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

In theory it would give users time to e.g., move tokens out of the runtime in case they don't agree with the upgrade (security parameter change/MRENCLAVE change/etc.). Of course there are ways to prevent this, so it may not be worth adding the complexity.

@kostko kostko force-pushed the kostko/docs/adr-rt-token-transfer branch from ed6039d to c653da3 Compare September 22, 2020 09:09
@kostko kostko force-pushed the kostko/docs/adr-rt-governance branch 2 times, most recently from ba3a25f to bcc2f9f Compare September 22, 2020 09:40
@kostko kostko force-pushed the kostko/docs/adr-rt-token-transfer branch from c653da3 to 6cf0773 Compare September 22, 2020 16:38
@kostko kostko force-pushed the kostko/docs/adr-rt-governance branch from bcc2f9f to 25b1b0e Compare September 22, 2020 17:00
@kostko kostko force-pushed the kostko/docs/adr-rt-token-transfer branch from 6cf0773 to d23d2f7 Compare September 25, 2020 10:42
@hydai
Copy link

hydai commented Sep 30, 2020

Hi, I would like to request a feature of the runtime governance.

Is it possible to limit the number of compute nodes per entity?

As a ParaTime developer, we will encounter the following issues if the number of compute nodes per entity is not limited:

  1. Someone can create as many as possible compute nodes with malicious modification. When the number of compute nodes per entity is not limited, all of the active compute nodes can be taken by this maliciously modified version in one epoch time.
  2. If we want to provide rewards for the active compute nodes, an entity which creates more compute nodes may increase its possibility to join the active group.

@kostko kostko force-pushed the kostko/docs/adr-rt-token-transfer branch from d23d2f7 to b6f9b83 Compare September 30, 2020 10:50
@kostko kostko force-pushed the kostko/docs/adr-rt-governance branch 2 times, most recently from 4c5be3e to 0b2325e Compare September 30, 2020 11:13
@kostko kostko force-pushed the kostko/docs/adr-rt-token-transfer branch from b6f9b83 to 2a9c80b Compare October 1, 2020 12:01
@kostko kostko force-pushed the kostko/docs/adr-rt-governance branch from 0b2325e to b3d6930 Compare October 1, 2020 12:12
@kostko kostko force-pushed the kostko/docs/adr-rt-token-transfer branch from 2a9c80b to 11151cb Compare October 2, 2020 09:12
@kostko kostko force-pushed the kostko/docs/adr-rt-governance branch from b3d6930 to 3ee37f4 Compare October 2, 2020 09:14
@kostko kostko force-pushed the kostko/docs/adr-rt-token-transfer branch from 11151cb to da9fcb1 Compare October 7, 2020 09:30
@kostko kostko force-pushed the kostko/docs/adr-rt-governance branch from 3ee37f4 to a8cb147 Compare October 7, 2020 09:31
@kostko kostko force-pushed the kostko/docs/adr-rt-governance branch from a8cb147 to 9b8f590 Compare October 13, 2020 15:32
@kostko kostko force-pushed the kostko/docs/adr-rt-token-transfer branch from da9fcb1 to 7ac17e6 Compare October 13, 2020 16:08
Base automatically changed from kostko/docs/adr-rt-token-transfer to master October 13, 2020 16:21
@kostko kostko force-pushed the kostko/docs/adr-rt-governance branch from 9b8f590 to 4566321 Compare October 13, 2020 16:27
@kostko kostko marked this pull request as ready for review October 13, 2020 16:27
@kostko kostko force-pushed the kostko/docs/adr-rt-governance branch 2 times, most recently from 30e50b2 to d386605 Compare October 14, 2020 08:36
@kostko kostko force-pushed the kostko/docs/adr-rt-governance branch 2 times, most recently from ce517e1 to 9a4a61f Compare October 21, 2020 19:45
@kostko kostko force-pushed the kostko/docs/adr-rt-governance branch from 9a4a61f to 348c154 Compare October 22, 2020 13:22
@kostko kostko merged commit 4df0aa6 into master Oct 22, 2020
@kostko kostko deleted the kostko/docs/adr-rt-governance branch October 22, 2020 13:24
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
c:docs/adr Category: documentation/ADR
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

6 participants