Skip to content

Conversation

@shmartel
Copy link
Contributor

This PR moves the private branch that was created into public. The branch has already been reviewed, to view the status, please go to the OpenAPI Hub.

@openapi-pipeline-app
Copy link

openapi-pipeline-app bot commented Sep 10, 2020

[Staging] Swagger Validation Report

️✔️BreakingChange [Detail]
 There are no breaking changes. 
️✔️LintDiff [Detail]
 Validation passes for LintDiff. 
️✔️Avocado [Detail]
 Validation passes for Avocado. 
️✔️ModelValidation [Detail]
 Validation passes for ModelValidation. 
️✔️SemanticValidation [Detail]
 Validation passes for SemanticValidation. 
Posted by Swagger Pipeline | How to fix these errors?

@azure-pipelines
Copy link

Azure Pipelines successfully started running 1 pipeline(s).

@openapi-sdkautomation
Copy link

openapi-sdkautomation bot commented Sep 10, 2020

azure-sdk-for-java - Release

⚠️ warning [Logs] [Expand Details]
  • ⚠️ Generate from 1ae67f3 with merge commit 995320f. SDK Automation 13.0.17.20200909.5
  • ⚠️sdk/healthcareapis/mgmt-v2018_08_20_preview [View full logs]  [Release SDK Changes]
      [mvn] [ERROR] COMPILATION ERROR : 
      [mvn] [ERROR] /z/work/azure-sdk-for-java/sdk/healthcareapis/mgmt-v2018_08_20_preview/src/main/java/com/microsoft/azure/management/healthcareapis/v2018_08_20_preview/implementation/ServicesInner.java:[19,73] cannot find symbol
      [mvn]   symbol:   class ErrorDetailsInnerException
      [mvn]   location: package com.microsoft.azure.management.healthcareapis.v2018_08_20_preview
      [mvn] [ERROR] /z/work/azure-sdk-for-java/sdk/healthcareapis/mgmt-v2018_08_20_preview/src/main/java/com/microsoft/azure/management/healthcareapis/v2018_08_20_preview/implementation/OperationsInner.java:[15,73] cannot find symbol
      [mvn]   symbol:   class ErrorDetailsInnerException
      [mvn]   location: package com.microsoft.azure.management.healthcareapis.v2018_08_20_preview
      [mvn] [ERROR] /z/work/azure-sdk-for-java/sdk/healthcareapis/mgmt-v2018_08_20_preview/src/main/java/com/microsoft/azure/management/healthcareapis/v2018_08_20_preview/implementation/OperationResultsInner.java:[13,73] cannot find symbol
      [mvn]   symbol:   class ErrorDetailsInnerException
      [mvn]   location: package com.microsoft.azure.management.healthcareapis.v2018_08_20_preview
      [mvn] [ERROR] Failed to execute goal org.apache.maven.plugins:maven-compiler-plugin:3.1:compile (default-compile) on project azure-mgmt-healthcareapis: Compilation failure: Compilation failure: 
      [mvn] [ERROR] /z/work/azure-sdk-for-java/sdk/healthcareapis/mgmt-v2018_08_20_preview/src/main/java/com/microsoft/azure/management/healthcareapis/v2018_08_20_preview/implementation/ServicesInner.java:[19,73] cannot find symbol
      [mvn] [ERROR]   symbol:   class ErrorDetailsInnerException
      [mvn] [ERROR]   location: package com.microsoft.azure.management.healthcareapis.v2018_08_20_preview
      [mvn] [ERROR] /z/work/azure-sdk-for-java/sdk/healthcareapis/mgmt-v2018_08_20_preview/src/main/java/com/microsoft/azure/management/healthcareapis/v2018_08_20_preview/implementation/OperationsInner.java:[15,73] cannot find symbol
      [mvn] [ERROR]   symbol:   class ErrorDetailsInnerException
      [mvn] [ERROR]   location: package com.microsoft.azure.management.healthcareapis.v2018_08_20_preview
      [mvn] [ERROR] /z/work/azure-sdk-for-java/sdk/healthcareapis/mgmt-v2018_08_20_preview/src/main/java/com/microsoft/azure/management/healthcareapis/v2018_08_20_preview/implementation/OperationResultsInner.java:[13,73] cannot find symbol
      [mvn] [ERROR]   symbol:   class ErrorDetailsInnerException
      [mvn] [ERROR]   location: package com.microsoft.azure.management.healthcareapis.v2018_08_20_preview
      [mvn] [ERROR] -> [Help 1]
      [mvn] [ERROR] 
      [mvn] [ERROR] To see the full stack trace of the errors, re-run Maven with the -e switch.
      [mvn] [ERROR] Re-run Maven using the -X switch to enable full debug logging.
      [mvn] [ERROR] 
      [mvn] [ERROR] For more information about the errors and possible solutions, please read the following articles:
      [mvn] [ERROR] [Help 1] http://cwiki.apache.org/confluence/display/MAVEN/MojoFailureException
    • ⚠️sdk/healthcareapis/mgmt-v2019-09-16 [View full logs]  [Release SDK Changes]
        [mvn] [ERROR] Failed to execute goal org.apache.maven.plugins:maven-javadoc-plugin:2.8:jar (default-cli) on project azure-mgmt-healthcareapis: MavenReportException: Error while creating archive: 
        [mvn] [ERROR] Exit code: 1 - javadoc: error - Illegal package name: "com.microsoft.azure.management.healthcareapis.v2019-09-16"
        [mvn] [ERROR] 
        [mvn] [ERROR] Command line was: /usr/lib/jvm/java-8-openjdk-amd64/jre/../bin/javadoc @options @packages
        [mvn] [ERROR] 
        [mvn] [ERROR] Refer to the generated Javadoc files in '/z/work/azure-sdk-for-java/sdk/healthcareapis/mgmt-v2019-09-16/target/apidocs' dir.
        [mvn] [ERROR] -> [Help 1]
        [mvn] [ERROR] 
        [mvn] [ERROR] To see the full stack trace of the errors, re-run Maven with the -e switch.
        [mvn] [ERROR] Re-run Maven using the -X switch to enable full debug logging.
        [mvn] [ERROR] 
        [mvn] [ERROR] For more information about the errors and possible solutions, please read the following articles:
        [mvn] [ERROR] [Help 1] http://cwiki.apache.org/confluence/display/MAVEN/MojoExecutionException

      @openapi-sdkautomation
      Copy link

      openapi-sdkautomation bot commented Sep 10, 2020

      azure-sdk-for-js - Release

      ️✔️ succeeded [Logs] [Expand Details]
      • ️✔️ Generate from 1ae67f3 with merge commit 995320f. SDK Automation 13.0.17.20200909.5
      • ️✔️@azure/arm-healthcareapis [View full logs]  [Release SDK Changes]
        [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/healthcareApisManagementClient.js → ./dist/arm-healthcareapis.js...
        [npmPack] created ./dist/arm-healthcareapis.js in 258ms
        [npmPack] npm notice 
        [npmPack] npm notice package: @azure/[email protected]
        [npmPack] npm notice === Tarball Contents === 
        [npmPack] npm notice 50.9kB  dist/arm-healthcareapis.js                        
        [npmPack] npm notice 16.7kB  dist/arm-healthcareapis.min.js                    
        [npmPack] npm notice 1.7kB   esm/healthcareApisManagementClient.js             
        [npmPack] npm notice 2.6kB   esm/healthcareApisManagementClientContext.js      
        [npmPack] npm notice 345B    esm/models/index.js                               
        [npmPack] npm notice 445B    esm/operations/index.js                           
        [npmPack] npm notice 17.3kB  esm/models/mappers.js                             
        [npmPack] npm notice 2.0kB   esm/operations/operationResults.js                
        [npmPack] npm notice 464B    esm/models/operationResultsMappers.js             
        [npmPack] npm notice 2.3kB   esm/operations/operations.js                      
        [npmPack] npm notice 479B    esm/models/operationsMappers.js                   
        [npmPack] npm notice 2.3kB   esm/models/parameters.js                          
        [npmPack] npm notice 1.0kB   rollup.config.js                                  
        [npmPack] npm notice 12.7kB  esm/operations/services.js                        
        [npmPack] npm notice 792B    esm/models/servicesMappers.js                     
        [npmPack] npm notice 1.7kB   package.json                                      
        [npmPack] npm notice 457B    tsconfig.json                                     
        [npmPack] npm notice 102.6kB dist/arm-healthcareapis.js.map                    
        [npmPack] npm notice 15.6kB  dist/arm-healthcareapis.min.js.map                
        [npmPack] npm notice 710B    esm/healthcareApisManagementClient.d.ts.map       
        [npmPack] npm notice 834B    esm/healthcareApisManagementClient.js.map         
        [npmPack] npm notice 526B    esm/healthcareApisManagementClientContext.d.ts.map
        [npmPack] npm notice 1.4kB   esm/healthcareApisManagementClientContext.js.map  
        [npmPack] npm notice 6.7kB   esm/models/index.d.ts.map                         
        [npmPack] npm notice 181B    esm/operations/index.d.ts.map                     
        [npmPack] npm notice 126B    esm/models/index.js.map                           
        [npmPack] npm notice 196B    esm/operations/index.js.map                       
        [npmPack] npm notice 1.1kB   esm/models/mappers.d.ts.map                       
        [npmPack] npm notice 9.7kB   esm/models/mappers.js.map                         
        [npmPack] npm notice 785B    esm/operations/operationResults.d.ts.map          
        [npmPack] npm notice 1.1kB   esm/operations/operationResults.js.map            
        [npmPack] npm notice 217B    esm/models/operationResultsMappers.d.ts.map       
        [npmPack] npm notice 230B    esm/models/operationResultsMappers.js.map         
        [npmPack] npm notice 1.0kB   esm/operations/operations.d.ts.map                
        [npmPack] npm notice 1.4kB   esm/operations/operations.js.map                  
        [npmPack] npm notice 228B    esm/models/operationsMappers.d.ts.map             
        [npmPack] npm notice 241B    esm/models/operationsMappers.js.map               
        [npmPack] npm notice 541B    esm/models/parameters.d.ts.map                    
        [npmPack] npm notice 1.6kB   esm/models/parameters.js.map                      
        [npmPack] npm notice 3.7kB   esm/operations/services.d.ts.map                  
        [npmPack] npm notice 6.5kB   esm/operations/services.js.map                    
        [npmPack] npm notice 364B    esm/models/servicesMappers.d.ts.map               
        [npmPack] npm notice 377B    esm/models/servicesMappers.js.map                 
        [npmPack] npm notice 3.5kB   README.md                                         
        [npmPack] npm notice 1.2kB   esm/healthcareApisManagementClient.d.ts           
        [npmPack] npm notice 1.7kB   src/healthcareApisManagementClient.ts             
        [npmPack] npm notice 832B    esm/healthcareApisManagementClientContext.d.ts    
        [npmPack] npm notice 2.3kB   src/healthcareApisManagementClientContext.ts      
        [npmPack] npm notice 17.7kB  esm/models/index.d.ts                             
        [npmPack] npm notice 129B    esm/operations/index.d.ts                         
        [npmPack] npm notice 17.1kB  src/models/index.ts                               
        [npmPack] npm notice 413B    src/operations/index.ts                           
        [npmPack] npm notice 1.5kB   esm/models/mappers.d.ts                           
        [npmPack] npm notice 13.8kB  src/models/mappers.ts                             
        [npmPack] npm notice 1.7kB   esm/operations/operationResults.d.ts              
        [npmPack] npm notice 3.2kB   src/operations/operationResults.ts                
        [npmPack] npm notice 154B    esm/models/operationResultsMappers.d.ts           
        [npmPack] npm notice 420B    src/models/operationResultsMappers.ts             
        [npmPack] npm notice 2.1kB   esm/operations/operations.d.ts                    
        [npmPack] npm notice 4.2kB   src/operations/operations.ts                      
        [npmPack] npm notice 169B    esm/models/operationsMappers.d.ts                 
        [npmPack] npm notice 445B    src/models/operationsMappers.ts                   
        [npmPack] npm notice 616B    esm/models/parameters.d.ts                        
        [npmPack] npm notice 2.2kB   src/models/parameters.ts                          
        [npmPack] npm notice 10.2kB  esm/operations/services.d.ts                      
        [npmPack] npm notice 19.8kB  src/operations/services.ts                        
        [npmPack] npm notice 482B    esm/models/servicesMappers.d.ts                   
        [npmPack] npm notice 782B    src/models/servicesMappers.ts                     
        [npmPack] npm notice 1.1kB   LICENSE.txt                                       
        [npmPack] npm notice === Tarball Details === 
        [npmPack] npm notice name:          @azure/arm-healthcareapis               
        [npmPack] npm notice version:       1.0.0                                   
        [npmPack] npm notice filename:      azure-arm-healthcareapis-1.0.0.tgz      
        [npmPack] npm notice package size:  49.1 kB                                 
        [npmPack] npm notice unpacked size: 380.0 kB                                
        [npmPack] npm notice shasum:        0a1478411c0e6c4aad2d2dd3d72f587fe46b6f77
        [npmPack] npm notice integrity:     sha512-cmQTfhfs+SDFs[...]RNZUNkMSmZr5A==
        [npmPack] npm notice total files:   69                                      
        [npmPack] npm notice 

      @openapi-sdkautomation
      Copy link

      openapi-sdkautomation bot commented Sep 10, 2020

      Azure CLI Extension 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

      @openapi-sdkautomation
      Copy link

      openapi-sdkautomation bot commented Sep 10, 2020

      azure-sdk-for-go - Release

      ⚠️ warning [Logs] [Expand Details]

      @openapi-sdkautomation
      Copy link

      openapi-sdkautomation bot commented Sep 10, 2020

      azure-sdk-for-python - Release

      ️✔️ succeeded [Logs] [Expand Details]
      • ️✔️ Generate from 1ae67f3 with merge commit 995320f. SDK Automation 13.0.17.20200909.5
      • ️✔️azure-mgmt-healthcareapis [View full logs]  [Release SDK Changes]
        [build_conf] INFO:packaging_tools:Building template azure-mgmt-healthcareapis
        [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-healthcareapis
        [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] /usr/lib/python3.6/distutils/dist.py:261: UserWarning: Unknown distribution option: 'long_description_content_type'
        [build_package]   warnings.warn(msg)
        [breaking_change_setup] Ignoring mock: markers 'python_version <= "2.7"' don't match your environment
        [ChangeLog] Size of delta 14.253% size of original (original: 10952 chars, delta: 1561 chars)
        [ChangeLog] **Features**
        [ChangeLog] 
        [ChangeLog]   - Model ServicesProperties has a new parameter export_configuration
        [ChangeLog]   - Model ServiceCosmosDbConfigurationInfo has a new parameter key_vault_key_uri
        [ChangeLog]   - Model ServicesDescription has a new parameter identity
        [ChangeLog]   - Model Resource has a new parameter identity

      @openapi-sdkautomation
      Copy link

      openapi-sdkautomation bot commented Sep 10, 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 Sep 10, 2020

      azure-sdk-for-net - Release

      failed [Logs] [Expand Details]
      • Generate from 1ae67f3 with merge commit 995320f. SDK Automation 13.0.17.20200909.5
        [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.HealthcareApis [View full logs]  [Release SDK Changes]
          Failed to create the package Microsoft.Azure.Management.HealthcareApis.
          Error: dotnet msbuild build.proj /t:CreateNugetPackage /p:Scope=healthcareapis /v:n /p:SkipTests=true , {} 

        @openapi-sdkautomation
        Copy link

        openapi-sdkautomation bot commented Sep 10, 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

        @openapi-sdkautomation
        Copy link

        openapi-sdkautomation bot commented Sep 10, 2020

        azure-resource-manager-schemas - Release

        ⚠️ warning [Logs] [Expand Details]

        @yungezz
        Copy link
        Member

        yungezz commented Sep 11, 2020

        hi @shmartel could you pls share PR link in private branch? thanks.

        @shmartel
        Copy link
        Contributor Author

        hi @shmartel could you pls share PR link in private branch? thanks.
        Hi, @yungezz , yes: https://github.com/Azure/azure-rest-api-specs-pr/pull/1679

        @shmartel
        Copy link
        Contributor Author

        @yungezz yungezz added WaitForARMFeedback <valid label in PR review process> add this label when ARM review is required and removed WaitForARMFeedback <valid label in PR review process> add this label when ARM review is required labels Sep 14, 2020
        @yungezz
        Copy link
        Member

        yungezz commented Sep 15, 2020

        The PR is reviewed in private repo https://github.com/Azure/azure-rest-api-specs-pr/pull/1679. I diffed swagger file, they're identical. So remove "WaitForARMFeedback" label.

        hi @shmartel , SDK Net task doesn't block PR merging. Is this PR ready to merge?

        @shmartel
        Copy link
        Contributor Author

        shmartel commented Sep 15, 2020

        The PR is reviewed in private repo Azure/azure-rest-api-specs-pr#1679. I diffed swagger file, they're identical. So remove "WaitForARMFeedback" label.

        hi @shmartel , SDK Net task doesn't block PR merging. Is this PR ready to merge?

        Hi, @yungezz . OK, good to know. Yes, this PR is ready to merge -- please do so. Thanks!

        @yungezz yungezz merged commit 995320f into master Sep 16, 2020
        nitegupt pushed a commit to nitegupt/azure-rest-api-specs that referenced this pull request Sep 25, 2020
        Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

        Labels

        None yet

        Projects

        None yet

        Development

        Successfully merging this pull request may close these issues.

        3 participants