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

Make ACS chat events "version" property as int64 #10984

Merged
merged 1 commit into from
Sep 29, 2020
Merged

Make ACS chat events "version" property as int64 #10984

merged 1 commit into from
Sep 29, 2020

Conversation

vikrampraveen
Copy link
Contributor

No description provided.

@openapi-pipeline-app
Copy link

openapi-pipeline-app bot commented Sep 29, 2020 •
edited
Loading

[Staging] Swagger Validation Report

️❌BreakingChange: 2 Errors, 0 Warnings [Detail] [Expand]
Rule Message
❌ 1023 - TypeFormatChanged The new version has a different format than the previous one.
New: Microsoft.Communication/stable/2018-01-01/AzureCommunicationServices.json#L239:9
Old: Microsoft.Communication/stable/2018-01-01/AzureCommunicationServices.json#L239:9
❌ 1023 - TypeFormatChanged The new version has a different format than the previous one.
New: Microsoft.Communication/stable/2018-01-01/AzureCommunicationServices.json#L275:9
Old: Microsoft.Communication/stable/2018-01-01/AzureCommunicationServices.json#L274:9
️️✔️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 29, 2020 •
edited
Loading

azure-sdk-for-python - Release

️✔️ succeeded [Logs] [Expand Details]
  • ️✔️ Generate from b2c8a66 with merge commit 5caf552. SDK Automation 13.0.17.20200918.2
  • ️✔️azure-eventgrid [View full logs]  [Release SDK Changes] 
    [build_conf] INFO:packaging_tools:Building template azure-eventgrid
[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-eventgrid
[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] Traceback (most recent call last):
[ChangeLog]   File "/usr/lib/python3.6/runpy.py", line 193, in _run_module_as_main
[ChangeLog]     "__main__", mod_spec)
[ChangeLog]   File "/usr/lib/python3.6/runpy.py", line 85, in _run_code
[ChangeLog]     exec(code, run_globals)
[ChangeLog]   File "/z/work/azure-sdk-for-python/tools/azure-sdk-tools/packaging_tools/change_log.py", line 200, in 
[ChangeLog]     change_log = build_change_log(old_report, new_report)
[ChangeLog]   File "/z/work/azure-sdk-for-python/tools/azure-sdk-tools/packaging_tools/change_log.py", line 147, in build_change_log
[ChangeLog]     change_log.operation(diff_line)
[ChangeLog]   File "/z/work/azure-sdk-for-python/tools/azure-sdk-tools/packaging_tools/change_log.py", line 38, in operation
[ChangeLog]     _, operation_name, *remaining_path = path
[ChangeLog] ValueError: not enough values to unpack (expected at least 2, got 1)
[ChangeLog] Size of delta 9.152% size of original (original: 96048 chars, delta: 8790 chars)

@openapi-sdkautomation
Copy link

openapi-sdkautomation bot commented Sep 29, 2020 •
edited
Loading

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 29, 2020 •
edited
Loading

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 29, 2020 •
edited
Loading

azure-sdk-for-js - Release

❌ failed [Logs] [Expand Details]
  • ❌ Generate from b2c8a66 with merge commit 5caf552. SDK Automation 13.0.17.20200918.2
  • ❌@azure/eventgrid [View full logs]  [Release 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] npm WARN deprecated resolve-url@0.2.1: https://github.com/lydell/resolve-url#deprecated
[npmPack] npm WARN deprecated urix@0.1.0: Please see https://github.com/lydell/urix#deprecated
[npmPack] npm notice created a lockfile as package-lock.json. You should commit this file.
[npmPack] npm ERR! code ELIFECYCLE
[npmPack] npm ERR! errno 2
[npmPack] npm ERR! @azure/eventgrid@3.0.0-beta.3 build: `tsc && rollup -c rollup.config.js && npm run minify`
[npmPack] npm ERR! Exit status 2
[npmPack] npm ERR! 
[npmPack] npm ERR! Failed at the @azure/eventgrid@3.0.0-beta.3 build script.
[npmPack] npm ERR! This is probably not a problem with npm. There is likely additional logging output above.
[npmPack] 
[npmPack] npm ERR! A complete log of this run can be found in:
[npmPack] npm ERR!     /home/sdk/.npm/_logs/2020-09-29T22_59_40_359Z-debug.log
[npmPack] npm ERR! code ELIFECYCLE
[npmPack] npm ERR! errno 2
[npmPack] npm ERR! @azure/eventgrid@3.0.0-beta.3 prepack: `npm install && npm run build`
[npmPack] npm ERR! Exit status 2
[npmPack] npm ERR! 
[npmPack] npm ERR! Failed at the @azure/eventgrid@3.0.0-beta.3 prepack script.
[npmPack] npm ERR! This is probably not a problem with npm. There is likely additional logging output above.
[npmPack] 
[npmPack] npm ERR! A complete log of this run can be found in:
[npmPack] npm ERR!     /home/sdk/.npm/_logs/2020-09-29T22_59_44_212Z-debug.log
Failed to create the package @azure/eventgrid.
Error: npm pack 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.
npm WARN deprecated resolve-url@0.2.1: https://github.com/lydell/resolve-url#deprecated
npm WARN deprecated urix@0.1.0: Please see https://github.com/lydell/urix#deprecated
npm notice created a lockfile as package-lock.json. You should commit this file.
npm ERR! code ELIFECYCLE
npm ERR! errno 2
npm ERR! @azure/eventgrid@3.0.0-beta.3 build: `tsc && rollup -c rollup.config.js && npm run minify`
npm ERR! Exit status 2
npm ERR! 
npm ERR! Failed at the @azure/eventgrid@3.0.0-beta.3 build script.
npm ERR! This is probably not a problem with npm. There is likely additional logging output above.

      npm ERR! A complete log of this run can be found in:
      
npm ERR!     /home/sdk/.npm/_logs/2020-09-29T22_59_40_359Z-debug.log
      
npm ERR! code ELIFECYCLE
      
npm ERR! errno 2
      
npm ERR! @azure/eventgrid@3.0.0-beta.3 prepack: npm install && npm run build
      
npm ERR! Exit status 2
      
npm ERR!
      
npm ERR! Failed at the @azure/eventgrid@3.0.0-beta.3 prepack script.
      
npm ERR! This is probably not a problem with npm. There is likely additional logging output above.
      

      npm ERR! A complete log of this run can be found in:
      
npm ERR!     /home/sdk/.npm/_logs/2020-09-29T22_59_44_212Z-debug.log
      
, {}

    @openapi-sdkautomation
    Copy link

    openapi-sdkautomation bot commented Sep 29, 2020 •
    edited
    Loading

    azure-sdk-for-go - Release

    ️✔️ succeeded [Logs] [Expand Details]

    @openapi-sdkautomation
    Copy link

    openapi-sdkautomation bot commented Sep 29, 2020 •
    edited
    Loading

    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 29, 2020 •
    edited
    Loading

    azure-sdk-for-net - 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 29, 2020 •
    edited
    Loading

    azure-sdk-for-java - 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 29, 2020 •
    edited
    Loading

    azure-resource-manager-schemas - 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

    This was referenced Sep 29, 2020
    Closed
    Closed
    lmazuel
    lmazuel approved these changes Sep 29, 2020
    View reviewed changes
    Copy link
    Member

    @lmazuel lmazuel left a comment

    Choose a reason for hiding this comment

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

    Changes have been checked with service team and RestAPI behavior

    @lmazuel lmazuel merged commit 5caf552 into master Sep 29, 2020
    @lmazuel lmazuel deleted the user/vipravee/acschateventsversionformat branch September 29, 2020 22:57
    @YijunXieMS YijunXieMS mentioned this pull request Oct 1, 2020
    Closed
    iscai-msft added a commit to iscai-msft/azure-rest-api-specs that referenced this pull request Oct 5, 2020
    @iscai-msft
    Merge branch 'master' of https://github.com/Azure/azure-rest-api-specs …
    edef031 
    …into fix_kv_python_readme

* 'master' of https://github.com/Azure/azure-rest-api-specs: (364 commits)
  Update pull_request_assignment.yml (Azure#11026)
  [Ready For Review] a new version with changes on compute instance/vm sizes (Azure#10542)
  correct the description (Azure#11009)
  [Data plane][Azure Cognitive Search] Add encryptionKey property to indexer, datasource and skillset metadata (Azure#10839)
  Adding reseller Id (Azure#11013)
  Update Phone Number Administration Swagger to remove unnecessary parameters, and adjust API responses. (Azure#11010)
  update all swaggers to 2020-05-01 API version (Azure#10960)
  Changed the Instance Field Description to Support Primitive Data type (Azure#10995)
  Fix the acronym casing for ACSSMS (Azure#10988)
  Update pull_request_assignment.yml (Azure#10974)
  Add sdk_customization.md (Azure#10861)
  [Hub Generated] Review request for Microsoft.DataMigration to add version preview/2018-07-15-preview (Azure#10494)
  Make version int64 (Azure#10984)
  Microsoft.Security/iotDefenderSettings (2020-08-06-preview) (Azure#10810)
  added incident alerts and incident bookmarks APIs (Azure#10806)
  add publicNetworkAccess property to datafactory (Azure#10956)
  Fix CDN matchValues and selector fields to be optional (Azure#10950)
  [Microsoft.StorageSync] Introduce 2020-09-01 API version (Azure#10836)
  strings update (Azure#10969)
  Update QnAMaker.json - response code correction (Azure#10828)
  ...
    Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
    Reviewers

    @Kishp01 Kishp01 Kishp01 approved these changes

    @lmazuel lmazuel lmazuel approved these changes

    @ahamad-MS ahamad-MS Awaiting requested review from ahamad-MS

    Assignees

    @anuchandy anuchandy

    Labels
    None yet
    Projects
    None yet
    Milestone
    No milestone
    Development

    Successfully merging this pull request may close these issues.
    4 participants
    @vikrampraveen @lmazuel @Kishp01 @anuchandy