feat: Set logs exporter default to OTLP (#1857)

The OTLP logs exporter is available on RubyGems.
This changes the default to match the other signals.

Author: kaylareopelle

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

@@ -36,8 +36,7 @@ def configure_log_record_processors
         end
 
         def wrapped_log_exporters_from_env
-          # TODO: set default to OTLP to match traces, default is console until other exporters merged
-          exporters = ENV.fetch('OTEL_LOGS_EXPORTER', 'console')
+          exporters = ENV.fetch('OTEL_LOGS_EXPORTER', 'otlp')
 
           exporters.split(',').map do |exporter|
             case exporter.strip

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

@@ -35,22 +35,21 @@
     end
 
     describe 'processors' do
-      it 'defaults to a simple processor with a console exporter' do
+      it 'defaults to a batch processor with an otlp exporter' do
+        skip 'OTLP exporter not compatible with JRuby' if RUBY_ENGINE == 'jruby'
         configurator.configure
 
         processors = OpenTelemetry.logger_provider.instance_variable_get(:@log_record_processors)
 
         assert_equal 1, processors.size
         processor = processors[0]
 
-        assert_instance_of OpenTelemetry::SDK::Logs::Export::SimpleLogRecordProcessor, processor
-        assert_instance_of OpenTelemetry::SDK::Logs::Export::ConsoleLogRecordExporter, processor.instance_variable_get(:@log_record_exporter)
+        assert_instance_of OpenTelemetry::SDK::Logs::Export::BatchLogRecordProcessor, processor
+        assert_instance_of OpenTelemetry::Exporter::OTLP::Logs::LogsExporter, processor.instance_variable_get(:@exporter)
       end
 
       it 'can be set by environment variable' do
-        skip 'OTLP exporter not compatible with JRuby' if RUBY_ENGINE == 'jruby'
-
-        OpenTelemetry::TestHelpers.with_env('OTEL_LOGS_EXPORTER' => 'otlp') do
+        OpenTelemetry::TestHelpers.with_env('OTEL_LOGS_EXPORTER' => 'console') do
           configurator.configure
         end
 
@@ -59,8 +58,8 @@
         assert_equal 1, processors.size
         processor = processors[0]
 
-        assert_instance_of OpenTelemetry::SDK::Logs::Export::BatchLogRecordProcessor, processor
-        assert_instance_of OpenTelemetry::Exporter::OTLP::Logs::LogsExporter, processor.instance_variable_get(:@exporter)
+        assert_instance_of OpenTelemetry::SDK::Logs::Export::SimpleLogRecordProcessor, processor
+        assert_instance_of OpenTelemetry::SDK::Logs::Export::ConsoleLogRecordExporter, processor.instance_variable_get(:@log_record_exporter)
       end
 
       it 'supports "none" as an environment variable' do