-
Notifications
You must be signed in to change notification settings - Fork 87
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
v3.0.19 #2586
Merged
Merged
v3.0.19 #2586
Conversation
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
ironoa
reviewed
Feb 13, 2024
ironoa
reviewed
Feb 13, 2024
ironoa
reviewed
Feb 13, 2024
ironoa
reviewed
Feb 16, 2024
This comment was marked as resolved.
This comment was marked as resolved.
implement Factory and Template patterns
This comment was marked as resolved.
This comment was marked as resolved.
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
Candidates Config and DB Changes:
slotId
andkyc
field (set to false if they have not been set yet) https://github.com/w3f/1k-validators-be-PRIVATE/issues/16unique
on candidates not set yet - there needs to be separations of non-candidates telemetry nodes (which will otherwise havenull
slotIds to get implemented first)slotId
andkyc
fieldslotId
field can be merged / changed without any need to adjust things manually in the db. There may still need to be an editing of existing duplicates, though. https://github.com/w3f/1k-validators-be-PRIVATE/issues/3Misc Refactoring:
Bootstrap.ts
job (was intended to be run from local instance to query prod instance and set data to what's returned, but isn't used).Config.ts
to be able to get a candidates list either from a url (prod version is a github url), or a local path (for when running locally)location
andinfrastructureData
fields get removed from each candidate db entry on startup in the cleaning hook (as they are instead fetched fresh from thelocation
db records)controller
tobondedAddress
innominator.ts
in core, as controllers are deprecated https://github.com/w3f/1k-validators-be-PRIVATE/issues/36cron.ts
andjobs.ts
(both only run as monolith, as functions imported from the separateWorker
package, as cron jobs) -> into new/scorekeeper/jobs/cron/
folderconstraints.ts
into multiple files (incore/src/constraints
) https://github.com/w3f/1k-validators-be-PRIVATE/issues/23common
andcore
Consistency FIxes:
EraPointsJob
has been updated to query for all eras until the beginning of the chain - a candidates rank is the max of the number of eras there exists era points for of any identity address that is apart of a validators sub/super identity. This has removed rank updates from scorekeeper rounds https://github.com/w3f/1k-validators-be-PRIVATE/issues/33Nomination
db records to be created when a non-proxy nomination tx happens, or when a proxy tx is executed rather than when the proxy announcement is made https://github.com/w3f/1k-validators-be-PRIVATE/issues/34Scoring Changes:
nominatedAt
score from 30 -> 80 https://github.com/w3f/1k-validators-be-PRIVATE/issues/24API Changes:
/candidates/rank
endpoint, that returns the list of all candidates (valid and invalid), ordered by rank https://github.com/w3f/1k-validators-be-PRIVATE/issues/33/locationstats/valid
endpoint, which returns the list of locations from only valid nodes, along with the associated score (this should map 1:1 with a candidates score - so long as the score is also updated around the same time as the endpoint is fetched from. The endpoint calculates it on the fly, so this is a more recent source of 'truth' - when the score of a candidate updates it should be the same as what's gotten from this endpoint when queried from) https://github.com/w3f/1k-validators-be-PRIVATE/issues/32/nominators
endpoint to list the validators of the lastNomination
record - this reflects the validators that are currently on-chain, rather than the validators that will be nominated in the next round. https://github.com/w3f/1k-validators-be-PRIVATE/issues/34Testing:
test/api
to be Jest tests - these are a mix of integration tests and mocks. These are first just to get something working as well as a part of enabling it as a CircleCI check (these tests may be flakey, so if those fail that shouldn't be a blocker yet). So far the tests there pass. This also addstestCore
as a check to CircleCI https://github.com/w3f/1k-validators-be-PRIVATE/issues/2Note: since this adds new CI checks for the candidates file - #2597 should be merged before this so that CI checks for the candidates file will pass (as this introduces both the checks and the updated candidates files)