fix stale comments

p2p/protocol/circuitv2/relay/constraints.go

@@ -45,8 +45,8 @@ func newConstraints(rc *Resources) *constraints {
 	}
 }
 
-// AddReservation adds a reservation for a given peer with a given multiaddr.
-// If adding this reservation violates IP constraints, an error is returned.
+// Reserve adds a reservation for a given peer with a given multiaddr.
+// If adding this reservation violates IP, ASN, or total reservation constraints, an error is returned.
 func (c *constraints) Reserve(p peer.ID, a ma.Multiaddr, expiry time.Time) error {
 	c.mutex.Lock()
 	defer c.mutex.Unlock()

p2p/protocol/circuitv2/relay/constraints_test.go

@@ -62,10 +62,10 @@ func TestConstraints(t *testing.T) {
 			t.Fatal(err)
 		}
 		if err := c.Reserve(p2, ipAddr, expiry); err != errTooManyReservationsForIP {
-			t.Fatalf("expected to run into total reservation limit, got %v", err)
+			t.Fatalf("expected to run into IP reservation limit as this IP has already been reserved by a different peer, got %v", err)
 		}
 		if err := c.Reserve(p, randomIPv4Addr(t), expiry); err != nil {
-			t.Fatalf("expected to update reservation for peer, got %v", err)
+			t.Fatalf("expected to update existing reservation for peer, got %v", err)
 		}
 		if err := c.Reserve(p2, ipAddr, expiry); err != nil {
 			t.Fatalf("expected reservation for different peer to be possible, got %v", err)

p2p/protocol/circuitv2/relay/resources.go

@@ -53,7 +53,7 @@ func DefaultResources() Resources {
 		MaxCircuits:     16,
 		BufferSize:      2048,
 
-		MaxReservationsPerPeer: 4,
+		MaxReservationsPerPeer: 1,
 		MaxReservationsPerIP:   8,
 		MaxReservationsPerASN:  32,
 	}