Skip to content

Commit

Permalink
simplify
Browse files Browse the repository at this point in the history
Signed-off-by: Tim Vaillancourt <[email protected]>
  • Loading branch information
timvaillancourt committed Feb 13, 2025
1 parent c7ae686 commit ccae715
Show file tree
Hide file tree
Showing 4 changed files with 8 additions and 8 deletions.
10 changes: 5 additions & 5 deletions test/ci_workflow_gen.go
Original file line number Diff line number Diff line change
Expand Up @@ -32,7 +32,8 @@ const (
mysql80 mysqlVersion = "mysql80"
mysql84 mysqlVersion = "mysql84"

cores16Runner = "vitess-ubuntu20-16cpu-1"
cores16RunnerName = "vitess-ubuntu20-16cpu-1"
defaultRunnerName = "ubuntu-24.04"
defaultMySQLVersion = mysql80
)

Expand Down Expand Up @@ -165,14 +166,13 @@ type clusterTest struct {
Name, Shard, Platform string
FileName string
BuildTag string
RunsOn string
MemoryCheck bool
MakeTools, InstallXtraBackup bool
Docker bool
LimitResourceUsage bool
EnableBinlogTransactionCompression bool
PartialKeyspace bool
Cores16 bool
Cores16Runner string
}

type vitessTesterTest struct {
Expand Down Expand Up @@ -256,12 +256,12 @@ func generateClusterWorkflows(list []string, tpl string) {
Name: fmt.Sprintf("Cluster (%s)", cluster),
Shard: cluster,
BuildTag: buildTag[cluster],
RunsOn: defaultRunnerName,
}
cores16Clusters := canonnizeList(clusterRequiring16CoresMachines)
for _, cores16Cluster := range cores16Clusters {
if cores16Cluster == cluster {
test.Cores16 = true
test.Cores16Runner = cores16Runner
test.RunsOn = cores16RunnerName
break
}
}
Expand Down
2 changes: 1 addition & 1 deletion test/templates/cluster_endtoend_test.tpl
Original file line number Diff line number Diff line change
Expand Up @@ -15,7 +15,7 @@ jobs:
build:
timeout-minutes: 60
name: Run endtoend tests on {{.Name}}
runs-on: {{if .Cores16}}{{ .Cores16Runner }}{{else}}ubuntu-24.04{{end}}
runs-on: {{.RunsOn}}

steps:
- name: Skip CI
Expand Down
2 changes: 1 addition & 1 deletion test/templates/cluster_endtoend_test_docker.tpl
Original file line number Diff line number Diff line change
Expand Up @@ -6,7 +6,7 @@ permissions: read-all
jobs:
build:
name: Run endtoend tests on {{.Name}}
runs-on: {{if .Cores16}}{{ .Cores16Runner }}{{else}}ubuntu-24.04{{end}}
runs-on: {{.RunsOn}}

steps:
- name: Skip CI
Expand Down
2 changes: 1 addition & 1 deletion test/templates/cluster_endtoend_test_mysql57.tpl
Original file line number Diff line number Diff line change
Expand Up @@ -19,7 +19,7 @@ env:
jobs:
build:
name: Run endtoend tests on {{.Name}}
runs-on: {{if .Cores16}}{{ .Cores16Runner }}{{else}}ubuntu-24.04{{end}}
runs-on: {{.RunsOn}}

steps:
- name: Skip CI
Expand Down

0 comments on commit ccae715

Please sign in to comment.