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

Add generic member_data to simplify operator governance #1657

Merged
merged 12 commits into from
Sep 28, 2020

Conversation

eddyashton
Copy link
Member

As described in #1597.

This adds a member_data field for each member, in the same pattern as user_data, storing arbitrary JSON for the governance scripts to interpret how they wish. operator_gov.lua is updated to use this, creating an object with is_operator = true for operating members, rather than hardcoding them in the constitution.

#1597 mentioned exposing the proposer's ID to the constitution, but I don't think this is actually necessary. We don't currently block proposals on any grounds, so we'd have to add that if we wanted to gate the creation of specific proposals to members/non-members. Instead we care about the operatoricity of each voter. In the case of proposals submitted by operators (which are freely submitted like any other members'), their single yes vote is sufficient to pass an 'operator proposal' (one of the constitution-defined calls), but this works purely by vote-iteration and doesn't need to consider the proposer separately.

@eddyashton eddyashton requested a review from a team as a code owner September 25, 2020 16:00
@ghost
Copy link

ghost commented Sep 25, 2020

member_data@13193 aka 20200928.14 vs master ewma over 50 builds from 12648 to 13190
images

tests/memberclient.py Outdated Show resolved Hide resolved
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