diff --git a/runner-manager/cfd/cmd/drive/get_job_config.go b/runner-manager/cfd/cmd/drive/get_job_config.go index c3a6ff4..bd89a01 100644 --- a/runner-manager/cfd/cmd/drive/get_job_config.go +++ b/runner-manager/cfd/cmd/drive/get_job_config.go @@ -68,6 +68,9 @@ type CIVar struct { Value string } +// match images w/ docker domain, or no domain (i.e. docker by default) +var domainRegex = regexp.MustCompile(`^((registry-\d+|index)?\.?docker\.io\/|[^.]*(:|$))`) + type VcapAppData struct { CFApi string `json:"cf_api"` OrgID string `json:"org_id"` @@ -197,14 +200,11 @@ func (cfg *JobConfig) processImage(img Image, m *cloudgov.AppManifest) { if img.Name != "" { m.Docker.Image = img.Name - // match images w/ docker domain, or no domain (i.e. docker by default) - re := regexp.MustCompile(`^((registry-\d+|index)?\.?docker\.io\/|[^.]*(:|$))`) - // TODO: #95 if strings.Contains(img.Name, "registry.gitlab.com") { m.Docker.Username = cfg.CIRegistryUser m.Docker.Password = cfg.CIRegistryPass - } else if re.FindString(img.Name) != "" { + } else if domainRegex.FindString(img.Name) != "" { m.Docker.Username = cfg.DockerHubUser m.Docker.Password = cfg.DockerHubToken }