feat(api): add cds.template.version var for current template version (#4927)

Coenen Benjamin · web-flow · commit 0bcfe739e9c2 · 2020-01-29T10:03:23.000+01:00
Signed-off-by: Benjamin Coenen &lt;5719034+bnjjj@users.noreply.github.com&gt;
diff --git a/cli/cdsctl/template.go b/cli/cdsctl/template.go
@@ -27,7 +27,7 @@ func template() *cobra.Command {
 		cli.NewCommand(templateBulkCmd, templateBulkRun, nil, withAllCommandModifiers()...),
 		cli.NewCommand(templatePullCmd, templatePullRun, nil, withAllCommandModifiers()...),
 		cli.NewCommand(templatePushCmd, templatePushRun, nil, withAllCommandModifiers()...),
-		cli.NewCommand(templateDeleteCmd, templateDeleteRun, nil, withAllCommandModifiers()...),
+		cli.NewDeleteCommand(templateDeleteCmd, templateDeleteRun, nil, withAllCommandModifiers()...),
 		cli.NewListCommand(templateInstancesCmd, templateInstancesRun, nil, withAllCommandModifiers()...),
 		cli.NewCommand(templateDetachCmd, templateDetachRun, nil, withAllCommandModifiers()...),
 	})
@@ -212,21 +212,28 @@ var templateDeleteCmd = cli.Command{
 func templateDeleteRun(v cli.Values) error {
 	wt, err := getTemplateFromCLI(v)
 	if err != nil {
+		if v.GetBool("force") && sdk.ErrorIs(err, sdk.ErrNotFound) {
+			return nil
+		}
 		return err
 	}
 	if wt == nil {
 		wt, err = suggestTemplate()
 		if err != nil {
+			if v.GetBool("force") && sdk.ErrorIs(err, sdk.ErrNotFound) {
+				return nil
+			}
 			return err
 		}
 	}
 
 	if err := client.TemplateDelete(wt.Group.Name, wt.Slug); err != nil {
+		if v.GetBool("force") && sdk.ErrorIs(err, sdk.ErrNotFound) {
+			return nil
+		}
 		return err
 	}
 
-	fmt.Println("Template successfully deleted")
-
 	return nil
 }
 
diff --git a/engine/api/workflow/process_parameters.go b/engine/api/workflow/process_parameters.go
@@ -220,6 +220,10 @@ func getNodeRunBuildParameters(ctx context.Context, proj *sdk.Project, wr *sdk.W
 	tmp["cds.run.number"] = fmt.Sprintf("%d", run.Number)
 	tmp["cds.run.subnumber"] = fmt.Sprintf("%d", run.SubNumber)
 
+	if wr.Workflow.TemplateInstance != nil {
+		tmp["cds.template.version"] = fmt.Sprintf("%d", wr.Workflow.TemplateInstance.WorkflowTemplateVersion)
+	}
+
 	_, next := observability.Span(ctx, "workflow.interpolate")
 	params = make([]sdk.Parameter, 0, len(tmp))
 	for k, v := range tmp {
diff --git a/engine/api/workflow_run.go b/engine/api/workflow_run.go
@@ -11,6 +11,7 @@ import (
 	"time"
 
 	"github.com/ovh/cds/engine/api/permission"
+	"github.com/ovh/cds/engine/api/workflowtemplate"
 
 	"github.com/go-gorp/gorp"
 	"github.com/gorilla/mux"
@@ -924,6 +925,10 @@ func (api *API) postWorkflowRunHandler() service.Handler {
 				return sdk.WrapError(errWf, "unable to load workflow %s", name)
 			}
 
+			if err := workflowtemplate.AggregateTemplateInstanceOnWorkflow(ctx, api.mustDB(), wf); err != nil {
+				return sdk.WrapError(err, "cannot load workflow template")
+			}
+
 			// Check node permission
 			if isService := isService(ctx); !isService && !permission.AccessToWorkflowNode(ctx, api.mustDB(), wf, &wf.WorkflowData.Node, getAPIConsumer(ctx), sdk.PermissionReadExecute) {
 				return sdk.WrapError(sdk.ErrNoPermExecution, "not enough right on node %s", wf.WorkflowData.Node.Name)
diff --git a/tests/04_sc_workflow_template_run.yml b/tests/04_sc_workflow_template_run.yml
@@ -0,0 +1,49 @@
+name: Push a template and run this template
+version: "2"
+testcases:
+- name: prepare test
+  steps:
+  - script: "{{.cdsctl}} -f {{.cdsctl.config}} template delete shared.infra/demo-workflow-hello-world --force"
+  - script: "{{.cdsctl}} -f {{.cdsctl.config}} project remove --force 04SCWORKFLOWTEMPLATERUN"
+  - script: "{{.cdsctl}} -f {{.cdsctl.config}} project remove --force 04SCWORKFLOWTEMPLATERUN"
+  - script: "{{.cdsctl}} -f {{.cdsctl.config}} group remove --force 04scworkflowtemplaterun "
+  - script: "{{.cdsctl}} -f {{.cdsctl.config}} project add 04SCWORKFLOWTEMPLATERUN 04scworkflowtemplaterun"
+
+- name: template push
+  steps:
+  - script: "{{.cdsctl}} -f {{.cdsctl.config}} template push https://raw.githubusercontent.com/ovh/cds/master/contrib/workflow-templates/demo-workflow-hello-world/demo-workflow-hello-world.yml"
+    assertions:
+    - result.code ShouldEqual 0
+    - "result.systemout ShouldContainSubstring successfully pushed"
+
+- name: create workflow from template
+  steps:
+  - script: "{{.cdsctl}} -f {{.cdsctl.config}} workflow applyTemplate 04SCWORKFLOWTEMPLATERUN testtemplate shared.infra/demo-workflow-hello-world --import-push -n"
+    assertions:
+    - result.code ShouldEqual 0
+    - "result.systemout ShouldContainSubstring has been created"
+
+- name: run workflow
+  steps:
+  - script:  {{.cdsctl}} -f {{.cdsctl.config}} workflow run 04SCWORKFLOWTEMPLATERUN testtemplate
+    assertions:
+    - result.code ShouldEqual 0
+    - "result.systemout ShouldContainSubstring has been launched"
+
+- name: check workflow
+  steps:
+  - script:  {{.cdsctl}} -f {{.cdsctl.config}} workflow status 04SCWORKFLOWTEMPLATERUN testtemplate 1 --format json
+    retry: 20
+    delay: 10
+    assertions:
+    - result.code ShouldEqual 0
+    - result.systemoutjson.last_execution ShouldNotBeEmpty
+    - result.systemoutjson.start ShouldNotBeEmpty
+    - result.systemoutjson.num ShouldContainSubstring 1
+    - result.systemoutjson.status ShouldEqual Success
+    - result.systemoutjson.tags ShouldContainSubstring triggered_by
+
+- name: check workflow log
+  steps:
+    - script: {{.cdsctl}} -f {{.cdsctl.config}} workflow logs download 04SCWORKFLOWTEMPLATERUN testtemplate 1 --pattern build
+    - script: grep "CDS_TEMPLATE_VERSION=0" *.log
