Skip to content

Commit

Permalink
Support krane annotations for predeployed
Browse files Browse the repository at this point in the history
  • Loading branch information
adrianna-chang-shopify committed Aug 28, 2019
1 parent e03a947 commit fb8ec30
Show file tree
Hide file tree
Showing 2 changed files with 13 additions and 5 deletions.
Original file line number Diff line number Diff line change
Expand Up @@ -56,6 +56,11 @@ def prunable?
prunable == "true"
end

def predeployed?
predeployed = predeployed_annotation
predeployed.nil? || predeployed == "true"
end

def prunable_annotation
if @definition.dig("metadata", "annotations").key?("kubernetes-deploy.shopify.io/prunable")
@definition.dig("metadata", "annotations", "kubernetes-deploy.shopify.io/prunable")
Expand All @@ -64,9 +69,12 @@ def prunable_annotation
end
end

def predeployed?
predeployed = @definition.dig("metadata", "annotations", "kubernetes-deploy.shopify.io/predeployed")
predeployed.nil? || predeployed == "true"
def predeployed_annotation
if @definition.dig("metadata", "annotations").key?("kubernetes-deploy.shopify.io/predeployed")
@definition.dig("metadata", "annotations", "kubernetes-deploy.shopify.io/predeployed")
elsif @definition.dig("metadata", "annotations").key?("krane.shopify.io/predeployed")
@definition.dig("metadata", "annotations", "krane.shopify.io/predeployed")
end
end

def rollout_conditions
Expand Down
4 changes: 2 additions & 2 deletions test/integration-serial/serial_deploy_test.rb
Original file line number Diff line number Diff line change
Expand Up @@ -225,12 +225,12 @@ def test_custom_resources_predeployed

things = f.dig("things.yml", "CustomResourceDefinition").first
things["metadata"]["annotations"] = {
"kubernetes-deploy.shopify.io/predeployed" => "true",
"krane.shopify.io/predeployed" => "true",
}

widgets = f.dig("widgets.yml", "CustomResourceDefinition").first
widgets["metadata"]["annotations"] = {
"kubernetes-deploy.shopify.io/predeployed" => "false",
"krane.shopify.io/predeployed" => "false",
}
end)
reset_logger
Expand Down

0 comments on commit fb8ec30

Please sign in to comment.