search

Azure / azure-openapi-validator

Azure Open API Validator
MIT License
44 stars 43 forks source link

LintDiff Process canceled due to failure could not read file #673

Closed TimLovellSmith closed 3 months ago

TimLovellSmith commented 4 months ago

Before filling a bug

Describe the bug

Automatic LintDiff continuous integration is giving errors in the following PR.

https://github.com/Azure/azure-rest-api-specs-pr/pull/16988

❌ AutoRest exception | "tag":"default","details":"Process() cancelled due to failure" |   ❌ AutoRest exception | "tag":"default","details":"!Error: Could not read 'file:///mnt/vss/_work/1/azure-rest-api-specs-pr/specification/windowsesu/resource-manager/Microsoft.WindowsESU/preview/2019-09-16/windowsesu.json'." |   ❌ AutoRest exception | "tag":"default","details":"Autorest completed with an error. If you think the error message is unclear,or is a bug,please declare an issues at https://github.com/Azure/autorest/issues with the error message you are seeing."

Please provide as much information as you can. This would be:
- OpenAPI files having the issues
- Autorest command used

Expected behavior It should instead give normal linter output shouldn't it? At least the file looks to be part of the PR.

Additional context Add any other context about the problem here. [must fix]The following errors/warnings are introduced by current PR

renesc-ms commented 4 months ago

This issue also occurred for PR https://github.com/Azure/azure-rest-api-specs-pr/pull/16561.

mikeharder commented 3 months ago

The first issue was a bug in the spec:

https://github.com/Azure/azure-rest-api-specs-pr/blob/983cac0c78ad3f65ea4ce550da1f0e05de108856/specification/windowsesu/resource-manager/readme.md?plain=1#L38

readme.md pointed to an input file that did not exist.

Appears this was fixed before the PR was merged.

mikeharder commented 3 months ago

@renesc-ms: Your spec has a similar bug and was already merged to RPSaaSDev.

https://github.com/Azure/azure-rest-api-specs-pr/blob/8ee60dd313c05717d1c291256405bb1d598354a9/specification/supplychain/resource-manager/readme.md#L38

Your readme.md points to an input file that doesn't exist. The version should be 2023-02-22-preview instead of 2023-02-22.

renesc-ms commented 3 months ago

@mikeharder Thank you for your follow-up which clarifies the issue for us to address.