Merge pull request #1349 from dappnode/pablo/remove-staker-migration

Disallow setStakerConfig migration
dappnode · Jan 10, 2023 · 86b9d72 · 86b9d72
2 parents 4097a92 + 522b25d
commit 86b9d72
Showing 1 changed file with 5 additions and 2 deletions.
diff --git a/packages/dappmanager/src/modules/migrations/index.ts b/packages/dappmanager/src/modules/migrations/index.ts
@@ -1,4 +1,3 @@
-import { setDefaultStakerConfig } from "./setDefaultStakerConfig";
 import AggregateError = require("aggregate-error");
 import { migrateUserActionLogs } from "./migrateUserActionLogs";
 import { removeLegacyDockerAssets } from "./removeLegacyDockerAssets";
@@ -55,7 +54,10 @@ export async function executeMigrations(): Promise<void> {
     })
   );
 
-  setDefaultStakerConfig().catch(e =>
+  /**
+   * WARNING! This migration is dangerous if user has not set a staker config, the docker daemon can crash and is safer to not do it automatically
+   *  
+   setDefaultStakerConfig().catch(e =>
     migrationErrors.push({
       migration: "set default staker configuration",
       coreVersion: "v0.2.58",
@@ -64,6 +66,7 @@ export async function executeMigrations(): Promise<void> {
       stack: e.stack
     })
   );
+   */
 
   switchEthClientIfOpenethereumOrGethLight().catch(e =>
     migrationErrors.push({