Skip to content

Commit

Permalink
Remove hostnameprefix from properties and make checknameavailability … (
Browse files Browse the repository at this point in the history
#1054)

* Remove hostnameprefix from properties and make checknameavailability regional

* Remove 201 response declaration from the swagger spec
  • Loading branch information
jlautman authored and sanjaiganesh committed Jan 15, 2020
1 parent 98ec3bd commit a85c88f
Show file tree
Hide file tree
Showing 7 changed files with 20 additions and 51 deletions.
Original file line number Diff line number Diff line change
@@ -1,6 +1,7 @@
{
"parameters": {
"subscriptionId": "12345",
"location": "westus2",
"api-version": "2019-10-10-preview",
"nameAvailabilityParameters": {
"type": "Microsoft.SpoolService/spools",
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -7,8 +7,7 @@
"parameters": {
"location": "West US",
"properties": {
"hostNamePrefix": "MySpoolService"
}
}
}
},
"responses": {
Expand All @@ -22,27 +21,8 @@
"type": "Microsoft.SpoolService/spools",
"location": "West US",
"properties": {
"provisioningState": "Updating",
"hostName": "MySpoolService.service.spool.net",
"hostNamePrefix": "MySpoolService",
"version": "0.2.0",
"optionalFeatures": []
}
}
},
"201": {
"headers": {
"Azure-AsyncOperation": "http://management.azure.com/subscriptions/12345/providers/Microsoft.SpoolService/operations/67890"
},
"body": {
"id": "/subscriptions/12345/resourceGroups/MyResourceGroup/providers/Microsoft.SpoolService/spools/MySpoolResource",
"name": "MySpoolResource",
"type": "Microsoft.SpoolService/spools",
"location": "West US",
"properties": {
"provisioningState": "Creating",
"hostName": "MySpoolService.service.spool.net",
"hostNamePrefix": "MySpoolService",
"provisioningState": "Succeeded",
"hostName": "myspoolservice.service.spool.net",
"version": "0.2.0",
"optionalFeatures": []
}
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -14,8 +14,7 @@
"location": "West US",
"properties": {
"provisioningState": "Running",
"hostNamePrefix": "MySpoolService",
"hostName": "MySpoolService.service.spool.net",
"hostName": "myspoolservice.service.spool.net",
"version": "0.2.0",
"optionalFeatures": []
}
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -11,8 +11,7 @@
{
"properties": {
"provisioningState": "Running",
"hostName": "MySpoolService.service.spool.net",
"hostNamePrefix": "MySpoolService",
"hostName": "myspoolservice.service.spool.net",
"version": "0.2.0",
"optionalFeatures": []
}
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -10,8 +10,7 @@
{
"properties": {
"provisioningState": "Running",
"hostName": "MySpoolService.service.spool.net",
"hostNamePrefix": "MySpoolService",
"hostName": "myspoolservice.service.spool.net",
"version": "0.2.0",
"optionalFeatures": []
}
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -7,8 +7,7 @@
"parameters": {
"location": "West US",
"properties": {
"hostNamePrefix": "MySpoolService"
}
}
}
},
"responses": {
Expand All @@ -20,8 +19,7 @@
"location": "West US",
"properties": {
"provisioningState": "Running",
"hostName": "MySpoolService.service.spool.net",
"hostNamePrefix": "MySpoolService",
"hostName": "myspoolservice.service.spool.net",
"version": "0.2.0",
"optionalFeatures": []
}
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -49,7 +49,7 @@
}
}
},
"/subscriptions/{subscriptionId}/providers/Microsoft.SpoolService/checkNameAvailability": {
"/subscriptions/{subscriptionId}/providers/Microsoft.SpoolService/locations/{location}/checkNameAvailability": {
"post": {
"description": "Checks that the Spool name is valid and is not already in use.",
"operationId": "Spool_CheckNameAvailability",
Expand All @@ -69,6 +69,13 @@
},
{
"$ref": "#/parameters/SubscriptionIdParameter"
},
{
"name": "location",
"in": "path",
"description": "Region in which the name must be available.",
"required": true,
"type": "string"
}
],
"consumes": [ "application/json" ],
Expand Down Expand Up @@ -301,12 +308,6 @@
"$ref": "#/definitions/SpoolResource"
}
},
"201": {
"description": "Created. The response describes the new service and contains an Azure-AsyncOperation header to query the operation result.",
"schema": {
"$ref": "#/definitions/SpoolResource"
}
},
"default": {
"description": "Error response describing why the operation failed.",
"schema": {
Expand Down Expand Up @@ -778,12 +779,8 @@
"modelAsString": true
}
},
"hostNamePrefix": {
"description": "Prefix for the hostName of the Spool service. \r\nThe hostname will be of format: <hostNamePrefix>.spool.azure.net.",
"type": "string"
},
"hostName": {
"description": "FQDN of the Spool service instance. Format: xxx.service.Spool.net",
"description": "FQDN of the Spool service instance. Format: xxx.westus2.service.Spool.net",
"type": "string",
"readOnly": true
},
Expand All @@ -807,10 +804,7 @@
}
}
}
},
"required": [
"hostNamePrefix"
]
}
},
"SpoolKeys": {
"description": "A class representing the access keys of Spool service.",
Expand Down Expand Up @@ -888,5 +882,4 @@
"x-ms-parameter-location": "method"
}
}

}
}

0 comments on commit a85c88f

Please sign in to comment.