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

Registries public preview 2022-10-01-preview #20200

Conversation

fahdkmsft
Copy link
Contributor

@fahdkmsft fahdkmsft commented Aug 11, 2022

MSFT employees can try out our new experience at OpenAPI Hub - one location for using our validation tools and finding your workflow.

Azure 1st Party Service can try out the Shift Left experience to initiate API design review from ADO code repo. If you are interested, may request engineering support by filling in with the form https://aka.ms/ShiftLeftSupportForm.

Changelog

Add a changelog entry for this PR by answering the following questions:

  1. What's the purpose of the update?
    • new service onboarding
    • new API version
    • update existing version for new feature
    • update existing version to fix swagger quality issue in s360
    • Other, please clarify
  2. When are you targeting to deploy the new service/feature to public regions? Please provide the date or, if the date is not yet available, the month.
  3. When do you expect to publish the swagger? Please provide date or, the the date is not yet available, the month.
  4. If updating an existing version, please select the specific language SDKs and CLIs that must be refreshed after the swagger is published.
    • SDK of .NET (need service team to ensure code readiness)
    • SDK of Python
    • SDK of Java
    • SDK of Js
    • SDK of Go
    • PowerShell
    • CLI
    • Terraform
    • No refresh required for updates in this PR

Contribution checklist:

If any further question about AME onboarding or validation tools, please view the FAQ.

ARM API Review Checklist

Applicability: ⚠️

If your changes encompass only the following scenarios, you should SKIP this section, as these scenarios do not require ARM review.

  • Change to data plane APIs
  • Adding new properties
  • All removals

Otherwise your PR may be subject to ARM review requirements. Complete the following:

  • Check this box if any of the following appy to the PR so that the label "ARMReview" and "WaitForARMFeedback" will be added by bot to kick off ARM API Review. Missing to check this box in the following scenario may result in delays to the ARM manifest review and deployment.

    • Adding a new service
    • Adding new API(s)
    • Adding a new API version
      -[ ] To review changes efficiently, ensure you are using OpenAPIHub to initialize the PR for adding a new version. More details, refer to the wiki.
  • Ensure you've reviewed following guidelines including ARM resource provider contract and REST guidelines. Estimated time (4 hours). This is required before you can request review from ARM API Review board.

  • If you are blocked on ARM review and want to get the PR merged with urgency, please get the ARM oncall for reviews (RP Manifest Approvers team under Azure Resource Manager service) from IcM and reach out to them.

Breaking Change Review Checklist

If any of the following scenarios apply to the PR, request approval from the Breaking Change Review Board as defined in the Breaking Change Policy.

  • Removing API(s) in a stable version
  • Removing properties in a stable version
  • Removing API version(s) in a stable version
  • Updating API in a stable or public preview version with Breaking Change Validation errors
  • Updating API(s) in public preview over 1 year (refer to Retirement of Previews)

Action: to initiate an evaluation of the breaking change, create a new intake using the template for breaking changes. Addition details on the process and office hours are on the Breaking change Wiki.

Please follow the link to find more details on PR review process.

@openapi-pipeline-app
Copy link

openapi-pipeline-app bot commented Aug 11, 2022

Swagger Validation Report

️️✔️BreakingChange succeeded [Detail] [Expand]
There are no breaking changes.
️⚠️Breaking Change(Cross-Version): 7 Warnings warning [Detail]
compared swaggers (via Oad v0.9.6)] new version base version
machineLearningServices.json 2022-10-01-preview(b70bb75) 2022-05-01(main)
machineLearningServices.json 2022-10-01-preview(b70bb75) 2022-06-01-preview(main)
mfe.json 2022-10-01-preview(b70bb75) 2022-05-01(main)
mfe.json 2022-10-01-preview(b70bb75) 2022-06-01-preview(main)
workspaceFeatures.json 2022-10-01-preview(b70bb75) 2022-05-01(main)
workspaceFeatures.json 2022-10-01-preview(b70bb75) 2022-06-01-preview(main)

The following breaking changes are detected by comparison with the latest stable version:

Rule Message
⚠️ 1032 - DifferentAllOf The new version has a different 'allOf' property than the previous one.
New: Microsoft.MachineLearningServices/preview/2022-10-01-preview/mfe.json#L6729:9
Old: Microsoft.MachineLearningServices/stable/2022-05-01/mfe.json#L5815:9
⚠️ 1032 - DifferentAllOf The new version has a different 'allOf' property than the previous one.
New: Microsoft.MachineLearningServices/preview/2022-10-01-preview/mfe.json#L6393:5
Old: Microsoft.MachineLearningServices/stable/2022-05-01/mfe.json#L5508:5
⚠️ 1032 - DifferentAllOf The new version has a different 'allOf' property than the previous one.
New: Microsoft.MachineLearningServices/preview/2022-10-01-preview/mfe.json#L6456:5
Old: Microsoft.MachineLearningServices/stable/2022-05-01/mfe.json#L5568:5
⚠️ 1032 - DifferentAllOf The new version has a different 'allOf' property than the previous one.
New: Microsoft.MachineLearningServices/preview/2022-10-01-preview/mfe.json#L6493:5
Old: Microsoft.MachineLearningServices/stable/2022-05-01/mfe.json#L5602:5
⚠️ 1032 - DifferentAllOf The new version has a different 'allOf' property than the previous one.
New: Microsoft.MachineLearningServices/preview/2022-10-01-preview/mfe.json#L6577:5
Old: Microsoft.MachineLearningServices/stable/2022-05-01/mfe.json#L5666:5
⚠️ 1032 - DifferentAllOf The new version has a different 'allOf' property than the previous one.
New: Microsoft.MachineLearningServices/preview/2022-10-01-preview/mfe.json#L7633:9
Old: Microsoft.MachineLearningServices/stable/2022-05-01/mfe.json#L6388:9
⚠️ 1032 - DifferentAllOf The new version has a different 'allOf' property than the previous one.
New: Microsoft.MachineLearningServices/preview/2022-10-01-preview/mfe.json#L15198:9
Old: Microsoft.MachineLearningServices/stable/2022-05-01/mfe.json#L9911:9
️⚠️LintDiff: 21 Warnings warning [Detail]
compared tags (via openapi-validator v1.13.0) new version base version
package-preview-2022-10 package-preview-2022-10(b70bb75) package-preview-2022-10(dev-machinelearningservices-Microsoft.MachineLearningServices-2022-10-01-preview)

The following errors/warnings are introduced by current PR:

Rule Message
⚠️ R2001 - AvoidNestedProperties Consider using x-ms-client-flatten to provide a better end user experience
Location: Microsoft.MachineLearningServices/preview/2022-10-01-preview/registries.json#L429
⚠️ R2001 - AvoidNestedProperties Consider using x-ms-client-flatten to provide a better end user experience
Location: Microsoft.MachineLearningServices/preview/2022-10-01-preview/registries.json#L599
⚠️ R2001 - AvoidNestedProperties Consider using x-ms-client-flatten to provide a better end user experience
Location: Microsoft.MachineLearningServices/preview/2022-10-01-preview/registries.json#L637
⚠️ R3018 - EnumInsteadOfBoolean Booleans are not descriptive and make them hard to use. Consider using string enums with allowed set of values defined. Property: storageAccountHnsEnabled
Location: Microsoft.MachineLearningServices/preview/2022-10-01-preview/registries.json#L819
⚠️ R3027 - TrackedResourceListByResourceGroup The tracked resource, 'RegistryTrackedResource', must have a list by resource group operation.(This rule does not apply for tenant level resources.)
Location: Microsoft.MachineLearningServices/preview/2022-10-01-preview/registries.json#L575
⚠️ R4002 - LocationMustHaveXmsMutability Property 'location' must have ''x-ms-mutability':['read', 'create']' extension defined. Resource Model: 'RegistryTrackedResource'
Location: Microsoft.MachineLearningServices/preview/2022-10-01-preview/registries.json#L380
⚠️ R4021 - DescriptionAndTitleMissing 'PartialUserAssignedIdentity' model/property lacks 'description' and 'title' property. Consider adding a 'description'/'title' element. Accurate description/title is essential for maintaining reference documentation.
Location: Microsoft.MachineLearningServices/preview/2022-10-01-preview/registries.json#L466
⚠️ R4021 - DescriptionAndTitleMissing 'publicNetworkAccess' model/property lacks 'description' and 'title' property. Consider adding a 'description'/'title' element. Accurate description/title is essential for maintaining reference documentation.
Location: Microsoft.MachineLearningServices/preview/2022-10-01-preview/registries.json#L507
⚠️ R4021 - DescriptionAndTitleMissing 'discoveryUrl' model/property lacks 'description' and 'title' property. Consider adding a 'description'/'title' element. Accurate description/title is essential for maintaining reference documentation.
Location: Microsoft.MachineLearningServices/preview/2022-10-01-preview/registries.json#L511
⚠️ R4021 - DescriptionAndTitleMissing 'intellectualPropertyPublisher' model/property lacks 'description' and 'title' property. Consider adding a 'description'/'title' element. Accurate description/title is essential for maintaining reference documentation.
Location: Microsoft.MachineLearningServices/preview/2022-10-01-preview/registries.json#L515
⚠️ R4021 - DescriptionAndTitleMissing 'mlFlowRegistryUri' model/property lacks 'description' and 'title' property. Consider adding a 'description'/'title' element. Accurate description/title is essential for maintaining reference documentation.
Location: Microsoft.MachineLearningServices/preview/2022-10-01-preview/registries.json#L524
⚠️ R4021 - DescriptionAndTitleMissing 'privateLinkCount' model/property lacks 'description' and 'title' property. Consider adding a 'description'/'title' element. Accurate description/title is essential for maintaining reference documentation.
Location: Microsoft.MachineLearningServices/preview/2022-10-01-preview/registries.json#L528
⚠️ R4021 - DescriptionAndTitleMissing 'RegistryTrackedResource' model/property lacks 'description' and 'title' property. Consider adding a 'description'/'title' element. Accurate description/title is essential for maintaining reference documentation.
Location: Microsoft.MachineLearningServices/preview/2022-10-01-preview/registries.json#L575
⚠️ R4021 - DescriptionAndTitleMissing 'ResourceBase' model/property lacks 'description' and 'title' property. Consider adding a 'description'/'title' element. Accurate description/title is essential for maintaining reference documentation.
Location: Microsoft.MachineLearningServices/preview/2022-10-01-preview/registries.json#L629
⚠️ R4021 - DescriptionAndTitleMissing 'SystemCreatedAcrAccount' model/property lacks 'description' and 'title' property. Consider adding a 'description'/'title' element. Accurate description/title is essential for maintaining reference documentation.
Location: Microsoft.MachineLearningServices/preview/2022-10-01-preview/registries.json#L796
⚠️ R4021 - DescriptionAndTitleMissing 'acrAccountSku' model/property lacks 'description' and 'title' property. Consider adding a 'description'/'title' element. Accurate description/title is essential for maintaining reference documentation.
Location: Microsoft.MachineLearningServices/preview/2022-10-01-preview/registries.json#L799
⚠️ R4021 - DescriptionAndTitleMissing 'SystemCreatedStorageAccount' model/property lacks 'description' and 'title' property. Consider adding a 'description'/'title' element. Accurate description/title is essential for maintaining reference documentation.
Location: Microsoft.MachineLearningServices/preview/2022-10-01-preview/registries.json#L811
⚠️ R4021 - DescriptionAndTitleMissing 'storageAccountHnsEnabled' model/property lacks 'description' and 'title' property. Consider adding a 'description'/'title' element. Accurate description/title is essential for maintaining reference documentation.
Location: Microsoft.MachineLearningServices/preview/2022-10-01-preview/registries.json#L819
⚠️ R4021 - DescriptionAndTitleMissing 'UserCreatedAcrAccount' model/property lacks 'description' and 'title' property. Consider adding a 'description'/'title' element. Accurate description/title is essential for maintaining reference documentation.
Location: Microsoft.MachineLearningServices/preview/2022-10-01-preview/registries.json#L830
⚠️ R4021 - DescriptionAndTitleMissing 'UserCreatedStorageAccount' model/property lacks 'description' and 'title' property. Consider adding a 'description'/'title' element. Accurate description/title is essential for maintaining reference documentation.
Location: Microsoft.MachineLearningServices/preview/2022-10-01-preview/registries.json#L841
⚠️ R4030 - UniqueXmsExample Do not have duplicate name of x-ms-example, make sure every x-ms-example name unique. Duplicate x-ms-example: List registries.
Location: Microsoft.MachineLearningServices/preview/2022-10-01-preview/registries.json#L83


The following errors/warnings exist before current PR submission:

Only 30 items are listed, please refer to log for more details.

Rule Message
R2008 - MutabilityWithReadOnly When property is modeled as 'readOnly': true then x-ms-mutability extension can only have 'read' value. When property is modeled as 'readOnly': false then applying x-ms-mutability extension with only 'read' value is not allowed. Extension contains invalid values: 'read'.
Location: Microsoft.MachineLearningServices/preview/2022-10-01-preview/mfe.json#L6725
R2008 - MutabilityWithReadOnly When property is modeled as 'readOnly': true then x-ms-mutability extension can only have 'read' value. When property is modeled as 'readOnly': false then applying x-ms-mutability extension with only 'read' value is not allowed. Extension contains invalid values: 'read'.
Location: Microsoft.MachineLearningServices/preview/2022-10-01-preview/mfe.json#L6811
R2008 - MutabilityWithReadOnly When property is modeled as 'readOnly': true then x-ms-mutability extension can only have 'read' value. When property is modeled as 'readOnly': false then applying x-ms-mutability extension with only 'read' value is not allowed. Extension contains invalid values: 'read'.
Location: Microsoft.MachineLearningServices/preview/2022-10-01-preview/mfe.json#L8963
R2008 - MutabilityWithReadOnly When property is modeled as 'readOnly': true then x-ms-mutability extension can only have 'read' value. When property is modeled as 'readOnly': false then applying x-ms-mutability extension with only 'read' value is not allowed. Extension contains invalid values: 'read'.
Location: Microsoft.MachineLearningServices/preview/2022-10-01-preview/mfe.json#L10681
R2008 - MutabilityWithReadOnly When property is modeled as 'readOnly': true then x-ms-mutability extension can only have 'read' value. When property is modeled as 'readOnly': false then applying x-ms-mutability extension with only 'read' value is not allowed. Extension contains invalid values: 'read'.
Location: Microsoft.MachineLearningServices/preview/2022-10-01-preview/mfe.json#L11503
R2008 - MutabilityWithReadOnly When property is modeled as 'readOnly': true then x-ms-mutability extension can only have 'read' value. When property is modeled as 'readOnly': false then applying x-ms-mutability extension with only 'read' value is not allowed. Extension contains invalid values: 'read'.
Location: Microsoft.MachineLearningServices/preview/2022-10-01-preview/mfe.json#L11521
R2008 - MutabilityWithReadOnly When property is modeled as 'readOnly': true then x-ms-mutability extension can only have 'read' value. When property is modeled as 'readOnly': false then applying x-ms-mutability extension with only 'read' value is not allowed. Extension contains invalid values: 'read'.
Location: Microsoft.MachineLearningServices/preview/2022-10-01-preview/mfe.json#L12407
R2008 - MutabilityWithReadOnly When property is modeled as 'readOnly': true then x-ms-mutability extension can only have 'read' value. When property is modeled as 'readOnly': false then applying x-ms-mutability extension with only 'read' value is not allowed. Extension contains invalid values: 'read'.
Location: Microsoft.MachineLearningServices/preview/2022-10-01-preview/mfe.json#L12507
R2008 - MutabilityWithReadOnly When property is modeled as 'readOnly': true then x-ms-mutability extension can only have 'read' value. When property is modeled as 'readOnly': false then applying x-ms-mutability extension with only 'read' value is not allowed. Extension contains invalid values: 'read'.
Location: Microsoft.MachineLearningServices/preview/2022-10-01-preview/mfe.json#L13613
R2008 - MutabilityWithReadOnly When property is modeled as 'readOnly': true then x-ms-mutability extension can only have 'read' value. When property is modeled as 'readOnly': false then applying x-ms-mutability extension with only 'read' value is not allowed. Extension contains invalid values: 'read'.
Location: Microsoft.MachineLearningServices/preview/2022-10-01-preview/mfe.json#L14431
R3008 - CollectionObjectPropertiesNaming Collection object 'AmlComputeNodesInformation' returned by list operation 'Compute_ListNodes' with 'x-ms-pageable' extension, has no property named 'value'.
Location: Microsoft.MachineLearningServices/preview/2022-10-01-preview/machineLearningServices.json#L1086
⚠️ R1003 - ListInOperationName Since operation 'QuotasUpdate' response has model definition 'array', it should be of the form 'list'. Note: If you have already shipped an SDK on top of this spec, fixing this warning may introduce a breaking change.
Location: Microsoft.MachineLearningServices/preview/2022-10-01-preview/machineLearningServices.json#L567
⚠️ R2001 - AvoidNestedProperties Consider using x-ms-client-flatten to provide a better end user experience
Location: Microsoft.MachineLearningServices/preview/2022-10-01-preview/machineLearningServices.json#L3263
⚠️ R2001 - AvoidNestedProperties Consider using x-ms-client-flatten to provide a better end user experience
Location: Microsoft.MachineLearningServices/preview/2022-10-01-preview/machineLearningServices.json#L3364
⚠️ R2001 - AvoidNestedProperties Consider using x-ms-client-flatten to provide a better end user experience
Location: Microsoft.MachineLearningServices/preview/2022-10-01-preview/machineLearningServices.json#L3459
⚠️ R2001 - AvoidNestedProperties Consider using x-ms-client-flatten to provide a better end user experience
Location: Microsoft.MachineLearningServices/preview/2022-10-01-preview/machineLearningServices.json#L3672
⚠️ R2001 - AvoidNestedProperties Consider using x-ms-client-flatten to provide a better end user experience
Location: Microsoft.MachineLearningServices/preview/2022-10-01-preview/machineLearningServices.json#L3861
⚠️ R2001 - AvoidNestedProperties Consider using x-ms-client-flatten to provide a better end user experience
Location: Microsoft.MachineLearningServices/preview/2022-10-01-preview/machineLearningServices.json#L3882
⚠️ R2001 - AvoidNestedProperties Consider using x-ms-client-flatten to provide a better end user experience
Location: Microsoft.MachineLearningServices/preview/2022-10-01-preview/machineLearningServices.json#L3950
⚠️ R2001 - AvoidNestedProperties Consider using x-ms-client-flatten to provide a better end user experience
Location: Microsoft.MachineLearningServices/preview/2022-10-01-preview/machineLearningServices.json#L3993
⚠️ R2001 - AvoidNestedProperties Consider using x-ms-client-flatten to provide a better end user experience
Location: Microsoft.MachineLearningServices/preview/2022-10-01-preview/machineLearningServices.json#L4013
⚠️ R2001 - AvoidNestedProperties Consider using x-ms-client-flatten to provide a better end user experience
Location: Microsoft.MachineLearningServices/preview/2022-10-01-preview/machineLearningServices.json#L4028
⚠️ R2001 - AvoidNestedProperties Consider using x-ms-client-flatten to provide a better end user experience
Location: Microsoft.MachineLearningServices/preview/2022-10-01-preview/machineLearningServices.json#L4298
⚠️ R2001 - AvoidNestedProperties Consider using x-ms-client-flatten to provide a better end user experience
Location: Microsoft.MachineLearningServices/preview/2022-10-01-preview/machineLearningServices.json#L5866
⚠️ R2001 - AvoidNestedProperties Consider using x-ms-client-flatten to provide a better end user experience
Location: Microsoft.MachineLearningServices/preview/2022-10-01-preview/machineLearningServices.json#L6046
⚠️ R2001 - AvoidNestedProperties Consider using x-ms-client-flatten to provide a better end user experience
Location: Microsoft.MachineLearningServices/preview/2022-10-01-preview/mfe.json#L6763
⚠️ R2001 - AvoidNestedProperties Consider using x-ms-client-flatten to provide a better end user experience
Location: Microsoft.MachineLearningServices/preview/2022-10-01-preview/mfe.json#L6851
⚠️ R2001 - AvoidNestedProperties Consider using x-ms-client-flatten to provide a better end user experience
Location: Microsoft.MachineLearningServices/preview/2022-10-01-preview/mfe.json#L7421
⚠️ R2001 - AvoidNestedProperties Consider using x-ms-client-flatten to provide a better end user experience
Location: Microsoft.MachineLearningServices/preview/2022-10-01-preview/mfe.json#L7478
⚠️ R2001 - AvoidNestedProperties Consider using x-ms-client-flatten to provide a better end user experience
Location: Microsoft.MachineLearningServices/preview/2022-10-01-preview/mfe.json#L7680
️️✔️Avocado succeeded [Detail] [Expand]
Validation passes for Avocado.
️❌ModelValidation: 10 Errors, 0 Warnings failed [Detail]
Rule Message
OBJECT_MISSING_REQUIRED_PROPERTY Missing required property: secrets
Url: Microsoft.MachineLearningServices/preview/2022-10-01-preview/mfe.json#L6060:39
ExampleUrl: 2022-10-01-preview/examples/Datastore/list.json
OBJECT_MISSING_REQUIRED_PROPERTY Missing required property: secrets
Url: Microsoft.MachineLearningServices/preview/2022-10-01-preview/mfe.json#L6060:39
ExampleUrl: 2022-10-01-preview/examples/Datastore/get.json
OBJECT_MISSING_REQUIRED_PROPERTY Missing required property: secrets
Url: Microsoft.MachineLearningServices/preview/2022-10-01-preview/mfe.json#L6060:39
ExampleUrl: examples/Datastore/AzureBlobWAccountKey/createOrUpdate.json
OBJECT_MISSING_REQUIRED_PROPERTY Missing required property: secrets
Url: Microsoft.MachineLearningServices/preview/2022-10-01-preview/mfe.json#L6060:39
ExampleUrl: examples/Datastore/AzureBlobWAccountKey/createOrUpdate.json
OBJECT_MISSING_REQUIRED_PROPERTY Missing required property: secrets
Url: Microsoft.MachineLearningServices/preview/2022-10-01-preview/mfe.json#L13847:45
ExampleUrl: examples/Datastore/AzureDataLakeGen1WServicePrincipal/createOrUpdate.json
OBJECT_MISSING_REQUIRED_PROPERTY Missing required property: secrets
Url: Microsoft.MachineLearningServices/preview/2022-10-01-preview/mfe.json#L13847:45
ExampleUrl: examples/Datastore/AzureDataLakeGen1WServicePrincipal/createOrUpdate.json
OBJECT_MISSING_REQUIRED_PROPERTY Missing required property: secrets
Url: Microsoft.MachineLearningServices/preview/2022-10-01-preview/mfe.json#L13847:45
ExampleUrl: examples/Datastore/AzureDataLakeGen2WServicePrincipal/createOrUpdate.json
OBJECT_MISSING_REQUIRED_PROPERTY Missing required property: secrets
Url: Microsoft.MachineLearningServices/preview/2022-10-01-preview/mfe.json#L13847:45
ExampleUrl: examples/Datastore/AzureDataLakeGen2WServicePrincipal/createOrUpdate.json
OBJECT_MISSING_REQUIRED_PROPERTY Missing required property: secrets
Url: Microsoft.MachineLearningServices/preview/2022-10-01-preview/mfe.json#L6060:39
ExampleUrl: examples/Datastore/AzureFileWAccountKey/createOrUpdate.json
OBJECT_MISSING_REQUIRED_PROPERTY Missing required property: secrets
Url: Microsoft.MachineLearningServices/preview/2022-10-01-preview/mfe.json#L6060:39
ExampleUrl: examples/Datastore/AzureFileWAccountKey/createOrUpdate.json
️️✔️SemanticValidation succeeded [Detail] [Expand]
Validation passes for SemanticValidation.
️️✔️SDK Track2 Validation succeeded [Detail] [Expand]
Validation passes for SDKTrack2Validation

  • The following tags are being changed in this PR
    • "https://github.com/Azure/azure-rest-api-specs/blob/b70bb7526455ba3b44e097ee4ac376491cc8273b/specification/machinelearningservices/resource-manager/readme.md#tag-package-preview-2022-10">machinelearningservices/resource-manager/readme.md#package-preview-2022-10
️️✔️PrettierCheck succeeded [Detail] [Expand]
Validation passes for PrettierCheck.
️️✔️SpellCheck succeeded [Detail] [Expand]
Validation passes for SpellCheck.
️️✔️CadlValidation succeeded [Detail] [Expand]
Validation passes for CadlValidation.
Posted by Swagger Pipeline | How to fix these errors?

@openapi-workflow-bot
Copy link

Hi, @fahdkmsft Thanks for your PR. I am workflow bot for review process. Here are some small tips.

  • Please ensure to do self-check against checklists in first PR comment.
  • PR assignee is the person auto-assigned and responsible for your current PR reviewing and merging.
  • For specs comparison cross API versions, Use API Specs Comparison Report Generator
  • If there is CI failure(s), to fix CI error(s) is mandatory for PR merging; or you need to provide justification in PR comment for explanation. How to fix?

  • Any feedback about review process or workflow bot, pls contact swagger and tools team. [email protected]

    @openapi-pipeline-app
    Copy link

    openapi-pipeline-app bot commented Aug 11, 2022

    Swagger Generation Artifacts

    ️️✔️ApiDocPreview succeeded [Detail] [Expand]
     Please click here to preview with your @microsoft account. 
    ️️✔️SDK Breaking Change Tracking succeeded [Detail] [Expand]

    Breaking Changes Tracking

    ️️✔️ azure-sdk-for-net-track2 succeeded [Detail] [Expand]
    • ️✔️Succeeded [Logs] Generate from 19f121c34bb7a4b7352771fd54cff853dbd4d8b6. SDK Automation 14.0.0
      command	pwsh ./eng/scripts/Automation-Sdk-Init.ps1 ../azure-sdk-for-net_tmp/initInput.json ../azure-sdk-for-net_tmp/initOutput.json
      warn	File azure-sdk-for-net_tmp/initOutput.json not found to read
      command	pwsh ./eng/scripts/Invoke-GenerateAndBuildV2.ps1 ../azure-sdk-for-net_tmp/generateInput.json ../azure-sdk-for-net_tmp/generateOutput.json
    • ️✔️Azure.ResourceManager.MachineLearning [View full logs]  [Preview SDK Changes]
      info	[Changelog]
    ️❌ azure-sdk-for-go failed [Detail]
    • Failed [Logs] Generate from 19f121c34bb7a4b7352771fd54cff853dbd4d8b6. SDK Automation 14.0.0
      command	sh ./eng/scripts/automation_init.sh ../../../../../azure-sdk-for-go_tmp/initInput.json ../../../../../azure-sdk-for-go_tmp/initOutput.json
      command	generator automation-v2 ../../../../../azure-sdk-for-go_tmp/generateInput.json ../../../../../azure-sdk-for-go_tmp/generateOutput.json
      cmderr	[generator automation-v2] [ERROR] total 1 error(s):
      cmderr	[generator automation-v2] [ERROR] failed to execute `go generate` '�[33;1mWARNING: Unable to find version for /mnt/vss/_work/1/s/azure-sdk-for-go/src/github.com/Azure/azure-sdk-for-go/sdk/resourcemanager/machinelearning/armmachinelearning�[0m
      cmderr	[generator automation-v2] [ERROR] Cannot find go module under resourcemanager/machinelearning/armmachinelearning, try to build in /mnt/vss/_work/1/s/azure-sdk-for-go/src/github.com/Azure/azure-sdk-for-go/sdk/resourcemanager/machinelearning/armmachinelearning
      cmderr	[generator automation-v2] [ERROR] ##[command]Cleaning auto-generated files in /mnt/vss/_work/1/s/azure-sdk-for-go/src/github.com/Azure/azure-sdk-for-go/sdk/resourcemanager/machinelearning/armmachinelearning
      cmderr	[generator automation-v2] [ERROR] ##[command]Executing autorest.go in  /mnt/vss/_work/1/s/azure-sdk-for-go/src/github.com/Azure/azure-sdk-for-go/sdk/resourcemanager/machinelearning/armmachinelearning
      cmderr	[generator automation-v2] [ERROR] autorest --version=3.8.2 --use=@autorest/[email protected] --go --track2 --output-folder=/mnt/vss/_work/1/s/azure-sdk-for-go/src/github.com/Azure/azure-sdk-for-go/sdk/resourcemanager/machinelearning/armmachinelearning --clear-output-folder=false --go.clear-output-folder=false --honor-body-placement=true ./autorest.md
      cmderr	[generator automation-v2] [ERROR] AutoRest code generation utility [cli version: 3.6.1; node: v14.20.0]
      cmderr	[generator automation-v2] [ERROR] (C) 2018 Microsoft Corporation.
      cmderr	[generator automation-v2] [ERROR] https://aka.ms/autorest
      cmderr	[generator automation-v2] [ERROR] info    | AutoRest core version selected from configuration: 3.8.2.
      cmderr	[generator automation-v2] [ERROR] There is a new version of AutoRest available (3.6.2).
      cmderr	[generator automation-v2] [ERROR] > You can install the newer version with with npm install -g autorest@latest
      cmderr	[generator automation-v2] [ERROR] info    |    Loading AutoRest core      '/home/cloudtest/.autorest/@[email protected]/nodemodules/@autorest/core/dist' (3.8.2)
      cmderr	[generator automation-v2] [ERROR] info    |    Installing AutoRest extension '@autorest/go' (4.0.0-preview.43 -> 4.0.0-preview.43)
      cmderr	[generator automation-v2] [ERROR] info    |    Installed AutoRest extension '@autorest/go' (4.0.0-preview.43->4.0.0-preview.43)
      cmderr	[generator automation-v2] [ERROR] info    |    Installing AutoRest extension '@autorest/modelerfour' (4.23.1 -> 4.23.1)
      cmderr	[generator automation-v2] [ERROR] info    |    Installed AutoRest extension '@autorest/modelerfour' (4.23.1->4.23.1)
      cmderr	[generator automation-v2] [ERROR] fatal   | Process() cancelled due to failure
      cmderr	[generator automation-v2] [ERROR] error   |   Error: Operation response '/paths/~1providers~1Microsoft.MachineLearningServices~1operations/get/responses/200' produces type couldn't be resolved. Operation is probably is missing a produces field and there isn't any global value. Please add "produces": [<contentType>]"
      cmderr	[generator automation-v2] [ERROR] error   | 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.
      cmderr	[generator automation-v2] [ERROR] ##[error]Error running autorest.go
      cmderr	[generator automation-v2] [ERROR] build.go:5: running "pwsh": exit status 1
      cmderr	[generator automation-v2] [ERROR] ': exit status 1
      error	Script return with result [failed] code [1] signal [null] cwd [azure-sdk-for-go/src/github.com/Azure/azure-sdk-for-go]: generator automation-v2
      warn	Skip package processing as generation is failed
    ️❌ azure-sdk-for-python-track2 failed [Detail]
    • Failed [Logs] Generate from 19f121c34bb7a4b7352771fd54cff853dbd4d8b6. SDK Automation 14.0.0
      command	sh scripts/automation_init.sh ../azure-sdk-for-python_tmp/initInput.json ../azure-sdk-for-python_tmp/initOutput.json
      cmderr	[automation_init.sh] WARNING: Skipping azure-nspkg as it is not installed.
      command	sh scripts/automation_generate.sh ../azure-sdk-for-python_tmp/generateInput.json ../azure-sdk-for-python_tmp/generateOutput.json
      cmderr	[automation_generate.sh]  config global `--global`, `--local` are deprecated. Use `--location=global` instead.
      cmdout	[automation_generate.sh] [Autorest] info    |    Installing AutoRest extension '@autorest/modelerfour' (4.23.5 -> 4.23.5)
      cmdout	[automation_generate.sh] [Autorest] info    |    Installed AutoRest extension '@autorest/modelerfour' (4.23.5->4.23.5)
      cmdout	[automation_generate.sh] [Autorest] info    |    Installing AutoRest extension '@autorest/python' (6.0.1 -> 6.0.1)
      cmdout	[automation_generate.sh] [Autorest] info    |    Installed AutoRest extension '@autorest/python' (6.0.1->6.0.1)
      cmdout	[automation_generate.sh] [Autorest] fatal   | Process() cancelled due to failure
      cmdout	[automation_generate.sh] [Autorest] error   |   Error: Operation response '/paths/~1providers~1Microsoft.MachineLearningServices~1operations/get/responses/200' produces type couldn't be resolved. Operation is probably is missing a produces field and there isn't any global value. Please add "produces": [<contentType>]"
      cmdout	[automation_generate.sh] [Autorest] error   | 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.
      cmdout	[automation_generate.sh] [Autorest]
      cmdout	[automation_generate.sh] [Autorest] info    |    Loading AutoRest core      '/home/cloudtest/.autorest/@[email protected]/nodemodules/@autorest/core/dist' (3.8.4)
      cmdout	[automation_generate.sh] [Autorest] info    |    Loading AutoRest extension '@autorest/modelerfour' (4.23.5->4.23.5)
      cmdout	[automation_generate.sh] [Autorest] info    |    Loading AutoRest extension '@autorest/python' (6.0.1->6.0.1)
      cmdout	[automation_generate.sh] [Autorest] fatal   | Process() cancelled due to failure
      cmdout	[automation_generate.sh] [Autorest] error   |   Error: Operation response '/paths/~1providers~1Microsoft.MachineLearningServices~1operations/get/responses/200' produces type couldn't be resolved. Operation is probably is missing a produces field and there isn't any global value. Please add "produces": [<contentType>]"
      cmdout	[automation_generate.sh] [Autorest] error   | 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.
      cmdout	[automation_generate.sh] [Autorest]/mnt/vss/_work/1/s/azure-sdk-for-python_tmp/venv-sdk/auto_temp.json does not exist!!!Error happened during codegen
      error	Script return with result [failed] code [1] signal [null] cwd [azure-sdk-for-python]: sh scripts/automation_generate.sh
      warn	Skip package processing as generation is failed
    ️❌ azure-sdk-for-java failed [Detail]
    • Failed [Logs] Generate from 19f121c34bb7a4b7352771fd54cff853dbd4d8b6. SDK Automation 14.0.0
      command	./eng/mgmt/automation/init.sh ../azure-sdk-for-java_tmp/initInput.json ../azure-sdk-for-java_tmp/initOutput.json
      command	./eng/mgmt/automation/generate.py ../azure-sdk-for-java_tmp/generateInput.json ../azure-sdk-for-java_tmp/generateOutput.json
    • azure-resourcemanager-machinelearning [View full logs]  [Preview SDK Changes]
    ️❌ azure-sdk-for-js failed [Detail]
    • Failed [Logs] Generate from 19f121c34bb7a4b7352771fd54cff853dbd4d8b6. SDK Automation 14.0.0
      command	sh .scripts/automation_init.sh ../azure-sdk-for-js_tmp/initInput.json ../azure-sdk-for-js_tmp/initOutput.json
      warn	File azure-sdk-for-js_tmp/initOutput.json not found to read
      command	sh .scripts/automation_generate.sh ../azure-sdk-for-js_tmp/generateInput.json ../azure-sdk-for-js_tmp/generateOutput.json
      cmderr	[automation_generate.sh] [ERROR] An error occurred while generating codes for readme file: "specification/machinelearningservices/resource-manager/readme.md":
      cmderr	[automation_generate.sh] [ERROR] Err: Error: Command failed: autorest --version=3.8.4 --typescript --modelerfour.lenient-model-deduplication --azure-arm --head-as-boolean=true --license-header=MICROSOFT_MIT_NO_VERSION --generate-test --typescript-sdks-folder=/mnt/vss/_work/1/s/azure-sdk-for-js ../azure-rest-api-specs/specification/machinelearningservices/resource-manager/readme.md --use=@autorest/[email protected]
      cmderr	[automation_generate.sh] [ERROR] Stderr: "null"
      cmderr	[automation_generate.sh] [ERROR] Stdout: "null"
      cmderr	[automation_generate.sh] [ERROR] ErrorStack: "Error: Command failed: autorest --version=3.8.4 --typescript --modelerfour.lenient-model-deduplication --azure-arm --head-as-boolean=true --license-header=MICROSOFT_MIT_NO_VERSION --generate-test --typescript-sdks-folder=/mnt/vss/_work/1/s/azure-sdk-for-js ../azure-rest-api-specs/specification/machinelearningservices/resource-manager/readme.md --use=@autorest/[email protected]
      cmderr	[automation_generate.sh] [ERROR]     at checkExecSyncError (child_process.js:790:11)
      cmderr	[automation_generate.sh] [ERROR]     at Object.execSync (child_process.js:863:15)
      cmderr	[automation_generate.sh] [ERROR]     at Object.<anonymous> (/opt/hostedtoolcache/node/14.20.0/x64/lib/node_modules/@azure-tools/js-sdk-release-tools/dist/hlc/generateMgmt.js:39:33)
      cmderr	[automation_generate.sh] [ERROR]     at Generator.next (<anonymous>)
      cmderr	[automation_generate.sh] [ERROR]     at /opt/hostedtoolcache/node/14.20.0/x64/lib/node_modules/@azure-tools/js-sdk-release-tools/node_modules/tslib/tslib.js:115:75
      cmderr	[automation_generate.sh] [ERROR]     at new Promise (<anonymous>)
      cmderr	[automation_generate.sh] [ERROR]     at Object.__awaiter (/opt/hostedtoolcache/node/14.20.0/x64/lib/node_modules/@azure-tools/js-sdk-release-tools/node_modules/tslib/tslib.js:111:16)
      cmderr	[automation_generate.sh] [ERROR]     at Object.generateMgmt (/opt/hostedtoolcache/node/14.20.0/x64/lib/node_modules/@azure-tools/js-sdk-release-tools/dist/hlc/generateMgmt.js:20:20)
      cmderr	[automation_generate.sh] [ERROR]     at /opt/hostedtoolcache/node/14.20.0/x64/lib/node_modules/@azure-tools/js-sdk-release-tools/dist/autoGenerateInPipeline.js:34:34
      cmderr	[automation_generate.sh] [ERROR]     at Generator.next (<anonymous>)"
      error	Script return with result [failed] code [1] signal [null] cwd [azure-sdk-for-js]: sh .scripts/automation_generate.sh
      warn	Skip package processing as generation is failed
    ️❌ azure-resource-manager-schemas failed [Detail]
    • Failed [Logs] Generate from 19f121c34bb7a4b7352771fd54cff853dbd4d8b6. Schema Automation 14.0.0
      command	.sdkauto/initScript.sh ../azure-resource-manager-schemas_tmp/initInput.json ../azure-resource-manager-schemas_tmp/initOutput.json
      cmderr	[initScript.sh] WARN old lockfile
      cmderr	[initScript.sh] npm WARN old lockfile The package-lock.json file was created with an old version of npm,
      cmderr	[initScript.sh] npm WARN old lockfile so supplemental metadata must be fetched from the registry.
      cmderr	[initScript.sh] npm WARN old lockfile
      cmderr	[initScript.sh] npm WARN old lockfile This is a one-time fix-up, please be patient...
      cmderr	[initScript.sh] npm WARN old lockfile
      warn	File azure-resource-manager-schemas_tmp/initOutput.json not found to read
      command	.sdkauto/generateScript.sh ../azure-resource-manager-schemas_tmp/generateInput.json ../azure-resource-manager-schemas_tmp/generateOutput.json
    • machinelearningservices/resource-manager [View full logs]  [Preview Schema Changes]
    ️❌ azure-powershell failed [Detail]
    • Failed [Logs] Generate from 19f121c34bb7a4b7352771fd54cff853dbd4d8b6. SDK Automation 14.0.0
      command	sh ./tools/SwaggerCI/init.sh ../azure-powershell_tmp/initInput.json ../azure-powershell_tmp/initOutput.json
      command	pwsh ./tools/SwaggerCI/psci.ps1 ../azure-powershell_tmp/generateInput.json ../azure-powershell_tmp/generateOutput.json
    • Az.machinelearningservices [View full logs]  [Preview SDK Changes]
    Posted by Swagger Pipeline | How to fix these errors?

    @openapi-pipeline-app
    Copy link

    openapi-pipeline-app bot commented Aug 11, 2022

    Swagger pipeline restarted successfully. If there is ApiView generated, it will be updated in this comment.

    @openapi-workflow-bot
    Copy link

    Hi @fahdkmsft, Your PR has some issues. Please fix the CI sequentially by following the order of Avocado, semantic validation, model validation, breaking change, lintDiff. If you have any questions, please post your questions in this channel https://aka.ms/swaggersupport.

    TaskHow to fixPriority
    AvocadoFix-AvocadoHigh
    Semantic validationFix-SemanticValidation-ErrorHigh
    Model validationFix-ModelValidation-ErrorHigh
    LintDiffFix-LintDiffhigh
    If you need further help, please feedback via swagger feedback.

    @openapi-workflow-bot
    Copy link

    Hi, @fahdkmsft your PR are labelled with WaitForARMFeedback. A notification email will be sent out shortly afterwards to notify ARM review board([email protected]).

    @jorgecotillo jorgecotillo added ARMSignedOff <valid label in PR review process>add this label when ARM approve updates after review and removed WaitForARMFeedback <valid label in PR review process> add this label when ARM review is required labels Aug 12, 2022
    @fahdkmsft
    Copy link
    Contributor Author

    The ModelValidation errors are false-negatives as mentioned here: #18645 (comment)

    @ArcturusZhang
    Copy link
    Member

    The model validation is a known false alarm. Suppressing it

    @ArcturusZhang ArcturusZhang merged commit 9f5b38d into Azure:dev-machinelearningservices-Microsoft.MachineLearningServices-2022-10-01-preview Aug 16, 2022
    Maulik003 pushed a commit that referenced this pull request Oct 4, 2022
    * Registries public preview 2022-10-01-preview
    
    * Fix validation errors
    
    * Fix LintDiff and Avocado errors
    
    * Fixing prettier errors
    
    * Fixing more LintDiff errors
    
    * Another LintDiff error
    
    * Another Lint error
    
    Co-authored-by: Fahd Kamal <[email protected]>
    weshaggard pushed a commit that referenced this pull request Oct 5, 2022
    …to add version preview/2022-10-01-preview (#20818)
    
    * Adds base for updating Microsoft.MachineLearningServices from version preview/2022-06-01-preview to version 2022-10-01-preview
    
    * Updates readme
    
    * Updates API version in new specs and examples
    
    * Registries public preview 2022-10-01-preview (#20200)
    
    * Registries public preview 2022-10-01-preview
    
    * Fix validation errors
    
    * Fix LintDiff and Avocado errors
    
    * Fixing prettier errors
    
    * Fixing more LintDiff errors
    
    * Another LintDiff error
    
    * Another Lint error
    
    Co-authored-by: Fahd Kamal <[email protected]>
    
    * Added schedule api (#20273)
    
    * added schedule api
    
    * add updateSchedules to examples parent folder
    
    * fix typo
    
    * change position of allOf
    
    * remove subnet from example compute create
    
    * after prettier run
    
    * Description change for recurrence freq
    
    * point to mfe for common objects
    
    * add hash operator
    
    * removed triggertype as well
    
    * change year
    
    * get rid of breaking changes since we didn't change the api
    
    Co-authored-by: Naman Agarwal <[email protected]>
    
    * Added Compute Instance OS Patching Properties (#20284)
    
    * Added CI Os Patching related properties
    
    * Updated descriptions as per Dennis's suggestions
    
    Co-authored-by: Srivatsa Sinha <[email protected]>
    
    * Maulik/sync preview (#20554)
    
    * Added changes to sync with main
    
    * Fixed Prettier Check
    
    Co-authored-by: maulikjoshi <[email protected]>
    
    * Remove update schedules REST call for compute instance (#20699)
    
    * change post call to put
    
    * change put to patch
    
    * remove updateschedules rest call
    
    Co-authored-by: Naman Agarwal <[email protected]>
    
    * Update connection type and auth type (#20493)
    
    * Update connection type and auth type
    
    * update connection category
    
    * update x-ms-secret tag
    
    * Add autologger_settings for CI (#20302)
    
    * Add autologger_settings for CI
    
    Add settings to opt-in/out of default mlflow autologger on CIs.
    
    * update machinelearningservices.json
    
    * Add for command job
    
    * fix comma
    
    * Add autologger for command job
    
    * fix CI failures
    
    * Remove mfe.json from PR
    
    * Remove whitespace
    
    * Update custom-words.txt
    
    * Delete incorrect custom-words.txt
    
    * Fixing auto-rest issues in registries (#20772)
    
    * Fixing auto-rest issues in registries
    
    * Removing location from the required list
    
    Co-authored-by: Fahd Kamal <[email protected]>
    
    * Maulik/mfe preview changes (#20372)
    
    * Added mfe.json preview changes
    
    * Added resources to Registries and Workspaces folder
    
    * removed unwanted resources
    
    * removed unwanted resources
    
    * Ran prettier check
    
    * removed unwanted resources
    
    * removed unwanted resources
    
    * Added ManagedOnlineDeployment resource
    
    * Added files to OnlineDeplyment
    
    * Added Model Validation changes
    
    * Added mfe.json changes to handle duplicate operation id changes
    
    * Updated mfe to fix underscore error
    
    * Update title of mfe.json
    
    * Renamed name path parameter
    
    * Updated examples
    
    * Fixed Semantic validations and examples
    
    * Fixed Semantic validations and examples
    
    * Fixed Semantic validations and examples
    
    * Removed name changes from workspaces
    
    * Changes related to Model Validation
    
    * Changes related to Model validation
    
    * Added consumes amd produces in registries.json
    
    * Removed changes from registries
    
    * Updated parameter mlFlowAutologger to mlflowAutologger in mfe.json
    
    * Changed property name to mlflowAutologger
    
    Co-authored-by: maulikjoshi <[email protected]>
    
    * Updated custom words to resolve merge conflict (#20833)
    
    * Updated custom words to resolve merge conflict
    
    * Updated custom words to resolve merge conflict
    
    Co-authored-by: maulikjoshi <[email protected]>
    
    * Resolved merge conflict (#20845)
    
    Co-authored-by: maulikjoshi <[email protected]>
    
    * Fixed LintDiff checks (#20847)
    
    * Fixed LintDiff checks
    
    * Fixed LintDiff checks
    
    * Fixed LintDiff checks
    
    * Fixed LintDiff checks
    
    * Updated examples
    
    Co-authored-by: maulikjoshi <[email protected]>
    
    * Resolved merge conflict of cutsom-words.txt (#20956)
    
    * Resolved merge conflict
    
    * Resolved merge conflict
    
    Co-authored-by: maulikjoshi <[email protected]>
    
    * [Hub Generated] Review request for Microsoft.MachineLearningServices to add version stable/2022-10-01 (#20800) (#20982)
    
    * Adds base for updating Microsoft.MachineLearningServices from version stable/2022-05-01 to version 2022-10-01
    
    * Updates readme
    
    * Updates API version in new specs and examples
    
    * Added readOnly property (#20506)
    
    * Added readOnly property
    
    * Added changes to sync with main
    
    * Fixed Prettier check
    
    Co-authored-by: maulikjoshi <[email protected]>
    
    * Add schedule related schemas for Compute Instance (#20697)
    
    * support for update schedules on Compute Instance
    
    * change put to patch
    
    * remove update schedules rest call
    
    Co-authored-by: Naman Agarwal <[email protected]>
    
    * Added mfe.json changes (#20514)
    
    * Added mfe.json changes
    
    * Added checks related changes
    
    * Updated mfe.json to handle online endpoint and deployment changes
    
    * Updated mfe.json with AutoRebuild changes
    
    Co-authored-by: maulikjoshi <[email protected]>
    
    * Removed duplicate definitions and updated with mfe references (#20840)
    
    * Removed duplicate definitions and updated with mfe references
    
    * Replaced Cron to CronTrigger, Recurrence to RecurrenceTrigger from mfe
    
    Co-authored-by: maulikjoshi <[email protected]>
    
    Co-authored-by: maulikjoshi <[email protected]>
    Co-authored-by: Naman Agarwal <[email protected]>
    Co-authored-by: Naman Agarwal <[email protected]>
    
    Co-authored-by: maulikjoshi <[email protected]>
    Co-authored-by: Naman Agarwal <[email protected]>
    Co-authored-by: Naman Agarwal <[email protected]>
    
    * Revert "[Hub Generated] Review request for Microsoft.MachineLearningS… (#20983)
    
    * Revert "[Hub Generated] Review request for Microsoft.MachineLearningServices to add version stable/2022-10-01 (#20800) (#20982)"
    
    This reverts commit 649ef83.
    
    * Updated readme.md to resolve merge conflict
    
    Co-authored-by: maulikjoshi <[email protected]>
    
    * Maulik/fix preview (#20994)
    
    * Merged readme from main
    
    * Added preview tag changes
    
    Co-authored-by: maulikjoshi <[email protected]>
    
    * shuffle readme tags (#21005)
    
    * conflict resolution (#21007)
    
    Co-authored-by: fahdkmsft <[email protected]>
    Co-authored-by: Fahd Kamal <[email protected]>
    Co-authored-by: Naman Agarwal <[email protected]>
    Co-authored-by: Naman Agarwal <[email protected]>
    Co-authored-by: srivatsasinha <[email protected]>
    Co-authored-by: Srivatsa Sinha <[email protected]>
    Co-authored-by: maulikjoshi <[email protected]>
    Co-authored-by: ZhidaLiu <[email protected]>
    Co-authored-by: prakharg-msft <[email protected]>
    Co-authored-by: Teddy Todorov <[email protected]>
    anandanthony pushed a commit to anandMicro/azure-rest-api-specs that referenced this pull request Dec 16, 2022
    …to add version preview/2022-10-01-preview (Azure#20818)
    
    * Adds base for updating Microsoft.MachineLearningServices from version preview/2022-06-01-preview to version 2022-10-01-preview
    
    * Updates readme
    
    * Updates API version in new specs and examples
    
    * Registries public preview 2022-10-01-preview (Azure#20200)
    
    * Registries public preview 2022-10-01-preview
    
    * Fix validation errors
    
    * Fix LintDiff and Avocado errors
    
    * Fixing prettier errors
    
    * Fixing more LintDiff errors
    
    * Another LintDiff error
    
    * Another Lint error
    
    Co-authored-by: Fahd Kamal <[email protected]>
    
    * Added schedule api (Azure#20273)
    
    * added schedule api
    
    * add updateSchedules to examples parent folder
    
    * fix typo
    
    * change position of allOf
    
    * remove subnet from example compute create
    
    * after prettier run
    
    * Description change for recurrence freq
    
    * point to mfe for common objects
    
    * add hash operator
    
    * removed triggertype as well
    
    * change year
    
    * get rid of breaking changes since we didn't change the api
    
    Co-authored-by: Naman Agarwal <[email protected]>
    
    * Added Compute Instance OS Patching Properties (Azure#20284)
    
    * Added CI Os Patching related properties
    
    * Updated descriptions as per Dennis's suggestions
    
    Co-authored-by: Srivatsa Sinha <[email protected]>
    
    * Maulik/sync preview (Azure#20554)
    
    * Added changes to sync with main
    
    * Fixed Prettier Check
    
    Co-authored-by: maulikjoshi <[email protected]>
    
    * Remove update schedules REST call for compute instance (Azure#20699)
    
    * change post call to put
    
    * change put to patch
    
    * remove updateschedules rest call
    
    Co-authored-by: Naman Agarwal <[email protected]>
    
    * Update connection type and auth type (Azure#20493)
    
    * Update connection type and auth type
    
    * update connection category
    
    * update x-ms-secret tag
    
    * Add autologger_settings for CI (Azure#20302)
    
    * Add autologger_settings for CI
    
    Add settings to opt-in/out of default mlflow autologger on CIs.
    
    * update machinelearningservices.json
    
    * Add for command job
    
    * fix comma
    
    * Add autologger for command job
    
    * fix CI failures
    
    * Remove mfe.json from PR
    
    * Remove whitespace
    
    * Update custom-words.txt
    
    * Delete incorrect custom-words.txt
    
    * Fixing auto-rest issues in registries (Azure#20772)
    
    * Fixing auto-rest issues in registries
    
    * Removing location from the required list
    
    Co-authored-by: Fahd Kamal <[email protected]>
    
    * Maulik/mfe preview changes (Azure#20372)
    
    * Added mfe.json preview changes
    
    * Added resources to Registries and Workspaces folder
    
    * removed unwanted resources
    
    * removed unwanted resources
    
    * Ran prettier check
    
    * removed unwanted resources
    
    * removed unwanted resources
    
    * Added ManagedOnlineDeployment resource
    
    * Added files to OnlineDeplyment
    
    * Added Model Validation changes
    
    * Added mfe.json changes to handle duplicate operation id changes
    
    * Updated mfe to fix underscore error
    
    * Update title of mfe.json
    
    * Renamed name path parameter
    
    * Updated examples
    
    * Fixed Semantic validations and examples
    
    * Fixed Semantic validations and examples
    
    * Fixed Semantic validations and examples
    
    * Removed name changes from workspaces
    
    * Changes related to Model Validation
    
    * Changes related to Model validation
    
    * Added consumes amd produces in registries.json
    
    * Removed changes from registries
    
    * Updated parameter mlFlowAutologger to mlflowAutologger in mfe.json
    
    * Changed property name to mlflowAutologger
    
    Co-authored-by: maulikjoshi <[email protected]>
    
    * Updated custom words to resolve merge conflict (Azure#20833)
    
    * Updated custom words to resolve merge conflict
    
    * Updated custom words to resolve merge conflict
    
    Co-authored-by: maulikjoshi <[email protected]>
    
    * Resolved merge conflict (Azure#20845)
    
    Co-authored-by: maulikjoshi <[email protected]>
    
    * Fixed LintDiff checks (Azure#20847)
    
    * Fixed LintDiff checks
    
    * Fixed LintDiff checks
    
    * Fixed LintDiff checks
    
    * Fixed LintDiff checks
    
    * Updated examples
    
    Co-authored-by: maulikjoshi <[email protected]>
    
    * Resolved merge conflict of cutsom-words.txt (Azure#20956)
    
    * Resolved merge conflict
    
    * Resolved merge conflict
    
    Co-authored-by: maulikjoshi <[email protected]>
    
    * [Hub Generated] Review request for Microsoft.MachineLearningServices to add version stable/2022-10-01 (Azure#20800) (Azure#20982)
    
    * Adds base for updating Microsoft.MachineLearningServices from version stable/2022-05-01 to version 2022-10-01
    
    * Updates readme
    
    * Updates API version in new specs and examples
    
    * Added readOnly property (Azure#20506)
    
    * Added readOnly property
    
    * Added changes to sync with main
    
    * Fixed Prettier check
    
    Co-authored-by: maulikjoshi <[email protected]>
    
    * Add schedule related schemas for Compute Instance (Azure#20697)
    
    * support for update schedules on Compute Instance
    
    * change put to patch
    
    * remove update schedules rest call
    
    Co-authored-by: Naman Agarwal <[email protected]>
    
    * Added mfe.json changes (Azure#20514)
    
    * Added mfe.json changes
    
    * Added checks related changes
    
    * Updated mfe.json to handle online endpoint and deployment changes
    
    * Updated mfe.json with AutoRebuild changes
    
    Co-authored-by: maulikjoshi <[email protected]>
    
    * Removed duplicate definitions and updated with mfe references (Azure#20840)
    
    * Removed duplicate definitions and updated with mfe references
    
    * Replaced Cron to CronTrigger, Recurrence to RecurrenceTrigger from mfe
    
    Co-authored-by: maulikjoshi <[email protected]>
    
    Co-authored-by: maulikjoshi <[email protected]>
    Co-authored-by: Naman Agarwal <[email protected]>
    Co-authored-by: Naman Agarwal <[email protected]>
    
    Co-authored-by: maulikjoshi <[email protected]>
    Co-authored-by: Naman Agarwal <[email protected]>
    Co-authored-by: Naman Agarwal <[email protected]>
    
    * Revert "[Hub Generated] Review request for Microsoft.MachineLearningS… (Azure#20983)
    
    * Revert "[Hub Generated] Review request for Microsoft.MachineLearningServices to add version stable/2022-10-01 (Azure#20800) (Azure#20982)"
    
    This reverts commit 649ef83.
    
    * Updated readme.md to resolve merge conflict
    
    Co-authored-by: maulikjoshi <[email protected]>
    
    * Maulik/fix preview (Azure#20994)
    
    * Merged readme from main
    
    * Added preview tag changes
    
    Co-authored-by: maulikjoshi <[email protected]>
    
    * shuffle readme tags (Azure#21005)
    
    * conflict resolution (Azure#21007)
    
    Co-authored-by: fahdkmsft <[email protected]>
    Co-authored-by: Fahd Kamal <[email protected]>
    Co-authored-by: Naman Agarwal <[email protected]>
    Co-authored-by: Naman Agarwal <[email protected]>
    Co-authored-by: srivatsasinha <[email protected]>
    Co-authored-by: Srivatsa Sinha <[email protected]>
    Co-authored-by: maulikjoshi <[email protected]>
    Co-authored-by: ZhidaLiu <[email protected]>
    Co-authored-by: prakharg-msft <[email protected]>
    Co-authored-by: Teddy Todorov <[email protected]>
    Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
    Labels
    Approved-ModelValidation ARMReview ARMSignedOff <valid label in PR review process>add this label when ARM approve updates after review CI-FixRequiredOnFailure resource-manager
    Projects
    None yet
    Development

    Successfully merging this pull request may close these issues.

    4 participants