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

Incorrect ref in mixedreality #8863

Open
lmazuel opened this issue Mar 27, 2020 · 8 comments
Open

Incorrect ref in mixedreality #8863

lmazuel opened this issue Mar 27, 2020 · 8 comments
Labels
Mgmt This issue is related to a management-plane library. Mixed Reality Service Attention Workflow: This issue is responsible by Azure service team.

Comments

@lmazuel
Copy link
Member

lmazuel commented Mar 27, 2020

Error: $ref to original location 'file:///C:/Users/lmazuel/Git/azure-rest-api-specs/specification/mixedreality/resource-manager/Microsoft.MixedReality/preview/2019-12-02-preview/common.json#/components/parameters/accountKeyRegenerateParameter' is not found in the new refs collection

@yungezz
Copy link
Member

yungezz commented Apr 1, 2020

this need Swagger fixing. but didn't find mixed relatity service attention label, will follow to add one

@yungezz yungezz added the Service Attention Workflow: This issue is responsible by Azure service team. label Apr 1, 2020
@triage-new-issues triage-new-issues bot removed the triage label Apr 1, 2020
@XiangyuL-Microsoft
Copy link
Contributor

@akning-ms Hi, Arthur, May I ask what I should do for this issue, please?

@akning-ms
Copy link
Contributor

HI @XiangyuL-Microsoft, You should follow the comment to fix the swagger issue. or re-assign to correct person in your team. seems there is a ref is missing or point to wrong place

@XiangyuL-Microsoft
Copy link
Contributor

@lmazuel could you please explain a little about the issue? It seems that it happens on your local machine? Honestly, I have no idea how to fix it for you. Thank you very much.

@lmazuel
Copy link
Member Author

lmazuel commented May 5, 2020

Actually, I start to wonder if this could be an autorest v3 false positive, I'm discussing this with @daviwil right now about this. Two things here:

  • Either we see it's a false positive and we will close the issue
  • Either we get more details, and I will update the issue to pinpoint the exact trouble

In the meantime, this should not be considered blocking for anything release or process.

@daviwil
Copy link
Contributor

daviwil commented May 6, 2020

This seems to be a bug in AutoRest Core, I believe the $ref is specified correctly in the Swagger. Working on a fix now.

@lmazuel lmazuel removed the Service Attention Workflow: This issue is responsible by Azure service team. label May 12, 2020
@akning-ms akning-ms added the Mgmt This issue is related to a management-plane library. label Jun 25, 2020
@akning-ms
Copy link
Contributor

Hi @daviwil, as it is a bug for autorest core, can you just give a link here to track the bug? so we can close issue in swagger repo.

@daviwil
Copy link
Contributor

daviwil commented Jul 17, 2020

Here it is: Azure/autorest#3546

@JackTn JackTn added the Service Attention Workflow: This issue is responsible by Azure service team. label Jan 8, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Mgmt This issue is related to a management-plane library. Mixed Reality Service Attention Workflow: This issue is responsible by Azure service team.
Projects
None yet
Development

No branches or pull requests

6 participants