From a134e18e691f408c534e9febf851db0a1a439a75 Mon Sep 17 00:00:00 2001 From: ameyajoshi1 Date: Thu, 9 Jul 2020 12:56:54 +0530 Subject: [PATCH 1/2] Added log lines for debugging purpose --- .../local/JenkinsBrowserStackLocal.java | 33 +++++++++++++++++-- 1 file changed, 31 insertions(+), 2 deletions(-) diff --git a/src/main/java/com/browserstack/automate/ci/jenkins/local/JenkinsBrowserStackLocal.java b/src/main/java/com/browserstack/automate/ci/jenkins/local/JenkinsBrowserStackLocal.java index 41dd87a5..ae901b0b 100644 --- a/src/main/java/com/browserstack/automate/ci/jenkins/local/JenkinsBrowserStackLocal.java +++ b/src/main/java/com/browserstack/automate/ci/jenkins/local/JenkinsBrowserStackLocal.java @@ -6,10 +6,12 @@ import hudson.Launcher; import jenkins.security.MasterToSlaveCallable; import java.io.PrintStream; +import java.io.PrintWriter; import org.apache.commons.lang.StringUtils; import java.io.IOException; import java.io.Serializable; +import java.io.StringWriter; import java.util.ArrayList; import java.util.HashMap; import java.util.List; @@ -80,10 +82,34 @@ private String[] processLocalArguments(final String argString, String buildTag) } public void start() throws Exception { + PluginLogger.log(logger,""); + PluginLogger.log(logger,"Checking local options"); Map localOptions = new HashMap(); localOptions.put("key", accesskey); - if (binarypath != null && binarypath.length() > 0) localOptions.put("binarypath", binarypath); - super.start(localOptions); + + if(accesskey == null || accesskey.length()==0) { + PluginLogger.log(logger,"Access key is empty"); + }else { + PluginLogger.log(logger,"Access key is not empty"); + } + + if (binarypath != null && binarypath.length() > 0) { + localOptions.put("binarypath", binarypath); + PluginLogger.log(logger,"binarypath is "+binarypath); + }else { + PluginLogger.log(logger,"binarypath is not added "+binarypath); + } + + PluginLogger.log(logger,"Starting local"); + try { + super.start(localOptions); + } catch (Exception e) { + StringWriter sw = new StringWriter(); + e.printStackTrace(new PrintWriter(sw)); + String exceptionAsString = sw.toString(); + PluginLogger.log(logger,"Actual Exception is\n" + exceptionAsString); + throw e; + } } public void stop() throws Exception { @@ -94,10 +120,13 @@ public void stop() throws Exception { } public void start(Launcher launcher) throws Exception { + PluginLogger.log(logger,"will be fetching launcher channels and calling start"); launcher.getChannel().call(new MasterToSlaveCallable() { @Override public Void call() throws Exception { + PluginLogger.log(logger,"will call actual start method"); JenkinsBrowserStackLocal.this.start(); + PluginLogger.log(logger,"start call completed"); return null; } }); From ffcf5058e5b4180a297cd9b4c1a8ed565369b51a Mon Sep 17 00:00:00 2001 From: ameyajoshi1 Date: Thu, 9 Jul 2020 14:36:23 +0530 Subject: [PATCH 2/2] Updated log lines --- .../automate/ci/jenkins/local/JenkinsBrowserStackLocal.java | 3 +-- 1 file changed, 1 insertion(+), 2 deletions(-) diff --git a/src/main/java/com/browserstack/automate/ci/jenkins/local/JenkinsBrowserStackLocal.java b/src/main/java/com/browserstack/automate/ci/jenkins/local/JenkinsBrowserStackLocal.java index ae901b0b..64bbb5ac 100644 --- a/src/main/java/com/browserstack/automate/ci/jenkins/local/JenkinsBrowserStackLocal.java +++ b/src/main/java/com/browserstack/automate/ci/jenkins/local/JenkinsBrowserStackLocal.java @@ -82,7 +82,6 @@ private String[] processLocalArguments(final String argString, String buildTag) } public void start() throws Exception { - PluginLogger.log(logger,""); PluginLogger.log(logger,"Checking local options"); Map localOptions = new HashMap(); localOptions.put("key", accesskey); @@ -97,7 +96,7 @@ public void start() throws Exception { localOptions.put("binarypath", binarypath); PluginLogger.log(logger,"binarypath is "+binarypath); }else { - PluginLogger.log(logger,"binarypath is not added "+binarypath); + PluginLogger.log(logger,"binarypath is not added"); } PluginLogger.log(logger,"Starting local");