diff --git a/ipam/allocator.go b/ipam/allocator.go index 7900bbcb..e8f8dc13 100644 --- a/ipam/allocator.go +++ b/ipam/allocator.go @@ -167,7 +167,7 @@ func (alloc *Allocator) spaceRequestDenied(sender router.PeerName, r address.Ran for i := 0; i < len(alloc.pendingClaims); { claim := alloc.pendingClaims[i].(*claim) if r.Contains(claim.addr) { - claim.DeniedBy(alloc, sender) + claim.deniedBy(alloc, sender) alloc.pendingClaims = append(alloc.pendingClaims[:i], alloc.pendingClaims[i+1:]...) continue } diff --git a/ipam/claim.go b/ipam/claim.go index 64b22a02..9525b956 100644 --- a/ipam/claim.go +++ b/ipam/claim.go @@ -47,7 +47,7 @@ func (c *claim) Try(alloc *Allocator) bool { // If our ring doesn't know, it must be empty. if c.noErrorOnUnknown { alloc.infof("Claim %s for %s: address allocator still initializing; will try later.", c.addr, c.ident) - c.sendResult(nil) // don't make the caller wait + c.sendResult(nil) } else { c.sendResult(fmt.Errorf("%s is in the range %s, but the allocator is not initialized yet", c.addr, alloc.universe.AsCIDRString())) } @@ -60,7 +60,7 @@ func (c *claim) Try(alloc *Allocator) bool { alloc.infof("Claim %s for %s: %s; will try later.", c.addr, c.ident, err) c.sendResult(nil) } else { // just tell the user they can't do this. - c.DeniedBy(alloc, owner) + c.deniedBy(alloc, owner) } } return false @@ -86,7 +86,7 @@ func (c *claim) Try(alloc *Allocator) bool { return true } -func (c *claim) DeniedBy(alloc *Allocator, owner router.PeerName) { +func (c *claim) deniedBy(alloc *Allocator, owner router.PeerName) { name, found := alloc.nicknames[owner] if found { name = " (" + name + ")"