forked from forgejo/runner
990cf93c71
The only reason docker is really required by now, is that act_runner ping docker. This change only pings docker if a label with `docker://` is added to the runner. Plain labels without `:` like `self-hosted` are run directly on the system. Previously the pseudo non docker label `-self-hosted` have been required like this `self-hosted:docker://-self-hosted`, but due to docker ping this still required a dockerd to be pingable. Co-authored-by: Christopher Homberger <christopher.homberger@web.de> Reviewed-on: https://gitea.com/gitea/act_runner/pulls/16 Reviewed-by: Jason Song <i@wolfogre.com> Reviewed-by: Lunny Xiao <xiaolunwen@gmail.com> Co-authored-by: ChristopherHX <christopherhx@noreply.gitea.io> Co-committed-by: ChristopherHX <christopherhx@noreply.gitea.io>
11 lines
247 B
Go
11 lines
247 B
Go
package cmd
|
|
|
|
import "testing"
|
|
|
|
func TestValidateLabels(t *testing.T) {
|
|
labels := []string{"ubuntu-latest:docker://node:16-buster", "self-hosted"}
|
|
if err := validateLabels(labels); err != nil {
|
|
t.Errorf("validateLabels() error = %v", err)
|
|
}
|
|
}
|