Skip to content

Comments

Generate .NET SDK for Microsoft.Insights metric alerts 2024-03-01-preview version#53565

Draft
violetVo wants to merge 1294 commits intosdk-gen-trigger-metricAlert-preview-2024-03from
main
Draft

Generate .NET SDK for Microsoft.Insights metric alerts 2024-03-01-preview version#53565
violetVo wants to merge 1294 commits intosdk-gen-trigger-metricAlert-preview-2024-03from
main

Conversation

@violetVo
Copy link
Contributor

Contributing to the Azure SDK

Please see our CONTRIBUTING.md if you are not familiar with contributing to this repository or have questions.

For specific information about pull request etiquette and best practices, see this section.

Copy link
Member

@mrm9084 mrm9084 left a comment

Choose a reason for hiding this comment

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

Update for release

Copilot AI and others added 15 commits February 10, 2026 16:06
* Initial plan

* Run dotnet format on sdk/devopsinfrastructure projects

Co-authored-by: m-nash <64171366+m-nash@users.noreply.github.com>

---------

Co-authored-by: copilot-swe-agent[bot] <198982749+Copilot@users.noreply.github.com>
Co-authored-by: m-nash <64171366+m-nash@users.noreply.github.com>
* Initial plan

* Run dotnet format on sdk/devspaces

Co-authored-by: m-nash <64171366+m-nash@users.noreply.github.com>

---------

Co-authored-by: copilot-swe-agent[bot] <198982749+Copilot@users.noreply.github.com>
Co-authored-by: m-nash <64171366+m-nash@users.noreply.github.com>
* Initial plan

* Run dotnet format on sdk/devtestlabs

Co-authored-by: m-nash <64171366+m-nash@users.noreply.github.com>

---------

Co-authored-by: copilot-swe-agent[bot] <198982749+Copilot@users.noreply.github.com>
Co-authored-by: m-nash <64171366+m-nash@users.noreply.github.com>
* Initial plan

* Run dotnet format on sdk/resourcemover

Co-authored-by: m-nash <64171366+m-nash@users.noreply.github.com>

---------

Co-authored-by: copilot-swe-agent[bot] <198982749+Copilot@users.noreply.github.com>
Co-authored-by: m-nash <64171366+m-nash@users.noreply.github.com>
* Initial plan

* Run dotnet format on sdk/disconnectedoperations projects

Co-authored-by: m-nash <64171366+m-nash@users.noreply.github.com>

---------

Co-authored-by: copilot-swe-agent[bot] <198982749+Copilot@users.noreply.github.com>
Co-authored-by: m-nash <64171366+m-nash@users.noreply.github.com>
* Initial plan

* Run dotnet format on sdk/dns projects

Co-authored-by: m-nash <64171366+m-nash@users.noreply.github.com>

---------

Co-authored-by: copilot-swe-agent[bot] <198982749+Copilot@users.noreply.github.com>
Co-authored-by: m-nash <64171366+m-nash@users.noreply.github.com>
* Initial plan

* Run dotnet format on sdk/dnsresolver projects

Co-authored-by: m-nash <64171366+m-nash@users.noreply.github.com>

---------

Co-authored-by: copilot-swe-agent[bot] <198982749+Copilot@users.noreply.github.com>
Co-authored-by: m-nash <64171366+m-nash@users.noreply.github.com>
* Initial plan

* Run dotnet format on sdk/durabletask

Co-authored-by: m-nash <64171366+m-nash@users.noreply.github.com>

---------

Co-authored-by: copilot-swe-agent[bot] <198982749+Copilot@users.noreply.github.com>
Co-authored-by: m-nash <64171366+m-nash@users.noreply.github.com>
* Initial plan

* Run dotnet format on sdk/dynatrace

Co-authored-by: m-nash <64171366+m-nash@users.noreply.github.com>

---------

Co-authored-by: copilot-swe-agent[bot] <198982749+Copilot@users.noreply.github.com>
Co-authored-by: m-nash <64171366+m-nash@users.noreply.github.com>
* Initial plan

* Run dotnet format on sdk/easm projects

Co-authored-by: m-nash <64171366+m-nash@users.noreply.github.com>

---------

Co-authored-by: copilot-swe-agent[bot] <198982749+Copilot@users.noreply.github.com>
Co-authored-by: m-nash <64171366+m-nash@users.noreply.github.com>
* Initial plan

* Run dotnet format on sdk/edgeactions

Co-authored-by: m-nash <64171366+m-nash@users.noreply.github.com>

---------

Co-authored-by: copilot-swe-agent[bot] <198982749+Copilot@users.noreply.github.com>
Co-authored-by: m-nash <64171366+m-nash@users.noreply.github.com>
* Initial plan

* Run dotnet format on sdk/edgeorder projects

Co-authored-by: m-nash <64171366+m-nash@users.noreply.github.com>

---------

Co-authored-by: copilot-swe-agent[bot] <198982749+Copilot@users.noreply.github.com>
Co-authored-by: m-nash <64171366+m-nash@users.noreply.github.com>
* Initial plan

* Run dotnet format on sdk/edgezones

Co-authored-by: m-nash <64171366+m-nash@users.noreply.github.com>

---------

Co-authored-by: copilot-swe-agent[bot] <198982749+Copilot@users.noreply.github.com>
Co-authored-by: m-nash <64171366+m-nash@users.noreply.github.com>
* Initial plan

* Run dotnet format on sdk/frontdoor

Co-authored-by: m-nash <64171366+m-nash@users.noreply.github.com>

---------

Co-authored-by: copilot-swe-agent[bot] <198982749+Copilot@users.noreply.github.com>
Co-authored-by: m-nash <64171366+m-nash@users.noreply.github.com>
* Initial plan

* Run dotnet format on sdk/grafana projects

Co-authored-by: m-nash <64171366+m-nash@users.noreply.github.com>

---------

Co-authored-by: copilot-swe-agent[bot] <198982749+Copilot@users.noreply.github.com>
Co-authored-by: m-nash <64171366+m-nash@users.noreply.github.com>
ryazhang-microsoft and others added 30 commits February 17, 2026 20:09
…er (#56283)

* Add ManagedIdentityCredential support in ConfigurableCredentialProvider (#55502)

* Add ConfigurableCredential support for ManagedIdentityCredential tests

Implements #55502: Adds ManagedIdentityCredential to the ConfigurableCredential
test infrastructure. Refactors base test class to use virtual factory methods,
adds CC test subclass and creation tests. Probe-specific tests are skipped in
CC pending chained credential support (#56233).

* Add ManagedIdentityIdType/ManagedIdentityId config properties

Introduce unified ManagedIdentityIdType (SystemAssigned|ClientId|ResourceId|
ObjectId) and ManagedIdentityId config properties, taking priority over legacy
per-type properties. Creation tests expanded to 18 with separate assertions
for both identity type and ID value smuggled through to ManagedIdentityId.

* Rename ManagedIdentityIdType config property to ManagedIdentityIdKind
…x when using the identity-binding feature (#56237)

* Fix custom CA certificate handling in KubernetesProxyHttpHandler to avoid NullReferenceException on Linux

* Add Linux-specific tests for KubernetesProxyHttpHandler to verify X509Chain fix and update certificate validation callback behavior

* Improve Linux tests

* Initial plan

* Fix certificate validation tests - use runtime-generated certs and optimize Linux tests

Co-authored-by: JonathanCrd <17486462+JonathanCrd@users.noreply.github.com>

* Fix certificate generation - ensure CA and leaf are from same pair

Co-authored-by: JonathanCrd <17486462+JonathanCrd@users.noreply.github.com>

* Address code review feedback - improve certificate disposal and add comments

Co-authored-by: JonathanCrd <17486462+JonathanCrd@users.noreply.github.com>

* Remove unnecessary null assignments after disposal

Co-authored-by: JonathanCrd <17486462+JonathanCrd@users.noreply.github.com>

* Add runtime certificate generation for NET462 and enhance validation callback tests

* reverse custom cert logic in KubernetesProxyHttpHandler

---------

Co-authored-by: copilot-swe-agent[bot] <198982749+Copilot@users.noreply.github.com>
Co-authored-by: JonathanCrd <17486462+JonathanCrd@users.noreply.github.com>
* Increment package version after release of Azure.Security.CodeTransparency

* Increment package version after release of Azure.Security.ConfidentialLedger
* Foundry Agent Generated

* Add model deployment to test resource, preview API to client, and begin test agent development

* First test batch

* remove agent tests

* spelling error fix

* Removed Agent Execution/Response Handling

* Changed Cognitive Services Token Scope

* Agent Config based on JS PR

* removed file as it relates to agent connection/execution


* cspell and warning issue fix

* Refactor VoiceLiveClient.WebSockets.cs to eliminate code duplication

* pull from main

* add functions around repetive code

* ran scripts eng\scripts\Update-Snippets.ps1 ai,  eng\scripts\Export-API.ps1 ai, + dotnet build /t:GenerateCode

* missed cspell

* update correct cspell

* updated api definitions

* Update FOUNDRY SETUP tests to use v2, be consistnent with Agent Config (in JS code and C#)

* Mimick JS PR: Azure/azure-sdk-for-js#37283 to support model / agent classes

* multiple agent test not necessary right now. But showed needed adjustments to specify agent version

* ALL Foundry Agent Setup Tests PAss

* added tests... however there are some service-side issues marked with warning. Consider completetly removing some tests?

* Fix Parallel Test

* update commit hash for tsp-location

* typespec update

* foundry agent as tool tests / functionality removed.

RUNS IF I DO: dotnet build /p:RunApiCompatForSrc=false -- issue in typespec?

* build works with this change...

* Interim Tests

* generate build

* update tsp location commit hash

* ran Generate code, Update snippets, and export api

* revert tsp-location changes ... will have to update soon

* change tsp-location repo + commit

* merge from main, generate build, and 2 scripts

* builds with interimresponse added classes

* updated snippets!!!

* revert change

---------

Co-authored-by: Ryan Hurey <rhurey@microsoft.com>
Co-authored-by: Emily Jiji <emilyjiji@microsoft.com>
#56315)

* Replace `azd config list` with `azd auth status` in TROUBLESHOOTING.md

The `azd config list` command is no longer the recommended way to verify the
logged-in Azure Developer CLI account. Update the troubleshooting guide to use
`azd auth status` instead.

Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>

* Update azd login commands

* React to feedback from Charles

---------

Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>
Add execution and creation tests for ApiKeyCredential support in
ConfigurableCredential. Execution tests validate GetToken/GetTokenAsync
return the configured key. Creation tests verify config properties
propagate to the internal _apiKeyToken field.

Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>
…vider (#56327)

* Add InteractiveBrowserCredential support to ConfigurableCredentialProvider

- Add configurable IB tests inheriting from base InteractiveBrowserCredentialTests
- Make IB-specific settings configurable via IConfiguration: DisableAutomaticAuthentication,
  LoginHint, BrowserCustomization, AuthenticationRecord, UseDefaultBrokerAccount
- Add creation tests validating all properties pass through correctly
- Move broker-path tests to Azure.Identity.Broker test project
- Share ConfigurableCredentialTestHelper across projects via Compile Include
- Fix DisableAutomaticAuthentication not flowing through internal IBC constructor

Closes #55503

Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>

* Fix CI failure: inline TestCase parameter to avoid MSBuild log error matching

The ProcessMsBuildLogLine CI script matches '*: error*' in test output.
When GeneralExceptions_With_CertainErrors test is skipped on Linux, the
TestCase parameter containing 'TS003: Error' in the test name triggers
a false positive error. Move the string inline since there is only one
test case.

Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>

* Fix broker test failure on Linux: catch AuthenticationFailedException

On Linux CI, MSAL token cache persistence fails with missing
libsecret-1.so.0, which surfaces as AuthenticationFailedException
rather than CredentialUnavailableException. The test only validates
that BeforeBuildClient was invoked, so catching either exception is
correct.

Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>

---------

Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>
* Make exporter classes and options public

* Add sdk part

* run sdk command

* add enablestandardmetrics

* remove unintentional change

* add changelog entry

* delete unnecessary api file

* Update sdk/monitor/Azure.Monitor.OpenTelemetry.Exporter/CHANGELOG.md

---------

Co-authored-by: Matthew Sainsbury <msainsbury@microsoft.com>
Co-authored-by: Emily Jiji <emilyjiji@microsoft.com>
* Update UnbrandedGeneratorVersion to 1.0.0-alpha.20260218.5

* regen

* rename tsp-location due to spec compliance issues

* upgrade authoring

* disable azure aip oai

* update commit

* update

* regen

* clientinit

* shorten

* regen

* regen

* shorten again

---------

Co-authored-by: Ubuntu <cloudtest@8c86c60dc000001.tkkhlgj2u51eln3mcb0lygmbye.dx.internal.cloudapp.net>
Co-authored-by: jolov <jolov@microsoft.com>
Co-authored-by: Jorge Rangel <jorgerangel@microsoft.com>
* Update Sample13_WebSearch.md with warning section

Added warning about costs and terms for Bing Custom Search tool.

* Update warning message for web search sample

* Update Sample16_Bing_Grounding.md with warning

Added warning about costs and compliance for Bing grounding.

* Update Sample17_Bing_Grounding_Streaming.md with warning

Added warning about costs and data flow for Bing grounding.

* Update Sample18_Bing_Custom_Search.md

* Update Sample13_WebSearch.md

* Update warning about Bing grounding terms

* Revise warning for Bing grounding in documentation

Updated warning about Bing Search tool grounding costs and terms.

* Refine warning for Bing Custom Search tool

Updated warning about Bing Custom Search to remove unnecessary link parameters.

* Update Sample13_WebSearch.md

* Fix formatting in Bing grounding sample documentation

* Fix formatting of warning section in markdown sample

* Fix formatting in Bing Custom Search sample documentation
Co-authored-by: Emily Jiji <emilyjiji@microsoft.com>
…alpha.20260219.1 (#56360)

* Regenerate repository SDK with TypeSpec build 20260219.1

* Update SDK code sc_wo_4

* Update SDK code ed_mo_2

* Update SDK code mo_re_3

* Update SDK code ad_co_0

* Update SDK code co_ed_1

* Upgrade easm commit

* regen

---------

Co-authored-by: jolov <jolov@microsoft.com>
* Update UnbrandedGeneratorVersion to 1.0.0-alpha.20260219.3

* Update UnbrandedGeneratorVersion to 1.0.0-alpha.20260219.3 (#56363)

* Initial plan

* Update XmlSerializableVisitor to use WriteXml method name and regenerate test projects

Co-authored-by: jorgerangel-msft <102122018+jorgerangel-msft@users.noreply.github.com>

* Revert IXmlSerializable.cs, update XmlSerializableVisitor for private WriteXml, regenerate test projects

Co-authored-by: jorgerangel-msft <102122018+jorgerangel-msft@users.noreply.github.com>

* Rename constants in XmlSerializableVisitor: IXmlSerializableWriteMethodName and WriteMethodName

Co-authored-by: jorgerangel-msft <102122018+jorgerangel-msft@users.noreply.github.com>

---------

Co-authored-by: copilot-swe-agent[bot] <198982749+Copilot@users.noreply.github.com>
Co-authored-by: jorgerangel-msft <102122018+jorgerangel-msft@users.noreply.github.com>

---------

Co-authored-by: Ubuntu <cloudtest@e0b7f29bc000000.tkkhlgj2u51eln3mcb0lygmbye.dx.internal.cloudapp.net>
Co-authored-by: Copilot <198982749+Copilot@users.noreply.github.com>
Co-authored-by: jorgerangel-msft <102122018+jorgerangel-msft@users.noreply.github.com>
* Initial plan

* Add XmlPaginationTests.cs for XML pageable spector scenarios

Co-authored-by: jorgerangel-msft <102122018+jorgerangel-msft@users.noreply.github.com>

---------

Co-authored-by: copilot-swe-agent[bot] <198982749+Copilot@users.noreply.github.com>
Co-authored-by: jorgerangel-msft <102122018+jorgerangel-msft@users.noreply.github.com>
* Ran the prepare release script

* Add missing changelog entry

* Update CHANGELOG.md

* Revert unintended change in csproj

---------

Co-authored-by: Scott Addie <10702007+scottaddie@users.noreply.github.com>
…56368)

* Fix AOT warning regression in Azure.Monitor.OpenTelemetry.Exporter

With change #54942, new AOT warnings are being introduced due to using ConfigurationBinder Bind calls without using the source generator. Prior to that, this warning was suppressed, which wasn't correct either.

The real fix is to use the source generator.

Fix #56365

* Respond to PR feedback
…#56334)

* Fix IsChainedCredential for single credential selection in DACFactory

When CredentialSource selects a specific single credential, each Create*
method now checks Options.CredentialSource to set IsChainedCredential to
false. This ensures proper exception behavior - non-chained credentials
throw AuthenticationFailedException for unknown errors instead of wrapping
them as CredentialUnavailableException.

Fixes #56324

Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>

* pr fb

* Validate CredentialSource in ConvertCredentialSource

ConvertCredentialSource now throws InvalidOperationException for unknown
values instead of passing them through. Null throws a 'required' error,
empty/unknown strings throw an 'unsupported' error. Updated tests to
expect these exceptions.

Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>

---------

Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>
…p in ArrayBackedPropertyBag (#56354)

* fix ctor

* fix indentation
Co-authored-by: Ubuntu <cloudtest@6c2521a7c000004.tkkhlgj2u51eln3mcb0lygmbye.dx.internal.cloudapp.net>
* Checkpoint from Copilot CLI for coding agent session

* Fix ns logic

* regen

* fix

* fix
* Initial plan

* Add AGENTS.md file with comprehensive AI agent guidelines

Co-authored-by: ronniegeraghty <28957151+ronniegeraghty@users.noreply.github.com>

* Fix relative link to use absolute URL format in copilot-instructions.md

Co-authored-by: ronniegeraghty <28957151+ronniegeraghty@users.noreply.github.com>

* Convert all relative links to absolute URLs in AGENTS.md

Co-authored-by: ronniegeraghty <28957151+ronniegeraghty@users.noreply.github.com>

* Address review comments on AGENTS.md

Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>

* Address jsquire review comments and fix broken link

Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>

* Sync copilot-instructions.md with main and fix relative link

Updated to match main branch content. Converted relative
verify-setup.instructions.md link to absolute URL to pass
link validation CI.

Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>

---------

Co-authored-by: copilot-swe-agent[bot] <198982749+Copilot@users.noreply.github.com>
Co-authored-by: ronniegeraghty <28957151+ronniegeraghty@users.noreply.github.com>
Co-authored-by: Ronnie Geraghty <rgeraghty@microsoft.com>
Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>
* separate out the analyzers from the source generator

* pr fb

* Apply PR review feedback for analyzer separation

- Use pattern matching for cleaner code in analyzers
- Fix typos (outter -> outer, Analzyzer -> Analyzer)
- Add null check separation in AbstractTypeWithoutProxyAnalyzer
- Add comment explaining why private protected is disallowed
- Add remarks about MultipleContextAnalyzer only running on build
- Add tests for partial class declarations with varying order
- Fix ParameterlessConstructor diagnostic reference
- Update test expectation for non-partial context diagnostic

* Move diagnostics from source generator to analyzers

- Remove diagnostic reporting from ModelReaderWriterContextGenerator
- The following diagnostics are now only reported by their respective analyzers:
  - MultipleContextsNotSupported (MultipleContextAnalyzer)
  - ContextMustBePartial (PartialModifierAnalyzer)
  - ParameterlessConstructor (ParameterlessConstructorAnalyzer)
  - AbstractTypeWithoutProxy (AbstractTypeWithoutProxyAnalyzer)
- Add IsModelReaderWriterContext filter to properly handle invalid contexts
- Use pattern matching for null-safe type extraction in MustImplementIPersistableModelAnalyzer
- Rename partial modifier tests for clarity

* Simplify base type loop in BuildableAttributeUsageAnalyzer

Use do-while loop pattern per PR feedback

* Extract InheritsFrom helper to reduce code duplication

Add SymbolExtensions.InheritsFrom() method and use it in:
- AbstractTypeWithoutProxyAnalyzer
- MustImplementIPersistableModelAnalyzer
- ParameterlessConstructorAnalyzer
- PartialModifierAnalyzer

* Remove unrelated Azure.Core changes

These changes were temporary workarounds that are no longer needed
as the fixes are now properly in main.

* Add analyzer tests and fix unbound/closed generic handling

- Add 17 new tests across all 6 analyzers covering unbound generics,
  closed generics, structs, interfaces, sealed classes, implicit ctors,
  multiple attributes, and indirect inheritance
- Fix AbstractTypeWithoutProxyAnalyzer to skip unbound generic types
- Fix MustImplementIPersistableModelAnalyzer to handle non-collection
  generic types (e.g. Foo<int>) instead of returning null
- Fix ParameterlessConstructorAnalyzer to skip unbound generic types

Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>

* Address remaining PR review comments

- Use AttributeClass?.OriginalDefinition for attribute comparisons
- Fix MultipleContextAnalyzer to detect indirect context subclasses
- Standardize SymbolEqualityComparer.Default.Equals(a, b) style
- Add IndirectSubclassesShouldFail test for MultipleContextAnalyzer

Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>

---------

Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

AI Model Inference AI Projects Client This issue is related to a non-management package CodeGen Issues that relate to code generation Event Grid Health Deidentification Image Analysis KeyVault Mgmt This issue is related to a management package. Monitor - Query Provisioning Schema Registry

Projects

Status: Untriaged

Development

Successfully merging this pull request may close these issues.