Merge pull request #88 from lwander/docker-digest-optional

provider/docker: Digests are optional
spinnaker · May 31, 2016 · 4880ea3 · 4880ea3
2 parents f455412 + 6a385bf
commit 4880ea3
Showing 1 changed file with 3 additions and 2 deletions.
diff --git a/...src/main/java/com/netflix/spinnaker/echo/pipelinetriggers/monitor/DockerEventMonitor.java b/...src/main/java/com/netflix/spinnaker/echo/pipelinetriggers/monitor/DockerEventMonitor.java
@@ -67,8 +67,9 @@ public void processEvent(Event event) {
   @Override
   protected boolean isSuccessfulTriggerEvent(final TriggerEvent event) {
     DockerEvent dockerEvent = (DockerEvent) event;
-    String digest = dockerEvent.getContent().getDigest();
-    return digest != null && !digest.isEmpty();
+    // The event should always report a tag
+    String tag = dockerEvent.getContent().getTag();
+    return tag != null && !tag.isEmpty();
   }
 
   @Override