Bypass finding master for api type (#381)

Issue calls directly against input host when API used due to issues with Cloud, EDE, ECK clusters.
elastic · Mar 12, 2020 · cedaa19 · cedaa19
1 parent 0e684c0
commit cedaa19
Show file tree

Hide file tree

Showing 3 changed files with 3 additions and 9 deletions.
diff --git a/pom.xml b/pom.xml
@@ -4,7 +4,7 @@
 
     <groupId>com.elasticsearch</groupId>
     <artifactId>support-diagnostics</artifactId>
-    <version>8.0.0</version>
+    <version>8.0.1</version>
     <packaging>jar</packaging>
     <name>Support Diagnostics Utilities</name>
     <properties>

diff --git a/src/main/java/com/elastic/support/diagnostics/chain/DiagnosticChainExec.java b/src/main/java/com/elastic/support/diagnostics/chain/DiagnosticChainExec.java
@@ -21,7 +21,8 @@ public static void runDiagnostic(DiagnosticContext context, String type) {
                 case Constants.api :
                     new CheckElasticsearchVersion().execute(context);
                     new CheckUserAuthLevel().execute(context);
-                    new CheckPlatformDetails().execute(context);
+                    // Removed temporarily due to issues with finding and accessing cloud master
+                    //new CheckPlatformDetails().execute(context);
                     new RunClusterQueries().execute(context);
                     break;
 

diff --git a/src/main/java/com/elastic/support/diagnostics/commands/CheckPlatformDetails.java b/src/main/java/com/elastic/support/diagnostics/commands/CheckPlatformDetails.java
@@ -111,7 +111,6 @@ public void execute(DiagnosticContext context) {
                         targetOS = context.targetNode.os;
                     }
 
-
                     syscmd = new RemoteSystem(
                             targetOS,
                             context.diagnosticInputs.remoteUser,
@@ -153,12 +152,6 @@ public void execute(DiagnosticContext context) {
 
                     break;
 
-                case Constants.api:
-                    context.targetNode = findRemoteTargetNode(
-                            context.diagnosticInputs.host, nodeProfiles);
-                    context.runSystemCalls = false;
-                    break;
-
                 default:
                     // If it's not one of the above types it shouldn't be here but try to keep going...
                     context.runSystemCalls = false;