Skip to content

Conversation

@v-Ajnava
Copy link
Contributor

Latest improvements:

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

Contribution checklist:

  • I have reviewed the documentation for the workflow.
  • Validation tools were run on swagger spec(s) and have all been fixed in this PR.
  • The OpenAPI Hub was used for checking validation status and next steps.

ARM API Review Checklist

  • Service team MUST add the "WaitForARMFeedback" label if the management plane API changes fall into one of the below categories.
  • adding/removing APIs.
  • adding/removing properties.
  • adding/removing API-version.
  • adding a new service in Azure.

Failure to comply may result in delays for manifest application. Note this does not apply to data plane APIs.

  • If you are blocked on ARM review and want to get the PR merged urgently, please get the ARM oncall for reviews (RP Manifest Approvers team under Azure Resource Manager service) from IcM and reach out to them.
    Please follow the link to find more details on API review process.

@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 Feb 27, 2020

azure-sdk-for-go

⚠️ warning [Logs] [Expand Details]

@openapi-sdkautomation
Copy link

openapi-sdkautomation bot commented Feb 27, 2020

azure-sdk-for-net

️✔️ succeeded [Logs] [Expand Details]

@openapi-sdkautomation
Copy link

openapi-sdkautomation bot commented Feb 27, 2020

azure-sdk-for-js

️✔️ succeeded [Logs] [Expand Details]
  • ️✔️ Generate from 36bc47d with merge commit 9070df912fe01a857db2a7a550776d5ba2a303ab. SDK Automation 13.0.17.20191226.1
  • ️✔️@azure/arm-eventhub [Logs]  [Preview 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] loaded rollup.config.js with warnings
    [npmPack] (!) Unused external imports
    [npmPack] default imported from external module 'rollup' but never used
    [npmPack] 
    [npmPack] ./esm/eventHubManagementClient.js → ./dist/arm-eventhub.js...
    [npmPack] created ./dist/arm-eventhub.js in 395ms

@openapi-sdkautomation
Copy link

openapi-sdkautomation bot commented Feb 27, 2020

azure-sdk-for-java

⚠️ warning [Logs] [Expand Details]
  • ⚠️ Generate from 36bc47d with merge commit 9070df912fe01a857db2a7a550776d5ba2a303ab. SDK Automation 13.0.17.20191226.1
  • ️✔️sdk/eventhubs/mgmt-v2015_08_01 [Logs]  [Preview SDK Changes]
  • ️✔️sdk/eventhubs/mgmt-v2017_04_01 [Logs]  [Preview SDK Changes]
  • ⚠️sdk/eventhubs/mgmt-v2018_01_01_preview [Logs]  [Preview SDK Changes]
      [mvn] [ERROR] COMPILATION ERROR : 
      [mvn] [ERROR] /z/work/azure-sdk-for-java/sdk/eventhubs/mgmt-v2018_01_01_preview/src/main/java/com/microsoft/azure/management/eventhubs/v2018_01_01_preview/implementation/EventHub2018PreviewManagementClientImpl.java:[222,72] incompatible types: com.microsoft.azure.management.eventhubs.v2018_01_01_preview.implementation.EventHub2018PreviewManagementClientImpl cannot be converted to com.microsoft.azure.management.eventhubs.v2018_01_01_preview.implementation.EventHubManagementClientImpl
      [mvn] [ERROR] /z/work/azure-sdk-for-java/sdk/eventhubs/mgmt-v2018_01_01_preview/src/main/java/com/microsoft/azure/management/eventhubs/v2018_01_01_preview/implementation/EventHub2018PreviewManagementClientImpl.java:[223,68] incompatible types: com.microsoft.azure.management.eventhubs.v2018_01_01_preview.implementation.EventHub2018PreviewManagementClientImpl cannot be converted to com.microsoft.azure.management.eventhubs.v2018_01_01_preview.implementation.EventHubManagementClientImpl
      [mvn] [ERROR] /z/work/azure-sdk-for-java/sdk/eventhubs/mgmt-v2018_01_01_preview/src/main/java/com/microsoft/azure/management/eventhubs/v2018_01_01_preview/implementation/EventHub2018PreviewManagementClientImpl.java:[224,80] incompatible types: com.microsoft.azure.management.eventhubs.v2018_01_01_preview.implementation.EventHub2018PreviewManagementClientImpl cannot be converted to com.microsoft.azure.management.eventhubs.v2018_01_01_preview.implementation.EventHubManagementClientImpl
      [mvn] [ERROR] /z/work/azure-sdk-for-java/sdk/eventhubs/mgmt-v2018_01_01_preview/src/main/java/com/microsoft/azure/management/eventhubs/v2018_01_01_preview/implementation/EventHub2018PreviewManagementClientImpl.java:[225,72] incompatible types: com.microsoft.azure.management.eventhubs.v2018_01_01_preview.implementation.EventHub2018PreviewManagementClientImpl cannot be converted to com.microsoft.azure.management.eventhubs.v2018_01_01_preview.implementation.EventHubManagementClientImpl
      [mvn] [ERROR] Failed to execute goal org.apache.maven.plugins:maven-compiler-plugin:3.1:compile (default-compile) on project azure-mgmt-eventhubs: Compilation failure: Compilation failure: 
      [mvn] [ERROR] /z/work/azure-sdk-for-java/sdk/eventhubs/mgmt-v2018_01_01_preview/src/main/java/com/microsoft/azure/management/eventhubs/v2018_01_01_preview/implementation/EventHub2018PreviewManagementClientImpl.java:[222,72] incompatible types: com.microsoft.azure.management.eventhubs.v2018_01_01_preview.implementation.EventHub2018PreviewManagementClientImpl cannot be converted to com.microsoft.azure.management.eventhubs.v2018_01_01_preview.implementation.EventHubManagementClientImpl
      [mvn] [ERROR] /z/work/azure-sdk-for-java/sdk/eventhubs/mgmt-v2018_01_01_preview/src/main/java/com/microsoft/azure/management/eventhubs/v2018_01_01_preview/implementation/EventHub2018PreviewManagementClientImpl.java:[223,68] incompatible types: com.microsoft.azure.management.eventhubs.v2018_01_01_preview.implementation.EventHub2018PreviewManagementClientImpl cannot be converted to com.microsoft.azure.management.eventhubs.v2018_01_01_preview.implementation.EventHubManagementClientImpl
      [mvn] [ERROR] /z/work/azure-sdk-for-java/sdk/eventhubs/mgmt-v2018_01_01_preview/src/main/java/com/microsoft/azure/management/eventhubs/v2018_01_01_preview/implementation/EventHub2018PreviewManagementClientImpl.java:[224,80] incompatible types: com.microsoft.azure.management.eventhubs.v2018_01_01_preview.implementation.EventHub2018PreviewManagementClientImpl cannot be converted to com.microsoft.azure.management.eventhubs.v2018_01_01_preview.implementation.EventHubManagementClientImpl
      [mvn] [ERROR] /z/work/azure-sdk-for-java/sdk/eventhubs/mgmt-v2018_01_01_preview/src/main/java/com/microsoft/azure/management/eventhubs/v2018_01_01_preview/implementation/EventHub2018PreviewManagementClientImpl.java:[225,72] incompatible types: com.microsoft.azure.management.eventhubs.v2018_01_01_preview.implementation.EventHub2018PreviewManagementClientImpl cannot be converted to com.microsoft.azure.management.eventhubs.v2018_01_01_preview.implementation.EventHubManagementClientImpl
      [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

    @openapi-sdkautomation
    Copy link

    openapi-sdkautomation bot commented Feb 27, 2020

    azure-sdk-for-python

    ️✔️ succeeded [Logs] [Expand Details]
    • ️✔️ Generate from 36bc47d with merge commit 9070df912fe01a857db2a7a550776d5ba2a303ab. SDK Automation 13.0.17.20191226.1
    • ️✔️azure-mgmt-eventhub [Logs]  [Preview SDK Changes]
      [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)

    @azure-pipelines
    Copy link

    Azure Pipelines successfully started running 1 pipeline(s).

    @azure-pipelines
    Copy link

    Azure Pipelines successfully started running 1 pipeline(s).

    @azure-pipelines
    Copy link

    Azure Pipelines successfully started running 1 pipeline(s).

    @azure-pipelines
    Copy link

    Azure Pipelines successfully started running 1 pipeline(s).

    @v-Ajnava v-Ajnava added the WaitForARMFeedback <valid label in PR review process> add this label when ARM review is required label Feb 28, 2020
    "$ref": "#/parameters/subscriptionIdParameter"
    }
    ],
    "responses": {
    Copy link
    Contributor

    Choose a reason for hiding this comment

    The reason will be displayed to describe this comment to others. Learn more.

    Please include next link. If your service returns null for this field, that is perfectly fine. This will allow you to add paging later without a breaking change.

        "x-ms-pageable": {
          "nextLinkName": "nextLink"
        }
    

    Copy link
    Contributor Author

    Choose a reason for hiding this comment

    The reason will be displayed to describe this comment to others. Learn more.

    we are deprecating this API, its supported in older versions. removed from swagger.

    @azure-pipelines
    Copy link

    Azure Pipelines successfully started running 1 pipeline(s).

    @v-Ajnava
    Copy link
    Contributor Author

    v-Ajnava commented Mar 3, 2020

    @ryansbenson updated with changes and comments. One quick questions, is it okay to remove the changesrequested label to show up as review requested.

    @ravbhatnagar
    Copy link
    Contributor

    Signing off from ARM side as Ryan's comments have been addressed.

    @ravbhatnagar ravbhatnagar added ARMSignedOff <valid label in PR review process>add this label when ARM approve updates after review and removed ARMChangesRequested WaitForARMFeedback <valid label in PR review process> add this label when ARM review is required labels Mar 6, 2020
    @v-Ajnava v-Ajnava changed the title EventHub: Added New API to 2018-01-01-preview [DONOT MERGE] EventHub: Added New API to 2018-01-01-preview Mar 9, 2020
    @v-Ajnava v-Ajnava added the DoNotMerge <valid label in PR review process> use to hold merge after approval label Mar 9, 2020
    @azure-pipelines
    Copy link

    Azure Pipelines successfully started running 1 pipeline(s).

    @akning-ms akning-ms assigned ruowan and unassigned zikalino Nov 25, 2020
    @akning-ms
    Copy link
    Contributor

    akning-ms commented Nov 25, 2020

    Hi @v-Ajnava, is this PR still valid, as it is idle for a long time, looks this version(2018-01-01-preview) is already merged. if it is not in use. can you help close it?

    @akning-ms akning-ms added the Reviewed-ChangesRequested <valid label in PR review process>add this label when assignee request changes after review label Nov 25, 2020
    @v-Ajnava
    Copy link
    Contributor Author

    closing the PR as its not need know.

    @v-Ajnava v-Ajnava closed this Dec 14, 2020
    Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

    Labels

    ARMSignedOff <valid label in PR review process>add this label when ARM approve updates after review DoNotMerge <valid label in PR review process> use to hold merge after approval Reviewed-ChangesRequested <valid label in PR review process>add this label when assignee request changes after review

    Projects

    None yet

    Development

    Successfully merging this pull request may close these issues.

    7 participants