Skip to content

Azure Monitor Exporter - Update to Otel1.2.0-rc1#26355

Merged
rajkumar-rangaraj merged 4 commits intoAzure:mainfrom
rajkumar-rangaraj:rajrang/updateotel
Jan 15, 2022
Merged

Azure Monitor Exporter - Update to Otel1.2.0-rc1#26355
rajkumar-rangaraj merged 4 commits intoAzure:mainfrom
rajkumar-rangaraj:rajrang/updateotel

Conversation

@rajkumar-rangaraj
Copy link
Copy Markdown
Member

Modified AzureMonitorExporterLoggingExtensions from internal to public to allow the apps to consume AddOpenTelemetry inside LoggerFactory

Example

using var loggerFactory = LoggerFactory.Create(builder => builder
		.AddOpenTelemetry(loggerOptions =>
		{
			loggerOptions.SetResourceBuilder(resourceBuilder);
			loggerOptions.AddAzureMonitorLogExporter(exporterOptions =>
			{
				exporterOptions.ConnectionString = $"InstrumentationKey=Ikey;";
			});
		}));

@azure-sdk
Copy link
Copy Markdown
Collaborator

API changes have been detected in Azure.Monitor.OpenTelemetry.Exporter. You can review API changes here

API changes

+ namespace Microsoft.Extensions.Logging {
+     public static class AzureMonitorExporterLoggingExtensions {
+         public static OpenTelemetryLoggerOptions AddAzureMonitorLogExporter(this OpenTelemetryLoggerOptions loggerOptions, Action<AzureMonitorExporterOptions> configure = null);
+     }
+ }

@rajkumar-rangaraj rajkumar-rangaraj merged commit 443822b into Azure:main Jan 15, 2022
azure-sdk pushed a commit to azure-sdk/azure-sdk-for-net that referenced this pull request Oct 27, 2023
Update Search with x-ms-client-names and a few new requires (Azure#26355)

* Update Search with x-ms-client-names and a few new requires
* Make VectorizableTextQuery.text required
* Renamce SemanticSetting.defaultConfiguration to defaultConfigurationName to clarify it's a name of a configuration
* SemanticConfiguration.semanticPriorizedFields -> prioritizedFields as semantic is redundant
* Remove VectorQuery changes as they aren't final
* Remove SearchRequest.vectorQueries rename, fixed typo in SemanticSearch
* Add custom word
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Monitor - Exporter Monitor OpenTelemetry Exporter

Projects

None yet

Development

Successfully merging this pull request may close these issues.

5 participants