Skip to content

Commit

Permalink
fix(api): do not interpolate workflow pipelines requirements (#3085)
Browse files Browse the repository at this point in the history
  • Loading branch information
fsamin authored and sguiheux committed Jul 26, 2018
1 parent e10bd45 commit 1d1bdf1
Showing 1 changed file with 1 addition and 1 deletion.
2 changes: 1 addition & 1 deletion engine/api/workflow/execute_node_run.go
Original file line number Diff line number Diff line change
Expand Up @@ -347,7 +347,6 @@ func addJobsToQueue(ctx context.Context, db gorp.SqlExecutor, stage *sdk.Stage,
if errReq != nil {
errs.Join(*errReq)
}
job.Action.Requirements = jobRequirements

// add requirements in job parameters, to use them as {{.job.requirement...}} in job
_, next = tracing.Span(ctx, "workflow.prepareRequirementsToNodeJobRunParameters")
Expand All @@ -372,6 +371,7 @@ func addJobsToQueue(ctx context.Context, db gorp.SqlExecutor, stage *sdk.Stage,
Job: *job,
},
}
wjob.Job.Job.Action.Requirements = jobRequirements // Set the interpolated requirements on the job run only

if !stage.Enabled || !wjob.Job.Enabled {
wjob.Status = sdk.StatusDisabled.String()
Expand Down

0 comments on commit 1d1bdf1

Please sign in to comment.