From 2c686e882661004974ad2df6b9175026b6d207e9 Mon Sep 17 00:00:00 2001 From: Jason Song Date: Wed, 4 Jan 2023 16:43:40 +0800 Subject: [PATCH] fix: add restraints for models --- models/actions/run_job.go | 4 ++-- models/actions/runner.go | 2 +- models/actions/task_step.go | 2 +- models/migrations/v1_19/v238.go | 16 ++++++++-------- 4 files changed, 12 insertions(+), 12 deletions(-) diff --git a/models/actions/run_job.go b/models/actions/run_job.go index 9237d20ba2..ffb0331a69 100644 --- a/models/actions/run_job.go +++ b/models/actions/run_job.go @@ -25,10 +25,10 @@ type ActionRunJob struct { OwnerID int64 `xorm:"index"` CommitSHA string `xorm:"index"` IsForkPullRequest bool - Name string + Name string `xorm:"VARCHAR(255)"` Attempt int64 WorkflowPayload []byte - JobID string // job id in workflow, not job's id + JobID string `xorm:"VARCHAR(255)"` // job id in workflow, not job's id Needs []string `xorm:"JSON TEXT"` RunsOn []string `xorm:"JSON TEXT"` TaskID int64 // the latest task of the job diff --git a/models/actions/runner.go b/models/actions/runner.go index 7c4138c79e..a8665c943b 100644 --- a/models/actions/runner.go +++ b/models/actions/runner.go @@ -23,7 +23,7 @@ import ( type ActionRunner struct { ID int64 UUID string `xorm:"CHAR(36) UNIQUE"` - Name string `xorm:"VARCHAR(32)"` + Name string `xorm:"VARCHAR(255)"` OwnerID int64 `xorm:"index"` // org level runner, 0 means system Owner *user_model.User `xorm:"-"` RepoID int64 `xorm:"index"` // repo level runner, if orgid also is zero, then it's a global diff --git a/models/actions/task_step.go b/models/actions/task_step.go index 181fec226d..e512a39279 100644 --- a/models/actions/task_step.go +++ b/models/actions/task_step.go @@ -14,7 +14,7 @@ import ( // ActionTaskStep represents a step of ActionTask type ActionTaskStep struct { ID int64 - Name string + Name string `xorm:"VARCHAR(255)"` TaskID int64 `xorm:"index unique(task_number)"` Number int64 `xorm:"index unique(task_number)"` RepoID int64 `xorm:"index"` diff --git a/models/migrations/v1_19/v238.go b/models/migrations/v1_19/v238.go index a6e7ef066e..e4aaf8fbbf 100644 --- a/models/migrations/v1_19/v238.go +++ b/models/migrations/v1_19/v238.go @@ -14,7 +14,7 @@ func AddActionsTables(x *xorm.Engine) error { type ActionRunner struct { ID int64 UUID string `xorm:"CHAR(36) UNIQUE"` - Name string `xorm:"VARCHAR(32)"` + Name string `xorm:"VARCHAR(255)"` OwnerID int64 `xorm:"index"` // org level runner, 0 means system RepoID int64 `xorm:"index"` // repo level runner, if orgid also is zero, then it's a global Description string `xorm:"TEXT"` @@ -78,10 +78,10 @@ func AddActionsTables(x *xorm.Engine) error { OwnerID int64 `xorm:"index"` CommitSHA string `xorm:"index"` IsForkPullRequest bool - Name string + Name string `xorm:"VARCHAR(255)"` Attempt int64 WorkflowPayload []byte - JobID string // job id in workflow, not job's id + JobID string `xorm:"VARCHAR(255)"` // job id in workflow, not job's id Needs []string `xorm:"JSON TEXT"` RunsOn []string `xorm:"JSON TEXT"` TaskID int64 // the latest task of the job @@ -130,11 +130,11 @@ func AddActionsTables(x *xorm.Engine) error { type ActionTaskStep struct { ID int64 - Name string - TaskID int64 `xorm:"index unique(task_number)"` - Number int64 `xorm:"index unique(task_number)"` - RepoID int64 `xorm:"index"` - Status int `xorm:"index"` + Name string `xorm:"VARCHAR(255)"` + TaskID int64 `xorm:"index unique(task_number)"` + Number int64 `xorm:"index unique(task_number)"` + RepoID int64 `xorm:"index"` + Status int `xorm:"index"` LogIndex int64 LogLength int64 Started timeutil.TimeStamp