-
Notifications
You must be signed in to change notification settings - Fork 23
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 an ability to specify a node version in Cohort
#170
Comments
Is there an archived convo for the reasoning behind this? Seems like a reasonable enough idea. What kind of incentive structure exists for updating? Is there a chance we inadvertently incentivize nodes not to update? |
I created this issue after sharing the idea on Discord. The gist of it is that even if node operators are incentivized to update their nodes, there is a possibility that sampling will result in a set of incompatible nodes - There is a period of instability between releases where node operators have to update, etc. My line of thinking is that the policy creator should have finer control over that. |
Seems like a trade-off between For maximum uniformity it arguably should be a protocol-driven sampling constraint, rather than a discretionary parameter. I.e. you can only be selected if you've updated to the latest version, unless an adopter goes and changes the client code. The question then becomes, what about long-standing Cohorts in which some update and some do not? |
I'm increasingly finding myself with this view generally, not only for the purposes of this issue.
Definitely one of the hardcore challenges of long thinking. |
This becomes much more of a problem when a given app is creating lots of cohorts on the fly, whereas for early versions they're unlikely to have more than a single semi-perpetual cohort. On the staker side, we can assume a level of professionalism (given the smaller group) and sampling/incentive mitigations to make sure all nodes are on the latest version |
Will be addressed as part of #264 |
version
field to theCohort
object to select a group of nodes that satisfy certain node version/get_ursulas
:version
version
structure should follow semver, or whichever scheme we use to denote node versionThe text was updated successfully, but these errors were encountered: