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

[AutoPR containerservices/resource-manager] [AKS] add empty resource group validation #4119

Conversation

AutorestCI
Copy link
Contributor

Add minLength: 1 to resourceGroupName paramters

This avoids an inscrutable error in the CLI if the user specifies an
empty resource group name.
@AutorestCI
Copy link
Contributor Author

AutorestCI commented Jan 3, 2019

(message created by the CI based on PR content)

Installation instruction

Package azure-mgmt-containerservice

You can install the package azure-mgmt-containerservice of this PR using the following command:
pip install "git+https://github.com/Azure/azure-sdk-for-python@restapi_auto_4979#egg=azure-mgmt-containerservice&subdirectory=azure-mgmt-containerservice"

You can build a wheel to distribute for test using the following command:
pip wheel --no-deps "git+https://github.com/Azure/azure-sdk-for-python@restapi_auto_4979#egg=azure-mgmt-containerservice&subdirectory=azure-mgmt-containerservice"

If you have a local clone of this repository, you can also do:

  • git checkout restapi_auto_4979
  • pip install -e ./azure-mgmt-containerservice

Or build a wheel file to distribute for testing:

  • git checkout restapi_auto_4979
  • pip wheel --no-deps ./azure-mgmt-containerservice

Direct download

Your files can be directly downloaded here:

@AutorestCI AutorestCI merged commit 8ba3cef into restapi_auto_containerservices/resource-manager Jan 4, 2019
@AutorestCI AutorestCI deleted the restapi_auto_4979 branch January 4, 2019 23:55
@AutorestCI
Copy link
Contributor Author

This PR has been merged into #4064

lmazuel pushed a commit that referenced this pull request Jan 10, 2019
* Generated from 3d26bb89f70260fa213ca41ce21241e61f51f225 (#4062)

typo: containerservice/Microsoft.ContainerService

- chamge -> change
- clusteradmin -> cluster admin
- clusteruser -> cluster user
- agentpool -> an agent pool
- vnet's -> VNet's
- FDQN - > FQDN
- openshift -> OpenShift
- verison -> version

* [AutoPR containerservices/resource-manager] [AKS]: SP/AAD update operation for 2018-03-31 and 2018-08-01-preview (#4088)

* Generated from 705a889279c218f987a492183359414ebd6c205a

correct example fileName

* Generated from 739ebf423cf9fa4ff6c90e9056a080a5d19cfbac

add default response and 202, correct typo

* Generated from 4ea32ed77296ab86075f580f8211cf86a10a9a38 (#4119)

Add minLength: 1 to resourceGroupName paramters

This avoids an inscrutable error in the CLI if the user specifies an
empty resource group name.

* [AutoPR containerservices/resource-manager] [AKS]: Remove 200/OK response from reset SP/AAD api (#4099)

* Generated from abbffc45c89ec7df7a085668938d4d1c99de4d3f

rebase

* Generated from 68b822a898956ff4152d015407f6ee8ee9299360

long running operation

* Generated from db342271af8fbeef8464a0fa7e2fc5a4cd0f1662

return 202 with no body

* Generated from b39369cea532289fe4c841888bd6b04594921af8

forget to remove 202 body from 2018-08-01-preview api

* Generated from 9436137c879062700e3ade0d499603fc1437bfed

add 200 without schema

* Generated from df0936f255eeb8df56a204d5d89b012795f516bb

resolve conflict

* [AutoPR containerservices/resource-manager] [AKS] add missing description fields (#4120)

* Generated from a2ab98d9d7cafde90cb02ddc14ff6148560c0eb4

[AKS] add missing description fields

* Generated from 04a08f6fc7d53336a7ed2b39c3cb69f4a57d2966

[AKS] fix up reset endpoints in preview API

* Generated from 84cef65388454e9a36c4937b5afed77f12347ae8

[AKS] add missing description fields

* 4.4.0
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant