Fixing the DSC resource definitions

[vrdmr]

Both the resources are proxy resources and send all the Proxy Resource properties in the service. This change is to reflect the state of the service.

PR information

• The title of the PR is clear and informative.
• There are a small number of commits, each of which have an informative message. This means that previously merged commits do not appear in the history of the PR. For information on cleaning up the commits in your pull request, see this page.
• Except for special cases involving multiple contributors, the PR is started from a fork of the main repository, not a branch.
• If applicable, the PR references the bug/issue that it fixes.
• Swagger files are correctly named (e.g. the api-version in the path should match the api-version in the spec).

Quality of Swagger

• I have read the contribution guidelines.
• My spec meets the review criteria:
  • The spec conforms to the Swagger 2.0 specification.
  • The spec follows the guidelines described in the Swagger checklist document.
  • Validation tools were run on swagger spec(s) and have all been fixed in this PR.

[AutorestCI]

Automation for azure-sdk-for-go

Was unable to create SDK azure-sdk-for-go PR for this closed PR.

[AutorestCI]

Automation for azure-sdk-for-python

A PR has been created for you:
Azure/azure-sdk-for-python#1983

[lmazuel]

Disregard this approval...

[azuresdkciprbot]

Hi There,

I am the AutoRest Linter Azure bot. I am here to help. My task is to analyze the situation from the AutoRest linter perspective. Please review the below analysis result:

💡 Please review potentially introduced Error(s)/Warning(s): Analysis Report 💡

File: specification/automation/resource-manager/readme.md
Before the PR: Warning(s): 0 Error(s): 24
After the PR: Warning(s): 0 Error(s): 27

AutoRest Linter Guidelines | AutoRest Linter Issues | Send feedback

Thanks for your co-operation.

[lmazuel]

Duplicate properties

[lmazuel]

If adding proxyResource, please remove "name" and "id" (duplicate)

[vrdmr]

Thanks for catching this. I'll remove it.

[lmazuel]

If adding proxyResource, please remove "name" and "id" (duplicate)

[vrdmr]

Resolved.

[azuresdkciprbot]

Hi There,

I am the AutoRest Linter Azure bot. I am here to help. My task is to analyze the situation from the AutoRest linter perspective. Please review the below analysis result:

💡 Please review potentially introduced Error(s)/Warning(s): Analysis Report 💡

File: specification/automation/resource-manager/readme.md
Before the PR: Warning(s): 0 Error(s): 24
After the PR: Warning(s): 0 Error(s): 27

AutoRest Linter Guidelines | AutoRest Linter Issues | Send feedback

Thanks for your co-operation.

[lmazuel]

There is still linter issues, but I feel like the purpose of this specific PR is fine. Try to merge this as atomic improvement to the spec.

[vrdmr]

Thanks @lmazuel - We are in the process of fixing the Linter issues and would get them down to 0 soon.