Bump step-runner version to 0.29.0

This version adds job timeout to the RunRequest.

Author: Axel von Bertoldi

common/build.go

@@ -450,6 +450,7 @@ func (b *Build) executeStepStage(ctx context.Context, connector steps.Connector,
 
 			info := steps.JobInfo{
 				ID:         b.ID,
+				Timeout:    b.GetBuildTimeout(),
 				ProjectDir: b.FullProjectDir(),
 				Variables:  b.GetAllVariables(),
 			}

common/mocks.go

@@ -38,7 +38,7 @@ run:
 			Run(stepYml)
 		require.NoError(t, err)
 		assert.Equal(t, proto.StepResult_success, res.Status)
-		require.Contains(t, logs, `Running step "hello_script_legacy"`)
+		require.Contains(t, logs, `Running step name=hello_script_legacy`)
 		require.Contains(t, logs, "Hello from script_legacy")
 		require.Contains(t, logs, "Second command")
 	})
@@ -240,9 +240,9 @@ run:
 			Run(stepYml)
 		require.NoError(t, err)
 		assert.Equal(t, proto.StepResult_success, res.Status)
-		require.Contains(t, logs, `Running step "step1"`)
+		require.Contains(t, logs, `Running step name=step1`)
 		require.Contains(t, logs, "First step")
-		require.Contains(t, logs, `Running step "step2"`)
+		require.Contains(t, logs, `Running step name=step2`)
 		require.Contains(t, logs, "Second step")
 		assert.Len(t, res.SubStepResults, 2)
 	})

functions/script_legacy/script_legacy_test.go

@@ -80,7 +80,7 @@ require (
 	gitlab.com/gitlab-org/golang-cli-helpers v0.0.0-20220124161940-198f30295e7e
 	gitlab.com/gitlab-org/labkit v1.34.0
 	gitlab.com/gitlab-org/moa v0.0.0-20251209091627-66342f721c88
-	gitlab.com/gitlab-org/step-runner v0.27.0
+	gitlab.com/gitlab-org/step-runner v0.29.0
 	go.mozilla.org/pkcs7 v0.9.0
 	go.uber.org/automaxprocs v1.6.0
 	go.yaml.in/yaml/v3 v3.0.4

go.mod

@@ -604,8 +604,8 @@ gitlab.com/gitlab-org/labkit v1.34.0 h1:wJrUZdfxbZrA9+qvnpk5gYJtceW+p7ceSJSg1BPZ
 gitlab.com/gitlab-org/labkit v1.34.0/go.mod h1:JqQLdgjV/KKAZJ6gvNodaLStmWeTT9mxgJPIEi66VHI=
 gitlab.com/gitlab-org/moa v0.0.0-20251209091627-66342f721c88 h1:GVlo8Pr4wfXI/6UF+Rmi0Yv2l7lwVgFiNBeImgWVeko=
 gitlab.com/gitlab-org/moa v0.0.0-20251209091627-66342f721c88/go.mod h1:024490ksS75/Bi9UoJTu59qY44JuFBAfi5bzGsLIhtY=
-gitlab.com/gitlab-org/step-runner v0.27.0 h1:8wrnKWv7x3CcSXhJUDLHYkWimWlCp65T5JDmceTiPaw=
-gitlab.com/gitlab-org/step-runner v0.27.0/go.mod h1:KZ7aYYlc/DFt3xxhfygrr0TTlXWdnN+LJEsFC9W/1Sw=
+gitlab.com/gitlab-org/step-runner v0.29.0 h1:cVklPW741QGMblJEXE4WgyiXwYhmp7hNzQdmnl31Hxw=
+gitlab.com/gitlab-org/step-runner v0.29.0/go.mod h1:KZ7aYYlc/DFt3xxhfygrr0TTlXWdnN+LJEsFC9W/1Sw=
 go.etcd.io/bbolt v1.3.5 h1:XAzx9gjCb0Rxj7EoqcClPD1d5ZBxZJk0jbuoPHenBt0=
 go.etcd.io/bbolt v1.3.5/go.mod h1:G5EMThwa9y8QZGBClrRx5EY+Yw9kAhnjy3bSjsnlVTQ=
 go.mozilla.org/pkcs7 v0.9.0 h1:yM4/HS9dYv7ri2biPtxt8ikvB37a980dg69/pKmS+eI=

go.sum

@@ -23,6 +23,7 @@ type Connector interface {
 
 type JobInfo struct {
 	ID         int64
+	Timeout    time.Duration
 	ProjectDir string
 	Variables  spec.Variables
 }

steps/execute.go

@@ -30,6 +30,7 @@ func NewRequest(jobInfo JobInfo, steps []schema.Step) (*client.RunRequest, error
 
 	return &client.RunRequest{
 		Id:        strconv.FormatInt(jobInfo.ID, 10),
+		Timeout:   &jobInfo.Timeout,
 		WorkDir:   jobInfo.ProjectDir,
 		BuildDir:  jobInfo.ProjectDir,
 		Env:       map[string]string{},

steps/steps.go

@@ -5,11 +5,13 @@ package steps
 import (
 	"encoding/json"
 	"testing"
+	"time"
 
 	"github.com/stretchr/testify/assert"
 
-	"gitlab.com/gitlab-org/gitlab-runner/common/spec"
 	"gitlab.com/gitlab-org/step-runner/schema/v1"
+
+	"gitlab.com/gitlab-org/gitlab-runner/common/spec"
 )
 
 func Test_addStepsPreamble(t *testing.T) {
@@ -50,6 +52,18 @@ func Test_addStepsPreamble(t *testing.T) {
 	}
 }
 
+func Test_NewRequest_Adds_Timeout(t *testing.T) {
+	step := `[{"name":"script", "script": "foo bar baz"}]`
+	var steps []schema.Step
+	assert.NoError(t, json.Unmarshal([]byte(step), &steps))
+	to := time.Second * 42
+
+	ji := JobInfo{Timeout: to}
+	got, err := NewRequest(ji, steps)
+	assert.NoError(t, err)
+	assert.Equal(t, to, *got.Timeout)
+}
+
 func Test_addVariables_Omits(t *testing.T) {
 	keysToVars := func(keys []string) spec.Variables {
 		jobVars := spec.Variables{}