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

Adding Create, Delete and List Namespaces Endpoints for Event Hubs Cluster Resource #5565

Merged
merged 24 commits into from
Apr 29, 2019

Conversation

FBettati
Copy link
Contributor

@FBettati FBettati commented Apr 5, 2019

Latest improvements:

MSFT employees can try out our new experience at OpenAPI Hub - one location for using our validation tools and finding your workflow.

Contribution checklist:

  • I have reviewed the documentation for the workflow.
  • Validation tools were run on swagger spec(s) and have all been fixed in this PR.
  • The OpenAPI Hub was used for checking validation status and next steps.

ARM API Review Checklist

  • Service team MUST add the "WaitForARMFeedback" label if the management plane API changes fall into one of the below categories.
  • adding/removing APIs.
  • adding/removing properties.
  • adding/removing API-version.
  • adding a new service in Azure.

Failure to comply may result in delays for manifest application. Note this does not apply to data plane APIs.

  • If you are blocked on ARM review and want to get the PR merged urgently, please get the ARM oncall for reviews (RP Manifest Approvers team under Azure Resource Manager service) from IcM and reach out to them.
    Please follow the link to find more details on API review process.

@msftclas
Copy link

msftclas commented Apr 5, 2019

CLA assistant check
All CLA requirements met.

@AutorestCI
Copy link

AutorestCI commented Apr 5, 2019

Automation for azure-sdk-for-python

The initial PR has been merged into your service PR:
Azure/azure-sdk-for-python#4977

@AutorestCI
Copy link

AutorestCI commented Apr 5, 2019

Automation for azure-sdk-for-js

The initial PR has been merged into your service PR:
Azure/azure-sdk-for-js#2552

@AutorestCI
Copy link

AutorestCI commented Apr 5, 2019

Automation for azure-sdk-for-ruby

Nothing to generate for azure-sdk-for-ruby

@AutorestCI
Copy link

AutorestCI commented Apr 5, 2019

Automation for azure-sdk-for-java

Encountered a Subprocess error: (azure-sdk-for-java)

Command: ['/usr/local/bin/autorest', '/tmp/tmp_cuota3l/rest/specification/eventhub/resource-manager/readme.md', '--azure-libraries-for-java-folder=/tmp/tmp_cuota3l/sdk', '--java', '--multiapi', '[email protected]/[email protected]', '--verbose']
Finished with return code 1
and output:

AutoRest code generation utility [version: 2.0.4283; node: v8.12.0]
(C) 2018 Microsoft Corporation.
https://aka.ms/autorest
   Loading AutoRest core      '/root/.autorest/@[email protected]/node_modules/@microsoft.azure/autorest-core/dist' (2.0.4302)
   Loading AutoRest extension '@microsoft.azure/autorest.java' (2.1.85->2.1.85)
   Loading AutoRest extension '@microsoft.azure/autorest.modeler' (2.3.38->2.3.38)
Processing batch task - {"tag":"package-2018-01-preview"} .
FATAL: System.NotImplementedException: Handling return type 'Map<String, Integer>' for OtherMethod is not implemented
   at AutoRest.Java.Azure.Fluent.Model.OtherMethod.get_InnerReturnType() in /home/vsts/work/1/s/src/azurefluent/Model/FluentCommon/OtherMethods/OtherMethod.cs:line 69
   at AutoRest.Java.Azure.Fluent.Model.OtherMethod.get_ReturnModel() in /home/vsts/work/1/s/src/azurefluent/Model/FluentCommon/OtherMethods/OtherMethod.cs:line 94
   at AutoRest.Java.Azure.Fluent.Model.OtherMethods.<>c.<get_OtherFluentModels>b__12_0(OtherMethod om) in /home/vsts/work/1/s/src/azurefluent/Model/FluentCommon/OtherMethods/OtherMethods.cs:line 170
   at System.Linq.Enumerable.SelectListIterator`2.MoveNext()
   at System.Linq.Enumerable.WhereEnumerableIterator`1.MoveNext()
   at System.Linq.Enumerable.Any[TSource](IEnumerable`1 source)
   at AutoRest.Java.Azure.Fluent.Model.OtherMethods.get_ImportsForInterface() in /home/vsts/work/1/s/src/azurefluent/Model/FluentCommon/OtherMethods/OtherMethods.cs:line 47
   at AutoRest.Java.Azure.Fluent.Model.SegmentFluentMethodGroup.get_ImportsForInterface() in /home/vsts/work/1/s/src/azurefluent/Model/FluentCommon/FluentMethodGroup/SegmentFluentMethodGroup/SegmentFluentMethodGroup.cs:line 310
   at AutoRest.Java.Azure.Fluent.Model.ProxyFluentMethodGroup.get_ImportsForInterface() in /home/vsts/work/1/s/src/azurefluent/Model/FluentCommon/FluentMethodGroup/ProxyFluentMethodGroup/ProxyFluentMethodGroup.cs:line 117
   at AutoRest.Java.Azure.Fluent.Model.ProxyFluentMethodGroup.get_ImportsForInterface() in /home/vsts/work/1/s/src/azurefluent/Model/FluentCommon/FluentMethodGroup/ProxyFluentMethodGroup/ProxyFluentMethodGroup.cs:line 117
   at AutoRest.Java.Azure.Fluent.Model.ClientFluentMethodGroupInterface.get_Imports() in /home/vsts/work/1/s/src/azurefluent/Model/FluentCommon/FluentMethodGroup/ClientFluentMethodGroupInterface.cs:line 39
   at AutoRest.Java.azurefluent.Templates.FluentCommon.FluentMethodGroupInterfaceTemplate.<ExecuteAsync>d__1.MoveNext() in /home/vsts/work/1/s/src/obj/Razor/azurefluent/Templates/FluentCommon/FluentMethodGroupInterfaceTemplate.cshtml:line 21
--- End of stack trace from previous location where exception was thrown ---
   at System.Runtime.ExceptionServices.ExceptionDispatchInfo.Throw()
   at System.Runtime.CompilerServices.TaskAwaiter.HandleNonSuccessAndDebuggerNotification(Task task)
   at AutoRest.Core.CodeGenerator.<Write>d__12.MoveNext() in /home/vsts/work/1/s/autorest.common/src/CodeGenerator.cs:line 119
--- End of stack trace from previous location where exception was thrown ---
   at System.Runtime.ExceptionServices.ExceptionDispatchInfo.Throw()
   at System.Runtime.CompilerServices.TaskAwaiter.HandleNonSuccessAndDebuggerNotification(Task task)
   at AutoRest.Java.Azure.Fluent.CodeGeneratorJvaf.<Generate>d__6.MoveNext() in /home/vsts/work/1/s/src/azurefluent/CodeGeneratorJvaf.cs:line 63
--- End of stack trace from previous location where exception was thrown ---
   at System.Runtime.ExceptionServices.ExceptionDispatchInfo.Throw()
   at System.Runtime.CompilerServices.TaskAwaiter.HandleNonSuccessAndDebuggerNotification(Task task)
   at AutoRest.Java.Program.<ProcessInternal>d__3.MoveNext() in /home/vsts/work/1/s/src/Program.cs:line 114
--- End of stack trace from previous location where exception was thrown ---
   at System.Runtime.ExceptionServices.ExceptionDispatchInfo.Throw()
   at System.Runtime.CompilerServices.TaskAwaiter.HandleNonSuccessAndDebuggerNotification(Task task)
   at System.Runtime.CompilerServices.TaskAwaiter`1.GetResult()
   at NewPlugin.<Process>d__20.MoveNext() in /home/vsts/work/1/s/autorest.common/src/Plugins/NewPlugin.cs:line 163
FATAL: java/generate - FAILED
FATAL: Error: Plugin java reported failure.
Process() cancelled due to exception : Plugin java reported failure.
Failure during batch task - {"tag":"package-2018-01-preview"} -- Error: Plugin java reported failure..
  Error: Plugin java reported failure.

@AutorestCI
Copy link

AutorestCI commented Apr 5, 2019

Automation for azure-sdk-for-go

The initial PR has been merged into your service PR:
Azure/azure-sdk-for-go#4695

@azuresdkci
Copy link
Contributor

Can one of the admins verify this patch?

@praries880 praries880 added the WaitForARMFeedback <valid label in PR review process> add this label when ARM review is required label Apr 10, 2019
@praries880
Copy link

@FBettati
There are a bunch of un-referenced files in the API spec .

Did you miss adding a reference to them or are the files redundant now?

…view/2018-01-01-preview/EventHub-preview.json

Co-Authored-By: FBettati <[email protected]>
@praries880
Copy link

@KrisBash Can you see if the latest commits take care of your review comments?

Copy link
Contributor

@KrisBash KrisBash left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Please see comment regarding response on GET for available clusters

@FBettati
Copy link
Contributor Author

FBettati commented Apr 22, 2019

Hello @KrisBash, could you confirm that the changes you would like to see applied to this PR is changing the response for the availableClusters GET call to a collection response of the form "value": [ { "westus": 5 }, { "eastus": 4 } ]?

And adding on to that question, if we have a GET call that returns a collection response, wouldn't ARM guidelines be to add ARM IDs, required properties like location etc., and individual scope support to the items in the collection?

@KrisBash KrisBash added ARMSignedOff <valid label in PR review process>add this label when ARM approve updates after review and removed ARMChangesRequested <valid label in PR review process>add this label when require changes after ARM review WaitForARMFeedback <valid label in PR review process> add this label when ARM review is required labels Apr 23, 2019
Copy link
Contributor

@KrisBash KrisBash left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM thanks

@FBettati
Copy link
Contributor Author

Great, thanks! @praries880 could you merge this PR?

@praries880
Copy link

@jianghaolu Can you help me understand what's breaking here for the java SDK generation :

Processing batch task - {"tag":"package-2018-01-preview"} .
FATAL: System.NotImplementedException: Handling return type 'Map<String, Integer>' for OtherMethod is not implemented
   at AutoRest.Java.Azure.Fluent.Model.OtherMethod.get_InnerReturnType() in /home/vsts/work/1/s/src/azurefluent/Model/FluentCommon/OtherMethods/OtherMethod.cs:line 69

@praries880
Copy link

opened this issue to fix the java autogenerator issue : Azure/autorest.java#346

@lmazuel
Copy link
Member

lmazuel commented Apr 29, 2019

Merging, since the issue is in the Java generator. Checked with @praries880 first.

@lmazuel lmazuel merged commit 0c33653 into Azure:master Apr 29, 2019
dsgouda pushed a commit that referenced this pull request May 6, 2019
* Refactor enum list of /definitions/ReadableType (#5747)

- Remove unnecessary "Regular Expression Entity Extractor"

* Update rest-api-spec-scripts to 0.2 and remote travis-ci jobs (except… (#5742)

* Update rest-api-spec-scripts to 0.2 and remote travis-ci jobs (except branch check)

* return semantic and model

* minor

* modelValidation and semanticValidation use avocado diff.

* Install .Net

* semantic and model

* restore travis

* minor

* Fix documentation issues. (#5721)

* Introduce toBeDetached flag for Datadisks for ApiVersion 2019-03-01 (#5704)

* ToBeDetached added in Json and an example added

* Removed from OS disk and added to dataDisk

* Initial stable version (#5683)

* NFSAAS-2185 update to R4 standard

* NFSAAS-2185 update to R4 RP standard

* NFSAAS-2185 update to R4 RP standard

* NFSAAS-2185 update to R4 RP standard

* NFSAAS-2185 update to RP R4 standard

* NFSAAS-2185 update to R4 RP standard

* NFSAAS-2185 update to R4 RP standard

* NFSAAS-2185 update to R4 RP standard

* NFSAAS-2185 update to R4 RP standard

* Adds Partners GET (#5726)

*  Fixing csharp multi profiles  (#4092)

* Fixing csharp multi profiles

* minor fix

* fixing network

* unused

* unused

* FIxing dns

* Moved AdditionalCapabilities out from VM profile to VMSS and updated SKUs API to support zonal capabilities (#5753)

* Updated comments and introduced AvailabilitySetSkuType to help users use predefined constants

* Moved AdditionalCapabilities out from VM profile to VMSS and updated SKUs API to support zonal capabilities

* Fixed indentation

* Fixed indentation in skus.json

* QnAMaker DTO changes for Multiturn (#5738)

* DTO updates for multiturn support

* fix message

* max len of display text

* Resolve comments

* missed additional property

* Fix Namespace in Ruby Readme Qnamaker

* [Hub Generated] Review request for Microsoft.ApiManagement to add version stable/2019-01-01 (#5732)

* Policy Format

* update example of creating revisions from api

* upper case XML

* Add hyperVGeneration enum property to ImageProperties (#5740)

* added hyperV property along with hyperVGenerationType

* Updated Compute.json by chaging HyperVGenerationTypes to V1,v2

* Update compute.json

* Update compute.json to link enum to outer HyperVGeneration

* Update compute.json

Ad trailing ,

* Adding 2019-04-01 API version for Microsoft.DevSpaces RP (#5713)

* Adding old spec and examples

* Adding 2019-04-01 API version

* Fix readmes for SDK

* Add missing response code in swagger example

* 2019-02-01-preview (#5698)

* 2019-02-01-preview

* Added ruby spec. Removed Node spec. Fixed spelling error in 'utilizing'. Removed BOM from the beginning of example files.

* Sqldb serverless (#5760)

* Add auto pause delay param to 2017-01-01-preview get/database

* Add minCapacity param to 2017-01-01-preview database

* Scripts v 0.3.9 (#5761)

* Scripts v 0.3.9

* Update preproduction-azure-pipelines.yml

* Remove SDK generation rom preprod.

* branch protection (#5764)

* branch protection

* script

* test

* failOnstderr

* on private

* Added swagger spec for Microsoft.DynamicsTelemetry preview (#5161)

* Added swagger spec for Microsoft.DynamicsTelemetry preview

* host changed

* Fix Operations_List

* Update dynamicstelemetry.json

Host changed to management.azure.com

* "No Content" response added

* change blobduration data type to "date-time"

* Changed blobDuration format to "format":"duration"

* add 204 response to the sample

* fix validation error

* modelValidation2 and semanticValidation2 (#5765)

* modelValidation2 and semanticValidation2

* modelvalidation2

* rename

* [TimeSeriesInsights] Spec bugfix for GetEvents API - projected properties are a list of EventProperty and not string. (#5681)

* Fix or suppress spec violations
Fix R2016 by making PATCH body properties optional. Suppress the remaining 4 violations that are confirmed as false positives.

* Try removing 2nd top-level heading, which _may_ have been causing a linter warning.

* Fix extra indention which prevented linter from running.

* Added dataStringComparisonBehavior parameter to reference data put/update spec.

* Fixed grammar.

* adding partitionkeyProperty to environmentInfo

* changing property name

* changing modelAsString

* Initial checkin of spec. Unchanged from GA release other than version.

* Remove .orig files

* Fix validation error by removing IoTHub ES from List ES example response

* Add extra example that was missing from preview spec.

* Add long-term environment to resource hierarchy, and update examples.

* Switch to model-as-string for enums. Restore EnvironmentResource's inheritance from TrackedResource.

* Upsate sku description

* Add new TSI preview api-version to readme.md.

* Add suppressions for false-positive linter failures.

* Test suppression failures

* Adding preview package tag to several other places in readme.md

* Add preview package to the readme.go.md file

* Remove invalid required property from EnvironmentCreateOrUpdateParameters

* Initial version of data plane swagger spec for PAYG (Preview) SKU.

* Added missing Readme.md

* Fixed name of readme

* Fixed path to swagger json

* Attempt to fix readme

* Apply suggestions from code review

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

* Update timeseriesinsights.json

* Update timeseriesinsights.json

* Added link to oav issue

* bug fix - get events payload projected properties to take in list of EventProperty and not string.

* Fixed inconsistencies between examples and spec.

* Fixed bug in spec specifying TSID array.

* Updating the schema to adhere to the new design changes (#5769)

* add storage package-2019-04 to Go SDK (#5773)

* add netapp package-netapp-2019-05-01 to Go SDK (#5774)

* fixes (#5770)

* revert not support formats (#5763)

* [Cogs Face] Align with latest Face API v1.12 (#5722)

* Sync with APIM descriptions.

* Remove description end with '\n' and add add face to person in person gourp descriptions.

* Update

* [Hub Generated] Review request for Microsoft.OperationalInsights to add version preview/2015-11-01-preview (#5645)

* Update OperationalInsights.json

* Adds suppression to readme

* Adds suppression to readme

* Adds suppression to readme

* Adds suppression to readme

* Adds suppression to readme

* Revert "Adds suppression to readme"

This reverts commit 8c0e9bd.

* Revert "Adds suppression to readme"

This reverts commit ecf7615.

* Revert "Adds suppression to readme"

This reverts commit c8a0d5c.

* Revert "Adds suppression to readme"

This reverts commit 7f54236.

* Revert "Adds suppression to readme"

This reverts commit 350be00.

* Personalizer v1.0 preview (#5743)

* Initial setup of Personalizer SDK

* Add readme.md

* fix some comments

* Fix readme.md

* fix enums and examples, work in progress

* handle error codes

* fix typos

* resolve comments

* using UTF-8 encoding

* checking formatted Personalizer.json

* use ASCII

* use parameter names

* remove example from definitions

* Update examples

* update operation names to noun_verb format

* update generated swagger

* deleting earlier example files

* add back application/octet-stream

* Change model type back to file

* Update readonly properties

* Update readonly properties

* remove IActionResult

* [Datafactory] Adding getFeatureValue API in factory scope (#5591)

* Adding getFeatureValue API in factory scope

* Updating examples for IfNoneMatch header on get calls

* Fixing example validation errors

* remove ifNoneMatch=null from examples

* More example validation fixes

* One more IR example fix

* Deleted wrong property

* Fixing last example

* Restoring previous example

* Testing example for validation

* Reverting last commit

* Compute Python 2019-04 (#5775)

* [ACR] Support MSI in Task [NEW API VERSION] (#5561)

* examples for new sdk

* update examples

* update examples

* add identity to core examples

* Add opaque example

* add references to examples

* spelling fix

* update tag in all readmes

* update jave readme

* fix RunRequest examples

* Delete extra file.

* Delete extra file.

* Fix examples from previous API version

* fix example errors

* FileTaskRunRequest fix

* Add RunErrorMessage from previous version

* update api-version in examples

* old version: 2018-09-01

* add new API version: 2019-04-01

* Support Scheduler Trigger

* fix build errors

* rename to TimerTrigger

* nit rename comment

* Rename Gremlin container by graph (#5744)

* Rename Gremlin container by graph

* Replace rid in url with name

* Replace mongo with mongodb

* Replace rid with name in examples

* Fix bug

* Update MongoDB name

* Update file name by git mv as windows doesn't work

* Update MongDB name in example

* Adding all sync methods to Personalizer API (#5777)

* add stream as return type for get model

* fix response type for correct op

* change type file to stream

* remove stream ref

* Adding all sync methods to Personalizer API

* Adding all sync methods to Personalizer API

* Fix missing proxy override enum for managed instance swagger (#5780)

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

* Fix missing enum for proxy override

* Make Microsoft.Subscription.Admin api consistent. (#5771)

* Make Microsoft.Subscription.Admin api consistent.

* Remove location delete api

* Change Metrics to metrics

* Change MetricDefinitions to metricDefinitions

* Remove the Delete.json

* Fix examples (#5778)

* chore: jsonfmt addons (#5785)

Ran `jsonfmt -w specification/addons/**/*.json`

* Add v3.0-preview APIs for LUIS (#5724)

* Add v3.0-preview APIs

* Remove the timezoneoffset property

* Add the datetimeReference property

* - Add description to the definitions and the APIs
- Fix naming bugs (show-all-intents,dynamicLists)
- Rename the definitions for consistency

* [Merge is planed on 2019/04/29] Move scripts from this repo. (#5719)

* Breaking Change2

* Experimental lintdiff

* No more model/semantic/syntax validations in travis-CI.

* Scripts 0.1.8

* Use scripts from @azure/rest-api-specs-scripts

* ts-utils

* fix breaking changes

* momentOfTruthPostProcessing

* momentOfTruthPostProcessing.ts

* remove 'ts-utils.ts'

* Remove `utils`

* postToGitHub.ts

* switch to momentOfTruthUtils

* minor

* Syntax.ts

* semanticValidation.ts

* another try

* minor update

* URL/fix

* OAD ^0.2.13

* update to new scripts.

* Use scripts from `sergey/move-from-specs-repo`.

* simplified model validation.

* preprod

* postToGitHub

* momentOfTruthPostProcessing

* Semantic Validation

* getSwaggers()

* Use `getExamples()` and `getSwaggers()`.

* Update package.json

* Remove travis-CI jobs except branch strategy.

* remove model and semantic validations version 2.

* chore: Renormalize line endings (#5783)

Ran `git add --renormalize .`

* Adding Create, Delete and List Namespaces Endpoints for Event Hubs Cluster Resource (#5565)

* Adding ClusterPut.json example

* Create ListNamespaceInClusterPost.json

* Create ClusterDelete.json

* Update EventHub-preview.json

* Adding valid responses to Cluster PUT example.

* Update specification/eventhub/resource-manager/Microsoft.EventHub/preview/2018-01-01-preview/EventHub-preview.json

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

* Adding Available Regions API

* Adding ListAvailableClusters POST example.

* Modifying response in AvailableClusterList

* Updating LIstAvailableClustersPost.json example

* Delete ListAvailableClustersPost.json

* Create ListAvailableClustersGet.json

* Delete ListNamespaceInClusterPost.json

* Create ListNamespacesInClusterGet.json

* Update EventHub-preview.json

* Modify availableClusters API to have a valid ARM collection response

* Updating AvailableClusters collection response example

* Updating /cluster/namespaces and /availableClusters calls

* Adding parameters to ListAvailableClusters example.

* Adding extra layer to namespace IDs in ListNamespacesInCluster example.

* Removing comma from ListAvailableClusters example

* Added regultory compliance to package-composite-v3 (#5786)

* Added regultory compliance to package-composite-v3

* Added regultory compliance to package-composite-v3

* Improve documentation of sql database & elastic pool skus (#5756)

* Improve documentation of db and pool skus

* Update documentation for database 2017-10-01-preview

* Update documentation for elastic pool 2017-10-01-preview

* Fixed syntax error in databases 2017-10-01

* Fixed syntax error in elasticPools 2017-10-01

* Remove CreateVCoreDatabase example that is no longer referenced

* Remove DW DTU capacity example due to feedback

* Fixed broken Sku reference

* Fixed model validation issues

* Fixed another model validation error

* Additional doc tweaks

* Create new resource type: Microsoft.ResourceGraph/queries (#5478)

* 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

* Updating Profile format (#5768)

* Updating Profile format

Updating profile spec format to include actual path for resource spec

* Update 2019-03-01-hybrid.json

updating resourceName to resourceType

* [Azure Search] New GA data-plane API spec for version 2019-05-06 (#5782)

* [Azure Search] Copy 2017-11-11-Preview specs to 2019-05-06

* [Azure Search] Update 2019-05-06 spec to reflect new GA API

Made the following changes to specs and examples:
  1. Changed version number from 2017-11-11-Preview to 2019-05-06.
  2. Removed encryptionKey from Index and SynonymMap, as well as supporting
     types and examples. Encryption with customer-managed keys is still in
     preview.
  3. Added statusCode to IndexError since this was added to the 2019-05-06
     API.
  4. Made DataSourceCredentials.connectionString an optional property to
     suppress generation of client-side validation code. This will make it
     easier for customers to write correct code that updates their data
     sources. This is especially important now that Create and CreateOrUpdate
     do not return the connection string.

No further changes are required because all other features from
2017-11-11-Preview are now GA in 2019-05-06.

* [SRP] Expose lease container in Apr19 API. (#5733)

* [SRP] Expose lease container in Apr19 API.

* Fix lease container example files; fix casing in Lease Container URL

* Update CODEOWNERS (#5794)

assigning as one of the owners of profile directory

* [Microsoft.ResourceGraph] Returning null for nextLink in example to preserve 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

* Update the default tag for LUIS Runtime (#5793)

* Sync 2019-04-01 with "git add --renormalize ."call

* Fix style issues missed by automation
sergey-shandar pushed a commit that referenced this pull request May 23, 2019
* Adds base for updating Microsoft.Network from version stable/2019-02-01 to version 2019-04-01

* Updates readme

* Updates API version in new specs and examples

* Adding custom port for probe in Application Gateways. (#5758)

* Adding custom port for probe in Application Gateways

* Fix JSON

* PR comments

* PR comments

* Add support for Get ResourceNavigationLinks and Get ServiceAssociationLinks (#5767)

* Initial commit

* Rename operation IDs

* Sync from master. Improved code style (#5802)

* Refactor enum list of /definitions/ReadableType (#5747)

- Remove unnecessary "Regular Expression Entity Extractor"

* Update rest-api-spec-scripts to 0.2 and remote travis-ci jobs (except… (#5742)

* Update rest-api-spec-scripts to 0.2 and remote travis-ci jobs (except branch check)

* return semantic and model

* minor

* modelValidation and semanticValidation use avocado diff.

* Install .Net

* semantic and model

* restore travis

* minor

* Fix documentation issues. (#5721)

* Introduce toBeDetached flag for Datadisks for ApiVersion 2019-03-01 (#5704)

* ToBeDetached added in Json and an example added

* Removed from OS disk and added to dataDisk

* Initial stable version (#5683)

* NFSAAS-2185 update to R4 standard

* NFSAAS-2185 update to R4 RP standard

* NFSAAS-2185 update to R4 RP standard

* NFSAAS-2185 update to R4 RP standard

* NFSAAS-2185 update to RP R4 standard

* NFSAAS-2185 update to R4 RP standard

* NFSAAS-2185 update to R4 RP standard

* NFSAAS-2185 update to R4 RP standard

* NFSAAS-2185 update to R4 RP standard

* Adds Partners GET (#5726)

*  Fixing csharp multi profiles  (#4092)

* Fixing csharp multi profiles

* minor fix

* fixing network

* unused

* unused

* FIxing dns

* Moved AdditionalCapabilities out from VM profile to VMSS and updated SKUs API to support zonal capabilities (#5753)

* Updated comments and introduced AvailabilitySetSkuType to help users use predefined constants

* Moved AdditionalCapabilities out from VM profile to VMSS and updated SKUs API to support zonal capabilities

* Fixed indentation

* Fixed indentation in skus.json

* QnAMaker DTO changes for Multiturn (#5738)

* DTO updates for multiturn support

* fix message

* max len of display text

* Resolve comments

* missed additional property

* Fix Namespace in Ruby Readme Qnamaker

* [Hub Generated] Review request for Microsoft.ApiManagement to add version stable/2019-01-01 (#5732)

* Policy Format

* update example of creating revisions from api

* upper case XML

* Add hyperVGeneration enum property to ImageProperties (#5740)

* added hyperV property along with hyperVGenerationType

* Updated Compute.json by chaging HyperVGenerationTypes to V1,v2

* Update compute.json

* Update compute.json to link enum to outer HyperVGeneration

* Update compute.json

Ad trailing ,

* Adding 2019-04-01 API version for Microsoft.DevSpaces RP (#5713)

* Adding old spec and examples

* Adding 2019-04-01 API version

* Fix readmes for SDK

* Add missing response code in swagger example

* 2019-02-01-preview (#5698)

* 2019-02-01-preview

* Added ruby spec. Removed Node spec. Fixed spelling error in 'utilizing'. Removed BOM from the beginning of example files.

* Sqldb serverless (#5760)

* Add auto pause delay param to 2017-01-01-preview get/database

* Add minCapacity param to 2017-01-01-preview database

* Scripts v 0.3.9 (#5761)

* Scripts v 0.3.9

* Update preproduction-azure-pipelines.yml

* Remove SDK generation rom preprod.

* branch protection (#5764)

* branch protection

* script

* test

* failOnstderr

* on private

* Added swagger spec for Microsoft.DynamicsTelemetry preview (#5161)

* Added swagger spec for Microsoft.DynamicsTelemetry preview

* host changed

* Fix Operations_List

* Update dynamicstelemetry.json

Host changed to management.azure.com

* "No Content" response added

* change blobduration data type to "date-time"

* Changed blobDuration format to "format":"duration"

* add 204 response to the sample

* fix validation error

* modelValidation2 and semanticValidation2 (#5765)

* modelValidation2 and semanticValidation2

* modelvalidation2

* rename

* [TimeSeriesInsights] Spec bugfix for GetEvents API - projected properties are a list of EventProperty and not string. (#5681)

* Fix or suppress spec violations
Fix R2016 by making PATCH body properties optional. Suppress the remaining 4 violations that are confirmed as false positives.

* Try removing 2nd top-level heading, which _may_ have been causing a linter warning.

* Fix extra indention which prevented linter from running.

* Added dataStringComparisonBehavior parameter to reference data put/update spec.

* Fixed grammar.

* adding partitionkeyProperty to environmentInfo

* changing property name

* changing modelAsString

* Initial checkin of spec. Unchanged from GA release other than version.

* Remove .orig files

* Fix validation error by removing IoTHub ES from List ES example response

* Add extra example that was missing from preview spec.

* Add long-term environment to resource hierarchy, and update examples.

* Switch to model-as-string for enums. Restore EnvironmentResource's inheritance from TrackedResource.

* Upsate sku description

* Add new TSI preview api-version to readme.md.

* Add suppressions for false-positive linter failures.

* Test suppression failures

* Adding preview package tag to several other places in readme.md

* Add preview package to the readme.go.md file

* Remove invalid required property from EnvironmentCreateOrUpdateParameters

* Initial version of data plane swagger spec for PAYG (Preview) SKU.

* Added missing Readme.md

* Fixed name of readme

* Fixed path to swagger json

* Attempt to fix readme

* Apply suggestions from code review

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

* Update timeseriesinsights.json

* Update timeseriesinsights.json

* Added link to oav issue

* bug fix - get events payload projected properties to take in list of EventProperty and not string.

* Fixed inconsistencies between examples and spec.

* Fixed bug in spec specifying TSID array.

* Updating the schema to adhere to the new design changes (#5769)

* add storage package-2019-04 to Go SDK (#5773)

* add netapp package-netapp-2019-05-01 to Go SDK (#5774)

* fixes (#5770)

* revert not support formats (#5763)

* [Cogs Face] Align with latest Face API v1.12 (#5722)

* Sync with APIM descriptions.

* Remove description end with '\n' and add add face to person in person gourp descriptions.

* Update

* [Hub Generated] Review request for Microsoft.OperationalInsights to add version preview/2015-11-01-preview (#5645)

* Update OperationalInsights.json

* Adds suppression to readme

* Adds suppression to readme

* Adds suppression to readme

* Adds suppression to readme

* Adds suppression to readme

* Revert "Adds suppression to readme"

This reverts commit 8c0e9bd.

* Revert "Adds suppression to readme"

This reverts commit ecf7615.

* Revert "Adds suppression to readme"

This reverts commit c8a0d5c.

* Revert "Adds suppression to readme"

This reverts commit 7f54236.

* Revert "Adds suppression to readme"

This reverts commit 350be00.

* Personalizer v1.0 preview (#5743)

* Initial setup of Personalizer SDK

* Add readme.md

* fix some comments

* Fix readme.md

* fix enums and examples, work in progress

* handle error codes

* fix typos

* resolve comments

* using UTF-8 encoding

* checking formatted Personalizer.json

* use ASCII

* use parameter names

* remove example from definitions

* Update examples

* update operation names to noun_verb format

* update generated swagger

* deleting earlier example files

* add back application/octet-stream

* Change model type back to file

* Update readonly properties

* Update readonly properties

* remove IActionResult

* [Datafactory] Adding getFeatureValue API in factory scope (#5591)

* Adding getFeatureValue API in factory scope

* Updating examples for IfNoneMatch header on get calls

* Fixing example validation errors

* remove ifNoneMatch=null from examples

* More example validation fixes

* One more IR example fix

* Deleted wrong property

* Fixing last example

* Restoring previous example

* Testing example for validation

* Reverting last commit

* Compute Python 2019-04 (#5775)

* [ACR] Support MSI in Task [NEW API VERSION] (#5561)

* examples for new sdk

* update examples

* update examples

* add identity to core examples

* Add opaque example

* add references to examples

* spelling fix

* update tag in all readmes

* update jave readme

* fix RunRequest examples

* Delete extra file.

* Delete extra file.

* Fix examples from previous API version

* fix example errors

* FileTaskRunRequest fix

* Add RunErrorMessage from previous version

* update api-version in examples

* old version: 2018-09-01

* add new API version: 2019-04-01

* Support Scheduler Trigger

* fix build errors

* rename to TimerTrigger

* nit rename comment

* Rename Gremlin container by graph (#5744)

* Rename Gremlin container by graph

* Replace rid in url with name

* Replace mongo with mongodb

* Replace rid with name in examples

* Fix bug

* Update MongoDB name

* Update file name by git mv as windows doesn't work

* Update MongDB name in example

* Adding all sync methods to Personalizer API (#5777)

* add stream as return type for get model

* fix response type for correct op

* change type file to stream

* remove stream ref

* Adding all sync methods to Personalizer API

* Adding all sync methods to Personalizer API

* Fix missing proxy override enum for managed instance swagger (#5780)

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

* Fix missing enum for proxy override

* Make Microsoft.Subscription.Admin api consistent. (#5771)

* Make Microsoft.Subscription.Admin api consistent.

* Remove location delete api

* Change Metrics to metrics

* Change MetricDefinitions to metricDefinitions

* Remove the Delete.json

* Fix examples (#5778)

* chore: jsonfmt addons (#5785)

Ran `jsonfmt -w specification/addons/**/*.json`

* Add v3.0-preview APIs for LUIS (#5724)

* Add v3.0-preview APIs

* Remove the timezoneoffset property

* Add the datetimeReference property

* - Add description to the definitions and the APIs
- Fix naming bugs (show-all-intents,dynamicLists)
- Rename the definitions for consistency

* [Merge is planed on 2019/04/29] Move scripts from this repo. (#5719)

* Breaking Change2

* Experimental lintdiff

* No more model/semantic/syntax validations in travis-CI.

* Scripts 0.1.8

* Use scripts from @azure/rest-api-specs-scripts

* ts-utils

* fix breaking changes

* momentOfTruthPostProcessing

* momentOfTruthPostProcessing.ts

* remove 'ts-utils.ts'

* Remove `utils`

* postToGitHub.ts

* switch to momentOfTruthUtils

* minor

* Syntax.ts

* semanticValidation.ts

* another try

* minor update

* URL/fix

* OAD ^0.2.13

* update to new scripts.

* Use scripts from `sergey/move-from-specs-repo`.

* simplified model validation.

* preprod

* postToGitHub

* momentOfTruthPostProcessing

* Semantic Validation

* getSwaggers()

* Use `getExamples()` and `getSwaggers()`.

* Update package.json

* Remove travis-CI jobs except branch strategy.

* remove model and semantic validations version 2.

* chore: Renormalize line endings (#5783)

Ran `git add --renormalize .`

* Adding Create, Delete and List Namespaces Endpoints for Event Hubs Cluster Resource (#5565)

* Adding ClusterPut.json example

* Create ListNamespaceInClusterPost.json

* Create ClusterDelete.json

* Update EventHub-preview.json

* Adding valid responses to Cluster PUT example.

* Update specification/eventhub/resource-manager/Microsoft.EventHub/preview/2018-01-01-preview/EventHub-preview.json

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

* Adding Available Regions API

* Adding ListAvailableClusters POST example.

* Modifying response in AvailableClusterList

* Updating LIstAvailableClustersPost.json example

* Delete ListAvailableClustersPost.json

* Create ListAvailableClustersGet.json

* Delete ListNamespaceInClusterPost.json

* Create ListNamespacesInClusterGet.json

* Update EventHub-preview.json

* Modify availableClusters API to have a valid ARM collection response

* Updating AvailableClusters collection response example

* Updating /cluster/namespaces and /availableClusters calls

* Adding parameters to ListAvailableClusters example.

* Adding extra layer to namespace IDs in ListNamespacesInCluster example.

* Removing comma from ListAvailableClusters example

* Added regultory compliance to package-composite-v3 (#5786)

* Added regultory compliance to package-composite-v3

* Added regultory compliance to package-composite-v3

* Improve documentation of sql database & elastic pool skus (#5756)

* Improve documentation of db and pool skus

* Update documentation for database 2017-10-01-preview

* Update documentation for elastic pool 2017-10-01-preview

* Fixed syntax error in databases 2017-10-01

* Fixed syntax error in elasticPools 2017-10-01

* Remove CreateVCoreDatabase example that is no longer referenced

* Remove DW DTU capacity example due to feedback

* Fixed broken Sku reference

* Fixed model validation issues

* Fixed another model validation error

* Additional doc tweaks

* Create new resource type: Microsoft.ResourceGraph/queries (#5478)

* 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

* Updating Profile format (#5768)

* Updating Profile format

Updating profile spec format to include actual path for resource spec

* Update 2019-03-01-hybrid.json

updating resourceName to resourceType

* [Azure Search] New GA data-plane API spec for version 2019-05-06 (#5782)

* [Azure Search] Copy 2017-11-11-Preview specs to 2019-05-06

* [Azure Search] Update 2019-05-06 spec to reflect new GA API

Made the following changes to specs and examples:
  1. Changed version number from 2017-11-11-Preview to 2019-05-06.
  2. Removed encryptionKey from Index and SynonymMap, as well as supporting
     types and examples. Encryption with customer-managed keys is still in
     preview.
  3. Added statusCode to IndexError since this was added to the 2019-05-06
     API.
  4. Made DataSourceCredentials.connectionString an optional property to
     suppress generation of client-side validation code. This will make it
     easier for customers to write correct code that updates their data
     sources. This is especially important now that Create and CreateOrUpdate
     do not return the connection string.

No further changes are required because all other features from
2017-11-11-Preview are now GA in 2019-05-06.

* [SRP] Expose lease container in Apr19 API. (#5733)

* [SRP] Expose lease container in Apr19 API.

* Fix lease container example files; fix casing in Lease Container URL

* Update CODEOWNERS (#5794)

assigning as one of the owners of profile directory

* [Microsoft.ResourceGraph] Returning null for nextLink in example to preserve 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

* Update the default tag for LUIS Runtime (#5793)

* Sync 2019-04-01 with "git add --renormalize ."call

* Fix style issues missed by automation

* Add support for availability zones for azure firewall (#5888)

* [Network]: get virtualnetwork gateway vpn connection health (#5883)

* [Network]: get virtualnetwork gateway vpn connection health

* [Network]: get virtualnetwork gateway vpn connection health

* [Network] Remove text like "Possible values are" from props that have enum (#5879)

* Remove text like "Possible values are" from props that have enum

* Remove additional text

* Top Level Waf Resource Swagger changes, Changes in PR#5906 by yanshu + some comments addressed (#5921)

* top level waf resource swagger changes, changes in PR#5906 with some comments addressed on the right branch

* Update specification/network/resource-manager/Microsoft.Network/stable/2019-04-01/webapplicationfirewall.json

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

* Update specification/network/resource-manager/Microsoft.Network/stable/2019-04-01/webapplicationfirewall.json

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

*  some more review comments addressed

* Addressing some more review comments

*  addressing more review comments

* Add type field to child resources. Fix routefilter reference. Add ipv6peering property (#5975)

* add type field to child resources, fix routefilter def. add ipv6 peering in routefilter

* updated examples

* Reverting the changes done by PR #5921 (#5986)

* Changes related to setting VpnClient AAD authentication options to Virtual network gateway resource & Advertising virtual Wan 2 released APIs. (#5935)

* Changes related to setting VpnClient AAD authentication options to Virtual network gateway resource.

* Advertising virtual Wan 2 released APIs through swagger

* Incorporate code review comments

* Minor format fixes

* Add newly added param:usePolicyBasedTrafficSelectors to VpnConnection resource.

* Updated descriptions with trailing dot (#6020)

* Updated descriptions with trailing dot

* Fix example

* Adding PrivateEndpoint (formerly called InterfaceEndpoint) and PrivateLinkService with operations. (#5860)

* rename interfaceEndpoint to privateEndpoint.

* Change the PrivateEndpoint properties by following private ip design document.

* modify some of array part of private endpoint and other minor fix.

* Add availablePrivateEndpointTypes and its examples.

* Changes for privateEndpoint example.

* 1. Add default response for privateEndpoint
2. Add availablePrivateEndpointTypes to readme.md file.

* Fix some error detect by autorest.

* Minor fix from Azure Pipeline model validation steps.

* 1. Add PrivateLinkService and its example.
2. Modify PrivateEndpoint and its example to match the PrivateLinkService.

* add privateLinkService to readme.md file.

* Modify/Add examples to demonstrate PrivateEndpoint with manualPrivateLinkServiceConnections.

* Fix some issue found by AzurePipelines model validator.

* fix for model validator error.

* Typo fix.

* Minor fix for PR comment and others.

* delete interfaceendpoint example.

* edit some of descriptions.

* Add some new features for private link service.

* fix the issue found by ModelValidation.

* Mdofiy update privateEndpointConnection method according to spec changes. Other minor fix.

* fix issue found by model validation.

* no comment.

* Add x-ms-client-flatten property to the nested structure.

* Use SubResource instead of creating new definition (#6005)

* Reduced warnings count (#6043)

* Suppress "expected" warnings

* Refactored ApiVersion, SubscriptionId params usage

* Fixed VMSS related warnings

* Resource Navigation Link and Service Association Link merge issues (#6074)

* Added missed features

* Fix validation issues

* Mark nextLink as read only

* chore: jsonfmt network 2019 (#5904)

Ran `jsonfmt -w "specification/network/**/stable/2019*/**/*.json"`

* Fixed common warnings (#6085)

* Swagger support for Bastion Host (#6023)

* Swagger support for Bastion Host

* Fixing example

* Incorporated feedback comment

* Incorporated feedback comment

* Incorporated feedback comment

* Fixes for descriptions; common suppression

* Removing list by vnet api as we are not going to support it

* Specs for ServiceTags resource (#6084)

* Specs for ServiceTags

* Add description

* Addressed PR feedback

* Add read-only flags

* Update Zones for Nat Gateway (#6096)

* fixed merging conflicts
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
ARMSignedOff <valid label in PR review process>add this label when ARM approve updates after review
Projects
None yet
Development

Successfully merging this pull request may close these issues.

7 participants