Add Go SDK generation config by vladbarosan · Pull Request #3403 · Azure/azure-rest-api-specs

vladbarosan · 2018-07-11T21:16:47Z

No description provided.

AutorestCI · 2018-07-11T21:17:11Z

Automation for azure-sdk-for-python

Nothing to generate for azure-sdk-for-python

vladbarosan · 2018-07-11T21:18:56Z

Fixes Azure/azure-sdk-for-go#1246

jhendrixMSFT · 2018-07-11T21:19:12Z

specification/security/resource-manager/readme.md

Copy/paste error.

AutorestCI · 2018-07-11T21:19:57Z

Automation for azure-sdk-for-ruby

Nothing to generate for azure-sdk-for-ruby

jhendrixMSFT · 2018-07-11T21:20:35Z

specification/security/resource-manager/readme.md

If you fix the namespace value you can replace the hard-coded security path/namespace with $(namespace). One day I'd like to see the API version be defined along with a tag so we don't have to duplicate it here.

AutorestCI · 2018-07-11T21:20:37Z

Automation for azure-sdk-for-go

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

Command: ['/usr/local/bin/autorest', '/tmp/tmposhukyk_/rest/specification/security/resource-manager/readme.md', '--go', '--go-sdk-folder=/tmp/tmposhukyk_/src/github.com/Azure/azure-sdk-for-go', '--multiapi', '--use=@microsoft.azure/autorest.go@~2.1.108', '--use-onever', '--verbose']
Finished with return code 1
and output:

AutoRest code generation utility [version: 2.0.4280; node: v8.11.3]
(C) 2018 Microsoft Corporation.
https://aka.ms/autorest

There is a new version of AutoRest available (2.0.4282).
 > You can install the newer version with with npm install -g autorest@latest

   Loading AutoRest core      '/root/.autorest/@microsoft.azure_autorest-core@2.0.4280/node_modules/@microsoft.azure/autorest-core/dist' (2.0.4280)
   Including configuration file 'file:///tmp/tmposhukyk_/rest/specification/security/resource-manager/readme.go.md'
   Loading AutoRest extension '@microsoft.azure/autorest.go' (~2.1.108->2.1.108)
   Loading AutoRest extension '@microsoft.azure/autorest.modeler' (2.3.38->2.3.38)
Processing batch task - {"tag":"package-composite-v1"} .
   Including configuration file 'file:///tmp/tmposhukyk_/rest/specification/security/resource-manager/readme.go.md'
FATAL: System.InvalidOperationException: Found incompatible property types ,  for property 'properties' in schema inheritance chain
   at AutoRest.Modeler.SchemaResolver.ExpandAllOf(Schema schema) in C:\Users\ci\AppData\Local\Temp\PUBLISH8kiit\38_20171116T010915\autorest.modeler\src\SchemaResolver.cs:line 157
   at AutoRest.Modeler.SchemaResolver.Unwrap(Schema schema) in C:\Users\ci\AppData\Local\Temp\PUBLISH8kiit\38_20171116T010915\autorest.modeler\src\SchemaResolver.cs:line 75
   at AutoRest.Modeler.SchemaBuilder.BuildServiceType(String serviceTypeName, Boolean required) in C:\Users\ci\AppData\Local\Temp\PUBLISH8kiit\38_20171116T010915\autorest.modeler\src\SchemaBuilder.cs:line 35
   at AutoRest.Modeler.SwaggerModeler.BuildCompositeTypes() in C:\Users\ci\AppData\Local\Temp\PUBLISH8kiit\38_20171116T010915\autorest.modeler\src\SwaggerModeler.cs:line 271
   at AutoRest.Modeler.SwaggerModeler.Build(ServiceDefinition serviceDefinition) in C:\Users\ci\AppData\Local\Temp\PUBLISH8kiit\38_20171116T010915\autorest.modeler\src\SwaggerModeler.cs:line 66
   at AutoRest.Modeler.Program.<ProcessInternal>d__2.MoveNext() in C:\Users\ci\AppData\Local\Temp\PUBLISH8kiit\38_20171116T010915\autorest.modeler\src\Program.cs:line 60
--- 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 NewPlugin.<Process>d__15.MoveNext()
FATAL: go/imodeler1 - FAILED
FATAL: Error: Plugin imodeler1 reported failure.
Process() cancelled due to exception : Plugin imodeler1 reported failure.
Failure during batch task - {"tag":"package-composite-v1"} -- Error: Plugin imodeler1 reported failure..
  Error: Plugin imodeler1 reported failure.

AutorestCI · 2018-07-11T21:21:12Z

AutoRest linter results for ARM Related Validation Errors/Warnings

These errors are reported by the ARM team's validation tools, reachout to ARM RP API Review directly for any questions or concerns.

File: specification/security/resource-manager/readme.md

⚠️

0 new Warnings.(1 total)

❌0 new Errors.(0 total)

AutoRest Linter Guidelines | AutoRest Linter Issues | Send feedback

Thanks for your co-operation.

AutorestCI · 2018-07-11T21:21:12Z

AutoRest linter results for SDK Related Validation Errors/Warnings

These errors are reported by the SDK team's validation tools, reachout to ADX Swagger Reviewers directly for any questions or concerns.

File: specification/security/resource-manager/readme.md

⚠️

0 new Warnings.(76 total)

❌0 new Errors.(0 total)

AutoRest Linter Guidelines | AutoRest Linter Issues | Send feedback

Thanks for your co-operation.

AutorestCI · 2018-07-11T21:21:22Z

Automation for azure-sdk-for-node

Nothing to generate for azure-sdk-for-node

AutorestCI · 2018-07-11T21:21:22Z

Automation for azure-sdk-for-java

Nothing to generate for azure-sdk-for-java

AutorestCI · 2018-07-11T22:27:49Z

AutoRest linter results for SDK Related Validation Errors/Warnings

These errors are reported by the SDK team's validation tools, reachout to ADX Swagger Reviewers directly for any questions or concerns.

File: specification/security/resource-manager/readme.md

⚠️

0 new Warnings.(76 total)

❌0 new Errors.(0 total)

AutoRest Linter Guidelines | AutoRest Linter Issues | Send feedback

Thanks for your co-operation.

AutorestCI · 2018-07-11T22:27:49Z

AutoRest linter results for ARM Related Validation Errors/Warnings

These errors are reported by the ARM team's validation tools, reachout to ARM RP API Review directly for any questions or concerns.

File: specification/security/resource-manager/readme.md

⚠️

0 new Warnings.(1 total)

❌0 new Errors.(0 total)

AutoRest Linter Guidelines | AutoRest Linter Issues | Send feedback

Thanks for your co-operation.

vladbarosan · 2018-07-11T22:39:33Z

@chlahav there seems to be a problem with the swagger, are the 2 api version specs compatible ?
The following error shows up when trying to generate the Go SDK:
FATAL: System.InvalidOperationException: Found incompatible property types , for property 'properties' in schema inheritance chain

lmazuel · 2018-07-11T23:38:15Z

@vladbarosan We try to move to language configuration in its own file readme.go.md. Could it be done this way? Thanks!
Not critical.
Example:
https://github.com/Azure/azure-rest-api-specs/blob/master/specification/cdn/resource-manager/readme.go.md

vladbarosan · 2018-07-11T23:48:45Z

@lmazuel sure will update either wont merge until the generation actually works

AutorestCI · 2018-07-13T21:45:55Z

AutoRest linter results for ARM Related Validation Errors/Warnings

These errors are reported by the ARM team's validation tools, reachout to ARM RP API Review directly for any questions or concerns.

File: specification/security/resource-manager/readme.md

⚠️

0 new Warnings.(1 total)

❌0 new Errors.(0 total)

AutoRest Linter Guidelines | AutoRest Linter Issues | Send feedback

Thanks for your co-operation.

AutorestCI · 2018-07-13T21:45:55Z

AutoRest linter results for SDK Related Validation Errors/Warnings

These errors are reported by the SDK team's validation tools, reachout to ADX Swagger Reviewers directly for any questions or concerns.

File: specification/security/resource-manager/readme.md

⚠️

0 new Warnings.(74 total)

❌0 new Errors.(0 total)

AutoRest Linter Guidelines | AutoRest Linter Issues | Send feedback

Thanks for your co-operation.

AutorestCI · 2018-07-13T21:58:29Z

AutoRest linter results for ARM Related Validation Errors/Warnings

These errors are reported by the ARM team's validation tools, reachout to ARM RP API Review directly for any questions or concerns.

File: specification/security/resource-manager/readme.md

⚠️

0 new Warnings.(1 total)

❌0 new Errors.(0 total)

AutoRest Linter Guidelines | AutoRest Linter Issues | Send feedback

Thanks for your co-operation.

AutorestCI · 2018-07-13T21:58:29Z

AutoRest linter results for SDK Related Validation Errors/Warnings

These errors are reported by the SDK team's validation tools, reachout to ADX Swagger Reviewers directly for any questions or concerns.

File: specification/security/resource-manager/readme.md

⚠️

0 new Warnings.(74 total)

❌0 new Errors.(0 total)

AutoRest Linter Guidelines | AutoRest Linter Issues | Send feedback

Thanks for your co-operation.

vladbarosan · 2018-07-13T22:32:15Z

@lmazuel is there a way to re trigger the go generation to see if it works ?

lmazuel · 2018-07-16T17:03:49Z

@AutorestCI rebuild azure-sdk-for-go

AutorestCI · 2018-07-16T17:04:12Z

Encountered a Subprocess error

Command: ['/usr/local/bin/autorest', '/tmp/tmpyk8g9yg0/rest/specification/security/resource-manager/readme.md', '--go', '--go-sdk-folder=/tmp/tmpyk8g9yg0/src/github.com/Azure/azure-sdk-for-go', '--multiapi', '--use=@microsoft.azure/autorest.go@~2.1.105', '--use-onever', '--verbose']
Finished with return code 1
and output:

AutoRest code generation utility [version: 2.0.4280; node: v8.11.3]
(C) 2018 Microsoft Corporation.
https://aka.ms/autorest
   Loading AutoRest core      '/root/.autorest/@microsoft.azure_autorest-core@2.0.4280/node_modules/@microsoft.azure/autorest-core/dist' (2.0.4280)
   Including configuration file 'file:///tmp/tmpyk8g9yg0/rest/specification/security/resource-manager/readme.go.md'
   Loading AutoRest extension '@microsoft.azure/autorest.go' (~2.1.105->2.1.105)
   Loading AutoRest extension '@microsoft.azure/autorest.modeler' (2.3.38->2.3.38)
Processing batch task - {"tag":"package-composite-v1"} .
   Including configuration file 'file:///tmp/tmpyk8g9yg0/rest/specification/security/resource-manager/readme.go.md'
FATAL: System.InvalidOperationException: Found incompatible property types ,  for property 'properties' in schema inheritance chain
   at AutoRest.Modeler.SchemaResolver.ExpandAllOf(Schema schema) in C:\Users\ci\AppData\Local\Temp\PUBLISH8kiit\38_20171116T010915\autorest.modeler\src\SchemaResolver.cs:line 157
   at AutoRest.Modeler.SchemaResolver.Unwrap(Schema schema) in C:\Users\ci\AppData\Local\Temp\PUBLISH8kiit\38_20171116T010915\autorest.modeler\src\SchemaResolver.cs:line 75
   at AutoRest.Modeler.SchemaBuilder.BuildServiceType(String serviceTypeName, Boolean required) in C:\Users\ci\AppData\Local\Temp\PUBLISH8kiit\38_20171116T010915\autorest.modeler\src\SchemaBuilder.cs:line 35
   at AutoRest.Modeler.SwaggerModeler.BuildCompositeTypes() in C:\Users\ci\AppData\Local\Temp\PUBLISH8kiit\38_20171116T010915\autorest.modeler\src\SwaggerModeler.cs:line 271
   at AutoRest.Modeler.SwaggerModeler.Build(ServiceDefinition serviceDefinition) in C:\Users\ci\AppData\Local\Temp\PUBLISH8kiit\38_20171116T010915\autorest.modeler\src\SwaggerModeler.cs:line 66
   at AutoRest.Modeler.Program.<ProcessInternal>d__2.MoveNext() in C:\Users\ci\AppData\Local\Temp\PUBLISH8kiit\38_20171116T010915\autorest.modeler\src\Program.cs:line 60
--- 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 NewPlugin.<Process>d__15.MoveNext()
FATAL: go/imodeler1 - FAILED
FATAL: Error: Plugin imodeler1 reported failure.
Process() cancelled due to exception : Plugin imodeler1 reported failure.
Failure during batch task - {"tag":"package-composite-v1"} -- Error: Plugin imodeler1 reported failure..
  Error: Plugin imodeler1 reported failure.

lmazuel · 2018-07-16T17:04:46Z

@vladbarosan it seems it doesn't :(
(if you didn't notice, you can click on "Encountered a Subprocess error" to get the full trace)

vladbarosan · 2018-07-16T17:21:52Z

@lmazuel thanks for the command. Yes I noticed the problem is that it seems to be a modeler error which seems to point to a swagger issue FATAL: System.InvalidOperationException: Found incompatible property types , for property 'properties' in schema inheritance chain. @chlahav can you help here ?

vladbarosan · 2018-07-24T03:40:41Z

Merging and #3453 should fix generation issues

azuresdkci assigned lmazuel Jul 11, 2018

azuresdkci requested a review from lmazuel July 11, 2018 21:17

vladbarosan requested review from chlahav and jhendrixMSFT July 11, 2018 21:17

jhendrixMSFT reviewed Jul 11, 2018

View reviewed changes

specification/security/resource-manager/readme.md Outdated

Copy link
Copy Markdown

Member

jhendrixMSFT Jul 11, 2018

Choose a reason for hiding this comment

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

Copy/paste error.

jhendrixMSFT reviewed Jul 11, 2018

View reviewed changes

jhendrixMSFT approved these changes Jul 11, 2018

View reviewed changes

chlahav approved these changes Jul 12, 2018

View reviewed changes

Vlad Barosan added 2 commits July 13, 2018 14:36

Add Go SDK generation config

80aecba

fix namespace for go

d789d99

vladbarosan force-pushed the securityGoSDK branch from 81aa75b to fbe3086 Compare July 13, 2018 21:41

Move go sdk config to its own file

f6eb179

vladbarosan force-pushed the securityGoSDK branch from fbe3086 to f6eb179 Compare July 13, 2018 21:50

vladbarosan merged commit c0cbb2c into master Jul 24, 2018

vladbarosan deleted the securityGoSDK branch October 29, 2018 20:40

Conversation

vladbarosan commented Jul 11, 2018 • edited by azuresdkci Loading Uh oh! There was an error while loading. Please reload this page.

Uh oh!

Uh oh!

AutorestCI commented Jul 11, 2018 • edited Loading Uh oh! There was an error while loading. Please reload this page.

Uh oh!

Automation for azure-sdk-for-python

Uh oh!

vladbarosan commented Jul 11, 2018

Uh oh!

jhendrixMSFT Jul 11, 2018

Choose a reason for hiding this comment

Uh oh!

AutorestCI commented Jul 11, 2018 • edited Loading Uh oh! There was an error while loading. Please reload this page.

Uh oh!

Automation for azure-sdk-for-ruby

Uh oh!

jhendrixMSFT Jul 11, 2018

Choose a reason for hiding this comment

Uh oh!

AutorestCI commented Jul 11, 2018 • edited Loading Uh oh! There was an error while loading. Please reload this page.

Uh oh!

Automation for azure-sdk-for-go

Uh oh!

AutorestCI commented Jul 11, 2018

AutoRest linter results for ARM Related Validation Errors/Warnings

Uh oh!

AutorestCI commented Jul 11, 2018

AutoRest linter results for SDK Related Validation Errors/Warnings

Uh oh!

AutorestCI commented Jul 11, 2018 • edited Loading Uh oh! There was an error while loading. Please reload this page.

Uh oh!

Automation for azure-sdk-for-node

Uh oh!

AutorestCI commented Jul 11, 2018 • edited Loading Uh oh! There was an error while loading. Please reload this page.

Uh oh!

Automation for azure-sdk-for-java

Uh oh!

AutorestCI commented Jul 11, 2018

AutoRest linter results for SDK Related Validation Errors/Warnings

Uh oh!

AutorestCI commented Jul 11, 2018

AutoRest linter results for ARM Related Validation Errors/Warnings

Uh oh!

vladbarosan commented Jul 11, 2018

Uh oh!

lmazuel commented Jul 11, 2018 • edited Loading Uh oh! There was an error while loading. Please reload this page.

Uh oh!

Uh oh!

vladbarosan commented Jul 11, 2018

Uh oh!

AutorestCI commented Jul 13, 2018

AutoRest linter results for ARM Related Validation Errors/Warnings

Uh oh!

AutorestCI commented Jul 13, 2018

AutoRest linter results for SDK Related Validation Errors/Warnings

Uh oh!

AutorestCI commented Jul 13, 2018

AutoRest linter results for ARM Related Validation Errors/Warnings

Uh oh!

AutorestCI commented Jul 13, 2018

AutoRest linter results for SDK Related Validation Errors/Warnings

Uh oh!

vladbarosan commented Jul 13, 2018

Uh oh!

lmazuel commented Jul 16, 2018

Uh oh!

AutorestCI commented Jul 16, 2018

Uh oh!

lmazuel commented Jul 16, 2018 • edited Loading Uh oh! There was an error while loading. Please reload this page.

Uh oh!

Uh oh!

vladbarosan commented Jul 16, 2018 • edited Loading Uh oh! There was an error while loading. Please reload this page.

Uh oh!

Uh oh!

vladbarosan commented Jul 24, 2018

Uh oh!

Reviewers

Assignees

Labels

vladbarosan commented Jul 11, 2018 •

edited by azuresdkci

Loading

AutorestCI commented Jul 11, 2018 •

edited

Loading

AutorestCI commented Jul 11, 2018 •

edited

Loading

AutorestCI commented Jul 11, 2018 •

edited

Loading

AutorestCI commented Jul 11, 2018 •

edited

Loading

AutorestCI commented Jul 11, 2018 •

edited

Loading

lmazuel commented Jul 11, 2018 •

edited

Loading

lmazuel commented Jul 16, 2018 •

edited

Loading

vladbarosan commented Jul 16, 2018 •

edited

Loading