-
Notifications
You must be signed in to change notification settings - Fork 4.9k
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Make Types in ResourceIdentifier public #22612
Merged
archerzz
merged 2 commits into
Azure:feature/mgmt-track2
from
fengzhou-msft:expose_types
Jul 14, 2021
Merged
Make Types in ResourceIdentifier public #22612
archerzz
merged 2 commits into
Azure:feature/mgmt-track2
from
fengzhou-msft:expose_types
Jul 14, 2021
Conversation
This file contains 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
fengzhou-msft
requested review from
allenjzhang,
m-nash,
markcowl,
nisha-bhatia and
YalinLi0312
as code owners
July 13, 2021 09:26
fengzhou-msft
commented
Jul 13, 2021
@@ -428,7 +428,7 @@ internal virtual string ToResourceString() | |||
StringBuilder builder = new StringBuilder(Parent.ToResourceString()); | |||
if (IsChild) | |||
{ | |||
builder.Append($"/{ResourceType.Types.Last()}"); | |||
builder.Append($"/{ResourceType.Types[ResourceType.Types.Count - 1]}"); |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
To avoid error CA1826: Do not use Enumerable methods on indexable collections. Instead use the collection directly
.
…net into expose_types
archerzz
approved these changes
Jul 14, 2021
m-nash
added a commit
that referenced
this pull request
Jul 14, 2021
* initial checkin for core / tests (#18968) * initial checkin for core / tests * Add readme.md file * Fix header in readme * remove autorest until we convert off hand written types * update azure.core reference remove sets to tags ignore tests which require credentials for now * Add the proto client which will be used until the autogen work is complete * Add missing readmes for proto client * update to require linq 4.0.0 so that don't have to upgrade bcl * add changlog for CI * change system.linq.async to update * update to use source vs binary for the temporary generation of Azure.ResourceManager.Authorization * Add changelog for azure.resourcemanager.authorization and update version to be beta * update typos in readme * update name of proto client sln * merge in changes from proto repo this week * fixed unit test failures missed a few files from the merge from proto repo updated compute and network sdk version to use latest azure.core with readonly list types * ignore tests which require live azure until 5122 is closed * Update failing scenario test * Subnet Container list still returns SubnetOperations and not Subnet #5584 (#19109) * 5584 * Update async test * Test * WIP: test * Update Proto.Authorization.csproj * Update Proto.Authorization.csproj * Update Proto.Authorization.csproj * Update Proto.Authorization.csproj * add param checking to resource (#19134) * Adding parameter check for track 2 Azure.ResourceManager.core (#19162) * initial checkin * adding test cases * nibhati-5142-add Params and testing for ApiVersionBase.cs (#19193) * add tests for ApiVersionBase * update methods * nibhati - 5142 - add tests for ArmBuilder.cs (#19231) * add tests for ArmBuilder * Update ArmBuilderTests.cs * Update SubscriptionContainer.cs * Update test project to consume framework for recorded tests (#19171) * Update test project to consume framework for recorded tests * Changes required to get instrumentation working for the new armClient added support for session recording for onetimesetup and onetimeteardown * move unit test files * update tests to not use username for prefix * update targets to separate existing track 2 libraries from new core until autorest changes are finished * merge in armbuildertest and update to use new framework * remove necessity on session files if no onetimesetup * Address review comments * Move ApiVersionOverrides to ARMClientOptions class (#19358) * Rename list operations (#19386) * tenant resource sample (#19195) * WIP: tenant resource * WIP: updates * Update ResourceIdentifier.cs * Update ResourceIdentifier.cs * Update ResourceType.cs * WIP update check * WIP * Add readme for billing * Stylecop * WIP * Remove style * wip: update * WIP: updates * WIP * WIP: update * WIP * Updates * WIP: updates * Update * WIP: updates * WIP * Update tests * WIP: updates * Remove reference for Azure.Identity (#19434) * Remove reference for Azure.Identity * fixing ArmClient construction in proto client * resync and fix build error * 5521 - Rename TryGetValue to TryGet/TryGetAsync (#19478) * Rename TryGetValue to TryGet/TryGetAsync * Update ContainerBase.cs * Update ContainerBase.cs * Update TryGet method and test class * Update CheckResourceExists.cs * Rename TryGetValue to TryGet/TryGetAsync * Update ContainerBase.cs * Update ContainerBase.cs * Update TryGet method and test class * Update CheckResourceExists.cs * Update ContainerBaseTest.cs * Update TryGet Test * Update tests * Update tests and framework to instrument all clients recursively (#19443) * Framework changes * Azure.ResourceManager.Core changes to support client interceptors * Test changes * proto client changes * updates after merge * Address review comments * WIP * updates after WIP * Update the implicit operator (#19720) * Remove System.Linq.Async dependency. (#19745) * Change the accessibility to virtual for Resource.Id (#19804) * Sku Serialization (#19819) * Sku Serialization * Divide Sku class * Update Sku * Add lookup mechanism for ID to apiversion (#19437) * WIP: updates * WIP: updates * Update test * Update ResourceListOperationsTest.cs * Update CreateSingleVmExample.cs * Update CreateSingleVmExample.cs * Update CreateSingleVmExample.cs * Update CreateSingleVmExample.cs * WIP: Updates * Add extra line * Update * WIP: * WIP: updates * WIp: updates * Remove API version extensions * WIP: updates * Updates * Update test * WIP updates * Updates * WIP * Remove blank lines * Update filtering * Updates * Update * WIP: updates * WIP: updates * Add space * Remove debug * Updates * WIP: updates * WIP * Change the accessbility to virtual for Resource.Id * WIP: updates * WIP Updates * WIP updates * WIP * WIP * WIP: * revert * WIP: updates * WIP: updates * WIP: updated * WIP Co-authored-by: YalinLi0312 <yall@microsoft.com> * Resource Identifier changes (#19636) * Resource Identifier changes * Adapting Container classes to base class change in Id property * Fixing xml documentation comments for ResourceType * Cleaning up resource id ToStrign() and fixing ListLocations client creation * Adjusting new tenant resource operations for resource id scheme * Responding to review comments * Add test coverage, retain string reresentation of resource id * Fixing additional merge issues * Updating comparison tests to use concrete variable types * Plan Serialization (#19820) * Plan Serialization * Divide Plan class * Update Plan * Mnash 5705 rename container (#19932) * Change the accessbility to virtual for Resource.Id * rename ArmClient, ArmClientOptions, Get[Resource]Container * updates after merge * missing one file for rename Co-authored-by: YalinLi0312 <yall@microsoft.com> Co-authored-by: m-nash <prognash@microsoft.com> * 2 missing renames (#19952) * Change the accessbility to virtual for Resource.Id * rename ArmClient, ArmClientOptions, Get[Resource]Container * updates after merge * missing one file for rename * Missed 2 renames Co-authored-by: YalinLi0312 <yall@microsoft.com> Co-authored-by: m-nash <prognash@microsoft.com> * nibhati - 5191 - add ClientContext.cs (#19602) * WIP * WIP * Update ResourceOperationsBase.cs * update for tenant extensions * update csproj * Update Azure.ResourceManager.Network.csproj * update csproj * update csproj * WIP * WIP * update csproj * update csproj * update csproj * add tests * update test * update * fix test cases to use the diagnostics case fix a missing mocking constructor * WIP * update framework for tests * Update Azure.Test.Perf.csproj * merge * revert list json files * revert * WIP * fix ci * Update Program.cs * update proto * fix test * Update ClientContextTests.cs * update proto * Update Azure.ResourceManager.Billing.csproj * update version * update ci * Update Azure.ResourceManager.EventHubs.csproj * add packable Co-authored-by: m-nash <prognash@microsoft.com> * GenericResource Serialization (#19898) * Resource Serialization * Plan Serialization * Divide Plan class * Divide Resource class * Update writer * Update Plan * TrackedResource Serialization * Update GenericResource * Clean Up * Update GenericResource * Remove read only * Update tests * TrackedResource Serialization (#19855) * Resource Serialization * Divide Resource class * Update writer * TrackedResource Serialization * Update Deserialization * Update Test * Update Tests * Update Tracked Resource * Fix PR * Remove readonly * Remove the Get[Resource]Operations methods except for client extensions (#20136) * Change the accessbility to virtual for Resource.Id * wip * ignore test temporarily * update to remove Get[Resource]Operations Co-authored-by: YalinLi0312 <yall@microsoft.com> Co-authored-by: m-nash <prognash@microsoft.com> * nibhati - 5572 - make Credentials and Options internal (#20243) * Change the accessbility to virtual for Resource.Id * add transport * update test * add jsons * add json * remove transport Co-authored-by: YalinLi0312 <yall@microsoft.com> * Remove ignore on tests that were failing due to faulted tasks in interceptor (#20341) * Change the accessbility to virtual for Resource.Id * remove ignore on tests and update intrumentation to handle faulted tasks Co-authored-by: YalinLi0312 <yall@microsoft.com> Co-authored-by: m-nash <micnash@microsoft.com> * Make the Reference type generic (#20358) * Change the accessbility to virtual for Resource.Id * Make the reference type generic Co-authored-by: m-nash <prognash@microsoft.com> Co-authored-by: m-nash <prognash@gmail.com> * nibhati 5401 - add CancellationToken to GetResource() (#20396) * Change the accessbility to virtual for Resource.Id * add token Co-authored-by: YalinLi0312 <yall@microsoft.com> * JonathanCrd 5574 - remove system.collections.immutable (#20437) * Change the accessbility to virtual for Resource.Id * Removing package reference to System.Collections.Immutable * Removing use of ImmutableList Co-authored-by: YalinLi0312 <yall@microsoft.com> Co-authored-by: m-nash <prognash@microsoft.com> Co-authored-by: m-nash <prognash@gmail.com> * JonathanCrd 5003 - comparison methods for ResourceIdentifier (#20456) * Change the accessbility to virtual for Resource.Id * Operators == and != overloads in ResourceIdentifier * Added EqualsToString Tests and TestCases Co-authored-by: YalinLi0312 <yall@microsoft.com> Co-authored-by: m-nash <prognash@microsoft.com> Co-authored-by: m-nash <prognash@gmail.com> * update constructors for reference types (#20477) * update constructors for reference types * fix stylecop issue * fixes after merge * add two way implicit for autorest deserialization (#20557) * update constructors for reference types * fix stylecop issue * add two way implicit for autorest deserialization * Add a common definition for sub-resource (#20524) * add SubResource * serialization of the two classes * protected internal; full subresource namespace Co-authored-by: Yeming Liu <yeliu@microsoft.com> * Add default constructors to resource classes (#20583) * update constructor signatures for autogen (#20607) * update constructors for reference types * fix stylecop issue * add two way implicit for autorest deserialization * update constructor signatures for autogen * Remove ProxyResource since its intended to be used any longer (#20608) * adding reference constructors attributes (#20611) * update constructors for reference types * fix stylecop issue * add two way implicit for autorest deserialization * update constructor signatures for autogen * add attributes to define ctor types for autogen * Cache the creation of HTTP pipeline in ArmClient (#20636) * Remove create from container base; remove ArmBuilder abstract class (#20684) * Remove create from container base; remove ArmBuilder abstract class * UnTypedContainer->Container; fix doc issues * make createAsync() virtual Co-authored-by: Yeming Liu <yeliu@microsoft.com> * fixes after merge * adding singleton pattern in Arm.Core and Proto (#20591) * adding initial singleton base classs and proto code * PR comment fix. * Completed singleton VMSS test case. * Fix merge issue around ArmBuilder removal * Fix build issue. * Fixing test failure * CR comment. * Flattend the builder to partial classes. * CR fix * Update arm response types to match current Azure.Core pattern (#20738) * WIP * wip * Change the accessbility to virtual for Resource.Id * update strawman for operation changes * finalize changes before testing * update ph overloads * move pipeline creation into armoperation * wip * updates based on new OperationInternals * address PR comments * update test classes to use new pattern * final fixes for arm core tests * use mock response instead of null * updates to proto code * add null check around basetype Co-authored-by: YalinLi0312 <yall@microsoft.com> Co-authored-by: m-nash <prognash@microsoft.com> * Fix a bug in ArmClient (#20897) * Move generic resource from resources to core (#20927) * initial changes to move generic resource items over * wip updating tests * more work on test cases * updates to adapt the operation changes * remove arm response / operation * remove unused classes now and finish move resource test * update resourcegroup operation tests * update tests for genericresource operations * add genericresource container tests * update proto solution * update proto comments * address naming comments * update comments which no longer have inheritance address wildcard comment in csproj * update ResourceContainerBase (#20981) Co-authored-by: Yeming Liu <yeliu@microsoft.com> * updates to subresource type (#21010) clean up extensions meant for Operation now * VM image, extension image (edgezone not included as it is new not in Compute sdk yet) (#20971) * update reference types and add unit tests to validate (#21031) * updates to subresource type clean up extensions meant for Operation now * need protected internal serialization constructor for reference types, added unit tests to validate for future * updates after merge * update policy copying for clone and convert (#21069) * 5771 - GetGenericResourceOperations in ArmClient (#21066) * Change the accessbility to virtual for Resource.Id * GetGenericResourceOperations extension method implementation * Create ArmClientTests.cs * Scenario Tests * GetGenericResourceOperations for single ID * Single ID tests * Addressing comments * Tests recordings * Tests recordings Co-authored-by: YalinLi0312 <yall@microsoft.com> Co-authored-by: m-nash <prognash@microsoft.com> Co-authored-by: m-nash <prognash@gmail.com> * 5844 Rename SubResourceWritable to WritableSubResource (#21092) * Rename symbol * Rename class * Rename serialization class * Merge Azure.ResourceManager.Resources.Models.Subscription with Azure.ResourceManager.Core.SubscriptionData (#20751) * Initial * Merge * Update SubscriptionOperations.cs * Fix error * Update using * Update test * Fix tests * Update json * Update ClientContextTests.cs * update based on comments * Add get test * Remove unnecessary file * Update LocationListResult.cs * Update ClientContextTests.cs * Update operation class & add tests * move models to model folder Co-authored-by: m-nash <prognash@gmail.com> * remove id type from container base to match operation base (#21150) * remove id type from container base to match operation base * update proto references * add support for root resource Id container (#21175) * remove id type from container base to match operation base * update proto references * add support for root resource Id container * Merge ResourceGroup model in Resources with ResourceGroupData in Core (#20904) * Change the accessbility to virtual for Resource.Id * Merge ResourceGroup model in Resources with Core ResourceGroupData * Add and update scenario tests * Resolve merge conflicts * Update scenario tests * Record ResourceGroupContainerTests * Initialize Tags in base class * Update ResourceGroupOperationsTests and re-record * Fix the errors in Proto.Client scenario tests * Move some files in Generated folder back * Address review comments * Add records * Fix the failing test StartExportTemplate * Address comments * Change tag lro operations to return its lro type * Change rg CreateOrUpdate lro operations to return its lro object * Update CreateOrUpdate lro object * Run tests Co-authored-by: m-nash <prognash@microsoft.com> Co-authored-by: m-nash <prognash@gmail.com> * Delete PhTaskDeferringAsyncPageable.cs (#21183) * Remove PhArmResponse (#21139) * Remove PhArmResponse part 1 * Remove PhArmResponse part 2 * Delete PhArmResponse.cs Co-authored-by: m-nash <prognash@gmail.com> * Remove PhArmOperation (#21290) * Remove PhArmOperation * Disable TestResourceTypeFilterParamCheck Test * Disable SA warning * remove unused file Co-authored-by: m-nash <prognash@gmail.com> * 5841 - Update List methods from proto clients. (#21314) * Remove List methods with inner Get() call in Compute * Remove List methods with inner Get() call in Network * Removing scenario tests referencing deleted List() methods * Add lambda wrapper in armclient for getting container objects in othe… (#21390) * Add lambda wrapper in armclient for getting container objects in other RPs * update xml comments * 5285 Remove TrackedResource<TIdentifier, TModel> class. (#21376) * Delete TrackedResource.cs * Removed class from Core added to individual Proto Libraries * Removing imported class in Proto.Client * Add missing new lines * update to allow external access to a constructor that doesn't have an operation base (#21395) * Add lambda wrapper in armclient for getting container objects in other RPs * update xml comments * update to add externally accesible constructor that doesn't have an operation base like armclient as the parent * Move Azure.ResourceManager.Resources Tenants objects to Azure.ResourceManager,Core (#21413) * Change the accessbility to virtual for Resource.Id * Move Azure.ResourceManager.Resources Tenants objects to Azure.ResourceManager.Core * Resolve conflicts Co-authored-by: m-nash <prognash@microsoft.com> Co-authored-by: m-nash <prognash@gmail.com> * Templetize SubResource and WritableSubResource classes, and added unit tests (#21397) * Change the accessbility to virtual for Resource.Id * Sub resource template and tests * public class , public constructor * some PR comments * PR comments and phonecall around silent errors and casting Co-authored-by: YalinLi0312 <yall@microsoft.com> Co-authored-by: m-nash <prognash@microsoft.com> Co-authored-by: m-nash <prognash@gmail.com> * Remove the suppression for namespace in mgmt sdks (#21534) * remove no warn AZC0001 * remove the namespace warn suppression from mgmt plane sdks * remove old todo * update tenant and subscription list resource hooks to follow the same pattern (#21568) * fix the attribute of subresource (#21685) * fix the attribute of writableSubResource (#21688) * Add ArmClient tests (#21620) * Change the accessbility to virtual for Resource.Id * Add tests for ArmClient * Address review comment * Update recordings Co-authored-by: m-nash <prognash@microsoft.com> Co-authored-by: m-nash <prognash@gmail.com> * Add GenericResourceContainer tests (#21759) * Change the accessbility to virtual for Resource.Id * Add CreateOrUpdate tests * Add StartCreateOrUpdate tests * Add Get tests * Add GetApiVersion and ListByResourceGroup tests * Update test records Co-authored-by: m-nash <prognash@microsoft.com> Co-authored-by: m-nash <prognash@gmail.com> * Bug System.ArgumentOutOfRangeException thrown in ResourceIdentifier - 5879 - nibhati (#21537) * add list resources apis to resourceGroupOperations (#21809) * [Azure.ResourceManager.Core] Introduce the `UseClientContext` to `SubscriptionOperations` and `ResourceGroupOperations` (#21811) * introduce the useClientContext to subscriptionOperations and ResourceGroupOperations * resolve comments * resolve comments * fix compile errors in other solutions * Add ResourceGroupContainer tests (#21832) * Change the accessbility to virtual for Resource.Id * Add ResourceGroupContainer tests Co-authored-by: m-nash <prognash@microsoft.com> Co-authored-by: m-nash <prognash@gmail.com> * remove ph classes (#21852) * remove ph classes * add temp files to proto * change Parent in ContainerBase to OperationsBase type (#21872) * Add GenericResourceOperations tests (#21828) * Change the accessbility to virtual for Resource.Id * Add GenericResourceOperations tests * Merge GenericResourceOperationsTests and GenericResourceTests Co-authored-by: m-nash <prognash@microsoft.com> Co-authored-by: m-nash <prognash@gmail.com> * Add ResourceGroupOperations tests (#21893) * Change the accessbility to virtual for Resource.Id * Add ResourceGroupOperations tests * Remove parallelizable tag * Update session records Co-authored-by: m-nash <prognash@microsoft.com> Co-authored-by: m-nash <prognash@gmail.com> * update to be inside core namespace (#21892) * update to be inside core namespace * update proto references to removed namespace * remove references from test proj * remove unused interfaces (#21898) * update to be inside core namespace * update proto references to removed namespace * remove unused interfaces * Add ResourceGroupBuilder tests (#21897) * Change the accessbility to virtual for Resource.Id * Add ResourceGroupBuilder tests * Address comment * Extract the test for Build() to a seperate method Co-authored-by: m-nash <prognash@microsoft.com> Co-authored-by: m-nash <prognash@gmail.com> * Update Track 2 Mgmt Plane Template (#21876) * Update Track 2 Mgmt Plane Template * Moving the package ref into the actual project file * Fixing Management tests and updating sln to include missed project * Update build for core.test and invert the mgmt shared test code include. * Fix CI * Add ResourceGroupDeleteOperation tests (#21927) * 5966 sync code coverage fix (#21925) * consolidate test classes * update interceptors to get code coverage on sync methods for LRO responses * Remove ArmResponse since that is no longer needed * update test cases that moved with recordings * remove TA dependency on core sln * put back ai dependency which we should remove * update to cover .net 5 case * consolidate code * remove extension sub classes since they aren't needed (#21945) rename some parameters * Mnash apiview cleanup (#21952) * remove extension sub classes since they aren't needed rename some parameters * update instrumentation for generic resource and tenant * remove Get() from ResourceOperationsBase (#21963) Co-authored-by: Yeming Liu <yeliu@microsoft.com> * update to use zero wait on playback (#22066) * update to use zero wait on playback * update resourcemanager ci to run when shared files change * one more mgmt shared folder * Update sub resource CC to 94%. (#22117) * Add ResourceGroupsRestOperations tests (#21981) * Added support for implicit provider resource type ResourceIdentifier p… (#22142) * Added support for impicit provider resource type ResourceIdentifier parsing * Update clearer list name based on CR comment * Azure.ResourceManager.Core readmes and samples (#21858) * Change the accessbility to virtual for Resource.Id * Readme and sample files * Fixing links * Fixing links * Fixing links * Delete OLD README.md * Migrating to new SDK * samples rename * Update ManagingResourceGroups.md Co-authored-by: Scott Addie <10702007+scottaddie@users.noreply.github.com> * Addressing comments in Managing Resource Groups example * Hello World added * Update Sample1_HelloWorld.md * Delete README.md * Create README.md * Addressing HelloWorld comments * Apply suggestions from code review Co-authored-by: Scott Addie <10702007+scottaddie@users.noreply.github.com> * Addressing comments in readme * Commenting relative links * Fix CI Link errors * Fixing CI errors * Snippets IDs deleted * Changing Prerequisites and Auth info * Don't use .Construct in rg example * Migration guides in future PR * Fix CI * Prerequisites * Remove Links comments temporarily * Typo in Readme * Apply suggestions from code review Co-authored-by: m-nash <64171366+m-nash@users.noreply.github.com> * Changing all links to absolute links * Spelling * Commenting not existing links * Fix CI * Typo * Update Sample1_HelloWorld.md * Apply suggestions from code review Co-authored-by: Scott Addie <10702007+scottaddie@users.noreply.github.com> * Apply suggestions from code review Co-authored-by: Scott Addie <10702007+scottaddie@users.noreply.github.com> Co-authored-by: YalinLi0312 <yall@microsoft.com> Co-authored-by: m-nash <prognash@microsoft.com> Co-authored-by: m-nash <prognash@gmail.com> Co-authored-by: Scott Addie <10702007+scottaddie@users.noreply.github.com> Co-authored-by: m-nash <64171366+m-nash@users.noreply.github.com> * Move Providers and ProviderOperations from Azure.ResourceManager.Resources into Azure.ResourceManager.Core-nibhati-5862 (#21337) * Change the accessbility to virtual for Resource.Id * WIP * WIP * WIP * wip * Update TenantOperations.cs * wip * WIP * wip * wip * wip * wip * fix formatting * wip * Update ProviderContainer.cs * Update ProviderContainer.cs * wip * wip * Update ProviderContainerTests.cs * wip * wip * update to interceptor to handle targetinvocation on a failed task * remove expand parameter from LoadApiVersion methods Co-authored-by: YalinLi0312 <yall@microsoft.com> Co-authored-by: m-nash <prognash@gmail.com> * remove need to copy policies (#22280) * Add ResourceListOperations tests (#22279) * Change the accessbility to virtual for Resource.Id * Add ResourceListOperations tests Co-authored-by: m-nash <prognash@microsoft.com> Co-authored-by: m-nash <prognash@gmail.com> * remove proto (#22282) * Follow up PR - 5593 Azure.ResourceManager.Core Readme (#22262) * Change the accessbility to virtual for Resource.Id * Changing temporary links to absolute ones. Co-authored-by: YalinLi0312 <yall@microsoft.com> Co-authored-by: m-nash <prognash@microsoft.com> Co-authored-by: m-nash <prognash@gmail.com> * Add PropertyReferenceType attribute in ResourceIdentity (#22255) * Add PropertyReferenceTypeAttribute attribute in ResourceIdentity * Feedback * remove resources dependency (#22293) * wip * remove remaining references to azure.resourcemanager.resources * update tests to handle new changes * update summary for gets or sets * Updating GenericResource.Lists to return GenericResourceExpanded. (#22284) * Updating GenericResourceList to return GenericResourceExpanded * Added missing expand and related test cases. * Adding missing test recordings. * cleanup the warnings that were supressed (#22318) * fix CA1036 * remove CA1067 * remove CA1065 * remove SA1028 * Exposing expand on ListAtContext as well in preparation to expose that in each of the generated ListAsGeneric methods in autorest (#22351) * Updating GenericResourceList to return GenericResourceExpanded * Added missing expand and related test cases. * Adding missing test recordings. * Exposing expand on ListAtContext as well in preparation to expose that in each of the generated ListAsGeneric methods in autorest * Catch TargetInvocationException in interceptor (#22207) * Change the accessbility to virtual for Resource.Id * Catch TargetInvocationException in interceptor Co-authored-by: m-nash <prognash@microsoft.com> Co-authored-by: m-nash <prognash@gmail.com> * Add JsonConverter for ResourceIdentity (#22373) * Add JsonConverter for ResourceIdentity * Feedback * Format file * Update to use common rest api for OperationsList (#22343) * wip changes to add common rest api * update variable name * address pr comments * update recording * Add SubscriptionContainer tests (#22348) * Add SubscriptionsRestOperations tests (#22482) * ADO-5881 Move Azure.ResourceManager.Resources Tags objects to Azure.ResourceManager.Core (#21439) * Initial * Intial part 2 * Update tests * Update Tags * Update operation and container * Update PreDefinedTags * Update ResourceIdentifier.cs * Update at scope Tag * Create TestUpdateAllTags.json * Update ResourceGroupOperations.cs * Update-1 * Update-2 * Update resource type * Update testing coverage * Update to use single * Update tags * Update-2 * Update-3 * Update-4 * Update for rename and move method * Update to use replacewith * Add test coverage * Update for remove LRO methods * Update TaggableResourceTests.cs * minor PR comment fixes * fix tag resource container ctor * update generic resource to use new implementation remove unused test recordings Co-authored-by: m-nash <prognash@gmail.com> * changes to arm core (#22524) * [Bug Fix] Methods in ResourceGroupContainer.cs in Azure.ResourceManager.Core passes in null params (#22551) * [BugFix] Methods in ResourceGroupContainer.cs in Azure.ResourceManager.Core passes in null params * Add ResponseExtensions tests (#22553) * Move feature operations to azure.resourcemanager.core (#22570) * move features operations to core * update xml docs * Remove GetResource() from ResourceOperationsBase (#22571) * Change the accessbility to virtual for Resource.Id * Add SubscriptionContainer tests (#22348) * Remove GetResource() from ResourceOperationsBase * Remove GetResource() in ResourceGroup * Remove GetResource() in Feature Co-authored-by: m-nash <prognash@microsoft.com> Co-authored-by: m-nash <prognash@gmail.com> * Updating code snippets in Samples and Readme (#22518) * Change the accessbility to virtual for Resource.Id * Initial setup * Adding Snippets to tests\Samples * Generate snippets in markdown files * Sample 3 code snippets * Update Sample2_ManagingResourceGroups.cs * Update Snippets with generator * Fix Sample2 snippets * Delete Azure - Backup.ResourceManager.Core.Tests.csproj * Ignoring dummy subscription id tests * Adding namespaces to examples * Modify code snippets in main readme * Add ignore tags * Adding Azure.ResourceManager.Core namespace * Update code snippets * Adjust indentation * Removing auto-gen snippets for Sample3 Co-authored-by: YalinLi0312 <yall@microsoft.com> Co-authored-by: m-nash <prognash@microsoft.com> Co-authored-by: m-nash <prognash@gmail.com> * nibhati-5273-merge location (#22483) * Change the accessbility to virtual for Resource.Id * wip * WIP * Update ResourceOperationsBase.cs * wip * wip * wip * wip * wip * update loc based on feedback * update after merge * update to core framework tests * update samples after location changes * update readme based on code changes Co-authored-by: YalinLi0312 <yall@microsoft.com> Co-authored-by: Nisha Bhatia <nibhati@microsoft.com> Co-authored-by: m-nash <prognash@gmail.com> * add management object group to core (#22592) * wip * Add mgmt group to core * remove temp files * update to not have nullable but share the same operations between ManagementGroup and ManagementGroupInfo * address PR comments * change location to string to return name instead of display name (#22606) * update location name and corresponding tests * update comments * Make the Parent of ResourceContainerBase to be the OperationsBase type (#22589) * Add WritableSubResource tests (#22618) * Change the accessbility to virtual for Resource.Id * Add SubscriptionContainer tests (#22348) * Change SubResource constructors to protected * Fix a bug in ReferenceType tests * Add WritableSubResource tests * Delete not in use class UpdateResourceGroupOperation * Update unit test * Update reference type tests Co-authored-by: m-nash <prognash@microsoft.com> Co-authored-by: m-nash <prognash@gmail.com> * organize classes - move custom code outside of generated folder - remove duplicate identity (#22626) * organize classes move custom code outside of generated folder remove duplicate identity * remove unused method * Add ListExtensions tests (#22628) * Change the accessbility to virtual for Resource.Id * Add SubscriptionContainer tests (#22348) * Add ListExtensions tests * Add a positive test Co-authored-by: m-nash <prognash@microsoft.com> Co-authored-by: m-nash <prognash@gmail.com> * Add missing sections from the QuickStart to the main Readme (#22634) * Change the accessbility to virtual for Resource.Id * Adding missing sections from workshop quickstart * Adding missing code snippets * Updating code snippets in main readme * Small fixes in comments in code snippets Co-authored-by: YalinLi0312 <yall@microsoft.com> Co-authored-by: m-nash <prognash@microsoft.com> Co-authored-by: m-nash <prognash@gmail.com> * remove default location from armclientoptions (#22633) * organize classes move custom code outside of generated folder remove duplicate identity * remove unused method * remove defaut location from armclientoptions * address pr comments * update arm core version * remove unneeded abstract property * make Types public (#22612) Co-authored-by: bquantump <53361486+bquantump@users.noreply.github.com> Co-authored-by: nisha-bhatia <67986960+nisha-bhatia@users.noreply.github.com> Co-authored-by: Allen Zhang <allenzhang@live.com> Co-authored-by: Yalin Li <yall@microsoft.com> Co-authored-by: Minghao Chen <30464227+HarveyLink@users.noreply.github.com> Co-authored-by: Mark Cowlishaw <markcowl@microsoft.com> Co-authored-by: m-nash <prognash@microsoft.com> Co-authored-by: Jonathan Cárdenas <JonathanCrd@users.noreply.github.com> Co-authored-by: Yeming Liu <Yeming.Liu@microsoft.com> Co-authored-by: Yeming Liu <yeliu@microsoft.com> Co-authored-by: George Arama <50641385+gearama@users.noreply.github.com> Co-authored-by: Arcturus <dapzhang@microsoft.com> Co-authored-by: Feng Zhou <55177366+fengzhou-msft@users.noreply.github.com> Co-authored-by: Scott Addie <10702007+scottaddie@users.noreply.github.com> Co-authored-by: ShivangiReja <45216704+ShivangiReja@users.noreply.github.com> Co-authored-by: Nisha Bhatia <nibhati@microsoft.com>
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.
Make
Types
inResourceIdentifier
public as we need it to parse the parameters in ids like/subscriptions/{subscriptionId}/resourcegroups/{resourceGroupName}/providers/{resourceProviderNamespace}/{parentResourcePath}/{resourceType}/{resourceName}/providers/Microsoft.Authorization/policyAssignments
All SDK Contribution checklist:
This checklist is used to make sure that common guidelines for a pull request are followed.
Draft
mode if it is:General Guidelines and Best Practices
Testing Guidelines
SDK Generation Guidelines
*.csproj
andAssemblyInfo.cs
files have been updated with the new version of the SDK. Please double check nuget.org current release version.Additional management plane SDK specific contribution checklist:
Note: Only applies to
Microsoft.Azure.Management.[RP]
orAzure.ResourceManager.[RP]
Management plane SDK Troubleshooting
If this is very first SDK for a services and you are adding new service folders directly under /SDK, please add
new service
label and/or contact assigned reviewer.If the check fails at the
Verify Code Generation
step, please ensure:generate.ps1/cmd
to generate this PR instead of callingautorest
directly.Please pay attention to the @microsoft.csharp version output after running
generate.ps1
. If it is lower than current released version (2.3.82), please run it again as it should pull down the latest version.Note: We have recently updated the PSH module called by
generate.ps1
to emit additional data. This would help reduce/eliminate the Code Verification check error. Please run following command:Old outstanding PR cleanup
Please note:
If PRs (including draft) has been out for more than 60 days and there are no responses from our query or followups, they will be closed to maintain a concise list for our reviewers.