Merge pull request #328 from keel-hq/feature/helm_approvals_deadline

fixed a problem with helm approvals not getting default deadline
keel-hq · Dec 14, 2018 · ac6f5d4 · ac6f5d4
2 parents d0fc99c + 881962e
commit ac6f5d4
Showing 1 changed file with 4 additions and 0 deletions.
diff --git a/provider/helm/approvals.go b/provider/helm/approvals.go
@@ -51,6 +51,10 @@ func (p *Provider) isApproved(event *types.Event, plan *UpdatePlan) (bool, error
 	if err != nil {
 		if err == cache.ErrNotFound {
 
+			if plan.Config.ApprovalDeadline == 0 {
+				plan.Config.ApprovalDeadline = types.KeelApprovalDeadlineDefault
+			}
+
 			// creating new one
 			approval := &types.Approval{
 				Provider:       types.ProviderTypeHelm,