Skip to content

Commit c64f19d

Browse files
Add Artifact Registry to the Daemon struct
1 parent c0927ac commit c64f19d

File tree

3 files changed

+36
-33
lines changed

3 files changed

+36
-33
lines changed

cmd/drone-acr/main.go

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -90,7 +90,7 @@ func main() {
9090
if publicUrl != "" {
9191
// Set this env variable if public URL for artifact is available
9292
// If not, we will fall back to registry url
93-
os.Setenv("DAEMON_REGISTRY", publicUrl)
93+
os.Setenv("ARTIFACT_REGISTRY", publicUrl)
9494
}
9595

9696
// invoke the base docker plugin binary

cmd/drone-docker/main.go

Lines changed: 18 additions & 17 deletions
Original file line numberDiff line numberDiff line change
@@ -113,10 +113,10 @@ func main() {
113113
EnvVar: "PLUGIN_DAEMON_OFF",
114114
},
115115
cli.StringFlag{
116-
Name: "daemon.registry",
117-
Usage: "daemon registry",
116+
Name: "artifact.registry",
117+
Usage: "artifact registry",
118118
Value: "https://index.docker.io/v1/",
119-
EnvVar: "DAEMON_REGISTRY,PLUGIN_REGISTRY,DOCKER_REGISTRY",
119+
EnvVar: "ARTIFACT_REGISTRY,PLUGIN_REGISTRY,DOCKER_REGISTRY",
120120
},
121121
cli.StringFlag{
122122
Name: "dockerfile",
@@ -352,20 +352,21 @@ func run(c *cli.Context) error {
352352
SSHAgentKey: c.String("ssh-agent-key"),
353353
},
354354
Daemon: docker.Daemon{
355-
Registry: c.String("daemon.registry"),
356-
Mirror: c.String("daemon.mirror"),
357-
StorageDriver: c.String("daemon.storage-driver"),
358-
StoragePath: c.String("daemon.storage-path"),
359-
Insecure: c.Bool("daemon.insecure"),
360-
Disabled: c.Bool("daemon.off"),
361-
IPv6: c.Bool("daemon.ipv6"),
362-
Debug: c.Bool("daemon.debug"),
363-
Bip: c.String("daemon.bip"),
364-
DNS: c.StringSlice("daemon.dns"),
365-
DNSSearch: c.StringSlice("daemon.dns-search"),
366-
MTU: c.String("daemon.mtu"),
367-
Experimental: c.Bool("daemon.experimental"),
368-
RegistryType: registryType,
355+
Registry: c.String("docker.registry"),
356+
Mirror: c.String("daemon.mirror"),
357+
StorageDriver: c.String("daemon.storage-driver"),
358+
StoragePath: c.String("daemon.storage-path"),
359+
Insecure: c.Bool("daemon.insecure"),
360+
Disabled: c.Bool("daemon.off"),
361+
IPv6: c.Bool("daemon.ipv6"),
362+
Debug: c.Bool("daemon.debug"),
363+
Bip: c.String("daemon.bip"),
364+
DNS: c.StringSlice("daemon.dns"),
365+
DNSSearch: c.StringSlice("daemon.dns-search"),
366+
MTU: c.String("daemon.mtu"),
367+
Experimental: c.Bool("daemon.experimental"),
368+
RegistryType: registryType,
369+
ArtifactRegistry: c.String("artifact.registry"),
369370
},
370371
}
371372

docker.go

Lines changed: 17 additions & 15 deletions
Original file line numberDiff line numberDiff line change
@@ -16,20 +16,21 @@ import (
1616
type (
1717
// Daemon defines Docker daemon parameters.
1818
Daemon struct {
19-
Registry string // Docker registry
20-
Mirror string // Docker registry mirror
21-
Insecure bool // Docker daemon enable insecure registries
22-
StorageDriver string // Docker daemon storage driver
23-
StoragePath string // Docker daemon storage path
24-
Disabled bool // DOcker daemon is disabled (already running)
25-
Debug bool // Docker daemon started in debug mode
26-
Bip string // Docker daemon network bridge IP address
27-
DNS []string // Docker daemon dns server
28-
DNSSearch []string // Docker daemon dns search domain
29-
MTU string // Docker daemon mtu setting
30-
IPv6 bool // Docker daemon IPv6 networking
31-
Experimental bool // Docker daemon enable experimental mode
32-
RegistryType drone.RegistryType // Docker registry type
19+
Registry string // Docker registry
20+
Mirror string // Docker registry mirror
21+
Insecure bool // Docker daemon enable insecure registries
22+
StorageDriver string // Docker daemon storage driver
23+
StoragePath string // Docker daemon storage path
24+
Disabled bool // DOcker daemon is disabled (already running)
25+
Debug bool // Docker daemon started in debug mode
26+
Bip string // Docker daemon network bridge IP address
27+
DNS []string // Docker daemon dns server
28+
DNSSearch []string // Docker daemon dns search domain
29+
MTU string // Docker daemon mtu setting
30+
IPv6 bool // Docker daemon IPv6 networking
31+
Experimental bool // Docker daemon enable experimental mode
32+
RegistryType drone.RegistryType // Docker registry type
33+
ArtifactRegistry string // Docker registry where artifact can be viewed
3334
}
3435

3536
// Login defines Docker login parameters.
@@ -231,7 +232,8 @@ func (p Plugin) Exec() error {
231232

232233
if p.ArtifactFile != "" {
233234
if digest, err := getDigest(p.Build.TempTag); err == nil {
234-
if err = drone.WritePluginArtifactFile(p.Daemon.RegistryType, p.ArtifactFile, p.Daemon.Registry, p.Build.Repo, digest, p.Build.Tags); err != nil {
235+
fmt.Printf("Got artifact reg %s and digest %s\n", p.Daemon.ArtifactRegistry, digest)
236+
if err = drone.WritePluginArtifactFile(p.Daemon.RegistryType, p.ArtifactFile, p.Daemon.ArtifactRegistry, p.Build.Repo, digest, p.Build.Tags); err != nil {
235237
fmt.Printf("failed to write plugin artifact file at path: %s with error: %s\n", p.ArtifactFile, err)
236238
}
237239
} else {

0 commit comments

Comments
 (0)