fix: compact Dalli attributes (#1548)

Author: nickamorim

instrumentation/dalli/lib/opentelemetry/instrumentation/dalli/patches/server.rb

@@ -15,9 +15,10 @@ def request(op, *args) # rubocop:disable Naming/MethodParameterName
             attributes = {
               'db.system' => 'memcached',
               'db.operation' => operation,
-              'net.peer.name' => hostname,
-              'net.peer.port' => port
+              'net.peer.name' => hostname
             }
+            attributes['net.peer.port'] = port if port
+
             if config[:db_statement] == :include
               attributes['db.statement'] = Utils.format_command(operation, args)
             elsif config[:db_statement] == :obfuscate

instrumentation/dalli/test/opentelemetry/instrumentation/dalli/instrumentation_test.rb

@@ -142,6 +142,24 @@
         _(span.name).must_equal 'gat'
         _(span.attributes['db.statement']).must_equal 'gat foo 0'
       end
+
+      it 'supports unix domain socket' do
+        dalli.version
+        exporter.reset
+
+        server = dalli.instance_variable_get(:@ring).servers.first
+        server.stub(:hostname, '/tmp/memcached.sock') do
+          server.stub(:port, nil) do
+            dalli.set('foo', 'bar')
+
+            puts exporter.finished_spans
+            _(exporter.finished_spans.size).must_equal 1
+            _(span.name).must_equal 'set'
+            _(span.attributes).wont_include 'net.peer.port'
+            _(span.attributes['net.peer.name']).must_equal '/tmp/memcached.sock'
+          end
+        end
+      end
     end
   end