feat(artifacts): add optional release status param to fetching artifa…

…cts (#837)
spinnaker · Jun 26, 2019 · aa0c104 · aa0c104
1 parent de6efc2
commit aa0c104
Show file tree

Hide file tree

Showing 3 changed files with 11 additions and 6 deletions.
diff --git a/gate-web/src/main/groovy/com/netflix/spinnaker/gate/controllers/ArtifactController.java b/gate-web/src/main/groovy/com/netflix/spinnaker/gate/controllers/ArtifactController.java
@@ -73,10 +73,13 @@ List<String> artifactVersions(
   }
 
   @ApiOperation(
-      value = "Retrieve the available artifact versions for an artifact provider and package name")
+      value = "Retrieve the available artifact versions for an artifact provider and package name",
+      notes = "releaseStatus is an optional comma separated list of statuses to filter on.")
   @RequestMapping(value = "/{provider}/{packageName}", method = RequestMethod.GET)
   List<String> getVersionsOfArtifactForProvider(
-      @PathVariable String provider, @PathVariable String packageName) {
-    return artifactService.getVersionsOfArtifactForProvider(provider, packageName);
+      @PathVariable String provider,
+      @PathVariable String packageName,
+      @RequestParam(required = false) String releaseStatus) {
+    return artifactService.getVersionsOfArtifactForProvider(provider, packageName, releaseStatus);
   }
 }
diff --git a/gate-web/src/main/groovy/com/netflix/spinnaker/gate/services/ArtifactService.java b/gate-web/src/main/groovy/com/netflix/spinnaker/gate/services/ArtifactService.java
@@ -98,15 +98,16 @@ public Void getArtifactContents(
         .execute();
   }
 
-  public List<String> getVersionsOfArtifactForProvider(String provider, String packageName) {
+  public List<String> getVersionsOfArtifactForProvider(
+      String provider, String packageName, String releaseStatus) {
     if (!igorService.isPresent()) {
       throw new IllegalStateException(
           "Cannot fetch artifact versions because Igor is not enabled.");
     }
 
     return stringListCommand(
             "artifactVersionsByProvider",
-            () -> igorService.get().getArtifactVersions(provider, packageName))
+            () -> igorService.get().getArtifactVersions(provider, packageName, releaseStatus))
         .execute();
   }
 }
diff --git a/gate-web/src/main/groovy/com/netflix/spinnaker/gate/services/internal/IgorService.groovy b/gate-web/src/main/groovy/com/netflix/spinnaker/gate/services/internal/IgorService.groovy
@@ -69,7 +69,8 @@ interface IgorService {
 
   @GET('/artifacts/{provider}/{packageName}')
   List<String> getArtifactVersions(@Path("provider") String provider,
-                                   @Path("packageName") String packageName);
+                                   @Path("packageName") String packageName,
+                                   @Query("releaseStatus") String releaseStatus);
 
   @GET('/concourse/{buildMaster}/teams/{team}/pipelines/{pipeline}/resources')
   List<String> getConcourseResources(@Path("buildMaster") String buildMaster, @Path("team") String team, @Path("pipeline") String pipeline);