Skip to content

Comments

[Hub Generated] Review request for Microsoft.ApiManagement to add version preview/2019-12-01-preview#8046

Closed
solankisamir wants to merge 7 commits intomasterfrom
apim-2019-12-01-preview
Closed

[Hub Generated] Review request for Microsoft.ApiManagement to add version preview/2019-12-01-preview#8046
solankisamir wants to merge 7 commits intomasterfrom
apim-2019-12-01-preview

Conversation

@solankisamir
Copy link
Member

@solankisamir solankisamir commented Dec 30, 2019

If you are a MSFT employee you can view your work branch via this link.

Contribution checklist:

solankisamir and others added 7 commits October 17, 2019 08:41
* ApiManagement - renaming /properties entity to /namedValues

* ApiManagement - renaming parameter name: propId to namedValueId
* ApiManagement - async operatins for NamedValues and ApiSchemas

* ApiManagement - client secrets in IdentityProvider

* ApiManagement - json format fixes

* ApiManagement - examples fixes

* ApiManagement - fixed identtyProviderContract

* ApiManagement - secrets in subscriptions

* ApiManagement - secrets in OpenIdConnectProviders and AuthServers

* ApiManagement - secrets in TenantAccess

* ApiManagement - fix example

* ApiManagement - fixes
* ApiManagement - tagDescription id

* ApiManagement - policyDescriptions

* ApiManagement - policyDescriptions small fix

* ApiManagement - policyDescriptions sample

* ApiManagement - revisions id and api schema doc

* ApiManagement - tag descriptions list example

* ApiManagement - ciphers info
* update error response for apim deployment

* disable gateway

* user assigned identities

* api version contraint

* update examples

* fix model issues in examples

* update example

* replace sasolankrp with apimService1

Co-authored-by: Samir Solanki <samirsolanki@outlook.com>
@azure-pipelines
Copy link

Azure Pipelines successfully started running 1 pipeline(s).

1 similar comment
@azure-pipelines
Copy link

Azure Pipelines successfully started running 1 pipeline(s).

@openapi-sdkautomation
Copy link

openapi-sdkautomation bot commented Dec 30, 2019

azure-sdk-for-java

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 Dec 30, 2019

azure-sdk-for-python

️✔️ succeeded [Logs] [Expand Details]
  • ️✔️ Generate from 0615132 with merge commit a34460827bfc0800ed623de420fc779d1a6c3683. SDK Automation 13.0.17.20191226.1
  • ️✔️azure-mgmt-apimanagement [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] 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'

@openapi-sdkautomation
Copy link

openapi-sdkautomation bot commented Dec 30, 2019

azure-sdk-for-net

failed [Logs] [Expand Details]
  • Generate from 0615132 with merge commit a34460827bfc0800ed623de420fc779d1a6c3683. SDK Automation 13.0.17.20191226.1
    [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.ApiManagement [Logs]  [Preview SDK Changes]
      Failed to create the package Microsoft.Azure.Management.ApiManagement.
      Error: dotnet msbuild build.proj /t:CreateNugetPackage /p:Scope=apimanagement /v:n /p:SkipTests=true , {} 

    @openapi-sdkautomation
    Copy link

    openapi-sdkautomation bot commented Dec 30, 2019

    azure-sdk-for-js

    ️✔️ succeeded [Logs] [Expand Details]
    • ️✔️ Generate from 0615132 with merge commit a34460827bfc0800ed623de420fc779d1a6c3683. SDK Automation 13.0.17.20191226.1
    • ️✔️@azure/arm-apimanagement [Logs]  [Preview SDK Changes]
      [npmPack] npm WARN deprecated rollup-plugin-node-resolve@5.2.0: 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/apiManagementClient.js → ./dist/arm-apimanagement.js...
      [npmPack] created ./dist/arm-apimanagement.js in 1.7s

    @openapi-sdkautomation
    Copy link

    openapi-sdkautomation bot commented Dec 30, 2019

    azure-sdk-for-go

    ⚠️ warning [Logs] [Expand Details]
    • ⚠️ Generate from 0615132 with merge commit a34460827bfc0800ed623de420fc779d1a6c3683. SDK Automation 13.0.17.20191226.1
    • ⚠️apimanagement/mgmt/2016-07-07 [Logs
        No file is changed.
      • ⚠️apimanagement/mgmt/2016-10-10 [Logs
          No file is changed.
        • ⚠️apimanagement/mgmt/2017-03-01 [Logs
            No file is changed.
          • ⚠️apimanagement/mgmt/2018-01-01 [Logs
              No file is changed.
            • ⚠️apimanagement/mgmt/2019-01-01 [Logs
                No file is changed.
              • ⚠️preview/apimanagement/mgmt/2018-06-01-preview [Logs
                  No file is changed.

                @azuresdkci
                Copy link
                Contributor

                Can one of the admins verify this patch?

                @azure-pipelines
                Copy link

                Azure Pipelines successfully started running 1 pipeline(s).

                @solankisamir
                Copy link
                Member Author

                @NullMDR .net sdk might require some breaking changes in the SchemaContract definition, as we had put in custom implementation there. So the failure is expected.

                @PhoenixHe-NV
                Copy link

                @isra-fel Another .net generation failure FYI. From log I guess it's related to the dotnet project settings.

                @solankisamir Everything else looks good to me. You can fix the prettier check by following https://aka.ms/AA6h31t

                @isra-fel
                Copy link
                Member

                isra-fel commented Jan 3, 2020

                Got it. I'm fine with .net SDK

                @solankisamir
                Copy link
                Member Author

                @NullMDR @isra-fel created a clone of this PR #8066, as I couldn't edit changes in this branch.

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

                Labels

                API Management WaitForARMFeedback <valid label in PR review process> add this label when ARM review is required

                Projects

                None yet

                Development

                Successfully merging this pull request may close these issues.

                5 participants