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

v3.0.19 #2586

Merged
merged 50 commits into from
Feb 19, 2024
Merged

v3.0.19 #2586

merged 50 commits into from
Feb 19, 2024

Conversation

wpank
Copy link
Contributor

@wpank wpank commented Feb 13, 2024

Candidates Config and DB Changes:

  • updates candidate files to have a slotId and kyc field (set to false if they have not been set yet) https://github.com/w3f/1k-validators-be-PRIVATE/issues/16
    • unique on candidates not set yet - there needs to be separations of non-candidates telemetry nodes (which will otherwise have null slotIds to get implemented first)
  • check for duplicates of stash or telemetry name at startup (will log list if duplicates are found) https://github.com/w3f/1k-validators-be-PRIVATE/issues/21
  • Updates CI check to make sure all candidates in the config file have a slotId and kyc field
  • adds scripts for helping generate a config json with those fields
  • After this is deployed, any candidate with the same slotId 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/3

Misc Refactoring:

Consistency FIxes:

Scoring Changes:

API Changes:

  • adds /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
  • adds /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
  • adjust the /nominators endpoint to list the validators of the last Nomination 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/34

Testing:

  • refactor some of the existing tests that were in 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 adds testCore as a check to CircleCI https://github.com/w3f/1k-validators-be-PRIVATE/issues/2

Note: 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)

@wpank wpank changed the title v3.0.14 v3.0.15 Feb 14, 2024
@wpank wpank changed the title v3.0.15 v3.0.16 Feb 14, 2024
@wpank wpank changed the title v3.0.16 v3.0.17 Feb 14, 2024
ironoa referenced this pull request Feb 16, 2024
fix nominator without current targets
@wpank wpank changed the title v3.0.17 v3.0.19 Feb 16, 2024
@ironoa

This comment was marked as resolved.

@w3f w3f deleted a comment from gitguardian bot Feb 19, 2024

This comment was marked as resolved.

@wpank wpank merged commit ce25b97 into master Feb 19, 2024
10 checks passed
@wpank wpank deleted the will-v3.0.14 branch February 19, 2024 20:29
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.

2 participants