Skip to content

Remove deprecated Azure Log Analytics sample functions#34

Closed
daviburg wants to merge 3 commits intomainfrom
feature/remove-loganalytics
Closed

Remove deprecated Azure Log Analytics sample functions#34
daviburg wants to merge 3 commits intomainfrom
feature/remove-loganalytics

Conversation

@daviburg
Copy link
Copy Markdown
Member

@daviburg daviburg commented May 5, 2026

Summary

Remove sample functions and configuration for the deprecated Azure Log Analytics connector.

The Azure Log Analytics connector is deprecated by Microsoft. Users should use Azure Monitor Logs instead.

Changes

  • Deleted \AzureLogAnalyticsFunctions.cs\ — sample Azure Functions for ListSubscriptions and ListWorkspaces
  • Removed DI registration from \Program.cs\ (AzureloganalyticsClient singleton + using)
  • Removed \AzureLogAnalyticsOptions\ class and property from \ConnectorOptions.cs\
  • Removed connection settings from \local.settings.json.template\
  • Updated \README.md\ — removed Azure Log Analytics row from samples table

Related PRs

Validation

  • \dotnet build\ — 0 errors, 0 warnings

daviburg and others added 2 commits May 1, 2026 18:21
)

Consuming Connectors-NET-SDK namespace-rename branch via local project
reference. Will revert to NuGet package reference after SDK v0.9.0 publish.

Co-authored-by: Dobby <dobby@microsoft.com>
The Azure Log Analytics connector is deprecated by Microsoft. Removed sample functions, DI registration, configuration options, and settings template entries. Use Azure Monitor Logs connector instead.

Co-authored-by: Dobby <dobby@microsoft.com>
Copilot AI review requested due to automatic review settings May 5, 2026 05:54
@daviburg daviburg requested a review from a team as a code owner May 5, 2026 05:54
Copy link
Copy Markdown

Copilot AI left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Pull request overview

This PR removes the deprecated Azure Log Analytics connector sample from the DirectConnector/ Azure Functions app and updates the remaining samples to use the Microsoft.Azure.Connectors.Sdk.* connector namespaces instead of Microsoft.Azure.Connectors.DirectClient.*.

Changes:

  • Deleted the Azure Log Analytics sample functions and removed its DI/options/config entries.
  • Migrated sample code imports and XML docs from DirectClient namespaces to Microsoft.Azure.Connectors.Sdk.*.
  • Updated the root README samples table to drop the Azure Log Analytics entry.

Reviewed changes

Copilot reviewed 14 out of 14 changed files in this pull request and generated 9 comments.

Show a summary per file
File Description
README.md Removes Azure Log Analytics from the samples table (intro counts now need updating).
DirectConnector/AzureBlobFunctions.cs Updates Azure Blob connector namespace to Microsoft.Azure.Connectors.Sdk.*.
DirectConnector/AzureLogAnalyticsFunctions.cs Deletes the deprecated Azure Log Analytics sample functions.
DirectConnector/Configuration/ConnectorOptions.cs Removes AzureLogAnalyticsOptions and the AzureLogAnalytics options property.
DirectConnector/ConnectorFunctions.cs Updates Office365/SharePoint/Teams connector namespaces to Microsoft.Azure.Connectors.Sdk.*.
DirectConnector/ConnectorTriggerMetadataAttribute.cs Updates XML doc reference to the new SDK namespace.
DirectConnector/DirectConnector.csproj Switches from NuGet package reference to a local SDK ProjectReference (currently breaks clean builds).
DirectConnector/local.settings.json.template Removes Azure Log Analytics connection settings.
DirectConnector/MqFunctions.cs Updates MQ connector namespace to Microsoft.Azure.Connectors.Sdk.*.
DirectConnector/MsGraphFunctions.cs Updates MS Graph connector namespace to Microsoft.Azure.Connectors.Sdk.*.
DirectConnector/Office365UsersFunctions.cs Updates Office365 Users connector namespace to Microsoft.Azure.Connectors.Sdk.*.
DirectConnector/OneDriveFunctions.cs Updates OneDrive connector namespace + alias to Microsoft.Azure.Connectors.Sdk.*.
DirectConnector/Program.cs Removes Azure Log Analytics DI registration and migrates remaining connector usings to Microsoft.Azure.Connectors.Sdk.*.
DirectConnector/SmtpFunctions.cs Updates SMTP connector namespace to Microsoft.Azure.Connectors.Sdk.* and adjusts XML docs.

Comment thread DirectConnector/DirectConnector.csproj
Comment thread README.md
Comment thread DirectConnector/AzureBlobFunctions.cs Outdated
Comment thread DirectConnector/ConnectorFunctions.cs Outdated
Comment thread DirectConnector/MsGraphFunctions.cs Outdated
Comment thread DirectConnector/MqFunctions.cs Outdated
Comment thread DirectConnector/Office365UsersFunctions.cs Outdated
Comment thread DirectConnector/OneDriveFunctions.cs Outdated
Comment thread DirectConnector/SmtpFunctions.cs Outdated
Copy link
Copy Markdown

Copilot AI left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Pull request overview

Copilot reviewed 14 out of 14 changed files in this pull request and generated 1 comment.

Comment on lines +17 to 20
<!-- Temporary: consume local SDK branch for namespace-rename validation.
Revert to NuGet package reference after SDK v0.9.0 is published. -->
<ProjectReference Include="..\..\Connectors-NET-SDK-ns-rename\src\Microsoft.Azure.Connectors.Sdk\Microsoft.Azure.Connectors.Sdk.csproj" />
</ItemGroup>
@daviburg daviburg self-assigned this May 5, 2026
@daviburg
Copy link
Copy Markdown
Member Author

daviburg commented May 5, 2026

Looks like we have problem we pending SDK side changes on namespace and other breaking change refactoring, so we might need to combine this PR with the next clean update to the SDK.

@daviburg daviburg marked this pull request as draft May 5, 2026 06:59
daviburg added a commit to Azure/Connectors-NET-SDK that referenced this pull request May 5, 2026
## Summary

Remove the deprecated Azure Log Analytics connector
(`azureloganalytics`) from the SDK.

The [Azure Log Analytics
connector](https://learn.microsoft.com/en-us/connectors/azureloganalytics/)
and all its user-facing operations are deprecated by Microsoft. The
generated client contained only internal discovery methods
(ListSubscriptions, ListResourceGroups, ListWorkspaceNames,
ListArmQueryResultsSchema) — hidden helpers with no user-facing API
surface. Users should use the [Azure Monitor
Logs](https://learn.microsoft.com/en-us/connectors/azuremonitorlogs/)
connector instead.

## Changes

- **Deleted** `AzureloganalyticsExtensions.cs` (generated client) and
`AzureloganalyticsClientTests.cs` (tests)
- **Removed** `Azureloganalytics` from `ConnectorNames.cs` and
`ManagedConnectors.cs`
- **Updated** CHANGELOG.md, README.md, ROADMAP.md, connection-setup
SKILL.md

## Related PRs

- **Samples:** Azure/Connectors-NET-Samples#34 —
removes sample functions
- **Generator guardrail:**
https://msazure.visualstudio.com/One/_git/AzureUX-BPM/pullrequest/15620549
— prevents generating clients for effectively deprecated connectors

## Validation

- `dotnet build` — 0 errors, 0 warnings
- `dotnet test` — 202 passed, 0 failed
(ConnectorNames_AllConstantsAreRegistered sync test passes)

---------

Co-authored-by: Dobby <dobby@microsoft.com>
@daviburg
Copy link
Copy Markdown
Member Author

daviburg commented May 8, 2026

Closing — superseded by #40 which consolidates all SDK v0.9.0-preview.1 breaking changes, namespace migrations, and new features into a single PR.

@daviburg daviburg closed this May 8, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants