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

Improve r/gnoland/valopers to onboard new validators #3567

Open
moul opened this issue Jan 20, 2025 · 4 comments
Open

Improve r/gnoland/valopers to onboard new validators #3567

moul opened this issue Jan 20, 2025 · 4 comments
Assignees
Labels
in focus Core team is prioritizing this work 🧾 package/realm Tag used for new Realms or Packages.

Comments

@moul
Copy link
Member

moul commented Jan 20, 2025

The current version of the validator registration contract can be found here. We need to make a few changes, starting with removing the dependency on r/sys/validators. This contract should not be directly tied to the r/sys contracts; it is intended for validator operators to self-register, similar to a "call for papers." Later, a separate external contract (a glue contract) will interact with both this contract and r/sys/validators to propose templated changes to GovDAO. We can also remove the /v2 suffix since we are eliminating the dependency on r/sys/validators.

The contract should focus on allowing validators to register and update parameters as needed. It may optionally link with r/demo/profile and should include a list of questions inspired by the recent Atom One launch, using their GitHub or Google Forms as a reference. The contract should display a list of current proposals, along with a brief introductory text and a link to a page that can be easily updated—either a README or a meta issue, as the explanations will need to be refined.

cc @sw360cab @albttx @zivkovicmilos

@VT-Cosmos
Copy link

Here are the reference to two initiatives for collecting validators details on two initiatives, GovGen validators and AiB ATONE Delegation Program.

https://github.com/atomone-hub/govgen-validators
https://github.com/allinbits/AiB-ATONE-Delegation-Program

@albttx
Copy link
Member

albttx commented Jan 21, 2025

Why does a "glue" contract is needed ? Can't just a gno script do the trick ?

Also, why not just add a link in the Valoper struct to a r/demo/profile ?


On another topic, on the Update function, i don't understand why there is no security checking if it's the validator that self-update his informations (with std.PrevRealm / std.GetOrigCaller) ?

I'm probably missing something

@moul moul added this to the 🚀 Mainnet beta launch milestone Jan 21, 2025
@Kouteki Kouteki moved this from Triage to Todo in 🧙‍♂️gno.land core team Jan 21, 2025
@Kouteki Kouteki added in focus Core team is prioritizing this work 🧾 package/realm Tag used for new Realms or Packages. labels Jan 21, 2025
@moul
Copy link
Member Author

moul commented Jan 21, 2025

Glue contracts offer better auditability. While a Gno script could accomplish the task, I prefer to release my recurring script needs as contracts, which could be called "templated actions contracts." I will reserve my more private and unique scripting needs for Gno scripts.

@costinberty
Copy link

We confirm we can work on it at Berty. We'll be back with updates shortly.

@Kouteki Kouteki assigned jefft0 and D4ryl00 and unassigned jefft0 Jan 23, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
in focus Core team is prioritizing this work 🧾 package/realm Tag used for new Realms or Packages.
Projects
Development

No branches or pull requests

7 participants