Tweaks.

src/OpenTelemetry.Extensions.EventSource/OpenTelemetryEventSourceLoggerOptionsExtensions.cs

@@ -47,7 +47,7 @@ public static OpenTelemetryLoggerOptions AddEventSourceLogEmitter(
             Guard.ThrowIfNull(options);
             Guard.ThrowIfNull(shouldListenToFunc);
 
-            options.Services.TryAddSingleton<EventSourceManager>();
+            options.ConfigureServices(services => services.TryAddSingleton<EventSourceManager>());
 
             options.ConfigureProvider((sp, provider) =>
             {

src/OpenTelemetry/Logs/OpenTelemetryLoggingExtensions.cs

@@ -73,10 +73,8 @@ public static ILoggingBuilder AddOpenTelemetry(this ILoggingBuilder builder, Act
                  * needed.
                  */
 
-                var options = new OpenTelemetryLoggerOptions
-                {
-                    Services = builder.Services,
-                };
+                var options = new OpenTelemetryLoggerOptions(builder.Services);
+
                 configure(options);
 
                 /*

src/OpenTelemetry/Logs/Options/OpenTelemetryLoggerOptions.cs

@@ -34,16 +34,29 @@ public class OpenTelemetryLoggerOptions
         internal readonly List<BaseProcessor<LogRecord>> Processors = new();
         internal ResourceBuilder? ResourceBuilder;
         internal List<Action<IServiceProvider, OpenTelemetryLoggerProvider>>? ConfigurationActions = new();
-        internal IServiceCollection? Services;
 
         private const bool DefaultIncludeScopes = false;
         private const bool DefaultIncludeFormattedMessage = false;
         private const bool DefaultParseStateValues = false;
 
+        private IServiceCollection? services;
         private bool? includeScopes;
         private bool? includeFormattedMessage;
         private bool? parseStateValues;
 
+        /// <summary>
+        /// Initializes a new instance of the <see cref="OpenTelemetryLoggerOptions"/> class.
+        /// </summary>
+        public OpenTelemetryLoggerOptions()
+            : this(services: null)
+        {
+        }
+
+        internal OpenTelemetryLoggerOptions(IServiceCollection? services)
+        {
+            this.services = services;
+        }
+
         /// <summary>
         /// Gets or sets a value indicating whether or not log scopes should be
         /// included on generated <see cref="LogRecord"/>s. Default value:
@@ -82,6 +95,8 @@ public bool ParseStateValues
             set => this.parseStateValues = value;
         }
 
+        internal IServiceCollection? Services => this.services;
+
         /// <summary>
         /// Adds processor to the options.
         /// </summary>
@@ -173,7 +188,7 @@ public OpenTelemetryLoggerOptions ConfigureServices(
         {
             Guard.ThrowIfNull(configure);
 
-            var services = this.Services;
+            var services = this.services;
 
             if (services == null)
             {
@@ -208,6 +223,29 @@ public OpenTelemetryLoggerOptions ConfigureProvider(
             return this;
         }
 
+        internal OpenTelemetryLoggerProvider Build()
+        {
+            var services = this.services;
+
+            if (services == null)
+            {
+                throw new NotSupportedException("LoggerProviderBuilder build method cannot be called multiple times.");
+            }
+
+            this.services = null;
+
+            var serviceProvider = services.BuildServiceProvider();
+
+            var finalOptions = serviceProvider.GetRequiredService<IOptionsMonitor<OpenTelemetryLoggerOptions>>().CurrentValue;
+
+            this.ApplyTo(finalOptions);
+
+            return new OpenTelemetryLoggerProvider(
+                finalOptions,
+                serviceProvider,
+                ownsServiceProvider: true);
+        }
+
         internal void ApplyTo(OpenTelemetryLoggerOptions other)
         {
             Debug.Assert(other != null, "other instance was null");

src/OpenTelemetry/Logs/Options/OpenTelemetryLoggerOptionsSdk.cs

@@ -18,43 +18,16 @@
 
 using System;
 using Microsoft.Extensions.DependencyInjection;
-using Microsoft.Extensions.Options;
 
 namespace OpenTelemetry.Logs;
 
 internal sealed class OpenTelemetryLoggerOptionsSdk : OpenTelemetryLoggerOptions
 {
     public OpenTelemetryLoggerOptionsSdk(Action<OpenTelemetryLoggerOptions>? configure)
+        : base(new ServiceCollection())
     {
-        var services = new ServiceCollection();
-
-        services.AddOptions();
-
-        this.Services = services;
+        this.ConfigureServices(services => services.AddOptions());
 
         configure?.Invoke(this);
     }
-
-    public OpenTelemetryLoggerProvider Build()
-    {
-        var services = this.Services;
-
-        if (services == null)
-        {
-            throw new NotSupportedException("LoggerProviderBuilder build method cannot be called multiple times.");
-        }
-
-        this.Services = null;
-
-        var serviceProvider = services.BuildServiceProvider();
-
-        var finalOptions = serviceProvider.GetRequiredService<IOptionsMonitor<OpenTelemetryLoggerOptions>>().CurrentValue;
-
-        this.ApplyTo(finalOptions);
-
-        return new OpenTelemetryLoggerProvider(
-            finalOptions,
-            serviceProvider,
-            ownsServiceProvider: true);
-    }
 }

test/OpenTelemetry.Tests/Logs/OpenTelemetryLoggingExtensionsTests.cs

@@ -275,7 +275,7 @@ public void ServiceCollectionAddOpenTelemetryProcessorThroughDependencyWithRegis
             {
                 if (!registerOutside)
                 {
-                    options.Services!.AddSingleton<CustomProcessor>();
+                    options.ConfigureServices(services => services.AddSingleton<CustomProcessor>());
                 }
 
                 options.AddProcessor<CustomProcessor>();