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
Draft
Generate .NET SDK for Microsoft.Insights metric alerts 2024-03-01-preview version#53565violetVo wants to merge 1294 commits intosdk-gen-trigger-metricAlert-preview-2024-03from
violetVo wants to merge 1294 commits intosdk-gen-trigger-metricAlert-preview-2024-03from
Conversation
* 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>
…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>
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
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.