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

JS Automation Tool Fails to Process Multiple Packages #9829

Open
raych1 opened this issue Feb 13, 2025 · 4 comments · Fixed by #9851 · May be fixed by #9857
Open

JS Automation Tool Fails to Process Multiple Packages #9829

raych1 opened this issue Feb 13, 2025 · 4 comments · Fixed by #9851 · May be fixed by #9857
Assignees
Labels
Automation Tool Mgmt-EngSys Engineering System (Management Plane Specific)

Comments

@raych1
Copy link
Member

raych1 commented Feb 13, 2025

This is a batch run for all the typespecs. Part of the errors are as follows:

Err: Error: Find unexpected changed package directory. Length: 4. Value: sdk/documentintelligence/ai-document-intelligence-rest, sdk/face/ai-vision-face-rest, sdk/vision/ai-vision-image-analysis-rest, sdk/healthinsights/azurehealthinsightsradiologyinsights-rest. Please only change files in one directory
2025-02-13T19:19:27.8957298Z 07:19:27.895 �[4m�[32mcmdout�[39m�[24m 	[automation_generate.sh] Stderr: "undefined"
2025-02-13T19:19:27.8958200Z 07:19:27.895 �[4m�[32mcmdout�[39m�[24m 	[automation_generate.sh] Stdout: "undefined"
2025-02-13T19:19:27.8960076Z 07:19:27.895 �[4m�[32mcmdout�[39m�[24m 	[automation_generate.sh] ErrorStack: "Error: Find unexpected changed package directory. Length: 4. Value: sdk/documentintelligence/ai-document-intelligence-rest, sdk/face/ai-vision-face-rest, sdk/vision/ai-vision-image-analysis-rest, sdk/healthinsights/azurehealthinsightsradiologyinsights-rest. Please only change files in one directory
2025-02-13T19:19:27.8961798Z 07:19:27.895 �[4m�[32mcmdout�[39m�[24m 	[automation_generate.sh]     at generateRLCInPipeline (/opt/hostedtoolcache/node/22.13.0/x64/lib/node_modules/@azure-tools/js-sdk-release-tools/dist/llc/generateRLCInPipeline/generateRLCInPipeline.js:184:23)
2025-02-13T19:19:27.8963516Z 07:19:27.895 �[4m�[32mcmdout�[39m�[24m 	[automation_generate.sh]     at process.processTicksAndRejections (node:internal/process/task_queues:105:5)
2025-02-13T19:19:27.8964776Z 07:19:27.896 �[4m�[32mcmdout�[39m�[24m 	[automation_generate.sh]     at async automationGenerateInPipeline (/opt/hostedtoolcache/node/22.13.0/x64/lib/node_modules/@azure-tools/js-sdk-release-tools/dist/autoGenerateInPipeline.js:38:17)".
2025-02-13T19:19:27.8968675Z ##[error][ERROR]: Please check out https://github.com/Azure/autorest.typescript/blob/main/packages/typespec-ts/CONTRIBUTING.md#how-to-debug to troubleshoot the issue.
2025-02-13T19:19:27.8973035Z ##[error][ERROR]: Failed to generate SDK for package 'default' due to Error: Find unexpected changed package directory. Length: 4. Value: sdk/documentintelligence/ai-document-intelligence-rest, sdk/face/ai-vision-face-rest, sdk/vision/ai-vision-image-analysis-rest, sdk/healthinsights/azurehealthinsightsradiologyinsights-rest. Please only change files in one directory
2025-02-13T19:19:27.8974813Z 07:19:27.896 �[4m�[32mcmdout�[39m�[24m 	[automation_generate.sh]     at generateRLCInPipeline (/opt/hostedtoolcache/node/22.13.0/x64/lib/node_modules/@azure-tools/js-sdk-release-tools/dist/llc/generateRLCInPipeline/generateRLCInPipeline.js:184:23)
2025-02-13T19:19:27.8975745Z 07:19:27.896 �[4m�[32mcmdout�[39m�[24m 	[automation_generate.sh]     at process.processTicksAndRejections (node:internal/process/task_queues:105:5)
2025-02-13T19:19:27.8976742Z 07:19:27.896 �[4m�[32mcmdout�[39m�[24m 	[automation_generate.sh]     at async automationGenerateInPipeline (/opt/hostedtoolcache/node/22.13.0/x64/lib/node_modules/@azure-tools/js-sdk-release-tools/dist/autoGenerateInPipeline.js:38:17).
@raych1 raych1 added Automation Tool Mgmt-EngSys Engineering System (Management Plane Specific) labels Feb 13, 2025
@raych1 raych1 changed the title Failure in processing multiple packages scenario for JS automation tool JS Automation Tool Fails to Process Multiple Packages Feb 14, 2025
@github-project-automation github-project-automation bot moved this from Backlog to Done in API to SDK automation Feb 18, 2025
@wanlwanl
Copy link
Member

Hi @raych1, I remove the restriction that only allow 1 changed folder, so that js sdk automation supports multiple sdk automation.
Could you share how to run sdk automation in batch? Do you clone one repo for each RP? I'm asking this since JS sdk automation need to update rush.json which is in root folder, that all RPs might need to update it, if all RPs' sdk automation are run in the same repo, I may need to make it a critial section for this file

@raych1
Copy link
Member Author

raych1 commented Feb 18, 2025

@wanlwanl , batch run clones the repo once then generates SDK for each RP sequentially.

@wanlwanl
Copy link
Member

Great, I don't need to update the gneration script.

@wanlwanl wanlwanl linked a pull request Feb 19, 2025 that will close this issue
@wanlwanl
Copy link
Member

Reopen due to #9857

@wanlwanl wanlwanl reopened this Feb 19, 2025
@github-project-automation github-project-automation bot moved this from Done to Backlog in API to SDK automation Feb 19, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Automation Tool Mgmt-EngSys Engineering System (Management Plane Specific)
Projects
Status: Backlog
2 participants