From 21722e5619a11431c1a69f2ba0d2557dc4e6f85b Mon Sep 17 00:00:00 2001
From: David Hill <david.hill@mariadb.com>
Date: Thu, 8 Dec 2016 09:08:54 -0600
Subject: [PATCH] MCOL-453

---
 oamapps/mcsadmin/mcsadmin.cpp | 36 ++++++++++++++++++++++++++++++++++-
 1 file changed, 35 insertions(+), 1 deletion(-)

diff --git a/oamapps/mcsadmin/mcsadmin.cpp b/oamapps/mcsadmin/mcsadmin.cpp
index e333b8cbd..25d343102 100644
--- a/oamapps/mcsadmin/mcsadmin.cpp
+++ b/oamapps/mcsadmin/mcsadmin.cpp
@@ -1889,6 +1889,13 @@ int processCommand(string* arguments)
 
         case 14: // addDbroot parameters: dbroot-number
         {
+            parentOAMModule = getParentOAMModule();
+            if ( localModule != parentOAMModule ) {
+                //exit out since not on Parent OAM Module
+                cout << endl << "**** addDbroot Failed : only should be run on the Parent OAM Module, which is '" << parentOAMModule << "'" << endl;
+                break;
+            }
+            
 		string GlusterConfig = "n";
 		try {
 			oam.getSystemConfig( "GlusterConfig", GlusterConfig);
@@ -1978,6 +1985,13 @@ int processCommand(string* arguments)
 
         case 15: // removeDbroot parameters: dbroot-list
         {
+		parentOAMModule = getParentOAMModule();
+            if ( localModule != parentOAMModule ) {
+                //exit out since not on Parent OAM Module
+                cout << endl << "**** removeDbroot Failed : only should be run on the Parent OAM Module, which is '" << parentOAMModule << "'" << endl;
+                break;
+            }
+
 		string GlusterConfig = "n";
 		try {
 			oam.getSystemConfig( "GlusterConfig", GlusterConfig);
@@ -4990,6 +5004,12 @@ int processCommand(string* arguments)
         case 48: // addModule - parameters: Module type/Module Name, Number of Modules, Server Hostnames,
 					// Server root password optional
         {
+			parentOAMModule = getParentOAMModule();
+            if ( localModule != parentOAMModule ) {
+                // exit out since not on Parent OAM Module
+                cout << endl << "**** addModule Failed : only should be run on the Parent OAM Module, which is '" << parentOAMModule << "'" << endl;
+                break;
+            }
             if (arguments[1] == "")
             {
                 // need at least  arguments
@@ -6605,6 +6625,13 @@ int processCommand(string* arguments)
 
         case 65: // alterSystem-disableModule
         {
+			parentOAMModule = getParentOAMModule();
+            if ( localModule != parentOAMModule ) {
+                //exit out since not on Parent OAM Module
+                cout << endl << "**** alterSystem-disableModule Failed : only should be run on the Parent OAM Module, which is '" << parentOAMModule << "'" << endl;
+                break;
+            }
+
             if (arguments[1] == "")
             {
                 // need arguments
@@ -6748,7 +6775,14 @@ int processCommand(string* arguments)
 
         case 66: // alterSystem-enableModule
         {
-            if (arguments[1] == "")
+        	parentOAMModule = getParentOAMModule();
+            if ( localModule != parentOAMModule ) {
+                //exit out since not on Parent OAM Module
+                cout << endl << "**** alterSystem-enableModule Failed : only should be run on the Parent OAM Module, which is '" << parentOAMModule << "'" << endl;
+                break;
+            }
+
+		    if (arguments[1] == "")
             {
                 // need arguments
                 cout << endl << "**** alterSystem-enableModule Failed : Missing a required Parameter, enter 'help' for additional information" << endl;