Revert "Tank API V2 Clients Release (#245)"

This reverts commit ec74378.
intuit · Jun 30, 2023 · 08606ce · 08606ce
1 parent ec74378
commit 08606ce
Show file tree

Hide file tree

Showing 126 changed files with 424 additions and 3,547 deletions.
diff --git a/agent/agent_standalone/pom.xml b/agent/agent_standalone/pom.xml
@@ -16,46 +16,8 @@
   <dependencies>
     <dependency>
       <groupId>${project.groupId}</groupId>
-      <artifactId>tank-api</artifactId>
+      <artifactId>agent-client</artifactId>
       <version>${project.version}</version>
-      <exclusions>
-        <exclusion>
-            <groupId>org.springdoc</groupId>
-            <artifactId>springdoc-openapi-ui</artifactId>
-        </exclusion>
-        <exclusion>
-          <groupId>${project.groupId}</groupId>
-          <artifactId>script-engine</artifactId>
-        </exclusion>
-        <exclusion>
-          <groupId>${project.groupId}</groupId>
-          <artifactId>data-access</artifactId>
-        </exclusion>
-        <exclusion>
-          <groupId>${project.groupId}</groupId>
-          <artifactId>tank-vm-manager</artifactId>
-        </exclusion>
-        <exclusion>
-          <groupId>${project.groupId}</groupId>
-          <artifactId>tank-script-processor</artifactId>
-        </exclusion>
-        <exclusion>
-          <groupId>${project.groupId}</groupId>
-          <artifactId>api</artifactId>
-        </exclusion>
-        <exclusion>
-          <groupId>org.projectlombok</groupId>
-          <artifactId>lombok</artifactId>
-        </exclusion>
-        <exclusion>
-          <groupId>org.apache.logging.log4j</groupId>
-          <artifactId>log4j-core</artifactId>
-        </exclusion>
-        <exclusion>
-          <groupId>jakarta.xml.bind</groupId>
-          <artifactId>jakarta.xml.bind-api</artifactId>
-        </exclusion>
-      </exclusions>
     </dependency>
 
     <dependency>

diff --git a/...ent_standalone/src/main/java/com/intuit/tank/standalone/agent/StandaloneAgentStartup.java b/...ent_standalone/src/main/java/com/intuit/tank/standalone/agent/StandaloneAgentStartup.java
@@ -28,7 +28,7 @@
 import org.apache.logging.log4j.LogManager;
 import org.apache.logging.log4j.Logger;
 
-import com.intuit.tank.rest.mvc.rest.clients.AgentClient;
+import com.intuit.tank.AgentServiceClient;
 import com.intuit.tank.harness.HostInfo;
 import com.intuit.tank.vm.agent.messages.AgentAvailability;
 import com.intuit.tank.vm.agent.messages.AgentAvailabilityStatus;
@@ -37,23 +37,23 @@
 public class StandaloneAgentStartup implements Runnable {
 
     private static Logger LOG = LogManager.getLogger(StandaloneAgentStartup.class);
-    public static final String SERVICE_RELATIVE_PATH = "/v2/agent";
+    public static final String SERVICE_RELATIVE_PATH = "/rest/v1/agent-service";
     private static String API_HARNESS_COMMAND = "./startAgent.sh";
     public static final String METHOD_SETTINGS = "/settings";
-    public static final String METHOD_SUPPORT = "/support-files";
+    public static final String METHOD_SUPPORT = "/supportFiles";
     private static final long PING_TIME = 1000 * 60 * 5;// five minutes
 
     private String controllerBase;
     private AgentAvailability currentAvailability;
-    private AgentClient agentClient;
+    private AgentServiceClient agentClient;
     private String instanceId;
     private String hostname;
     private int capacity = 4000;
 
     @Override
     public void run() {
         CommandListener.startHttpServer(CommandListener.PORT, this);
-        agentClient = new AgentClient(controllerBase);
+        agentClient = new AgentServiceClient(controllerBase);
 
         if (hostname != null) {
             instanceId = hostname;
@@ -152,7 +152,7 @@ private void sendAvailability() {
                 currentAvailability.getInstanceUrl(), currentAvailability.getCapacity(),
                 currentAvailability.getAvailabilityStatus());
         LOG.info("Sending availaability: " + ToStringBuilder.reflectionToString(availability));
-        agentClient.setStandaloneAgentAvailability(availability);
+        agentClient.standaloneAgentAvailable(availability);
     }
 
     public static void main(String[] args) {

diff --git a/agent/agent_standalone_pkg/src/main/assembly/assembly.xml b/agent/agent_standalone_pkg/src/main/assembly/assembly.xml
@@ -16,11 +16,6 @@
       <useProjectArtifact>true</useProjectArtifact>
       <unpack>true</unpack>
       <scope>runtime</scope>
-      <unpackOptions>
-        <excludes>
-          <exclude>**/Log4j2Plugins.dat</exclude>
-        </excludes>
-      </unpackOptions>
     </dependencySet>
   </dependencySets>
 </assembly>
diff --git a/agent/agent_startup/src/main/java/com/intuit/tank/agent/AgentStartup.java b/agent/agent_startup/src/main/java/com/intuit/tank/agent/AgentStartup.java
@@ -34,10 +34,10 @@
 
 public class AgentStartup implements Runnable {
     private static final Logger logger = LogManager.getLogger(AgentStartup.class);
-    private static final String SERVICE_RELATIVE_PATH = "/v2/agent";
+    private static final String SERVICE_RELATIVE_PATH = "/rest/v1/agent-service";
     private static final String METHOD_SETTINGS = "/settings";
     private static final String API_HARNESS_COMMAND = "./startAgent.sh";
-    private static final String METHOD_SUPPORT = "/support-files";
+    private static final String METHOD_SUPPORT = "/supportFiles";
     private static final int[] FIBONACCI = new int[] { 1, 1, 2, 3, 5, 8, 13 };
 
     private final String controllerBaseUrl;
@@ -120,8 +120,5 @@ private static void usage() {
         System.out.println("Tank Test Startup Usage:");
         System.out.println("java -cp agent-startup-pkg-1.0-all.jar com/intuit/tank/agent/AgentStartup <options>");
         System.out.println("-controller=<controller_base_url>:  The url of the controller to get test info from");
-        System.out.println("Service Path: " + SERVICE_RELATIVE_PATH);
-        System.out.println("Settings Method: " + METHOD_SETTINGS);
-        System.out.println("Support Files Method: " + METHOD_SUPPORT);
     }
 }
diff --git a/agent/apiharness/pom.xml b/agent/apiharness/pom.xml
@@ -27,50 +27,6 @@
       <version>${project.version}</version>
     </dependency>
 
-    <dependency>
-      <groupId>${project.groupId}</groupId>
-      <artifactId>tank-api</artifactId>
-      <version>${project.version}</version>
-      <exclusions>
-        <exclusion>
-          <groupId>org.springdoc</groupId>
-          <artifactId>springdoc-openapi-ui</artifactId>
-        </exclusion>
-        <exclusion>
-          <groupId>${project.groupId}</groupId>
-          <artifactId>script-engine</artifactId>
-        </exclusion>
-        <exclusion>
-          <groupId>${project.groupId}</groupId>
-          <artifactId>data-access</artifactId>
-        </exclusion>
-        <exclusion>
-          <groupId>${project.groupId}</groupId>
-          <artifactId>tank-vm-manager</artifactId>
-        </exclusion>
-        <exclusion>
-          <groupId>${project.groupId}</groupId>
-          <artifactId>tank-script-processor</artifactId>
-        </exclusion>
-        <exclusion>
-          <groupId>${project.groupId}</groupId>
-          <artifactId>api</artifactId>
-        </exclusion>
-        <exclusion>
-          <groupId>org.projectlombok</groupId>
-          <artifactId>lombok</artifactId>
-        </exclusion>
-        <exclusion>
-          <groupId>org.apache.logging.log4j</groupId>
-          <artifactId>log4j-core</artifactId>
-        </exclusion>
-        <exclusion>
-          <groupId>jakarta.xml.bind</groupId>
-          <artifactId>jakarta.xml.bind-api</artifactId>
-        </exclusion>
-      </exclusions>
-    </dependency>
-
     <dependency>
       <groupId>${project.groupId}</groupId>
       <artifactId>agent-client</artifactId>
@@ -125,4 +81,5 @@
       <version>${project.version}</version>
     </dependency>
   </dependencies>
+
 </project>
diff --git a/agent/apiharness/src/main/java/com/intuit/tank/harness/APIMonitor.java b/agent/apiharness/src/main/java/com/intuit/tank/harness/APIMonitor.java
@@ -18,10 +18,10 @@
 import org.apache.logging.log4j.LogManager;
 import org.apache.logging.log4j.Logger;
 
-import com.intuit.tank.rest.mvc.rest.clients.AgentClient;
-import com.intuit.tank.vm.vmManager.models.CloudVmStatus;
-import com.intuit.tank.vm.vmManager.models.VMStatus;
-import com.intuit.tank.vm.vmManager.models.ValidationStatus;
+import com.intuit.tank.CloudServiceClient;
+import com.intuit.tank.api.model.v1.cloud.CloudVmStatus;
+import com.intuit.tank.api.model.v1.cloud.VMStatus;
+import com.intuit.tank.api.model.v1.cloud.ValidationStatus;
 import com.intuit.tank.harness.logging.LogUtil;
 import com.intuit.tank.reporting.api.TPSInfoContainer;
 import com.intuit.tank.vm.agent.messages.WatsAgentStatusResponse;
@@ -36,7 +36,7 @@ public class APIMonitor implements Runnable {
     private static final int MIN_REPORT_TIME = 15000;
     private static Logger LOG = LogManager.getLogger(APIMonitor.class);
     private static boolean doMonitor = true;
-    private static AgentClient client;
+    private static CloudServiceClient client;
     private static CloudVmStatus status;
     private long reportInterval = APIMonitor.MIN_REPORT_TIME;
     private boolean isLocal;
@@ -45,7 +45,7 @@ public APIMonitor(Boolean isLocal, CloudVmStatus vmStatus) {
         this.isLocal = isLocal;
         status = vmStatus;
         try {
-            client = new AgentClient(APITestHarness.getInstance().getTankConfig().getControllerBase());
+            client = new CloudServiceClient(APITestHarness.getInstance().getTankConfig().getControllerBase());
             reportInterval = Math.max(APITestHarness.getInstance().getTankConfig().getAgentConfig()
                     .getStatusReportIntervalMilis(reportInterval), MIN_REPORT_TIME);
         } catch (Exception e) {
@@ -65,15 +65,15 @@ public void run() {
                     newStatus.setTotalTps(tpsInfo.getTotalTps());
                     sendTps(tpsInfo);
                 }
-                if (!isLocal) client.setInstanceStatus(newStatus.getInstanceId(), newStatus);
+                if (!isLocal) client.setVmStatus(newStatus.getInstanceId(), newStatus);
                 APITestHarness.getInstance().checkAgentThreads();
                 Thread.sleep(reportInterval);
             } catch (Exception t) {
                 LOG.error(LogUtil.getLogMessage("Unable to send status metrics | " + t.getMessage()), t);
             }
         }
         CloudVmStatus newStatus = createStatus(APITestHarness.getInstance().getStatus());
-        client.setInstanceStatus(newStatus.getInstanceId(), newStatus);
+        client.setVmStatus(newStatus.getInstanceId(), newStatus);
     }
 
     private void sendTps(final TPSInfoContainer tpsInfo) {
@@ -136,7 +136,7 @@ public synchronized static void setJobStatus(JobStatus jobStatus) {
                         stats.getMaxVirtualUsers(),
                         stats.getCurrentNumberUsers(), status.getStartTime(), endTime);
                 status.setUserDetails(APITestHarness.getInstance().getUserTracker().getSnapshot());
-                client.setInstanceStatus(status.getInstanceId(), status);
+                client.setVmStatus(status.getInstanceId(), status);
             } catch (Exception e) {
                 LOG.error("Error sending status to controller: " + e.toString(), e);
             }

diff --git a/agent/apiharness/src/main/java/com/intuit/tank/harness/APITestHarness.java b/agent/apiharness/src/main/java/com/intuit/tank/harness/APITestHarness.java
@@ -41,10 +41,10 @@
 import org.apache.logging.log4j.core.config.LoggerConfig;
 import org.apache.logging.log4j.message.ObjectMessage;
 
-import com.intuit.tank.rest.mvc.rest.clients.AgentClient;
-import com.intuit.tank.vm.vmManager.models.CloudVmStatus;
-import com.intuit.tank.vm.vmManager.models.VMStatus;
-import com.intuit.tank.vm.vmManager.models.ValidationStatus;
+import com.intuit.tank.AgentServiceClient;
+import com.intuit.tank.api.model.v1.cloud.CloudVmStatus;
+import com.intuit.tank.api.model.v1.cloud.VMStatus;
+import com.intuit.tank.api.model.v1.cloud.ValidationStatus;
 import com.intuit.tank.harness.data.HDTestPlan;
 import com.intuit.tank.harness.data.HDWorkload;
 import com.intuit.tank.harness.logging.LogUtil;
@@ -252,7 +252,7 @@ private void startHttp(String baseUrl) {
         if (baseUrl == null) {
             baseUrl = AmazonUtil.getControllerBaseUrl();
         }
-        AgentClient client = new AgentClient(baseUrl);
+        AgentServiceClient client = new AgentServiceClient(baseUrl);
         String instanceUrl = null;
         int retryCount = 0;
         while (instanceUrl == null) {

diff --git a/agent/apiharness/src/main/java/com/intuit/tank/harness/UserTracker.java b/agent/apiharness/src/main/java/com/intuit/tank/harness/UserTracker.java
@@ -19,7 +19,7 @@
 import java.util.Map;
 import java.util.stream.Collectors;
 
-import com.intuit.tank.vm.vmManager.models.UserDetail;
+import com.intuit.tank.api.model.v1.cloud.UserDetail;
 
 /**
  * 

diff --git a/agent/apiharness/src/test/java/com/intuit/tank/harness/APIMonitorTest.java b/agent/apiharness/src/test/java/com/intuit/tank/harness/APIMonitorTest.java
@@ -1,8 +1,8 @@
 package com.intuit.tank.harness;
 
-import com.intuit.tank.vm.vmManager.models.CloudVmStatus;
-import com.intuit.tank.vm.vmManager.models.VMStatus;
-import com.intuit.tank.vm.vmManager.models.ValidationStatus;
+import com.intuit.tank.api.model.v1.cloud.CloudVmStatus;
+import com.intuit.tank.api.model.v1.cloud.VMStatus;
+import com.intuit.tank.api.model.v1.cloud.ValidationStatus;
 import com.intuit.tank.vm.api.enumerated.JobStatus;
 import com.intuit.tank.vm.api.enumerated.VMImageType;
 import com.intuit.tank.vm.api.enumerated.VMRegion;

diff --git a/agent/apiharness/src/test/java/com/intuit/tank/harness/UserTrackerTest.java b/agent/apiharness/src/test/java/com/intuit/tank/harness/UserTrackerTest.java
@@ -18,7 +18,7 @@
 
 import java.util.List;
 
-import com.intuit.tank.vm.vmManager.models.UserDetail;
+import com.intuit.tank.api.model.v1.cloud.UserDetail;
 import com.intuit.tank.test.TestGroups;
 import org.junit.jupiter.api.Tag;
 import org.junit.jupiter.api.Test;

diff --git a/agent/apiharness_pkg/src/main/assembly/assembly.xml b/agent/apiharness_pkg/src/main/assembly/assembly.xml
@@ -16,11 +16,6 @@
       <useProjectArtifact>true</useProjectArtifact>
       <unpack>true</unpack>
       <scope>runtime</scope>
-      <unpackOptions>
-        <excludes>
-          <exclude>**/Log4j2Plugins.dat</exclude>
-        </excludes>
-      </unpackOptions>
     </dependencySet>
   </dependencySets>
 </assembly>
diff --git a/api/src/main/java/com/intuit/tank/vm/api/service/v1/project/ProjectServiceUrlBuilder.java b/api/src/main/java/com/intuit/tank/vm/api/service/v1/project/ProjectServiceUrlBuilder.java
@@ -39,7 +39,7 @@ private ProjectServiceUrlBuilder() {
      */
     public static String getScriptXmlUrl(String jobId) {
         String baseUrl = new TankConfig().getControllerBase();
-        return baseUrl + "/v2/jobs/script/" + jobId;
+        return baseUrl + "/rest/v1/project-service/script/" + jobId;
     }
 
 }