Azure / azure-sdk-tools

Tools repository leveraged by the Azure SDK team.
MIT License
110 stars 172 forks source link

[SDK Automation] Establish base line for SDK automation for TypeSpecs #8858

Open raych1 opened 3 weeks ago

raych1 commented 3 weeks ago

Objective Establish a baseline for the SDK automation results, which will be used as a reference for future SDK automation runs.

Task

Spec PRs created covering all TypeSpecs https://github.com/test-repo-billy/azure-rest-api-specs/pull/3481 https://github.com/test-repo-billy/azure-rest-api-specs/pull/3482 https://github.com/test-repo-billy/azure-rest-api-specs/pull/3485 https://github.com/test-repo-billy/azure-rest-api-specs/pull/3486 https://github.com/test-repo-billy/azure-rest-api-specs/pull/3487 https://github.com/test-repo-billy/azure-rest-api-specs/pull/3489 https://github.com/test-repo-billy/azure-rest-api-specs/pull/3490

msyyc commented 3 weeks ago

Python fails for: https://github.com/test-repo-billy/azure-rest-api-specs/pull/3485/checks?check_run_id=28928592993 (fixed) https://github.com/test-repo-billy/azure-rest-api-specs/pull/3487/checks?check_run_id=28932298671 (fixed by https://github.com/Azure/azure-rest-api-specs/pull/30335) https://github.com/test-repo-billy/azure-rest-api-specs/pull/3489/checks?check_run_id=28975547996 (fixed by https://github.com/Azure/azure-rest-api-specs/pull/30336)

wanlwanl commented 3 weeks ago

JS SDK Generation Test Summary

sdk sutomation stage may mark success even though it contains generation failures.

Batches Status

Invalid emitter settings

Lack of required options. The correct settings should be:

Required Shared Parameters

e.g.

parameters:
  "service-dir":
    default: "sdk/openai"

If folders are shared between package, define e.g.

"dependencies":
"additionalDirectories":
- "specification/contosowidgetmanager/Contoso.WidgetManager.Shared/"
default: ""

Modular

e.g.

  "@azure-tools/typespec-ts":
    generateMetadata: true
    generateSample: true
    generateTest: true
    azureSdkForJs: true
    isModularLibrary: true
    experimentalExtensibleEnums: true
    enableOperationGroup: true
    hierarchyClient: false
    package-dir: "openai-assistants"
    packageDetails:
      name: "@azure/openai-assistants"
    flavor: azure

RLC

e.g.

  "@azure-tools/typespec-ts":
    generateMetadata: true
    generateSample: true
    generateTest: true
    azureSdkForJs: true
    package-dir: "ai-content-safety-rest"
    packageDetails:
      name: "@azure-rest/ai-content-safety"
      version: "1.0.0"
      description: "ContentSafety Service"
    flavor: azure