From e00e002fc28fa2f313a504cd15eb4a57d8b4decf Mon Sep 17 00:00:00 2001 From: Miek Gieben Date: Sun, 20 Mar 2016 17:54:21 +0000 Subject: [PATCH] more --- core/setup/etcd.go | 73 ++++++++++++++++++++++++++++++++++++++ middleware/etcd/backend.go | 9 +++-- middleware/etcd/etcd.go | 22 +++++------- 3 files changed, 88 insertions(+), 16 deletions(-) create mode 100644 core/setup/etcd.go diff --git a/core/setup/etcd.go b/core/setup/etcd.go new file mode 100644 index 000000000..76aed5249 --- /dev/null +++ b/core/setup/etcd.go @@ -0,0 +1,73 @@ +package setup + +import ( + "log" + "os" + + "github.com/miekg/coredns/middleware" + "github.com/miekg/coredns/middleware/file" + "github.com/miekg/dns" +) + +// File sets up the file middleware. +func File(c *Controller) (middleware.Middleware, error) { + zones, err := fileParse(c) + if err != nil { + return nil, err + } + return func(next middleware.Handler) middleware.Handler { + return file.File{Next: next, Zones: zones} + }, nil + +} + +func fileParse(c *Controller) (file.Zones, error) { + // Maybe multiple, each for each zone. + z := make(map[string]file.Zone) + names := []string{} + for c.Next() { + if c.Val() == "file" { + // file db.file [origin] + if !c.NextArg() { + return file.Zones{}, c.ArgErr() + } + fileName := c.Val() + + origin := c.ServerBlockHosts[c.ServerBlockHostIndex] + if c.NextArg() { + c.Next() + origin = c.Val() + } + // normalize this origin + origin = middleware.Host(origin).StandardHost() + + zone, err := parseZone(origin, fileName) + if err == nil { + z[origin] = zone + } + names = append(names, origin) + } + } + return file.Zones{Z: z, Names: names}, nil +} + +// +// parsrZone parses the zone in filename and returns a []RR or an error. +func parseZone(origin, fileName string) (file.Zone, error) { + f, err := os.Open(fileName) + if err != nil { + return nil, err + } + tokens := dns.ParseZone(f, origin, fileName) + zone := make([]dns.RR, 0, defaultZoneSize) + for x := range tokens { + if x.Error != nil { + log.Printf("[ERROR] failed to parse %s: %v", origin, x.Error) + return nil, x.Error + } + zone = append(zone, x.RR) + } + return file.Zone(zone), nil +} + +const defaultZoneSize = 20 // A made up number. diff --git a/middleware/etcd/backend.go b/middleware/etcd/backend.go index c601629d3..95990f719 100644 --- a/middleware/etcd/backend.go +++ b/middleware/etcd/backend.go @@ -11,6 +11,11 @@ import ( etcdc "github.com/coreos/etcd/client" ) +const ( + priority = 10 // default priority when nothing is set + ttl = 3600 // default ttl when nothing is set +) + func (g *Backend) Records(name string, exact bool) ([]msg.Service, error) { path, star := msg.PathWithWildcard(name) r, err := g.get(path, true) @@ -125,7 +130,7 @@ Nodes: serv.Key = n.Key serv.Ttl = g.calculateTtl(n, serv) if serv.Priority == 0 { - serv.Priority = int(g.config.Priority) + serv.Priority = priority } sx = append(sx, *serv) } @@ -139,7 +144,7 @@ func (g *Backend) calculateTtl(node *etcdc.Node, serv *msg.Service) uint32 { etcdTtl := uint32(node.TTL) if etcdTtl == 0 && serv.Ttl == 0 { - return g.config.Ttl + return ttl } if etcdTtl == 0 { return serv.Ttl diff --git a/middleware/etcd/etcd.go b/middleware/etcd/etcd.go index 84d5dd3a4..9f179228e 100644 --- a/middleware/etcd/etcd.go +++ b/middleware/etcd/etcd.go @@ -2,6 +2,7 @@ package etcd import ( + "github.com/miekg/coredns/middleware" "github.com/skynetservices/skydns/singleflight" etcd "github.com/coreos/etcd/client" @@ -10,25 +11,18 @@ import ( type ( Etcd struct { - Ttl uint32 - Priority uint16 - Backend *Backend + Next middleware.Handler + + client etcd.KeysAPI + ctx context.Context + inflight *singleflight.Group } ) -type Backend struct { - client etcd.KeysAPI - ctx context.Context - config *Config - inflight *singleflight.Group -} - -// NewBackend returns a new Backend. -func NewBackend(client etcd.KeysAPI, ctx context.Context, config *Config) *Backend { - return &Backend{ +func NewEtcd(client etcd.KeysAPI, ctx context.Context) Etcd { + return Etcd{ client: client, ctx: ctx, - config: config, inflight: &singleflight.Group{}, } }