-
Notifications
You must be signed in to change notification settings - Fork 4.9k
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
[SDK Automation] Spec repo azure-sdk-for-net-track2 CI failed for new RP #46862
Comments
There are two problem:
|
Root cause: |
@chunyu3 , the policy requires all the new service use TypeSpec. Is this service an exception to use Swagger? |
@chunyu3 is this a bug in the .NET automation scripts? |
Current new service of management-plane only can be generated from Typespec, so the mgmt-plane SDK scaffolding is only for TypeSpec project. But this is a swagger project, and the swagger have existed for a long time, but it does not generate .NET SDK. We cannot identify it is a green field service or brown field service. We need to decide if we need to support scaffolding for this kind of project. If not, .NET automation will throw an error for this, skip the generation. |
@chunyu3 ok, this is the change needs to make in .NET automation script. What is the other issue you mentioned above for the SDK automation framework? Does it still exist? |
@raych1 The issue is because mgmt-plane will not support scaffolding for swagger project. So we will throw error if it is a swagger project but it is a new PR (no sdk package folder), and the whole .NET sdk automation still fail. The automation framework issue does not exist. |
When there is a new RP onboard in azure-rest-api-specs, the CI failure
https://dev.azure.com/azure-sdk/internal/_build/results?buildId=4270662&view=logs&j=03afb3bb-7296-55ad-aa07-ceee610c73b2&t=0f320c29-1c90-5304-d2d3-c58a6bb76437&l=306
The text was updated successfully, but these errors were encountered: