feat: Update error message for invalid log record attribute (#1825)

* feat: Use logger.warn for log attribute validation

Update the way users are notified about invalid log record attribute
keys and values by using OpenTelemetry.logger.warn instead of
OpenTelemetry.handle_error. Also, include the key with the invalid
type in the message.

* test: Update tests for new log message

* chore: Rubocop Bundler/OrderedGems

* fix: Use a handle_error call to match Traces

* chore: Add missing :message kwarg

* chore: Add missing empty line at EOF

* chore: Remove mutex_m install

* Update logs_sdk/Gemfile

Author: kaylareopelle

logs_sdk/lib/opentelemetry/sdk/logs/log_record.rb

@@ -139,10 +139,12 @@ def validate_attributes(attrs)
           # Future refactor opportunity: https://github.com/open-telemetry/opentelemetry-ruby/issues/1739
           attrs.keep_if do |k, v|
             if !Internal.valid_key?(k)
-              OpenTelemetry.handle_error(message: "invalid log record attribute key type #{k.class} on record: '#{body}'")
+              OpenTelemetry.handle_error(message: "Invalid log record attribute key type #{k.class} for " \
+                "key #{k.inspect} on record: '#{body}'. Attribute keys must be Strings. Dropping attribute.")
               return false
             elsif !Internal.valid_value?(v)
-              OpenTelemetry.handle_error(message: "invalid log record attribute value type #{v.class} for key '#{k}' on record: '#{body}'")
+              OpenTelemetry.handle_error(message: "Invalid log record attribute value type #{v.class} for key '#{k}' " \
+                "on record: '#{body}'. Dropping attribute.")
               return false
             end
 

logs_sdk/test/opentelemetry/sdk/logs/log_record_test.rb

@@ -123,14 +123,14 @@
       it 'emits an error message if attribute key is invalid' do
         OpenTelemetry::TestHelpers.with_test_logger do |log_stream|
           logger.on_emit(attributes: { a: 'a' })
-          assert_match(/invalid log record attribute key type Symbol/, log_stream.string)
+          assert_match(/Invalid log record attribute key type Symbol/, log_stream.string)
         end
       end
 
       it 'emits an error message if the attribute value is invalid' do
         OpenTelemetry::TestHelpers.with_test_logger do |log_stream|
           logger.on_emit(attributes: { 'a' => Class.new })
-          assert_match(/invalid log record attribute value type Class/, log_stream.string)
+          assert_match(/Invalid log record attribute value type Class/, log_stream.string)
         end
       end