Don't delete all the non-tombstone entries every 10mins!

nameserver/entry.go

@@ -131,6 +131,7 @@ func (es *Entries) merge(incoming Entries) Entries {
 	return newEntries
 }
 
+// f returning true means keep the entry.
 func (es *Entries) tombstone(ourname router.PeerName, f func(*Entry) bool) Entries {
 	es.checkAndPanic()
 	defer es.checkAndPanic()

nameserver/nameserver.go

@@ -176,7 +176,7 @@ func (n *Nameserver) deleteTombstones() {
 	defer n.Unlock()
 	now := time.Now().Unix()
 	n.entries.filter(func(e *Entry) bool {
-		return now-e.Tombstone <= int64(tombstoneTimeout/time.Second)
+		return e.Tombstone == 0 || now-e.Tombstone <= int64(tombstoneTimeout/time.Second)
 	})
 }
 

nameserver/nameserver_test.go

@@ -230,6 +230,9 @@ func TestTombstoneDeletion(t *testing.T) {
 	require.Nil(t, err)
 	require.Equal(t, []address.Address{0}, nameserver.Lookup("hostname"))
 
+	nameserver.deleteTombstones()
+	require.Equal(t, []address.Address{0}, nameserver.Lookup("hostname"))
+
 	err = nameserver.Delete("hostname", "containerid", "", address.Address(0))
 	require.Nil(t, err)
 	require.Equal(t, []address.Address{}, nameserver.Lookup("hostname"))