[Logs] Sync up logger provider build-up scenarios with tracer provider

[CodeBlanch]

Changes

This PR syncs up the LoggerProvider scenarios (added on #3504) with the TracerProvider scenarios (#3533)

• AddExporter methods added
• AddOpenTelemetry is now safe to be called multiple times
• Removed the ability to register BaseProcessor<LogRecord>s & Action<IServiceProvider, OpenTelemetryLoggerProvider>s into the IServiceCollection

Public API

namespace OpenTelemetry.Logs
{
   public class OpenTelemetryLoggerOptions
   {
+     public OpenTelemetryLoggerOptions AddExporter(ExportProcessorType exportProcessorType, BaseExporter<LogRecord> exporter) {}
+     public OpenTelemetryLoggerOptions AddExporter(ExportProcessorType exportProcessorType, BaseExporter<LogRecord> exporter, Action<ExportLogRecordProcessorOptions> configure) {}
+     public OpenTelemetryLoggerOptions AddExporter<T>(ExportProcessorType exportProcessorType) where T : BaseExporter<LogRecord> {}
+     public OpenTelemetryLoggerOptions AddExporter<T>(ExportProcessorType exportProcessorType, Action<ExportLogRecordProcessorOptions> configure) where T : BaseExporter<LogRecord> {}
   }

    // This class is similar to MetricReaderOptions and is used by AddExporter extensions
+   public class ExportLogRecordProcessorOptions
+   {
+      public ExportProcessorType ExportProcessorType { get; set; }
+      public BatchExportLogRecordProcessorOptions BatchExportProcessorOptions { get; set; }
+   }

    // This class is similar to BatchExportActivityProcessorOptions and exists so separate environment variables may be used to configure batch log processor settings
+   public class BatchExportLogRecordProcessorOptions
+   {
+   }
}

TODOs

• Appropriate CHANGELOG.md updated for non-trivial changes
• Changes in public API reviewed
• Unit tests

[codecov]

Codecov Report

Merging #3596 (da15761) into main (9c3d1b1) will increase coverage by 0.20%.
The diff coverage is 91.22%.

Additional details and impacted files

@@            Coverage Diff             @@
##             main    #3596      +/-   ##
==========================================
+ Coverage   87.14%   87.34%   +0.20%     
==========================================
  Files         280      282       +2     
  Lines       10112    10132      +20     
==========================================
+ Hits         8812     8850      +38     
+ Misses       1300     1282      -18     

Impacted Files	Coverage Δ
...enTelemetry/Logs/OpenTelemetryLoggingExtensions.cs	100.00% <ø> (+2.77%)	⬆️
...metry/Logs/BatchExportLogRecordProcessorOptions.cs	60.00% <60.00%> (ø)
...lemetry/Logs/Options/OpenTelemetryLoggerOptions.cs	99.01% <97.56%> (-0.99%)	⬇️
...nTelemetry/Logs/ExportLogRecordProcessorOptions.cs	100.00% <100.00%> (ø)
.../OpenTelemetry/Logs/OpenTelemetryLoggerProvider.cs	93.82% <100.00%> (-0.80%)	⬇️
...emetry.Api/Internal/OpenTelemetryApiEventSource.cs	79.41% <0.00%> (-2.95%)	⬇️
...Telemetry/Internal/SelfDiagnosticsEventListener.cs	97.65% <0.00%> (+0.78%)	⬆️
src/OpenTelemetry/Logs/OpenTelemetryLogger.cs	88.88% <0.00%> (+2.22%)	⬆️
...metryProtocol/Implementation/ActivityExtensions.cs	95.05% <0.00%> (+3.29%)	⬆️
...xporter.OpenTelemetryProtocol/OtlpTraceExporter.cs	77.27% <0.00%> (+40.90%)	⬆️
... and 1 more

[github-actions]

This PR was marked stale due to lack of activity and will be closed in 7 days. Commenting or Pushing will instruct the bot to automatically remove the label. This bot runs once per day.

[alanwest]

I assume the spec will declare official environment variable names. You think we can be optimistic and remove DOTNET from these environment variable names and just go with OTEL_BLP_MAX_QUEUE_SIZE for example?

[alanwest]

oh scratch that i just saw the previous resolved conversation..

[alanwest]

LGTM

[CodeBlanch]

Merged to keep moving. Hoping to settle the environment variables before stable release.