-
Notifications
You must be signed in to change notification settings - Fork 5.1k
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
Fixing the spec for DSCConfigurations. #3749
Conversation
Automation for azure-sdk-for-javaNothing to generate for azure-sdk-for-java |
Automation for azure-sdk-for-pythonThe initial PR has been merged into your service PR: |
Automation for azure-sdk-for-rubyThe initial PR has been merged into your service PR: |
The serialization issue in dotnet/Go SDK is mentioned in details here (slack link), but tl;dr is that Autorest generates code which expects the output schema to be json and we return file. This causes serialization issues and is breaking out current SDK users (our SDK is still in the preview though). |
Automation for azure-sdk-for-goThe initial PR has been merged into your service PR: |
Automation for azure-sdk-for-nodeThe initial PR has been merged into your service PR: |
Can one of the admins verify this patch? |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
This is causing the linter to fail on the example. See the log. I think the body needs to be tweaked slightly, see https://github.com/Azure/azure-rest-api-specs/blob/master/specification/automation/resource-manager/Microsoft.Automation/stable/2015-10-31/examples/getRunbookDraftContent.json which is very similar.
@jhendrixMSFT I'll take a look and send the fix asap. |
@jhendrixMSFT The problem is with Autorest as well as our spec. We could have done better design and returned a JSON response. Also, autorest does not understand schema of When I run
For DSCConfiguration
|
Description
The correct type was not being returned. This is in line with the fixes done in #3363.
PR information
api-version
in the path should match theapi-version
in the spec).Quality of Swagger