Skip to content
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

[ReleasePR @azure/arm-storagecache] [Hub Generated] Review request for Microsoft.StorageCache to add version stable/2021-03-01 #12675

Conversation

openapi-sdkautomation[bot]
Copy link
Contributor

@openapi-sdkautomation openapi-sdkautomation bot commented Nov 24, 2020

Create to sync Azure/azure-rest-api-specs#12875
ReCreate this PR

Installation Instructions

You can install the package @azure/arm-storagecache of this PR using the following command:

curl -L "https://portal.azure-devex-tools.com/api/sdk-dl-pub?p=Azure/12875/azure-sdk-for-js/@azure_arm-storagecache/azure-arm-storagecache-3.0.0.tgz" -o azure-arm-storagecache-3.0.0.tgz
npm install azure-arm-storagecache-3.0.0.tgz

Direct Download

The generated package can be directly downloaded from here:

Fix s360 kpis for 2020-03-01. (Azure#11593)

* Fix s360 kpis for 2020-03-01.

* Delete Caches_Delete.json.bak

Remove backup file that was accidentally pushed.

* Delete Caches_Flush.json.bak

Remove backup file that was accidentally pushed.

* Delete Caches_Start.json.bak

Remove backup file that was accidentally pushed.

* Delete Caches_Stop.json.bak

Remove backup file that was accidentally pushed.

* Delete Caches_UpgradeFirmware.json.bak

Remove backup file that was accidentally pushed.

* Delete StorageTargets_Delete.json.bak

Remove backup file that was accidentally pushed.
@ghost ghost added the customer-reported Issues that are reported by GitHub users external to the Azure organization. label Nov 24, 2020
@ghost
Copy link

ghost commented Nov 24, 2020

Thank you for your contribution openapi-sdkautomation[bot]! We will review the pull request and get back to you soon.

@openapi-sdkautomation openapi-sdkautomation bot changed the title [ReleasePR @azure/arm-storagecache] Fix s360 kpis for 2020-03-01. [ReleasePR @azure/arm-storagecache] [Hub Generated] Review request for Microsoft.StorageCache to add version stable/2020-10-01 Dec 25, 2020
@openapi-sdkautomation openapi-sdkautomation bot changed the title [ReleasePR @azure/arm-storagecache] [Hub Generated] Review request for Microsoft.StorageCache to add version stable/2020-10-01 [ReleasePR @azure/arm-storagecache] s360 in 2020-03-01 remove empty schema in actions to match the RP. Feb 19, 2021
@openapi-sdkautomation openapi-sdkautomation bot changed the title [ReleasePR @azure/arm-storagecache] s360 in 2020-03-01 remove empty schema in actions to match the RP. [ReleasePR @azure/arm-storagecache] [Hub Generated] Review request for Microsoft.StorageCache to add version stable/2021-03-01 Mar 5, 2021
openapi-sdkautomation bot pushed a commit to AzureSDKAutomation/azure-sdk-for-js that referenced this pull request May 11, 2021
PR for new api-version: Microsoft.Cache/redis/stable/2020-12-01 [rebased + refactored] (Azure#14023)

* Adds base for updating Microsoft.Cache from version stable/2020-06-01 to version 2020-12-01

* Updates readme

* Updates API version in new specs and examples

* Adding Redis Version to PUT request (Azure#12675)

* Adding Redis Version to PUT request

* Include RedisVersion in PUT and PATCH, and update description to clarify major version

* Adding new properties isPrimary and replicasPerPrimary (Azure#13005)

* Adding new properties isPrimary and replicasPerPrimary

* Addressing PR feedback

Co-authored-by: Lavanya Singampalli <lasingam@microsoft.com>

* Refactor the latest redis.json API spec proposal to use the definitions ProxyResource, TrackedResource and OperationListResult from common-types/resource-management/v2. (I hope CI likes this!)

* Cleanup  now-unreferenced #definitions which were still flagged as duplicate definitions.

* Fix linter errors for camelCasing of API names and formatting of integer types.

* Fix more linter errors by adding pagable, refactoring LinkedServers and FirewallRules. These will be between-versions breaking changes for the SDK, in the name of more API consistency.

* Fix linter error 'PUT' operation 'PrivateEndpointConnections_Put' should use method name 'Create'. Note: If you have already shipped an SDK on top of this spec, fixing this warning may introduce a breaking change.

* Fix some duplicate example names.

* Revert "Fix linter error 'PUT' operation 'PrivateEndpointConnections_Put' should use method name 'Create'. Note: If you have already shipped an SDK on top of this spec, fixing this warning may introduce a breaking change." -- because it introduces new linter errors.

* Revert "Fix more linter errors by adding pagable, refactoring LinkedServers and FirewallRules. These will be between-versions breaking changes for the SDK, in the name of more API consistency."

This reverts commit c0e305128834947066bb59de2c4247b29feef5ee.

* Fix the example name for CheckNameAvailability.

* Add x-ms-pageable suggested by the linter.

* Revert the linter fix for CheckNameAvailabiltiy camelCasing to avoid introducing new linter errors.

* Use same FirewallRule response schema for PUT and GET, addressing linter errors, and have RedisFirewallRuleCreateParameters inherit it in hopes that helps with SDK backcompat.

* Fix indenting.

* Fix broken $ref reference syntax.

* Add back the description for RedisFirewallRuleCreateParameters.

* Actually run prettier.

* With print width 20

* Suppress the warnings that booleans are not descriptive, since to change them would be a breaking change.

* Delete unintended extra character

* Revert enum type-hinting and refactors to common-types that are getting flagged as breaking changes.

* Add back the refactorings to use Resource/ProxyResource/TrackedResource from common-types. Since otherwise we get "Duplicate Schema named Resource -- properties.id.description: undefined => "Fully qualified resource ID for the resource. Ex - /subscriptions/subscriptionId/resourceGroups/resourceGroupName/providers/resourceProviderNamespace/resourceType/resourceName",

properties.name.description: undefined => "The name of the resource","

Co-authored-by: MECHANDR <30642185+MECHANDR@users.noreply.github.com>
Co-authored-by: lasingam <68033674+lasingam@users.noreply.github.com>
Co-authored-by: Lavanya Singampalli <lasingam@microsoft.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
customer-reported Issues that are reported by GitHub users external to the Azure organization.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

0 participants