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.

Sign up for GitHub

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

Jump to bottom

Test for python readme configuration(Do not merge) #18146

Closed
wants to merge 3 commits into from
Closed

Test for python readme configuration(Do not merge) #18146

wants to merge 3 commits into from

Conversation

msyyc
Copy link
Member

@msyyc msyyc commented Mar 9, 2022 •
edited
Loading

@openapi-workflow-bot
Copy link

Hi, @msyyc 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. vscswagger@microsoft.com

    @openapi-workflow-bot
    Copy link

    [Call for Action] To better understand Azure service dev/test scenario, and support Azure service developer better on Swagger and REST API related tests in early phase, please help to fill in with this survey https://aka.ms/SurveyForEarlyPhase. It will take 5 to 10 minutes. If you already complete survey, please neglect this comment. Thanks.

    @openapi-pipeline-app
    Copy link

    openapi-pipeline-app bot commented Mar 9, 2022 •
    edited
    Loading

    Swagger Validation Report

    ️️✔️BreakingChange succeeded [Detail] [Expand] 
    There are no breaking changes.
    ️️✔️LintDiff succeeded [Detail] [Expand] 
    Validation passes for LintDiff.

    ️️✔️Avocado succeeded [Detail] [Expand] 
    Validation passes for Avocado.
    ️️✔️~[Staging] ApiReadinessCheck succeeded [Detail] [Expand]
    ️️✔️ModelValidation succeeded [Detail] [Expand] 
    Validation passes for ModelValidation.
    ️️✔️SemanticValidation succeeded [Detail] [Expand] 
    Validation passes for SemanticValidation.
    ️️✔️Cross-Version Breaking Changes succeeded [Detail] [Expand] 
    There are no breaking changes.
    ️️✔️CredScan succeeded [Detail] [Expand] 
    There is no credential detected.
    ️️✔️SDK Track2 Validation succeeded [Detail] [Expand] 
    Validation passes for SDKTrack2Validation
    ️️✔️PrettierCheck succeeded [Detail] [Expand] 
    Validation passes for PrettierCheck.
    ️️✔️SpellCheck succeeded [Detail] [Expand] 
    Validation passes for SpellCheck.
    ️️✔️Lint(RPaaS) succeeded [Detail] [Expand] 
    Validation passes for Lint(RPaaS).
    Posted by Swagger Pipeline | How to fix these errors?

    @openapi-pipeline-app
    Copy link

    openapi-pipeline-app bot commented Mar 9, 2022 •
    edited
    Loading

    Swagger Generation Artifacts

    ️️✔️ApiDocPreview succeeded [Detail] [Expand]
    ️❌SDK Breaking Change Tracking failed [Detail]

    Breaking Changes Tracking

    ❌azure-sdk-for-go-track2 - armdatadog - 0.3.0
    ️✔️azure-sdk-for-python-track2 - track2_azure-mgmt-testremove - first release
    ️✔️azure-sdk-for-go - datadog/mgmt/2021-03-01/datadog - v62.1.0
    ❌azure-sdk-for-go - preview/datadog/mgmt/2020-02-01-preview/datadog - v62.1.0
    +	Function `MonitorsClient.UpdateSender` return value(s) have been changed from `(*http.Response, error)` to `(MonitorsUpdateFuture, error)`
+	Function `MonitorsClient.Update` return value(s) have been changed from `(MonitorResource, error)` to `(MonitorsUpdateFuture, error)`
    ️️✔️ azure-sdk-for-net succeeded [Detail] [Expand]  
  
    • ️✔️Succeeded [Logs] Generate from fe004d69f02bbe7a585a26635990ac340cc78976. SDK Automation 14.0.0 
      warn	Skip initScript due to not configured
command	sudo apt-get install -y dotnet-sdk-6.0
command	autorest --version=2.0.4421 --csharp --reflect-api-versions --license-header=MICROSOFT_MIT_NO_VERSION --use=@microsoft.azure/autorest.csharp@2.3.82 --csharp-sdks-folder=/home/vsts/work/1/s/azure-sdk-for-net/sdk ../azure-rest-api-specs/specification/datadog/resource-manager/readme.md
cmderr	[Autorest] realpath(): Permission denied
cmderr	[Autorest] realpath(): Permission denied
cmderr	[Autorest] realpath(): Permission denied
cmderr	[Autorest] realpath(): Permission denied
cmderr	[Autorest] realpath(): Permission denied
cmderr	[Autorest] realpath(): Permission denied
    • ️✔️Microsoft.Azure.Management.Datadog        [View full logs]         [Preview SDK Changes]      
    ️⚠️ azure-sdk-for-python-track2 warning [Detail]   
  
    • ⚠️Warning [Logs] Generate from fe004d69f02bbe7a585a26635990ac340cc78976. 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] npm notice
cmderr	[automation_generate.sh] npm notice New minor version of npm available! 8.3.1 -> 8.5.3
cmderr	[automation_generate.sh] npm notice Changelog: <https://github.com/npm/cli/releases/tag/v8.5.3>
cmderr	[automation_generate.sh] npm notice Run `npm install -g npm@8.5.3` to update!
cmderr	[automation_generate.sh] npm notice
    • ️✔️track2_azure-mgmt-testremove        [View full logs]         [Preview SDK Changes]      
      info	[Changelog]   - Initial Release
    ️️✔️ azure-sdk-for-java succeeded [Detail] [Expand]  
  
    ️️✔️ azure-sdk-for-go succeeded [Detail] [Expand]  
  
    • ️✔️Succeeded [Logs] Generate from fe004d69f02bbe7a585a26635990ac340cc78976. SDK Automation 14.0.0 
      command	sh ./eng/initScript.sh ../../../../../azure-sdk-for-go_tmp/initInput.json ../../../../../azure-sdk-for-go_tmp/initOutput.json
command	generator automation ../../../../../azure-sdk-for-go_tmp/generateInput.json ../../../../../azure-sdk-for-go_tmp/generateOutput.json
    • ️✔️datadog/mgmt/2021-03-01/datadog        [View full logs]         [Preview SDK Changes]      
      info	[Changelog] ### Other Changes
info	[Changelog]
info	[Changelog] Total 0 breaking change(s), 0 additive change(s).
    • ️✔️preview/datadog/mgmt/2020-02-01-preview/datadog        [View full logs]       
      info	[Changelog] ### Breaking Changes
info	[Changelog]
info	[Changelog] - Function `MonitorsClient.Update` return value(s) have been changed from `(MonitorResource, error)` to `(MonitorsUpdateFuture, error)`
info	[Changelog] - Function `MonitorsClient.UpdateSender` return value(s) have been changed from `(*http.Response, error)` to `(MonitorsUpdateFuture, error)`
info	[Changelog]
info	[Changelog] ### Features Added
info	[Changelog]
info	[Changelog] - New function `*MonitorsUpdateFuture.UnmarshalJSON([]byte) error`
info	[Changelog] - New struct `MonitorsUpdateFuture`
info	[Changelog] - New field `Sku` in struct `MonitorResourceUpdateParameters`
info	[Changelog]
info	[Changelog] Total 2 breaking change(s), 4 additive change(s).
    ️️✔️ azure-sdk-for-go-track2 succeeded [Detail] [Expand]  
  
    • ️✔️Succeeded [Logs] Generate from fe004d69f02bbe7a585a26635990ac340cc78976. 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
    • ️✔️armdatadog        [View full logs]         [Preview SDK Changes]        Breaking Change Detected
      info	[Changelog] ### Breaking Changes
info	[Changelog]
info	[Changelog] - Function `ErrorDetail.MarshalJSON` has been removed
info	[Changelog] - Struct `ErrorAdditionalInfo` has been removed
info	[Changelog] - Struct `ErrorDetail` has been removed
info	[Changelog] - Struct `ErrorResponse` has been removed
info	[Changelog]
info	[Changelog] Total 7 breaking change(s), 0 additive change(s).
    ️⚠️ azure-sdk-for-js warning [Detail]   
  
    • ⚠️Warning [Logs] Generate from fe004d69f02bbe7a585a26635990ac340cc78976. 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
cmderr	[automation_init.sh]  read-package-tree@5.1.6: The functionality that this package provided is now in @npmcli/arborist
cmderr	[automation_init.sh] npm WARN deprecated uuid@3.4.0: Please upgrade  to version 7 or higher.  Older versions may use Math.random() in certain circumstances, which is known to be problematic.  See https://v8.dev/blog/math-random for details.
cmderr	[automation_init.sh] npm WARN deprecated @opentelemetry/types@0.2.0: Package renamed to @opentelemetry/api, see https://github.com/open-telemetry/opentelemetry-js
cmderr	[automation_init.sh] npm notice created a lockfile as package-lock.json. You should commit this file.
cmderr	[automation_init.sh] npm WARN @octokit/plugin-request-log@1.0.4 requires a peer of @octokit/core@>=3 but none is installed. You must install peer dependencies yourself.
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
    • ️✔️@azure/arm-datadog        [View full logs]         [Preview SDK Changes]      
      info	[Changelog]
error	breakingChangeTracking is enabled, but version or changelogItem is not found in output.
    ️⚠️ azure-resource-manager-schemas warning [Detail]   
  
    • ⚠️Warning [Logs] Generate from fe004d69f02bbe7a585a26635990ac340cc78976. 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]
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
warn	No file changes detected after generation
    • ️✔️datadog        [View full logs]       
    Posted by Swagger Pipeline | How to fix these errors?
          		
    
    
      

    
    


        
    

            

            
    
              
            
    
        
    
      
    

      
    
            
  
    
    
  
    
    

  
    
  
    


        
    

    


    



    


      
    
  
            
    
    
    
      
    
    
    
    

      

          @msyyc
msyyc




        changed the title
Update readme.python.md

Test for python readme configuration


      Mar 9, 2022

    
    
  
    


    
    
    
    
      
    
    
    
    

      

          @msyyc
msyyc




        changed the title
Test for python readme configuration

Test for python readme configuration(Do not merge)


      Mar 9, 2022

    
    
  
    


    


  
    
  
    
    
  
    

  
    
      
    
  This was referenced Mar 9, 2022

    


    
  


  
    
      
   Draft

  
    

    



      
    

    


    
  


  
    
      
   Draft

  
    

    



  
    

    




    

      
    
  
        



    

      
    
  
          
    
  
    
    
  
    

  
    
      
    
  This was referenced Mar 9, 2022

    


    
  


  
    
      
   Draft

  
    

    



      
    

    


    
  


  
    
      
   Draft

  
    

    



  
    

    

    
    
    
    
      
    
    
    
    

      

          @msyyc
msyyc




            closed this


      Mar 10, 2022

    
    
  
    

    
    

    
    
    
    
      
    
    
    
    

      

          @msyyc
msyyc




        reopened this

      Mar 10, 2022

    
    
  
    





    

      
    
  
        



    

      
    
  
          
    
  
    
    
  
    

  
    
      
    
    @openapi-sdkautomation
openapi-sdkautomation
bot


    mentioned this pull request
    
      Mar 10, 2022
    

    


    
  


  
    
      
   Draft

  
    

    




  
    

    

    
    
    
    
      
    
    
    
    

      

          @msyyc
msyyc




            closed this


      Mar 28, 2022

    
    
  
    

    
    

  
    
  
  
    
  
          @msyyc
  msyyc


    
    deleted the
    
      
        testremove-2022-03-09
    
    branch

    
      March 28, 2022 01:31
    
    

    

    



    





  
  
    
    
    
      

      
    



    
    

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


  

    

      
    
    
    
  
    


    
  
              

    


      
    
  
    
    
    
    
  
    
    Reviewers
  
    

    

    No reviews



    


    
    
  
    

      
  
    
    Assignees
  
    


        

    No one assigned




    


      

    
  


  
    
    Labels
  
    


    
    
      

    CI-BreakingChange-Go-V2



    


    


      

  
    
    
    
        
    
    Projects
  
    

        
    

    

    None yet




      
    


      

    
  
    
    
  
    
    Milestone
  
    

      No milestone


    


    
      
          

    
  
    
    
      
    
        
    
    
    Development
  
    


          


    Successfully merging this pull request may close these issues.
    


  


            
  
    

    

        
          

    

  
      
    
  
    
    
    
      1 participant
    
    
    
    
        
          @msyyc