Merge pull request #2905 from w3f/optimize-check-candidate

Optimize check candidate
w3f · Apr 26, 2024 · 0e8cda5 · 0e8cda5
2 parents 6f59566 + 628c1d3
commit 0e8cda5
Show file tree

Hide file tree

Showing 9 changed files with 12 additions and 11 deletions.
diff --git a/charts/otv-backend/Chart.yaml b/charts/otv-backend/Chart.yaml
@@ -1,5 +1,5 @@
 description: 1K Validators Backend
 name: otv-backend
-version: v3.2.1
-appVersion: v3.2.1
+version: v3.2.2
+appVersion: v3.2.2
 apiVersion: v2
diff --git a/packages/common/package.json b/packages/common/package.json
@@ -1,6 +1,6 @@
 {
   "name": "@1kv/common",
-  "version": "3.2.1",
+  "version": "3.2.2",
   "description": "Services for running the Thousand Validator Program.",
   "main": "build/index.js",
   "types": "build/index.d.ts",

diff --git a/packages/common/src/constraints/CheckCandidates.ts b/packages/common/src/constraints/CheckCandidates.ts
@@ -34,7 +34,6 @@ export const checkCandidate = async (
 
     const validateValid = await checkValidateIntention(
       constraints.config,
-      constraints.chaindata,
       candidate,
       validators,
     );
@@ -64,7 +63,10 @@ export const checkCandidate = async (
       );
     }
 
-    const identityValid = await checkIdentity(constraints.chaindata, candidate);
+    const identityValid =
+      constraints.config?.constraints?.skipIdentity == true
+        ? true
+        : (await checkIdentity(constraints.chaindata, candidate)) || false;
     if (!identityValid) {
       logger.info(`${candidate.name} identity not valid`, constraintsLabel);
     }

diff --git a/packages/common/src/constraints/ValidityChecks.ts b/packages/common/src/constraints/ValidityChecks.ts
@@ -44,7 +44,6 @@ export const checkOnline = async (candidate: Candidate): Promise<boolean> => {
 // Check the validate intention for a single validator
 export const checkValidateIntention = async (
   config: Config.ConfigSchema,
-  chaindata: ChainData,
   candidate: Candidate,
   validators: string[],
 ): Promise<boolean> => {

diff --git a/packages/core/package.json b/packages/core/package.json
@@ -1,6 +1,6 @@
 {
   "name": "@1kv/core",
-  "version": "3.2.1",
+  "version": "3.2.2",
   "description": "Services for running the Thousand Validator Program.",
   "main": "index.js",
   "scripts": {

diff --git a/packages/gateway/package.json b/packages/gateway/package.json
@@ -1,6 +1,6 @@
 {
   "name": "@1kv/gateway",
-  "version": "3.2.1",
+  "version": "3.2.2",
   "description": "Services for running the Thousand Validator Program.",
   "main": "build/index.js",
   "types": "build/index.d.ts",

diff --git a/packages/scorekeeper-status-ui/package.json b/packages/scorekeeper-status-ui/package.json
@@ -1,7 +1,7 @@
 {
   "name": "@1kv/scorekeeper-status-ui",
   "private": true,
-  "version": "3.2.1",
+  "version": "3.2.2",
   "type": "module",
   "scripts": {
     "dev": "vite",

diff --git a/packages/telemetry/package.json b/packages/telemetry/package.json
@@ -1,6 +1,6 @@
 {
   "name": "@1kv/telemetry",
-  "version": "3.2.1",
+  "version": "3.2.2",
   "description": "Services for running the Thousand Validator Program.",
   "main": "build/index.js",
   "types": "build/index.d.ts",

diff --git a/packages/worker/package.json b/packages/worker/package.json
@@ -1,6 +1,6 @@
 {
   "name": "@1kv/worker",
-  "version": "3.2.1",
+  "version": "3.2.2",
   "description": "Services for running the Thousand Validator Program.",
   "main": "build/index.js",
   "types": "build/index.d.ts",