Skip to content

Conversation

@navysingla
Copy link
Contributor

Fixing parameter name, API is not live yet for actual customers

@openapi-pipeline-app
Copy link

openapi-pipeline-app bot commented Aug 12, 2020

[Staging] Swagger Validation Report

BreakingChange: 6 Errors, 0 Warnings [Detail] [Expand]
Rule Message
1033 - RemovedProperty The new version is missing a property found in the old version. Was 'workloadType' renamed or removed?
New: Microsoft.Subscription/preview/2019-10-01-preview/subscriptions.json#L779:7
Old: Microsoft.Subscription/preview/2019-10-01-preview/subscriptions.json#L779:7
1033 - RemovedProperty The new version is missing a property found in the old version. Was 'workloadType' renamed or removed?
New: Microsoft.Subscription/preview/2019-10-01-preview/subscriptions.json#L779:7
Old: Microsoft.Subscription/preview/2019-10-01-preview/subscriptions.json#L779:7
1034 - AddedRequiredProperty The new version has new required property 'workload' that was not found in the old version.
New: Microsoft.Subscription/preview/2019-10-01-preview/subscriptions.json#L779:7
Old: Microsoft.Subscription/preview/2019-10-01-preview/subscriptions.json#L779:7
1034 - AddedRequiredProperty The new version has new required property 'workload' that was not found in the old version.
New: Microsoft.Subscription/preview/2019-10-01-preview/subscriptions.json#L766:9
Old: Microsoft.Subscription/preview/2019-10-01-preview/subscriptions.json#L766:9
1034 - AddedRequiredProperty The new version has new required property 'workload' that was not found in the old version.
New: Microsoft.Subscription/preview/2019-10-01-preview/subscriptions.json#L779:7
Old: Microsoft.Subscription/preview/2019-10-01-preview/subscriptions.json#L779:7
1034 - AddedRequiredProperty The new version has new required property 'workload' that was not found in the old version.
New: Microsoft.Subscription/preview/2019-10-01-preview/subscriptions.json#L772:5
Old: Microsoft.Subscription/preview/2019-10-01-preview/subscriptions.json#L772:5
️✔️LintDiff [Detail]
 Validation passes for LintDiff. 
️✔️Avocado [Detail]
 Validation passes for Avocado. 
Posted by Swagger Pipeline | How to fix these errors?

@azure-pipelines
Copy link

Azure Pipelines successfully started running 1 pipeline(s).

@azuresdkci
Copy link
Contributor

Can one of the admins verify this patch?

@openapi-sdkautomation
Copy link

openapi-sdkautomation bot commented Aug 12, 2020

azure-sdk-for-go - Release

⚠️ warning [Logs] [Expand Details]

@openapi-sdkautomation
Copy link

openapi-sdkautomation bot commented Aug 12, 2020

Azure CLI Extension Generation - Release

  • Installation instruction for service account [Preview Changes]
    az extension add --source=https://sdkautomation.blob.core.windows.net/sdkautomation-pipeline/Azure/azure-rest-api-specs/10439/azure-cli-extensions/Azure/azure-cli-extensions/account/account-0.1.0-py3-none-any.whl
️✔️ succeeded [Logs] [Expand Details]
  • ️✔️ Generate from a7e0eb3 with merge commit 16e95fa. Azure CLI Automation 13.0.17.20200807.2
    [AutoRest] WARNING (PreCheck/SchemaMissingType): The schema 'Location' with an undefined type and decalared properties is a bit ambigious. This has been auto-corrected to 'type:object'
    [AutoRest] WARNING (PreCheck/SchemaMissingType): The schema 'LocationListResult' with an undefined type and decalared properties is a bit ambigious. This has been auto-corrected to 'type:object'
    [AutoRest] WARNING (PreCheck/SchemaMissingType): The schema 'Subscription' with an undefined type and decalared properties is a bit ambigious. This has been auto-corrected to 'type:object'
    [AutoRest] WARNING (PreCheck/SchemaMissingType): The schema 'SubscriptionPolicies' with an undefined type and decalared properties is a bit ambigious. This has been auto-corrected to 'type:object'
    [AutoRest] WARNING (PreCheck/SchemaMissingType): The schema 'SubscriptionListResult' with an undefined type and decalared properties is a bit ambigious. This has been auto-corrected to 'type:object'
    [AutoRest] WARNING (PreCheck/SchemaMissingType): The schema 'TenantIdDescription' with an undefined type and decalared properties is a bit ambigious. This has been auto-corrected to 'type:object'
    [AutoRest] WARNING (PreCheck/SchemaMissingType): The schema 'TenantListResult' with an undefined type and decalared properties is a bit ambigious. This has been auto-corrected to 'type:object'
    [AutoRest] WARNING (PreCheck/SchemaMissingType): The schema 'CanceledSubscriptionId' with an undefined type and decalared properties is a bit ambigious. This has been auto-corrected to 'type:object'
    [AutoRest] WARNING (PreCheck/SchemaMissingType): The schema 'RenamedSubscriptionId' with an undefined type and decalared properties is a bit ambigious. This has been auto-corrected to 'type:object'
    [AutoRest] WARNING (PreCheck/SchemaMissingType): The schema 'EnabledSubscriptionId' with an undefined type and decalared properties is a bit ambigious. This has been auto-corrected to 'type:object'
    [AutoRest] WARNING (PreCheck/SchemaMissingType): The schema 'OperationListResult' with an undefined type and decalared properties is a bit ambigious. This has been auto-corrected to 'type:object'
    [AutoRest] WARNING (PreCheck/SchemaMissingType): The schema 'Operation-display' with an undefined type and decalared properties is a bit ambigious. This has been auto-corrected to 'type:object'
    [AutoRest] WARNING (PreCheck/SchemaMissingType): The schema 'PutAliasRequestProperties' with an undefined type and decalared properties is a bit ambigious. This has been auto-corrected to 'type:object'
    [AutoRest] WARNING (PreCheck/SchemaMissingType): The schema 'PutAliasResponse' with an undefined type and decalared properties is a bit ambigious. This has been auto-corrected to 'type:object'
    [AutoRest] WARNING (PreCheck/SchemaMissingType): The schema 'PutAliasResponseProperties' with an undefined type and decalared properties is a bit ambigious. This has been auto-corrected to 'type:object'
    [AutoRest] WARNING (PreCheck/SchemaMissingType): The schema 'PutAliasListResult' with an undefined type and decalared properties is a bit ambigious. This has been auto-corrected to 'type:object'
    [AutoRest] WARNING (): cli.auto-parameter-hidden is not true, skip visibility cleaner
    [AutoRest] 
    [AutoRest] NO TEST SCENARIO PROVIDED - DEFAULT WILL BE USED
    [AutoRest] ADD FOLLOWING SECTION TO readme.cli.md FILE TO MODIFY IT
    [AutoRest] --------------------------------------------------------
    [AutoRest]   test-scenario:
    [AutoRest]     - name: /Subscription/put/CreateAlias
    [AutoRest]     - name: /Subscription/get/GetAlias
    [AutoRest]     - name: /Subscription/get/GetAlias
    [AutoRest]     - name: /Subscription/post/cancelSubscription
    [AutoRest]     - name: /Subscription/post/enableSubscription
    [AutoRest]     - name: /Subscription/post/renameSubscription
    [AutoRest]     - name: /Subscriptions/get/getSubscription
    [AutoRest]     - name: /Subscriptions/get/listLocations
    [AutoRest]     - name: /Subscriptions/get/listSubscriptions
    [AutoRest]     - name: /Tenants/get/listTenants
    [AutoRest]     - name: /Subscription/delete/DeleteAlias
    [AutoRest] --------------------------------------------------------

@openapi-sdkautomation
Copy link

openapi-sdkautomation bot commented Aug 12, 2020

azure-sdk-for-js - Release

️✔️ succeeded [Logs] [Expand Details]
  • ️✔️ Generate from a7e0eb3 with merge commit 16e95fa. SDK Automation 13.0.17.20200807.2
  • ️✔️@azure/arm-subscriptions [View full logs]  [Release SDK Changes]
    Only show 100 items here, please refer to log for details.
    [npmPack] npm WARN deprecated [email protected]: This package has been deprecated and is no longer maintained. Please use @rollup/plugin-node-resolve.
    [npmPack] npm WARN deprecated [email protected]: https://github.com/lydell/resolve-url#deprecated
    [npmPack] npm WARN deprecated [email protected]: Please see https://github.com/lydell/urix#deprecated
    [npmPack] npm notice created a lockfile as package-lock.json. You should commit this file.
    [npmPack] loaded rollup.config.js with warnings
    [npmPack] (!) Unused external imports
    [npmPack] default imported from external module 'rollup' but never used
    [npmPack] 
    [npmPack] ./esm/subscriptionClient.js → ./dist/arm-subscriptions.js...
    [npmPack] created ./dist/arm-subscriptions.js in 266ms
    [npmPack] npm notice 
    [npmPack] npm notice package: @azure/[email protected]
    [npmPack] npm notice === Tarball Contents === 
    [npmPack] npm notice 45.6kB dist/arm-subscriptions.js                        
    [npmPack] npm notice 14.8kB dist/arm-subscriptions.min.js                    
    [npmPack] npm notice 345B   esm/models/index.js                              
    [npmPack] npm notice 483B   esm/operations/index.js                          
    [npmPack] npm notice 15.3kB esm/models/mappers.js                            
    [npmPack] npm notice 1.5kB  esm/operations/operations.js                     
    [npmPack] npm notice 458B   esm/models/operationsMappers.js                  
    [npmPack] npm notice 1.7kB  esm/models/parameters.js                         
    [npmPack] npm notice 990B   rollup.config.js                                 
    [npmPack] npm notice 1.6kB  esm/subscriptionClient.js                        
    [npmPack] npm notice 2.2kB  esm/subscriptionClientContext.js                 
    [npmPack] npm notice 7.5kB  esm/operations/subscriptionOperations.js         
    [npmPack] npm notice 651B   esm/models/subscriptionOperationsMappers.js      
    [npmPack] npm notice 3.8kB  esm/operations/subscriptions.js                  
    [npmPack] npm notice 498B   esm/models/subscriptionsMappers.js               
    [npmPack] npm notice 2.2kB  esm/operations/tenants.js                        
    [npmPack] npm notice 441B   esm/models/tenantsMappers.js                     
    [npmPack] npm notice 1.7kB  package.json                                     
    [npmPack] npm notice 457B   tsconfig.json                                    
    [npmPack] npm notice 98.5kB dist/arm-subscriptions.js.map                    
    [npmPack] npm notice 14.7kB dist/arm-subscriptions.min.js.map                
    [npmPack] npm notice 6.4kB  esm/models/index.d.ts.map                        
    [npmPack] npm notice 204B   esm/operations/index.d.ts.map                    
    [npmPack] npm notice 126B   esm/models/index.js.map                          
    [npmPack] npm notice 219B   esm/operations/index.js.map                      
    [npmPack] npm notice 1.2kB  esm/models/mappers.d.ts.map                      
    [npmPack] npm notice 8.9kB  esm/models/mappers.js.map                        
    [npmPack] npm notice 663B   esm/operations/operations.d.ts.map               
    [npmPack] npm notice 908B   esm/operations/operations.js.map                 
    [npmPack] npm notice 215B   esm/models/operationsMappers.d.ts.map            
    [npmPack] npm notice 228B   esm/models/operationsMappers.js.map              
    [npmPack] npm notice 443B   esm/models/parameters.d.ts.map                   
    [npmPack] npm notice 1.2kB  esm/models/parameters.js.map                     
    [npmPack] npm notice 699B   esm/subscriptionClient.d.ts.map                  
    [npmPack] npm notice 852B   esm/subscriptionClient.js.map                    
    [npmPack] npm notice 424B   esm/subscriptionClientContext.d.ts.map           
    [npmPack] npm notice 1.2kB  esm/subscriptionClientContext.js.map             
    [npmPack] npm notice 3.0kB  esm/operations/subscriptionOperations.d.ts.map   
    [npmPack] npm notice 4.6kB  esm/operations/subscriptionOperations.js.map     
    [npmPack] npm notice 340B   esm/models/subscriptionOperationsMappers.d.ts.map
    [npmPack] npm notice 353B   esm/models/subscriptionOperationsMappers.js.map  
    [npmPack] npm notice 1.8kB  esm/operations/subscriptions.d.ts.map            
    [npmPack] npm notice 2.4kB  esm/operations/subscriptions.js.map              
    [npmPack] npm notice 244B   esm/models/subscriptionsMappers.d.ts.map         
    [npmPack] npm notice 257B   esm/models/subscriptionsMappers.js.map           
    [npmPack] npm notice 1.0kB  esm/operations/tenants.d.ts.map                  
    [npmPack] npm notice 1.4kB  esm/operations/tenants.js.map                    
    [npmPack] npm notice 199B   esm/models/tenantsMappers.d.ts.map               
    [npmPack] npm notice 212B   esm/models/tenantsMappers.js.map                 
    [npmPack] npm notice 3.3kB  README.md                                        
    [npmPack] npm notice 17.3kB esm/models/index.d.ts                            
    [npmPack] npm notice 167B   esm/operations/index.d.ts                        
    [npmPack] npm notice 16.7kB src/models/index.ts                              
    [npmPack] npm notice 451B   src/operations/index.ts                          
    [npmPack] npm notice 1.6kB  esm/models/mappers.d.ts                          
    [npmPack] npm notice 12.4kB src/models/mappers.ts                            
    [npmPack] npm notice 1.1kB  esm/operations/operations.d.ts                   
    [npmPack] npm notice 2.3kB  src/operations/operations.ts                     
    [npmPack] npm notice 148B   esm/models/operationsMappers.d.ts                
    [npmPack] npm notice 422B   src/models/operationsMappers.ts                  
    [npmPack] npm notice 475B   esm/models/parameters.d.ts                       
    [npmPack] npm notice 1.7kB  src/models/parameters.ts                         
    [npmPack] npm notice 999B   esm/subscriptionClient.d.ts                      
    [npmPack] npm notice 1.6kB  src/subscriptionClient.ts                        
    [npmPack] npm notice 649B   esm/subscriptionClientContext.d.ts               
    [npmPack] npm notice 2.0kB  src/subscriptionClientContext.ts                 
    [npmPack] npm notice 6.0kB  esm/operations/subscriptionOperations.d.ts       
    [npmPack] npm notice 12.6kB src/operations/subscriptionOperations.ts         
    [npmPack] npm notice 341B   esm/models/subscriptionOperationsMappers.d.ts    
    [npmPack] npm notice 619B   src/models/subscriptionOperationsMappers.ts      
    [npmPack] npm notice 4.0kB  esm/operations/subscriptions.d.ts                
    [npmPack] npm notice 7.7kB  src/operations/subscriptions.ts                  
    [npmPack] npm notice 188B   esm/models/subscriptionsMappers.d.ts             
    [npmPack] npm notice 463B   src/models/subscriptionsMappers.ts               
    [npmPack] npm notice 2.0kB  esm/operations/tenants.d.ts                      
    [npmPack] npm notice 4.0kB  src/operations/tenants.ts                        
    [npmPack] npm notice 131B   esm/models/tenantsMappers.d.ts                   
    [npmPack] npm notice 406B   src/models/tenantsMappers.ts                     
    [npmPack] npm notice 1.1kB  LICENSE.txt                                      
    [npmPack] npm notice === Tarball Details === 
    [npmPack] npm notice name:          @azure/arm-subscriptions                
    [npmPack] npm notice version:       2.0.0                                   
    [npmPack] npm notice filename:      azure-arm-subscriptions-2.0.0.tgz       
    [npmPack] npm notice package size:  45.6 kB                                 
    [npmPack] npm notice unpacked size: 357.8 kB                                
    [npmPack] npm notice shasum:        cd3feee0a342563180cf5909c838e8c9c751801e
    [npmPack] npm notice integrity:     sha512-WX+ZU2ZDgnHAP[...]rl8FP3yHmJNcQ==

@openapi-sdkautomation
Copy link

openapi-sdkautomation bot commented Aug 12, 2020

azure-sdk-for-java - Release

No readme.md specification configuration files were found that are associated with the files modified in this pull request, or swagger_to_sdk section in readme.md is not configured

@openapi-sdkautomation
Copy link

openapi-sdkautomation bot commented Aug 12, 2020

azure-sdk-for-python - Release

- Breaking Change detected in SDK

⚠️ warning [Logs] [Expand Details]
  • ⚠️ Generate from a7e0eb3 with merge commit 16e95fa. SDK Automation 13.0.17.20200807.2
  • ⚠️azure-mgmt-subscription [View full logs]  [Release SDK Changes] Breaking Change Detected
    [build_conf] INFO:packaging_tools:Building template azure-mgmt-subscription
    [build_conf] INFO:packaging_tools.conf:Skipping default conf since the file exists
    [build_conf] INFO:packaging_tools:Skipping CHANGELOG.md template, since a previous one was found
    [build_conf] INFO:packaging_tools:Template done azure-mgmt-subscription
    [build_package] /usr/lib/python3.6/distutils/dist.py:261: UserWarning: Unknown distribution option: 'long_description_content_type'
    [build_package]   warnings.warn(msg)
    [build_package] warning: no files found matching '*.py' under directory 'tests'
    [build_package] warning: no files found matching '*.yaml' under directory 'tests'
    [build_package] /usr/lib/python3.6/distutils/dist.py:261: UserWarning: Unknown distribution option: 'long_description_content_type'
    [build_package]   warnings.warn(msg)
    [build_package] warning: no files found matching '*.py' under directory 'tests'
    [build_package] warning: no files found matching '*.yaml' under directory 'tests'
    [breaking_change_setup] Ignoring mock: markers 'python_version <= "2.7"' don't match your environment
    [ChangeLog] Size of delta 41.507% size of original (original: 9107 chars, delta: 3780 chars)
    [ChangeLog] **Features**
    [ChangeLog] 
    [ChangeLog]   - Added operation SubscriptionOperations.create_alias
    [ChangeLog]   - Added operation SubscriptionOperations.get_alias
    [ChangeLog]   - Added operation SubscriptionOperations.list_alias
    [ChangeLog]   - Added operation SubscriptionOperations.delete_alias
    [ChangeLog] 
    [ChangeLog] **Breaking changes**
    [ChangeLog] 
    [ChangeLog]   - Removed operation SubscriptionOperations.create_subscription
    [ChangeLog]   - Removed operation SubscriptionOperations.create_csp_subscription
    [ChangeLog]   - Removed operation SubscriptionOperations.create_subscription_in_enrollment_account
    [ChangeLog]   - Removed operation group SubscriptionOperationOperations

@openapi-sdkautomation
Copy link

openapi-sdkautomation bot commented Aug 12, 2020

azure-sdk-for-net - Release

failed [Logs] [Expand Details]
  • Generate from a7e0eb3 with merge commit 16e95fa. SDK Automation 13.0.17.20200807.2
    [AutoRest] realpath(): Permission denied
    [AutoRest] realpath(): Permission denied
    [AutoRest] realpath(): Permission denied
    [AutoRest] realpath(): Permission denied
    [AutoRest] realpath(): Permission denied
    [AutoRest] realpath(): Permission denied
  • Microsoft.Azure.Management.Subscription [View full logs]  [Release SDK Changes]
      Failed to create the package Microsoft.Azure.Management.Subscription.
      Error: dotnet msbuild build.proj /t:CreateNugetPackage /p:Scope=subscription /v:n /p:SkipTests=true , {} 

    @openapi-sdkautomation
    Copy link

    openapi-sdkautomation bot commented Aug 12, 2020

    Trenton Generation - Release

    No readme.md specification configuration files were found that are associated with the files modified in this pull request, or swagger_to_sdk section in readme.md is not configured

    @navysingla
    Copy link
    Contributor Author

    @qianwens Can we merge this PR too

    @qianwens qianwens added the Approved-BreakingChange DO NOT USE! OBSOLETE label. See https://github.com/Azure/azure-sdk-tools/issues/6374 label Aug 18, 2020
    @qianwens qianwens merged commit 16e95fa into Azure:master Aug 18, 2020
    @openapi-sdkautomation
    Copy link

    openapi-sdkautomation bot commented Aug 18, 2020

    azure-sdk-for-python-track2 - Release

    No readme.md specification configuration files were found that are associated with the files modified in this pull request, or swagger_to_sdk section in readme.md is not configured

    @openapi-sdkautomation
    Copy link

    openapi-sdkautomation bot commented Aug 18, 2020

    azure-resource-manager-schemas - Release

    ️✔️ succeeded [Logs] [Expand Details]
    • ️✔️ Generate from a7e0eb3 with merge commit 16e95fa. Schema Automation 13.0.17.20200807.2
    • ️✔️subscription [View full logs]  [Release Schema Changes]
        [after_scripts|node] PostProcessor finished successfully.
        [after_scripts|node]   5609 passing (1m)
        AzureResourceSchema has no registered package commands.

      Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

      Labels

      Approved-BreakingChange DO NOT USE! OBSOLETE label. See https://github.com/Azure/azure-sdk-tools/issues/6374 CI-BreakingChange-Python

      Projects

      None yet

      Development

      Successfully merging this pull request may close these issues.

      3 participants