Avocado validate data-plane readme shouldn't have azure-arm tag

[tianxchen-ms]

From: Mike Kistler <[email protected]>

Date: Thursday, April 28, 2022 at 11:16 AM
To: Anne Loomis Thompson <[email protected]>, Shivangi Reja <[email protected]>
Cc: Alexander Sher <[email protected]>
Subject: Re: azure-arm

Anne, Shivangi,

This is good to know! It looks like these files were copy/pasted over from the mgmt plane directory without really understanding what they do or whether they are needed.

With this knowledge, I will start looking for these in API reviews and questioning whether they are needed and whether they should be deleted.

Thanks!

Mike

From: Shivangi Reja <[email protected]>
Date: Wednesday, April 27, 2022 at 10:35 PM
To: Mike Kistler <[email protected]>, Anne Loomis Thompson <[email protected]>
Cc: Alexander Sher <[email protected]>
Subject: RE: azure-arm

Hi @Mike,

We don’t use readme.csharp.md config for any track-2 SDK in azure-sdk-for-net repo. We either use readme.md with tag(ex: autorest.md) or we use directly the swagger file(ex: autorest.md) in autorest.md for all the track-2 SDKs.

Also, we don’t use azure-arm: true flag for any dataplane libraries. This flag is used to generate management plane SDK.

Thanks,

Shivangi

From: Mike Kistler <[email protected]>
Sent: Wednesday, April 27, 2022 7:17 PM
To: Shivangi Reja <[email protected]>; Anne Loomis Thompson <[email protected]>
Subject: Re: azure-arm

I’m asking because I see this flag being used in a number of data-plane services:

Would you agree that these are “bugs” and should be fixed?

Mike