[Deprecation] Deprecate setupMaster in favor of setupClusterManager

As part of the meta issue opensearch-project/OpenSearch#2589 to track the plan and progress of applying inclusive naming across OpenSearch Repositories We are deprecating setupMaster in favor of setupClusterManager Signed-off-by: manasvinibs <[email protected]>
manasvinibs · Jun 15, 2022 · 13aed55 · 13aed55
1 parent 72a3424
commit 13aed55
Show file tree

Hide file tree

Showing 3 changed files with 5 additions and 2 deletions.
diff --git a/src/cli/cluster/cluster.mock.ts b/src/cli/cluster/cluster.mock.ts
@@ -78,5 +78,7 @@ class MockClusterFork extends EventEmitter {
 
 export class MockCluster extends EventEmitter {
   fork = jest.fn(() => new MockClusterFork(this));
+  /** @deprecated use setupClusterManager */
   setupMaster = jest.fn();
+  setupClusterManager = jest.fn();
 }
diff --git a/src/cli/cluster/cluster_manager.ts b/src/cli/cluster/cluster_manager.ts
@@ -135,7 +135,7 @@ export class ClusterManager {
     });
 
     // When receive that event from server worker
-    // forward a reloadLoggingConfig message to master
+    // forward a reloadLoggingConfig message to cluster manager
     // and all workers. This is only used by LogRotator service
     // when the cluster mode is enabled
     this.server.on('reloadLoggingConfigFromServerWorker', () => {

diff --git a/src/cli/cluster/worker.ts b/src/cli/cluster/worker.ts
@@ -30,6 +30,7 @@
 
 import _ from 'lodash';
 import cluster from 'cluster';
+import { setupMaster as setupClusterManager } from 'cluster';
 import { EventEmitter } from 'events';
 
 import { BinderFor } from './binder_for';
@@ -44,7 +45,7 @@ export type ClusterWorker = cluster.Worker & {
   exitCode?: number;
 };
 
-cluster.setupMaster({
+setupClusterManager({
   exec: cliPath,
   silent: false,
 });