From 5677fec63e4adb23de1928c1b2f6f0d4ecbd5f41 Mon Sep 17 00:00:00 2001 From: Eric Zimanyi Date: Mon, 21 May 2018 12:29:29 -0400 Subject: [PATCH] fix(artifacts): Include artifact reference in bake key (#263) This ensures that we are considering the artifacts when deciding whether a requested bake is a duplicate of a prior bake. --- .../spinnaker/rosco/providers/CloudProviderBakeHandler.groovy | 4 ++++ 1 file changed, 4 insertions(+) diff --git a/rosco-core/src/main/groovy/com/netflix/spinnaker/rosco/providers/CloudProviderBakeHandler.groovy b/rosco-core/src/main/groovy/com/netflix/spinnaker/rosco/providers/CloudProviderBakeHandler.groovy index 86c3195c5..499d12ef8 100644 --- a/rosco-core/src/main/groovy/com/netflix/spinnaker/rosco/providers/CloudProviderBakeHandler.groovy +++ b/rosco-core/src/main/groovy/com/netflix/spinnaker/rosco/providers/CloudProviderBakeHandler.groovy @@ -89,6 +89,10 @@ abstract class CloudProviderBakeHandler { // Package name is always part of key, even if it is an empty string keys << (package_name ? package_name.tokenize().join('|') : "") + // If any artifacts to bake were specified, include them in the key as well + if (package_artifacts) { + keys << package_artifacts.collect { it.getReference() ?: "" }.join('|') + } String providerSpecificBakeKeyComponent = produceProviderSpecificBakeKeyComponent(region, bakeRequest) if (providerSpecificBakeKeyComponent) {