diff --git a/modules/structs/runner.go b/modules/structs/runner.go index cecbb72400..5ac88f0488 100644 --- a/modules/structs/runner.go +++ b/modules/structs/runner.go @@ -25,8 +25,7 @@ type RegisterRunnerOptions struct { // RegisterRunnerResponse contains the details of the just registered runner. // swagger:model type RegisterRunnerResponse struct { - ID int64 `json:"id" binding:"Required"` - UUID string `json:"uuid" binding:"Required"` - Token string `json:"token" binding:"Required"` - Ephemeral bool `json:"ephemeral" binding:"Required"` + ID int64 `json:"id" binding:"Required"` + UUID string `json:"uuid" binding:"Required"` + Token string `json:"token" binding:"Required"` } diff --git a/routers/api/v1/shared/runners.go b/routers/api/v1/shared/runners.go index ad470215e9..a96a10ad30 100644 --- a/routers/api/v1/shared/runners.go +++ b/routers/api/v1/shared/runners.go @@ -170,10 +170,9 @@ func RegisterRunner(ctx *context.APIContext, ownerID, repoID int64) { } response := &structs.RegisterRunnerResponse{ - ID: runner.ID, - UUID: runner.UUID, - Token: runner.Token, - Ephemeral: runner.Ephemeral, + ID: runner.ID, + UUID: runner.UUID, + Token: runner.Token, } ctx.JSON(http.StatusCreated, response) } diff --git a/templates/swagger/v1_json.tmpl b/templates/swagger/v1_json.tmpl index 976c25afa7..7e83814a2d 100644 --- a/templates/swagger/v1_json.tmpl +++ b/templates/swagger/v1_json.tmpl @@ -28538,10 +28538,6 @@ "type": "object", "title": "RegisterRunnerResponse contains the details of the just registered runner.", "properties": { - "ephemeral": { - "type": "boolean", - "x-go-name": "Ephemeral" - }, "id": { "type": "integer", "format": "int64", diff --git a/tests/integration/api_admin_actions_test.go b/tests/integration/api_admin_actions_test.go index 02644401fd..c935d352f9 100644 --- a/tests/integration/api_admin_actions_test.go +++ b/tests/integration/api_admin_actions_test.go @@ -376,7 +376,6 @@ func TestAPIAdminActionsRunnerOperations(t *testing.T) { assert.Positive(t, registerRunnerResponse.ID) assert.Equal(t, gouuid.Version(4), gouuid.MustParse(registerRunnerResponse.UUID).Version()) assert.Regexp(t, "(?i)^[0-9a-f]{40}$", registerRunnerResponse.Token) - assert.False(t, registerRunnerResponse.Ephemeral) registeredRunner := unittest.AssertExistsAndLoadBean(t, &actions_model.ActionRunner{UUID: registerRunnerResponse.UUID}) assert.Equal(t, registerRunnerResponse.ID, registeredRunner.ID) @@ -402,8 +401,6 @@ func TestAPIAdminActionsRunnerOperations(t *testing.T) { var registerRunnerResponse *api.RegisterRunnerResponse DecodeJSON(t, response, ®isterRunnerResponse) - assert.True(t, registerRunnerResponse.Ephemeral) - registeredRunner := unittest.AssertExistsAndLoadBean(t, &actions_model.ActionRunner{UUID: registerRunnerResponse.UUID}) assert.Equal(t, registerRunnerResponse.UUID, registeredRunner.UUID) assert.True(t, registeredRunner.Ephemeral) diff --git a/tests/integration/api_org_actions_test.go b/tests/integration/api_org_actions_test.go index f21723a751..35183306c2 100644 --- a/tests/integration/api_org_actions_test.go +++ b/tests/integration/api_org_actions_test.go @@ -250,7 +250,6 @@ func TestAPIOrgActionsRunnerOperations(t *testing.T) { assert.Positive(t, registerRunnerResponse.ID) assert.Equal(t, gouuid.Version(4), gouuid.MustParse(registerRunnerResponse.UUID).Version()) assert.Regexp(t, "(?i)^[0-9a-f]{40}$", registerRunnerResponse.Token) - assert.False(t, registerRunnerResponse.Ephemeral) registeredRunner := unittest.AssertExistsAndLoadBean(t, &actions_model.ActionRunner{UUID: registerRunnerResponse.UUID}) assert.Equal(t, registerRunnerResponse.ID, registeredRunner.ID) @@ -276,8 +275,6 @@ func TestAPIOrgActionsRunnerOperations(t *testing.T) { var registerRunnerResponse *api.RegisterRunnerResponse DecodeJSON(t, response, ®isterRunnerResponse) - assert.True(t, registerRunnerResponse.Ephemeral) - registeredRunner := unittest.AssertExistsAndLoadBean(t, &actions_model.ActionRunner{UUID: registerRunnerResponse.UUID}) assert.Equal(t, registerRunnerResponse.UUID, registeredRunner.UUID) assert.True(t, registeredRunner.Ephemeral) diff --git a/tests/integration/api_repo_actions_test.go b/tests/integration/api_repo_actions_test.go index 35cbc5d864..dfe33379d4 100644 --- a/tests/integration/api_repo_actions_test.go +++ b/tests/integration/api_repo_actions_test.go @@ -532,7 +532,6 @@ func TestAPIRepoActionsRunnerOperations(t *testing.T) { assert.Positive(t, registerRunnerResponse.ID) assert.Equal(t, gouuid.Version(4), gouuid.MustParse(registerRunnerResponse.UUID).Version()) assert.Regexp(t, "(?i)^[0-9a-f]{40}$", registerRunnerResponse.Token) - assert.False(t, registerRunnerResponse.Ephemeral) registeredRunner := unittest.AssertExistsAndLoadBean(t, &actions_model.ActionRunner{UUID: registerRunnerResponse.UUID}) assert.Equal(t, registerRunnerResponse.ID, registeredRunner.ID) @@ -559,8 +558,6 @@ func TestAPIRepoActionsRunnerOperations(t *testing.T) { var registerRunnerResponse *api.RegisterRunnerResponse DecodeJSON(t, response, ®isterRunnerResponse) - assert.True(t, registerRunnerResponse.Ephemeral) - registeredRunner := unittest.AssertExistsAndLoadBean(t, &actions_model.ActionRunner{UUID: registerRunnerResponse.UUID}) assert.Equal(t, registerRunnerResponse.UUID, registeredRunner.UUID) assert.True(t, registeredRunner.Ephemeral) diff --git a/tests/integration/api_user_actions_test.go b/tests/integration/api_user_actions_test.go index bdb763d5b2..304004659c 100644 --- a/tests/integration/api_user_actions_test.go +++ b/tests/integration/api_user_actions_test.go @@ -248,7 +248,6 @@ func TestAPIUserActionsRunnerOperations(t *testing.T) { assert.Positive(t, registerRunnerResponse.ID) assert.Equal(t, gouuid.Version(4), gouuid.MustParse(registerRunnerResponse.UUID).Version()) assert.Regexp(t, "(?i)^[0-9a-f]{40}$", registerRunnerResponse.Token) - assert.False(t, registerRunnerResponse.Ephemeral) registeredRunner := unittest.AssertExistsAndLoadBean(t, &actions_model.ActionRunner{UUID: registerRunnerResponse.UUID}) assert.Equal(t, registerRunnerResponse.ID, registeredRunner.ID) @@ -274,8 +273,6 @@ func TestAPIUserActionsRunnerOperations(t *testing.T) { var registerRunnerResponse *api.RegisterRunnerResponse DecodeJSON(t, response, ®isterRunnerResponse) - assert.True(t, registerRunnerResponse.Ephemeral) - registeredRunner := unittest.AssertExistsAndLoadBean(t, &actions_model.ActionRunner{UUID: registerRunnerResponse.UUID}) assert.Equal(t, registerRunnerResponse.UUID, registeredRunner.UUID) assert.True(t, registeredRunner.Ephemeral)