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

Update readme.python.md #25320

Merged
merged 2 commits into from
Aug 16, 2023
Merged

Update readme.python.md #25320

merged 2 commits into from
Aug 16, 2023

Conversation

ChenxiJiang333
Copy link
Member

@openapi-workflow-bot
Copy link

Hi, @ChenxiJiang333! Thank you for your pull request. To help get your PR merged:

  • Ensure you reviewed the checklists in the PR description.
  • Know that PR assignee is the person auto-assigned and responsible for your current PR review and approval.
  • For convenient view of the API changes made by this PR, refer to the URLs provided in the table in the Generated ApiView comment added to this PR. You can use ApiView to show API versions diff.

    • @openapi-pipeline-app
    Copy link

    openapi-pipeline-app bot commented Aug 16, 2023 •
    edited
    Loading

    Next Steps to Merge

    ⚠️ This is an experimental comment. It may not always be up-to-date. ⚠️

    ✔️ All automated merging requirements have been met! Refer to step 4 in the PR workflow diagram (even if your PR is for data plane, not ARM).

    @openapi-pipeline-app
    Copy link

    openapi-pipeline-app bot commented Aug 16, 2023 •
    edited
    Loading

    Swagger Validation Report

    ️️✔️BreakingChange succeeded [Detail] [Expand] 
    There are no breaking changes.
    ️️✔️Breaking Change(Cross-Version) succeeded [Detail] [Expand] 
    There are no breaking changes.
    ️️✔️CredScan succeeded [Detail] [Expand] 
    There is no credential detected.
    ️️✔️LintDiff succeeded [Detail] [Expand] 
    Validation passes for LintDiff.
    compared tags (via openapi-validator v2.1.4) new version base version
    default default(76beafb) default(main)
    ️️✔️Avocado succeeded [Detail] [Expand] 
    Validation passes for Avocado.
    ️️✔️SwaggerAPIView succeeded [Detail] [Expand]
    ️️✔️TypeSpecAPIView succeeded [Detail] [Expand]
    ️️✔️ModelValidation succeeded [Detail] [Expand] 
    Validation passes for ModelValidation.
    ️️✔️SemanticValidation succeeded [Detail] [Expand] 
    Validation passes for SemanticValidation.
    ️️✔️PoliCheck succeeded [Detail] [Expand] 
    Validation passed for PoliCheck.
    ️️✔️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).
    ️️✔️PR Summary succeeded [Detail] [Expand] 
    Validation passes for Summary.
    ️️✔️Automated merging requirements met succeeded [Detail] [Expand]
    Posted by Swagger Pipeline | How to fix these errors?

    @openapi-pipeline-app
    Copy link

    openapi-pipeline-app bot commented Aug 16, 2023 •
    edited
    Loading

    Swagger Generation Artifacts

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

    Breaking Changes Tracking


    ❌azure-sdk-for-python-track2 - track2_azure-mgmt-iothub - Approved - 2.4.0
    +	Model RoutingCosmosDBSqlApiProperties has a new required parameter container_name
+	Model RoutingCosmosDBSqlApiProperties no longer has parameter collection_name
+	Model RoutingEndpoints no longer has parameter cosmos_db_sql_collections
    ️️✔️ azure-sdk-for-net-track2 succeeded [Detail] [Expand]
    • ️✔️Succeeded [Logs]Release - Generate from 1df6d6f. 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
command	pwsh ./eng/scripts/Invoke-GenerateAndBuildV2.ps1 ../azure-sdk-for-net_tmp/generateInput.json ../azure-sdk-for-net_tmp/generateOutput.json
    • ️✔️Azure.ResourceManager.IotHub [View full logs]  [Release SDK Changes] 
      info	[Changelog]
    ️⚠️ azure-sdk-for-python-track2 warning [Detail]
    • ⚠️Warning [Logs]Release - Generate from 1df6d6f. 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! 9.6.7 -> 9.8.1
cmderr	[automation_generate.sh] npm notice Changelog: <https://github.com/npm/cli/releases/tag/v9.8.1>
cmderr	[automation_generate.sh] npm notice Run `npm install -g npm@9.8.1` to update!
cmderr	[automation_generate.sh] npm notice
    • ️✔️track2_azure-mgmt-iothub [View full logs]  [Release SDK Changes] Breaking Change Detected
      info	[Changelog] ### Features Added
info	[Changelog]
info	[Changelog]   - Model RoutingEndpoints has a new parameter cosmos_db_sql_containers
info	[Changelog]
info	[Changelog] ### Breaking Changes
info	[Changelog]
info	[Changelog]   - Model RoutingCosmosDBSqlApiProperties has a new required parameter container_name
info	[Changelog]   - Model RoutingCosmosDBSqlApiProperties no longer has parameter collection_name
info	[Changelog]   - Model RoutingEndpoints no longer has parameter cosmos_db_sql_collections
    ️⚠️ azure-sdk-for-java warning [Detail]
    • ⚠️Warning [Logs]Release - Generate from 1df6d6f. 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
cmderr	[init.sh] [notice] A new release of pip is available: 23.0.1 -> 23.2.1
cmderr	[init.sh] [notice] To update, run: pip install --upgrade pip
cmderr	[init.sh] [notice] A new release of pip is available: 23.0.1 -> 23.2.1
cmderr	[init.sh] [notice] To update, run: pip install --upgrade pip
command	./eng/mgmt/automation/generate.py ../azure-sdk-for-java_tmp/generateInput.json ../azure-sdk-for-java_tmp/generateOutput.json
    • ️✔️azure-resourcemanager-iothub [View full logs]  [Release SDK Changes]
    ️️✔️ azure-sdk-for-go succeeded [Detail] [Expand]
    • ️✔️Succeeded [Logs]Release - Generate from 1df6d6f. 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
    • ️✔️sdk/resourcemanager/iothub/armiothub [View full logs]  [Release SDK Changes] Breaking Change Detected
      info	[Changelog] ### Breaking Changes
info	[Changelog]
info	[Changelog] - Field `CollectionName` of struct `RoutingCosmosDBSQLAPIProperties` has been removed
info	[Changelog] - Field `CosmosDBSQLCollections` of struct `RoutingEndpoints` has been removed
info	[Changelog]
info	[Changelog] ### Features Added
info	[Changelog]
info	[Changelog] - New field `ContainerName` in struct `RoutingCosmosDBSQLAPIProperties`
info	[Changelog] - New field `CosmosDBSQLContainers` in struct `RoutingEndpoints`
info	[Changelog]
info	[Changelog] Total 2 breaking change(s), 2 additive change(s).
    ️️✔️ azure-sdk-for-js succeeded [Detail] [Expand]
    • ️✔️Succeeded [Logs]Release - Generate from 1df6d6f. 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
    • ️✔️@azure/arm-iothub [View full logs]  [Release SDK Changes] 
      info	[Changelog] **Features**
info	[Changelog]
info	[Changelog]   - Added Interface EncryptionPropertiesDescription
info	[Changelog]   - Added Interface IotHubManualFailoverHeaders
info	[Changelog]   - Added Interface IotHubPropertiesDeviceStreams
info	[Changelog]   - Added Interface IotHubResourceCreateOrUpdateHeaders
info	[Changelog]   - Added Interface IotHubResourceDeleteHeaders
info	[Changelog]   - Added Interface IotHubResourceUpdateHeaders
info	[Changelog]   - Added Interface KeyVaultKeyProperties
info	[Changelog]   - Added Interface PrivateEndpointConnectionsDeleteHeaders
info	[Changelog]   - Added Interface PrivateEndpointConnectionsUpdateHeaders
info	[Changelog]   - Added Interface RootCertificateProperties
info	[Changelog]   - Added Interface RoutingCosmosDBSqlApiProperties
info	[Changelog]   - Added Type Alias IpVersion
info	[Changelog]   - Interface IotHubProperties has a new optional parameter deviceStreams
info	[Changelog]   - Interface IotHubProperties has a new optional parameter encryption
info	[Changelog]   - Interface IotHubProperties has a new optional parameter ipVersion
info	[Changelog]   - Interface IotHubProperties has a new optional parameter rootCertificate
info	[Changelog]   - Interface RoutingEndpoints has a new optional parameter cosmosDBSqlContainers
info	[Changelog]   - Add parameters of IotHubResourceUpdateHeaders to TypeAlias IotHubResourceUpdateResponse
info	[Changelog]   - Added Enum KnownIpVersion
info	[Changelog]   - Enum KnownRoutingSource has a new value DigitalTwinChangeEvents
info	[Changelog]   - Enum KnownRoutingSource has a new value MqttBrokerMessages
info	[Changelog]   - Added function getContinuationToken
    ️️✔️ azure-resource-manager-schemas succeeded [Detail] [Expand]
    • ️✔️Succeeded [Logs]Release - Generate from 1df6d6f. Schema Automation 14.0.0 
      command	.sdkauto/initScript.sh ../azure-resource-manager-schemas_tmp/initInput.json ../azure-resource-manager-schemas_tmp/initOutput.json
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
    • ️✔️iothub [View full logs]  [Release Schema Changes]
    ️❌ azure-powershell failed [Detail]
    • ❌Pipeline Framework Failed [Logs]Release - Generate from 1df6d6f. 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
SSL error: syscall failure: Broken pipe
Error: SSL error: syscall failure: Broken pipe
    • ⚠️Az.iothub.DefaultTag [View full logs]  
      error	Fatal error: SSL error: syscall failure: Broken pipe
error	The following packages are still pending:
error		Az.iothub.DefaultTag
    Posted by Swagger Pipeline | How to fix these errors?

    @openapi-pipeline-app
    Copy link

    openapi-pipeline-app bot commented Aug 16, 2023 •
    edited
    Loading

    Generated ApiView

    Language Package Name ApiView Link
    Go sdk/resourcemanager/iothub/armiothub https://apiview.dev/Assemblies/Review/332957eb44f24354ba19c295eca94e52
    .Net Azure.ResourceManager.IotHub https://apiview.dev/Assemblies/Review/45463c21753b4e7682bb3f5dea42a4cc
    Java azure-resourcemanager-iothub https://apiview.dev/Assemblies/Review/fa4c6fa3f3ee44af8954cf8b2bec2a54
    JavaScript @azure/arm-iothub https://apiview.dev/Assemblies/Review/3c1e0cce4fdf4f9b8a5e8f6d64df8ef0

    @azure-sdk azure-sdk mentioned this pull request Aug 16, 2023
    Closed
    @msyyc msyyc marked this pull request as ready for review August 16, 2023 02:58
    @msyyc msyyc self-requested a review as a code owner August 16, 2023 02:58
    @openapi-pipeline-app
    Copy link

    openapi-pipeline-app bot commented Aug 16, 2023 •
    edited
    Loading

    Automatic PR validation restarted. This comment will be populated with next steps to merge this PR once validation is completed. Please wait ⌛.

    @azure-sdk azure-sdk mentioned this pull request Aug 16, 2023
    Closed
    @openapi-pipeline-app
    Copy link

    Automatic PR validation started. This comment will be populated with next steps to merge this PR once validation is completed. Please wait ⌛.

    @ChenxiJiang333 ChenxiJiang333 assigned msyyc and unassigned rkmanda Aug 16, 2023
    This was referenced Aug 16, 2023
    Closed
    Closed
    Closed
    Closed
    @AzureRestAPISpecReview AzureRestAPISpecReview added ARMReview ReadyForApiTest <valid label in PR review process>add this label when swagger and service APIs are ready for test WaitForARMFeedback <valid label in PR review process> add this label when ARM review is required labels Aug 16, 2023
    @msyyc msyyc added the ARMSignedOff <valid label in PR review process>add this label when ARM approve updates after review label Aug 16, 2023
    @openapi-workflow-bot openapi-workflow-bot bot removed the WaitForARMFeedback <valid label in PR review process> add this label when ARM review is required label Aug 16, 2023
    @msyyc msyyc merged commit 1df6d6f into Azure:main Aug 16, 2023
    @openapi-pipeline-app
    Copy link

    Swagger pipeline restarted successfully, please wait for status update in this comment.

    @azure-sdk azure-sdk mentioned this pull request Aug 16, 2023
    Closed
    This was referenced Aug 15, 2023
    Closed
    Closed
    Closed
    Closed
    Closed
    Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
    Reviewers

    @msyyc msyyc msyyc approved these changes

    Assignees

    @msyyc msyyc

    Labels
    Approved-SdkBreakingChange-Python ARMReview ARMSignedOff <valid label in PR review process>add this label when ARM approve updates after review CI-BreakingChange-Go ReadyForApiTest <valid label in PR review process>add this label when swagger and service APIs are ready for test resource-manager
    Projects
    None yet
    Milestone
    No milestone
    Development

    Successfully merging this pull request may close these issues.
    4 participants
    @ChenxiJiang333 @msyyc @rkmanda @AzureRestAPISpecReview