Skip to content

Conversation

@akshataP
Copy link
Owner

@akshataP akshataP commented Jun 1, 2019

This checklist is used to make sure that common issues in a pull request are addressed. This will expedite the process of getting your pull request merged and avoid extra work on your part to fix issues discovered during the review process.

PR information

  • The title of the PR is clear and informative.
  • There are a small number of commits, each of which have an informative message. This means that previously merged commits do not appear in the history of the PR. For information on cleaning up the commits in your pull request, see this page.
  • Except for special cases involving multiple contributors, the PR is started from a fork of the main repository, not a branch.
  • If applicable, the PR references the bug/issue that it fixes.
  • Swagger files are correctly named (e.g. the api-version in the path should match the api-version in the spec).

Quality of Swagger

huizlAzure and others added 30 commits April 29, 2019 15:42
* [SRP] Expose lease container in Apr19 API.

* Fix lease container example files; fix casing in Lease Container URL
assigning as one of the owners of profile directory
…reserve the existing scenario (#5792)

* Adding Workbooks resource type.

* Rename workbookDelete.json to WorkbookDelete.json

* Fixed undefiend parameter issue with Delete verb.

* Update swagger to add subscriptionId

* Fixed duplicated operation id issue

* Removed links endpoint

* Removed unused samples

* Revert "Removed unused samples"

This reverts commit bed6bc3.

* Revert "Removed links endpoint"

This reverts commit c245a85.

* Removed link logic and added get by resourceid

* Applied PR comments

* Added Resource Group parameter for GET. Removed get by sourceId since we are not using sourceId anymore

* Resolved the merge conflicts with readme.md

* Remove a location parameter since ARM routes it as we are using an tracted resource type

* Created newer version (2018-04-01-preview) for workbook resource type.

* Added Microsoft/Insights/Operations

* Added Microsoft/Insights/Operations

* Corrected the api version.

* Made properties of PATCH operation as not required. Updated the version number to reflect the actual production one.

* Updated Resource properties with the correct fields. Created new definition for PATCH operation since location is not required field

* Update readme file

* Uses plural for operation group name

* Removed x-ms-mutability settings

* Removed the location settings for PATCH

* Added Microsoft.ResourceGraph/SavedQuery resource type

* Fixed various validation errors

* Updated ARG/resources to add a saved query support along with examples

* Updated ARM/resources based on the feedback

* Prepared for Swagger submittion

* Added saved query entry in readme.md

* Changed HTTP status code of 201 to 200 for DELETE

* Removed a name from payload since it is already specified in url

* Removed location since it is a global resource and added eTag

* Moved savedquery.json along with all examples into 2018-09-01-preview folder

* Update saved query metadata in OperationsList.json

* Updated readme.md file to address a version

* Fixed a version for saved query

* Fixed indentation of -input-file in readme

* Renamed Saved => Graph

* Applied PR comments

* Restored extended enum value in ResultKind since we are using it

* Removed the extended enum value again

* Apply ARM API team comments

* Fix examples

* Fix examples

* Added nextLinkName

* add Go SDK build

* make descriptions match

* Added nextLink definition in OperationListResult

* Removed a duplicated OperationListResult and reference to resourcegraph.json

* Added nextLinkName: nextLink for x-ms-pageable in operations

* fix formatting and description

* Removed extended from enum

* Remove nextLink from the example

* Returning null for nextLink in the example so that we can preserve the existing scenario.

* Removed nextLink entirely per PR comment
- analaysis -> analysis
- perdicted -> predicted
* Add Swagger and Examples for custom resource provider to Azure

* Apply suggestions from code review

Fix typos in strings for RP.

Co-Authored-By: jjbfour <[email protected]>
* Add dynamic metric alert api

* add new line at end of flie

* Fix file name

* Rename CreateOrUpdateDynamicMetricAlertMultipleResource.json to createOrUpdateDynamicMetricAlertMultipleResource.json

* Remove failing dependency

* Add back example dependency

* Rename microsoft.insight

* Update dynamic threshold criteria descriptions

* Fix PR comments: Revert name change + Use fake subscriptionId
…ane RPs (#5795)

* hooking up swagger-tools validation

* oav fixes

* update folder to generate arm subscription to subscriptionManagement2 for node.js

* Add profile-2019-03-01-hybrid tags in readme.md of the management plane RPs.

* Add path for resource type "connections" in the 2019-03-01-hybrid profile

* update tag name to profile-hybrid-2019-03-01

* Update DNS and authorization resource types
* Adds base for updating Microsoft.ContainerService from version stable/2019-02-01 to version 2019-04-01

* Updates readme

* Updates API version in new specs and examples

* Adding windowsProfile and maxAgentPools property in the new 2019-04-01 api version (#5465)

* add maxAgentPools and windowsprofile properties

* fixing the sdk generation error

* adding load balancer sku property (#5556)

* Add support VMSS agent nodes with public IP. (#5748)

* Add identity property in managedCluster definition and add an example (#5709)

* Add identity property in managedCluster definition and add an example

* Fix typo

* node resource group can be read-write (#5762)

* Update managed cluster upgrades (#5789)

* Update managed cluster upgrades

* Fix syntax type error

* including 2019-04-01 api version in 2019-04 package

* remove the unreferenced MSIManagedClusterCreate_Update.json file

* fixing typo per comments
* Add list_by_scope

* Add filter
* Making Ink Reconizer Service Swagger file public

* fixed bad tag information in md files

* YAML fix

* fixed sdk folder name

* readme updates
)

Examples had trailing commas.
Formatted file on save which replaced tabs with spaces
* Azure Gov Profile

* Azure Gov Profile
* Initial commit.

* Incorporating PR feedback

* Adding examples, cleanup.

* Fixing json syntax errors.

* Incorporating feedback.

* Refactor swagger to common type.

* Incorporating feedback.

* Moved to preview, refactor, fixes.

* Updated text.

* Minor text change.

* More minor text updates.

* Reverting accidental change

* Refactor

* Refactor, Adding GetModels REST API placeholder

* Refactor, fixes

* Refactor, Updated definitions.

* Fix type definition

* model* API endpoint updated

* Fix build issue, Reverting change to face swagger

* updated namespace

* Updated return type for /models api

* Update FormRecognizer.json

fix paths of methods (add custom) and fix path of hosttemplate

* Refactor, opid changes.

* removing multi as it causing autorest codegen issues with langs like go, ruby etc.

* reverting face.json changes.

* renamed example, fixed typo in example to make it valid json

* API review feedback, Reverting OpIds

* API review feedback, fixing examples for model verification

* Fixing examples for model validation

* Capturing feedback from API Review

* Capturing feedback from API review
…sion stable/2019-01-01 (#5824)

* Fix support for Api Schema Create Or update

* add example for schemas

* remove the example causing failures
* Adding new profile

* Fixes for resources RP
* Add checkIdentityHealth api in Microsoft.Subscription.Admin RP.

* Fix comments 1.
* Create billing.json

Adding swagger

* Update billing.json

fix error

* Feedback changes

Feedback changes

* Update subscriptions.json

Changing rename API to patch

* Update subscriptions.json

feedback changes

* Update subscriptions.json

feedback change

* Version change

Changing version to 2019-03-01-preview

* Update subscriptions.json

minor change

* Added examples

* Update subscriptions.json

Url change

* Update subscriptions.json

Operation name change

* Update readme.md

Changes in readme file

* Update readme.md

updated default package

* Update subscriptions.json

Fixed example links

* Feedback change

* Suppress

* Added operations swagger
* only use rank, reward and activate apis

* add back Reward request
* Copy 2017-09-30 spec

* Add isPreview for 2019-04-01 spec

* Fix required params

* Fix required params error
…on preview/2019-04-01-preview (#5622)

* Adds base for updating Microsoft.Consumption from version stable/2019-01-01 to version 2019-04-01-preview

* Updates readme

* Updates API version in new specs and examples

* Usage Details API Contract changes for 2019-04-01-preview

Usage Details API Contract changes
- Usage Details model changes
- Usage details sample changes for the new model
- Usage Details download operation details
- Sample for usage details download
- Removed used oprerations and samples

* Usage Details API contract changes

- Added new samples for the query parameters
- Changed the "in" from path to query since this is optional query parameter.

* Usage Details API contract changes for 2019-04-01-preview

Included the samples in the consumption list

* Usage Details API contract changes for 2019-04-01-preview

- changed the values in samples.

* Usage API contract changes

- Change in samples

* Update UsageDetailsExpand.json

- changed the sample values

* Update UsageDetailsExpand.json

Changed the sample value

* Update consumption.json

Upated the sample file name in consumption.json

* Scope parameters added for the samples

- added the scope parameters for the samples

* Scope parameter fix

Scope parameter fixed in samples

* Scope parameters

Scope parameters added to the samples

* changed the scope parameter

added scope parameter to the samples

* Correcting scope parameter

Scope parameter changes

* Update Tags.json

Tags scope changes

* Fixed download and mg aggregated cost issues

Fixed date and download issues

* UsageDetails_Download operation id change

UsageDetails_Download operation id change

* Changed the casing for usage downloads operations

Changed the casing for usage downloads operations as per review

* Usage Details API contract changes

- Removed ActualAndAmortizedCost query options
- Removed samples for - ActualAndAmortizedCost
- Removed boolean isAmortizedCost and isActualCost from the model response.
- Removed the isActualCost and isAmortizedCost from samples

* Usage Detail API contract changes

Removed the unused boolean flgs from samples.

* Usage Details API contract changes

- Added parameter for Usage Download as well.
- Added sample for usage downloads with parmaeter.

* Usage Details API contract changes

- Removed the bad sample file for Charges (as per the api version we will return only single object) - https://dev.azure.com/azure-sdk/public/_build/results?buildId=17699&view=logs&jobId=8d2f9c49-cd83-5a9d-f3fe-2fd651afa742&taskId=43324f1f-6724-5d61-785f-f0ac9fd256a1&lineStart=82&lineEnd=83&colStart=1&colEnd=1

- Removed the unused sample files as per - https://dev.azure.com/azure-sdk/public/_build/results?buildId=17699&view=logs&jobId=f10f5372-40ed-54fc-caa4-b1cb5bd1252b&taskId=d3b2b841-a93b-5fa5-47c3-d8c4f6eae6e3&lineStart=14&lineEnd=15&colStart=1&colEnd=1

* metricParameter changes

defined allowed values for the metricParameter enum

* Changed the samples for ValidTill

Changed the download samples accordingly

* Fixed the enum values for metric types

Fixed the enum values for metric types

* changed the sample to match the case

changed the sample to match the enum case

* As per review removed the OData-EntityId

Removed additional property for OData-EntityId.

* Changed the examples

Removed Odata-EntityId from samples

* OperationId change for Downloads

Based on the Doc Preview generated we want to change how the name appears. So changing the operation name for downloads.
juniwang and others added 24 commits May 29, 2019 14:19
* add new property of SignalR resource

* update description of features

* remove async-operation options
* Microsoft.HanaOnAzure: Add Delete HanaInstance API

Add example file for the new API
Note: if this PR is merged after #5945 a provisioning state of 'deleting' should be displayed in the example

* Mark as async; correct status code to Accepted; update and validate examples

* Support 204 when there is no HanaInstance to delete

* Remove response body from DeleteHanaInstance

* Fix issues with rebasing conflicts
Ran `jsonfmt -w "specification/imds/**/*.json"`
* chore: jsonfmt keyvault

Ran `jsonfmt -w "specification/keyvault/**/*.json"`

* typo: recoverylevel -> recoveryLevel

* chore: Remove double backslash in example path
…6125)

* adjust output location to match PR#5394 on azure-sdk-for-python

* updates for packages associated w/ PR#5398

* advisor, cdn, graphrbac

* frontdoor, iotcentral, dns, devtestlabs, datalake-analytics (mgmt, dp), datafactory, cosmos, containerservice, computer, commerce, cogservices, kusto

* retargeting python output location for logic, managedservices, maps, mediaservices, mixedreality, monitor, mgmt-loganalytics, operationalinsights, policyinsights, security, monitor, servicefabric, sql, storage
Ran `jsonfmt -w "specification/managedservices/**/*.json"`
Ran `jsonfmt -w "specification/machinelearningexperimentation/**/*.json"`
* copy from 2019-03-01.

* Add changes for management group deployment.

* add changes in readme.md.

* refactor.

* add example.

* more changes.

* revert the tagname and tagvalue change.

* reference exmple file.

* update example file.

* fixes

* Add descriptions.
* Initial commit.

* Incorporating PR feedback

* Adding examples, cleanup.

* Fixing json syntax errors.

* Incorporating feedback.

* Refactor swagger to common type.

* Incorporating feedback.

* Moved to preview, refactor, fixes.

* Updated text.

* Minor text change.

* More minor text updates.

* Reverting accidental change

* Refactor

* Refactor, Adding GetModels REST API placeholder

* Refactor, fixes

* Refactor, Updated definitions.

* Fix type definition

* model* API endpoint updated

* Fix build issue, Reverting change to face swagger

* updated namespace

* Updated return type for /models api

* Update FormRecognizer.json

fix paths of methods (add custom) and fix path of hosttemplate

* Refactor, opid changes.

* removing multi as it causing autorest codegen issues with langs like go, ruby etc.

* reverting face.json changes.

* renamed example, fixed typo in example to make it valid json

* API review feedback, Reverting OpIds

* API review feedback, fixing examples for model verification

* Fixing examples for model validation

* Capturing feedback from API Review

* Capturing feedback from API review

* Fix generated code path, Fix generated SDK class name

* Update title to be used for SDK code generation

* Updating help text.

* Update help text.

* Update help text.

* Initial commit.

* merging receipts with FR

* update for receipt

* Receipt merge update

* add receipt ref, update ocr json

* fix readme

* Fix Receipt swagger title field

* Fixing receipt API verification issues.

* fixing build issues

* removing empty lines.

* adding description for understandingResult

* Adding reference to Ocr swagger.

* Trimming OCR to remove unreferenced definitions.

* Adding missing definitions.

* Fixing build issues.

* Adding updated OCR reference.

* fix version

* fixing typo

* capturing feedback from API PR review.
* Added specs for Agreements API in version 2018-11-01-preview

Added specs for Agreements API in version 2018-11-01-preview

* Updated as per naming convension

Updated as per naming convension

* Corrected specs and examples for BillingRP

Corrected specs and examples for BillingRP

* Fixed the errors

Fixed the errors

* Fixing errors in spec

Fixing errors in spec

* Updated examples

Updated examples

* Incorporated review comments

Incorporated review comments

* Incorporated review comments

* Incorporated review comments

Incorporated review comments

* Updated the operation id for Billing spec

Updated the operation id for Billing spec

* Incorporated review comments

Incorporated review comments

* Incorporated review comments

Incorporated review comments

* Fixed the errors

Fixed the errors

* Added multiple inputs

Added multiple inputs

* Fixed build errors

Fixed build errors

* Incorporated review comments

Incorporated review comments

* Incorporated review comments

* Spec for LineOfCredit API and other corrections

Spec for LineOfCredit API and other corrections

* Fixed build error

* Incorporated review comments

Incorporated review comments

* Incorporated review comments

Incorporated review comments

* Updated the increase api

Updated the increase api

* Incorporated review comments

Incorporated review comments

* Updated the operation id

* Fixed errors

* Fixed errors

* Fixed error

Fixed error
* chore: Suppress Conditon in April Network WAF

Related to #6104

* chore: Cleanup passing cSpell exceptions

* chore: Update EditorConfig to match existing format
Ran `jsonfmt -w "specification/managementpartner/**/*.json"`
Ran `jsonfmt -w "specification/machinelearningcompute/**/*.json"`
Ran `jsonfmt -w "specification/healthcareapis/**/*.json"`
* chore: jsonfmt deviceprovisioningservices

Ran `jsonfmt -w "specification/deviceprovisioningservices/**/*.json"`

* fix: Remove "description" in examples for model validation
Ran `jsonfmt -w "specification/maps/**/*.json"`
* Add CreateFromJson POST operation to ScaleUnit

* Fix few minor review comments from Marta

* Add missing DeploymentJsonPhysicalNodeParameters definition

* Resolve code review commments. Fix IP address and put back clustername in the sample
…review/2018-11-01-preview (#6041)

* swagger for patching billing account.

* updated based on feedback.

* update example based on feedback.

* updated azure-async operation based on feedback.

* update based on feedback.
…review/2018-11-01-preview (#6090)

* make validateAddress top level api

* review comments

* update class
@akshataP akshataP merged commit b0957b1 into akshataP:master Jun 1, 2019
akshataP pushed a commit that referenced this pull request Jun 1, 2019
Issue #1 : Azure#4576 (comment)

Typo in the DeltaOperationType Enum.

Issue #2 : Azure#4844 (comment)

GlobalAdmins list is marked as an object, while it's an array.
akshataP pushed a commit that referenced this pull request Jun 1, 2019
…on 2019-01-01 (Azure#5139)

* Adds base for updating Microsoft.Consumption from version stable/2018-10-01 to version 2019-01-01

* Updates readme

* Updates API version in new specs and examples

* updated readme to support new stable version 2019-01-01

* version introduced for java readme.md

* Updating swaggeer to SDK section for java SDK generation

* introducing MG scope for tags/budgets
introduce scope (method) parameter for APIs that support numerous scopes (arm-team suggestion implemented)

* Fixing Java code-generation (#1)

* Fixing Java code-generation

* Removing duplicate directive from java section

* update indentation
akshataP pushed a commit that referenced this pull request Jun 1, 2019
…01 binilkk (Azure#5042)

* Fixes for:

Issue #1 : Azure#4576 (comment)

Typo in the DeltaOperationType Enum.

Issue #2 : Azure#4844 (comment)

GlobalAdmins list is marked as an object, while it's an array.

* Updated the Specification as example for Risky IP Download report.

* Added the Result details

* Fixed review comments, changed the URI scheme as well as changed get/post methods

* Addressing Review Feedback, changing POST call of blobUri -> generateBlobUri
akshataP pushed a commit that referenced this pull request Jun 1, 2019
* Adds base for updating Microsoft.Network from version stable/2018-12-01 to version 2018-12-01

* Updates readme

* Updates API version in new specs and examples

* Network: Duplicate tags in readme file. (Azure#5227)

* FI from `master` branch to `network-february-release`. (Azure#5262)

* DataFactory Swagger Change for SapOpenHub and Rest (Azure#5195)

* DataFactory Swagger Change for SapOpenHub and Rest

* correct indentation

* address comment of REST

* fix syntax issue

* Fix issue with AutoRest

* Add cognitive service AnomalyFinder (Azure#5126)

* add AnomalyFinder swagger json

* AnomalyFinder update swagger json

* add AnomalyFinder readme

* fix file path; change class name;

* make camelCase style

* fix typo; change operationId; remove hard code arg limitation;

* fix type of "sensitivity"; change description of "timestamp";

* Improving secretVersion description (Azure#5220)

* [Cogsvcs] LUIS - add text to simple entity description (Azure#5216)

* Add hwRevision field to HanaInstance definition (Azure#5196)

* [DataFactory] Swagger changes for ADF newly released features: (Azure#5218)

* Swagger changes for ADF newly released features:
1. I can load data from o365 by leveraging the REST endpoint of o365
2. Support copying incremental files based on lastModifiedTime for S3/File/Blob
3. Enable ADLS Gen2 connector as both source & sink
4. Support data migration from MongoDB to CosmosDB (MongoDB API)
5. Make Kusto source/sink better and support Azure Data exploer source/sink
6. Support copy from ADLS Gen1 to ADLS Gen2 with ACL
7. serviceUrl @ S3 linked service
8. I can copy from Dynamics AX
9. AAD OAuth for OData Linked Service
10. I can copy from Oracle Service Cloud and Google Adwords

* fix swagger definetion for MongoDB Srouce cursor method.

* fix Amazon S3 servie url definition

* fix CosmosDbMongoDbApiSink

* Fix several typos

* Add maxConcurrentConnections in CopySource and CopySink

* Node v 10. (Azure#5236)

* Remove API reference of Gender::genderless. (Azure#5188)

* Add an Operations API for Commitment Plan RP (Azure#5222)

* Add an Operations API for Commitment Plan RP

* Add OperationEntityListResult related definitions

* Fix case typo

* Fix tab to spaces

* Fix NewtonSoftJson Issue (Azure#5240)

* Support create pipeline run with recovery mode

* Update description of isLatest

* Update descriptions

* update autorest version to fix CI

* Add suppresses for Machine Learning Commitment Plan RP (Azure#5221)

* typo: datafactory/resource-manager/Microsoft.DataFactory/stable/2018-06-01/ (Azure#5238)

- Adwords -> AdWords
- connectioned to -> for the

* scripts (Azure#5213)

* scripts

* TRAVIS_PULL_REQUEST

* another try

* TRAVIS_PULL_REQUEST_SLUG

* TRAVIS_REPO_SLUG

* Swagger to SDK

* linter and ruby

* not really a slug

* pool

* indent

* strange

* URI

* env vars

* uri

* env var

* TRAVIS_PULL_REQUEST_SLUG

* matrix

* Azure OpenAPI name

* linter: autorest fix

* error handling and PR_ONLY = true

* syntax

* pipelines.

* job name

* job name

* Names

* params, no getStatus.js

* No CI, only PR validations.

* displayName

* revert back

* no need to show vars

* Fix for lintdiff tool memory issue. (Azure#5246)

* node options

* correct NODE_OPTIONS

* lintdiff travis-ci

* 2019 Azure Stack hybrid Profile. (Azure#5212)

* 2019 Azure Stack hybrid Profile.

* fixing changes for profile

* remove space

* formatted profile

* [Azure Search] Add Suggest and Index APIs to data plane Swagger spec (Azure#5219)

* Renaming Search|SuggestParametersPayload to *Request and making them internal

Made AutocompleteRequest internal too, and fixed some incorrect comments.

* Adding Suggest API to the Azure Search data plane spec

* Fixing validation error in Suggest example

* Adding Index API to the Azure Search data plane spec

* Make example names consistent

* Adding new properties to Databricks LS and custom activity (Azure#5214)

* typo: instantitate -> instantiate (Azure#5251)

* Users/laramume/addv2.1 swagger (Azure#5187)

* updated model names in preview version to be consistent with stable version to avoid breaking changes in client side code

* changed swagger files

* added v2.1 swagger

* added v2.1 swagger

* added v2.1 swagger

* added v2.1 swagger

* added v2.1 swagger

* resolved PR comments

* Update swagger file and examples with time zone parameter (Azure#5224)

* Update swagger documetation to mark Collation property as writable.

Change Managed Instance collation property from read only to read and create.
Add collation to properties section in ManagedInstanceCreateMax example.

VSTS task: 12336257

* Remove 'cosmetics' from swagger.

* Remove comma as redundant

* Return to the initial, generated state.

* Update swagger file and examples with time zone parameter

* Minor (but improtant) changes in Azure DevOps pipeline. (Azure#5253)

* empty parameters

* TRAVIS_BRANCH

* target branch

* trying to set env var

* use shell

* info

* more info

* Pull Request Number

* [Ready to MERGE] Swagger for Tenant Properties API (Azure#5038)

* added swagger for Tenant Properties

added swagger for Tenant Properties

* Fixed camel casing in billing.json file

Fixed camel casing in billing.json file

* Update billing.json

* changes in DiscoverTenants contracts

changes in DiscoverTenants contracts

* billing.Json trim blank space

billing.Json trim blank space

* Update billing.json

* removal of plural name

removal of plural name

* Updating the route

Updating the route as per suggestion

* Moved Swagger for Discover Tenants from billing to consumption

Moved Swagger for Discover Tenants from billing to consumption

* Change in Namespace followed by one additional parameter "billingAccountId"

Change in Namespace followed by one additional parameter "billingAccountId"

* Change of method name from discoverTenants to tanants.

Change of method name from discoverTenants to tanants.

* changes ion response type

changes ion response type

* added name after Id. Code review implementation

added name after Id. Code review implementation

* Add JavaScript/TypeScript generation details to resource-manager/netapp (Azure#5261)

* Azure Security Center - break-to-composite v1 (Azure#5197)

* Azure Security Center - break-to-composite v1

* Azure Security Center - composite api v1 - suppress email format

* Add globalReachEnabled, Add Get and list for peer express route circuit connections. Remove allowGlobalReach (Azure#5266)

* Added GlobalReachEnabled field. Added GET and List apis for peer express route circuit connections. Removed AllowGlobalReach as it was a dead field and abandoned in favor for GlobalReachEnabled

* adding allow gloabl reach back

* fixed typo

* Fix descriptions & remove unused/duplicate definitions (Azure#5243)

* Add support for Local Sku (Azure#5317)

* Add WAF as a top level resource (Azure#5341)

* Add WAF as a top level resource

* Fix merge

* Addressed comments

* Addressed comments

* Code review comments

* fix issue for powershell

* add health probe log

* address the comments

* increase Node.js memory

* double the memory

* print node_options

* mistake

* pass to docker

* FI from master to network-february-release (Azure#5391)

* Add mixedreality 2019-02-28 API version to Go SDK (Azure#5271)

* Remove ARM gateway Node.js configuration (Azure#5283)

* Updates to Scheduled Query Rule swagger (Azure#5175)

- making aznsaction paramter optional
  - removing hidden-link value from tags
pu

* Fix JavaScript configuration for ARM Private DNS SDK (Azure#5284)

* Fix JavaScript configuration for ARM Private DNS SDK

* Add quotes

* Add private preview API for Azure Image Builder (Azure#5055)

* Add private preview API for Azure Image Builder

* Review comments

 - Added list-by-resource-group and list-by-subscription
 - Added default error response
 - Removed explicit operation url (implied by async contract)
 - Adjusted 'delete' response codes to match expectations
 - Allow patch for tags
 - Added provider operations API

* Review comment: use "nextLink" for paging

* Adding readme files for SDK generation

* PR: rename 'tags' -> 'artifactTags' for distributor

* Add examples, fix minor issues in swagger

* Updating Blueprint spec to include resource group tags (Azure#5263)

* Add Showback model (Azure#5217)

* Add Showback model

* fix ShowbackRule.json example

* remove "," from line 1709

* Add Put Rule

* Fix Scope

* Update json with pull request changes

* change recurring to enum

* chore: Graphservice -> GraphService (Azure#5098)

* Enable SSL on custom domains using users certificate or CDN managed certificate (Azure#5276)

* Translator Text API V3 (Azure#5230)

* Translator Text API V3

* Fix path and override

* Fix a typo and few other improvements in descriptions.

* Remove after_scripts

* Update readme.md

* Update readme.md

* Update MariaDB api version (Azure#5280)

* Adding Swagger and examples for VirtualClusters operation group (Azure#5248)

* Adding Swagger file for VirtualClusters operation group, together with examples.

* Fixing api version, typos and location in examples.

* Adding virtualclusters.json to readme.md

* Security - Updating API versions of alerts and pricing types (Azure#5269)

* Security - Updating api versions of alerts and pricing types

* fix codegen paths for Go SDK

* typo: asscociation -> association (Azure#5252)

* Fix lintdiff issue (Azure#5299)

* fix Go SDK package path for mariadb stable release (Azure#5298)

* changes to improve documentation of endpoint health statuses for routing (Azure#5290)

* add netapp to Go SDK (Azure#5308)

* Update MariaDB default API version to 2018-06-01 (Azure#5302)

* new application insignts api version for unified schema in action (Azure#5292)

* Added Azure App Configuration event data schema. (Azure#5309)

* Added Azure App Configuration event data schema.

* Added a few missing pieces.

* Prefixed event names with our service prefix 'AppConfiguration'.
* Added Microsoft.AppConfiguration to the event grid data-plane readme.

* Updated api-version path to '2018-01-01' from '2018-02-01'.

* Apim Cache resource + DELETE long running + refactor to align with tooling. (Azure#5260)

* api import and service delete long running operation

* Bring your own Cache

align with spec generated from tooling

user and notification align

Refactor products and apis

* align with spec generated from tooling

user and notification align

Refactor products and apis

validating examples

validating examples

move all parameters to apimanagement.json

Fix reference issue with tagcontract and example DeleteCache.

* switch to plural example file names for LIST

* bring back updatehostname and updatecertificate

* Dev adhybridhealthservice microsoft.ad hybrid health service 2014 01 01 binilkk (Azure#5042)

* Fixes for:

Issue #1 : Azure#4576 (comment)

Typo in the DeltaOperationType Enum.

Issue #2 : Azure#4844 (comment)

GlobalAdmins list is marked as an object, while it's an array.

* Updated the Specification as example for Risky IP Download report.

* Added the Result details

* Fixed review comments, changed the URI scheme as well as changed get/post methods

* Addressing Review Feedback, changing POST call of blobUri -> generateBlobUri

* Update UserAssignedIdentities spec in Web (Azure#5288)

* Update UserAssignedIdentities spec in Web

Microsoft.Web recently added support for UserAssigned managed service identities (in preview), but the swagger spec is incorrect for the identity payload as it follows the old construct of sending an array of identityIds as opposed to the updated spec which instead sends a dictionary object called userAssignedIdentities. 

Spec + history outlined here:
http://sharepoint/sites/AzureUX/Sparta/_layouts/15/WopiFrame2.aspx?sourcedoc=%7b633632B4-02E4-4DE2-92A9-EDCAF9BE73D4%7d&file=ARM%2BMSI.docx&action=default 

Our swagger contains identityids[] because of previous work to future support UserAssigned identities, but never released the feature in that state, rather we opted to wait for the new spec to be supported and then integrated with that on our end. 

Our MSI implementation reflects Microsoft.Compute's spec for user assigned.

https://github.com/Azure/azure-rest-api-specs/blob/78e1feab142c605839aa3cec23442ca26b8ef04d/specification/compute/resource-manager/Microsoft.Compute/stable/2018-06-01/compute.json

* fix linterdiff tool

* Add private dns module name for java codegen (Azure#5312)

* Update Swagger Files For ManagedInstanceAdministrator (Azure#5095)

* Update Swagger Files For ManagedInstanceAdministrator

* Fix for double wording

* Comments resolved

* Bringing consistency between to "ResourceGroup" definition (Azure#5256)

* Bringing consistency between this "ResourceGroup" definition and the one in Common.

* Fixing broken references

* reverted security python readme to pre multi api (Azure#5301)

* [Azure Search] Add Search API to data plane Swagger spec (Azure#5315)

* [Azure Search] Updating some Swagger descriptions to be more doc-friendly

Sometimes we over-describe things, especially "comma-separated lists" in
query string parameters. These are modeled as arrays in Swagger, which are
already understood to be comma-separated lists. They are modeled as
collections in generated code, so they are decidedly not comma-separated
lists in that context. So removing this phrase is a win-win.

* [Azure Search] Adding some missing readOnly attributes

* Adding Search API to the Azure Search data plane spec

* [Azure Search] Use regexes to generate code for server-side paging

This change introduces some very hacky regexes to customize the generated
code for server-side paging of the GET and POST Search APIs. They work by
splitting the generated SearchGet and SearchPost operations in two, so that
the portion that operates on a URL and request body can be reused.

This workaround is necessary because AutoRest lacks flexibility around
paging. The current x-ms-pageable implementation doesn't work for us because
it doesn't model additional top-level response properties or continuation
POST requests (only nextLink/GET).

The only other alternative to this approach besides custom code would have
been to try to parse the nextLink URL query string to reconstitute parameters
to pass to the generated SearchGet and SearchPost operations. This is
unacceptable for Azure Search, since it would exclude custom parameters that
can't be modeled in Swagger, like personalization IDs for custom ranking.
Neither AutoRest nor apparently Open API support modeling arbitrary query
string parameters.

Ideally we'll find a pattern common to other data plane APIs so we don't have
to hack this with regexes for every target language, but it's good enough for
now.

* [Azure Search] Rename some client properties for consistency

* [Azure Search] Make AutocompleteMode non-nullable

* [Hub Generated] Review request for Microsoft.Web to add version 2018-02-01 (Azure#5137)

* Add Hosting Environment network dependencies

* Address PR description feedback

* Add Network Dependencies examples

* Fix casing

* Remove incorrect pagination from examples.

* Add missing MarketplaceOrdering operations to swagger (Azure#5279)

* Add missing MarketplaceOrdering operations to swagger

* Add examples

* Add example files

* Apply suggestions from code review

Clarify operation descriptions

Co-Authored-By: zachraMSFT <[email protected]>

* Fix Hosting Environment Network Dependencies pagination (Azure#5320)

* Add regex validation for ManagedClusterAgentPoolProfile.name (Azure#5324)

* Adding instance pools swagger (Azure#5225)

* Initializing instance pools swagger and examples - no usages / managed instances get by instance pool

* updates to use common resource and sku definitions

* Update instance pool swagger with typo fixes

* Updates to instance pool swagger and reverting readme.md to exclude instance pools reference

* Fix whitespace diff

* Making a whitespace change to re-try linter validation

* Updates to add list instances pools by subscription and update license type param to use enum which is limited to LicenseIncluded and BasePrice

* Enhanced A2ARecoveryPointDetails with "disks" (Azure#5307)

* [DiskRP] Adding swagger spec for 09-30-2018 (Azure#5232)

* commit for the older swagger version to make the PR easier to look at

* Adding the DiskRP 09-30-2018 swagger spec. This includes support for hyperVGeneration, encryptionSettingCollections, diskState, Upload Disks, and Write access

* adding the updated readme

* updating the examples to use 09-30-2018

* removing extraneous examples

* adding a description for the EncryptionSettingsElementand fixing linter issues

* [Computer Vision] Add read to CV OCR API. (Azure#5314)

* Add read to OCR API.

* Add description to OCR related JSON properties.

* Mark asyncBatchAnalyze as long running operation.

* Update page property.

* Revert "Mark asyncBatchAnalyze as long running operation."

This reverts commit 51742b3.

* added links to limits for application gateway (Azure#5265)

* typo: Winodws -> Windows (Azure#5321)

* MultiApi EventHub (Azure#5331)

* try 2 bootstrapping swagger to .net sdk tools (Azure#5334)

* bootstrapping swagger to .net sdk tools

* deleting nuget.exe binary. Updating configs to locally restore pacakges. Adding the directory that will be used in restoring packages locally to .gitignore

* changing directory

* updating feed location to a readonly directory

* Azure Search: Resource Manager: Support 'None' Identity type (Azure#5319)

* Azure Search: Resource Manager: Support 'None' Identity type

* Azure Search: Resource Manager: Adding serice identity creation\removal examples

* [ADLA] - Catalog - Add stream path to USqlTableFragment (Azure#5329)

* [Hub Generated] Review request for Microsoft.Consumption to add version 2019-01-01 (Azure#5311)

* Add GTM scopes for budgets API

* Removed the extra period at the end of the statement

* fixing stos.proj parameter (Azure#5338)

* change flag name from useUnifiedSchema into useCommonAlertSchema per UX requirement (Azure#5323)

* Enhanced A2A protection input and response details (Azure#5330)

Enhanced A2AEnableProtectionInput & A2AReplicationDetails objects with recoveryAvailabilityZone, recoveryAzureNetworkId, and recoverySubnetName.

* [Hub Generated] Review request for Microsoft.ContainerService to add version 2019-02-01 (Azure#5350)

* Adds base for updating Microsoft.ContainerService from version preview/2018-08-01-preview to version 2019-02-01

* Updates readme

* Updates API version in new specs and examples

* [AKS] add availabilityZones to agent pool profile (Azure#5120)

* including orchestratorVersion and provisioning state for agent pool (Azure#5156)

* including orchestratorVersion and provisioning state for agent pool

* fixing the readme.go.md

* including enablePodSecurityPoliy properties for managedcluster and updating examples

* fixing the example for enablePodSecurityPolicy

* including apiServerAuthorizedIPRanges (Azure#5286)

* including apiServerAuthorizedIPRanges

* support azure network policy

* mark the new features as PREVIEW in description  (Azure#5310)

* mark the new features as PREVIEW

* mark vmss as required for az

* Azure Event Grid: Add 2019-02-01-preview tags for Ruby and Go readme files (Azure#5349)

* Add 2019-02-01-preview tags for Ruby and Go readme files

* fix wrong preview subdirectory

* Update some descriptions. (Azure#5305)

* update description. (Azure#5335)

* Added VNet API (Azure#5351)

* EventHub : corrected typo  (Azure#5360)

* corrected typo in uri

* removed unused parameters

* Add annotations property to trigger (Azure#5332)

* EventHub: Revert the NetworkRuleSet  (Azure#5363)

* Revert "EventHub : corrected typo  (Azure#5360)"

This reverts commit 1495122.

* Revert "Added VNet API (Azure#5351)"

This reverts commit f2ab4a8.

* Remove the AzureDataLakeStoreCosmosStructuredStreamDataset since it's internal only (Azure#5277)

* Removing SDK generation from Travis-CI (Azure#5367)

* Microsoft.DataBoxEdge RP: Corrected comments of trigger and extendedinfo APIs, corrected examples (Azure#5322)

* Corrected comments of trigger and extendedinfo APIs, corrected examples

* Microsoft.DataBoxEdge RP: Have better examples

* Corrected parameter description in order API

* oad => @azure/oad (Azure#5362)

* oad => @azure/oad

* oad => @azure/oad

* SRP Swagger changes for 2018-11-01 (Azure#5300)

* SRP Swagger changes for 2018-11-01

* Address CR comments

* Restrict ManagementPolicyRule -> type property to have only values from defined enum

* Make ManagementPolicySchema/version an enum string

* Remove 'version' property from ManagementPolicy schema

* Added List Policies API to SecurityAlerts (Azure#5070)

* Added SecurityAlert List API

Added List api to:
Manged Server + Managed Database
Logical Server + Logical Database

* FIxed a description

this property is for databases+servers

* Added missing Database exmaples

Added missing examples for database security alert

* Added missing description change

For Server Security Alerts

* Revert "FIxed a description"

This reverts commit e01a332.

* Revert "Added missing description change"

This reverts commit 17aeab7.

* Update DatabaseSecurityAlertListByDatabase.json

* fixed examples

* Update DatabaseSecurityAlertCreateMin.json

Deleted wild comma

* Update readme.md

* Minor fixes

Changed tabs to spaces.
Added location for servers exmaples.

* Removed location

removed the location property from examples

* graph: improve API coverage on application & service principal  (Azure#5181)

* [graphrbac]: fix bugs in permission grant apis

* fix paging

* add missing property

* remove pageable

* author pagable in the right way

* breaking changes over a wrong spec

* remove additionalProperties which causes wrong code-gen

* add suppressions

* use code to surpress

* address review feedabck

* moev ahead with the established pattern anyway

* address review feedbacks

* clean up all

* update operation id

* add one more property per CLI user's ask

* Add insights 2019-03-01 package to Go SDK (Azure#5373)

* Add insights 2019-03-01 package to Go SDK

* move global settings before packages

* fix package tag for new API version (Azure#5378)

* Remove GZRS and RAGZRS sku from SRP 2018-11-01 because feature slipped. (Azure#5377)

* typo: upto -> up to (Azure#5376)

* typo: graphrbac/data-plane/Microsoft.GraphRbac/stable/1.6 (Azure#5375)

- requied -> required
- authorizaed -> authorized
- Organizatio -> Organization
- Allorganizations -> AllOrganizations
- Microsof -> Microsoft
- perperties -> prorperties
- servie -> service
- altenative -> alternative
- thubmbprint -> thumbprint

* Azure DevOps Pipeline workaround for private repository (Azure#5374)

* Fix PolicyInsights code generation issue (Azure#5389)

* Add replica creation parameters to Postgres ResourceProvider for SDK (Azure#5379)

*  Add replica creation parameters to Postgres ResourceProvider for SDK

* Add replica creation parameters to Postgres ResourceProvider for SDK

* Add replica creation parameters to Postgres ResourceProvider for SDK

* Add replica creation parameters to Postgres ResourceProvider for SDK

* Add replica creation parameters to Postgres ResourceProvider for SDK

* Revert "FI from master to network-february-release (Azure#5391)" (Azure#5393)

This reverts commit 8f28f7f.
akshataP pushed a commit that referenced this pull request Jun 1, 2019
* Fixes for:

Issue #1 : Azure#4576 (comment)

Typo in the DeltaOperationType Enum.

Issue #2 : Azure#4844 (comment)

GlobalAdmins list is marked as an object, while it's an array.

* Updated the Specification as example for Risky IP Download report.

* Added the Result details

* Fixed review comments, changed the URI scheme as well as changed get/post methods

* Addressing Review Feedback, changing POST call of blobUri -> generateBlobUri

* Added new operation to get connector metadata.

* Addressing PR Comments

* Fixing typo

* Updated examples with real values
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.