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

LogAnalytics: update readme and fix conflicts by autorest.java #11126

Closed
wants to merge 1 commit into from
Closed

LogAnalytics: update readme and fix conflicts by autorest.java #11126

wants to merge 1 commit into from

Conversation

xseeseesee
Copy link
Contributor

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

Contribution checklist:

If any further question about AME onboarding or validation tools, please view the FAQ.

ARM API Review Checklist

  • Ensure to check this box if one of the following scenarios meet updates in the PR, so that label “WaitForARMFeedback” will be added automatically to involve ARM API Review. Failure to comply may result in delays for manifest application. Note this does not apply to data plane APIs, all “removals” and “adding a new property” no more require ARM API review.

    • Adding new API(s)
    • Adding a new API version
    • Adding a new service

  • If you are blocked on ARM review and want to get the PR merged with urgency, please get the ARM oncall for reviews (RP Manifest Approvers team under Azure Resource Manager service) from IcM and reach out to them.

Breaking Change Review Checklist

If there are following updates in the PR, ensure to request an approval from API Review Board as defined in the Breaking Change Policy.

  • Removing API(s) in stable version
  • Removing properties in stable version
  • Removing API version(s) in stable version
  • Updating API in stable version with Breaking Change Validation errors
  • Updating API(s) in preview over 1 year

Please follow the link to find more details on PR review process.

@xseeseesee xseeseesee requested a review from hohershk October 10, 2020 05:49
@openapi-pipeline-app
Copy link

openapi-pipeline-app bot commented Oct 10, 2020 •
edited
Loading

[Staging] Swagger Validation Report

️❌BreakingChange: 1 Errors, 0 Warnings [Detail] [Expand]
Rule Message
❌ 1006 - RemovedDefinition The new version is missing a definition that was found in the old version. Was 'LinkedStorageAccounts' removed or renamed?
New: Microsoft.OperationalInsights/preview/2019-08-01-preview/OperationalInsights.json#L425:3
Old: Microsoft.OperationalInsights/preview/2019-08-01-preview/OperationalInsights.json#L425:3
️❌LintDiff: 1 Errors, 0 Warnings [Detail] [Expand]
Rule Message
❌ AutoRest Exception "details":" "Channel": "fatal",
"Text": "swagger-document/compose - FAILED" "Channel": "fatal",
"Text": "Error: '$.parameters.SubscriptionIdParameter.description' has incompatible values (---\nThe workspace's resource subscription ID.\n,
---\n>-\n Gets subscription credentials which uniquely identify Microsoft Azure\n subscription. The subscription ID forms part of the URI for every service\n call.\n)."(node:5841) UnhandledPromiseRejectionWarning: Error: '$.parameters.SubscriptionIdParameter.description' has incompatible values (---The workspace's resource subscription ID.",
"location":"https://github.com/Azure/azure-rest-api-specs/blob/fa184e5e2ffd1abd1cab37811730ecfae9ed2542/specification/operationalinsights/resource-manager/readme.md"
️️✔️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 Oct 10, 2020 •
edited
Loading

Azure CLI Extension Generation

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

azure-sdk-for-go

⚠️ warning [Logs] [Expand Details]

@openapi-sdkautomation
Copy link

openapi-sdkautomation bot commented Oct 10, 2020 •
edited
Loading

azure-sdk-for-java

⚠️ warning [Logs] [Expand Details]
  • ⚠️ Generate from fa184e5 with merge commit 3bc54e0a28e1853c23456ee3eb0e3ead27fe7a78. SDK Automation 13.0.17.20200918.2
  • ️✔️sdk/loganalytics/mgmt-v2015_03_20 [View full logs]  [Preview SDK Changes] 
    Error: spawn bundle ENOENT
Error: spawn rake ENOENT
  • ️✔️sdk/loganalytics/mgmt-v2015_11_01_preview [View full logs]  [Preview SDK Changes] 
    Error: spawn bundle ENOENT
Error: spawn rake ENOENT
  • ⚠️sdk/loganalytics/mgmt-v2019_08_01_preview [View full logs]  [Preview SDK Changes] 
      Error: spawn bundle ENOENT
Error: spawn rake ENOENT
[mvn] [ERROR] COMPILATION ERROR : 
[mvn] [ERROR] /z/work/azure-sdk-for-java/sdk/loganalytics/mgmt-v2019_08_01_preview/src/main/java/com/microsoft/azure/management/loganalytics/v2019_08_01_preview/implementation/LinkedStorageAccountsImpl.java:[64,106] cannot find symbol
[mvn]   symbol:   class DataSourceType
[mvn]   location: class com.microsoft.azure.management.loganalytics.v2019_08_01_preview.implementation.LinkedStorageAccountsImpl
[mvn] [ERROR] /z/work/azure-sdk-for-java/sdk/loganalytics/mgmt-v2019_08_01_preview/src/main/java/com/microsoft/azure/management/loganalytics/v2019_08_01_preview/implementation/LinkedStorageAccountsImpl.java:[80,84] cannot find symbol
[mvn]   symbol:   class DataSourceType
[mvn]   location: class com.microsoft.azure.management.loganalytics.v2019_08_01_preview.implementation.LinkedStorageAccountsImpl
[mvn] [ERROR] Failed to execute goal org.apache.maven.plugins:maven-compiler-plugin:3.1:compile (default-compile) on project azure-mgmt-loganalytics: Compilation failure: Compilation failure: 
[mvn] [ERROR] /z/work/azure-sdk-for-java/sdk/loganalytics/mgmt-v2019_08_01_preview/src/main/java/com/microsoft/azure/management/loganalytics/v2019_08_01_preview/implementation/LinkedStorageAccountsImpl.java:[64,106] cannot find symbol
[mvn] [ERROR]   symbol:   class DataSourceType
[mvn] [ERROR]   location: class com.microsoft.azure.management.loganalytics.v2019_08_01_preview.implementation.LinkedStorageAccountsImpl
[mvn] [ERROR] /z/work/azure-sdk-for-java/sdk/loganalytics/mgmt-v2019_08_01_preview/src/main/java/com/microsoft/azure/management/loganalytics/v2019_08_01_preview/implementation/LinkedStorageAccountsImpl.java:[80,84] cannot find symbol
[mvn] [ERROR]   symbol:   class DataSourceType
[mvn] [ERROR]   location: class com.microsoft.azure.management.loganalytics.v2019_08_01_preview.implementation.LinkedStorageAccountsImpl
[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
    • ⚠️sdk/loganalytics/mgmt-v2020_03_01_preview [View full logs]  [Preview SDK Changes] 
        Error: spawn bundle ENOENT
Error: spawn rake ENOENT
[mvn] [ERROR] COMPILATION ERROR : 
[mvn] [ERROR] /z/work/azure-sdk-for-java/sdk/loganalytics/mgmt-v2020_03_01_preview/src/main/java/com/microsoft/azure/management/loganalytics/v2020_03_01_preview/implementation/LinkedStorageAccountsImpl.java:[64,111] cannot find symbol
[mvn]   symbol:   class DataSourceType
[mvn]   location: class com.microsoft.azure.management.loganalytics.v2020_03_01_preview.implementation.LinkedStorageAccountsImpl
[mvn] [ERROR] /z/work/azure-sdk-for-java/sdk/loganalytics/mgmt-v2020_03_01_preview/src/main/java/com/microsoft/azure/management/loganalytics/v2020_03_01_preview/implementation/LinkedStorageAccountsImpl.java:[80,84] cannot find symbol
[mvn]   symbol:   class DataSourceType
[mvn]   location: class com.microsoft.azure.management.loganalytics.v2020_03_01_preview.implementation.LinkedStorageAccountsImpl
[mvn] [ERROR] Failed to execute goal org.apache.maven.plugins:maven-compiler-plugin:3.1:compile (default-compile) on project azure-mgmt-loganalytics: Compilation failure: Compilation failure: 
[mvn] [ERROR] /z/work/azure-sdk-for-java/sdk/loganalytics/mgmt-v2020_03_01_preview/src/main/java/com/microsoft/azure/management/loganalytics/v2020_03_01_preview/implementation/LinkedStorageAccountsImpl.java:[64,111] cannot find symbol
[mvn] [ERROR]   symbol:   class DataSourceType
[mvn] [ERROR]   location: class com.microsoft.azure.management.loganalytics.v2020_03_01_preview.implementation.LinkedStorageAccountsImpl
[mvn] [ERROR] /z/work/azure-sdk-for-java/sdk/loganalytics/mgmt-v2020_03_01_preview/src/main/java/com/microsoft/azure/management/loganalytics/v2020_03_01_preview/implementation/LinkedStorageAccountsImpl.java:[80,84] cannot find symbol
[mvn] [ERROR]   symbol:   class DataSourceType
[mvn] [ERROR]   location: class com.microsoft.azure.management.loganalytics.v2020_03_01_preview.implementation.LinkedStorageAccountsImpl
[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
      • ⚠️sdk/loganalytics/mgmt-v2020_08_01 [View full logs]  [Preview SDK Changes] 
          Error: spawn bundle ENOENT
Error: spawn rake ENOENT
[mvn] [ERROR] COMPILATION ERROR : 
[mvn] [ERROR] /z/work/azure-sdk-for-java/sdk/loganalytics/mgmt-v2020_08_01/src/main/java/com/microsoft/azure/management/loganalytics/v2020_08_01/implementation/LinkedStorageAccountsImpl.java:[64,111] cannot find symbol
[mvn]   symbol:   class DataSourceType
[mvn]   location: class com.microsoft.azure.management.loganalytics.v2020_08_01.implementation.LinkedStorageAccountsImpl
[mvn] [ERROR] /z/work/azure-sdk-for-java/sdk/loganalytics/mgmt-v2020_08_01/src/main/java/com/microsoft/azure/management/loganalytics/v2020_08_01/implementation/LinkedStorageAccountsImpl.java:[80,84] cannot find symbol
[mvn]   symbol:   class DataSourceType
[mvn]   location: class com.microsoft.azure.management.loganalytics.v2020_08_01.implementation.LinkedStorageAccountsImpl
[mvn] [ERROR] Failed to execute goal org.apache.maven.plugins:maven-compiler-plugin:3.1:compile (default-compile) on project azure-mgmt-loganalytics: Compilation failure: Compilation failure: 
[mvn] [ERROR] /z/work/azure-sdk-for-java/sdk/loganalytics/mgmt-v2020_08_01/src/main/java/com/microsoft/azure/management/loganalytics/v2020_08_01/implementation/LinkedStorageAccountsImpl.java:[64,111] cannot find symbol
[mvn] [ERROR]   symbol:   class DataSourceType
[mvn] [ERROR]   location: class com.microsoft.azure.management.loganalytics.v2020_08_01.implementation.LinkedStorageAccountsImpl
[mvn] [ERROR] /z/work/azure-sdk-for-java/sdk/loganalytics/mgmt-v2020_08_01/src/main/java/com/microsoft/azure/management/loganalytics/v2020_08_01/implementation/LinkedStorageAccountsImpl.java:[80,84] cannot find symbol
[mvn] [ERROR]   symbol:   class DataSourceType
[mvn] [ERROR]   location: class com.microsoft.azure.management.loganalytics.v2020_08_01.implementation.LinkedStorageAccountsImpl
[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 Oct 10, 2020 •
        edited
        Loading

        azure-sdk-for-js

        ⚠️ warning [Logs] [Expand Details]
        • ⚠️ Generate from fa184e5 with merge commit 3bc54e0a28e1853c23456ee3eb0e3ead27fe7a78. SDK Automation 13.0.17.20200918.2 
          Failed to find any diff after autorest so no changed packages was found.

        @openapi-sdkautomation
        Copy link

        openapi-sdkautomation bot commented Oct 10, 2020 •
        edited
        Loading

        azure-sdk-for-python

        ⚠️ warning [Logs] [Expand Details]
        • ⚠️ Generate from fa184e5 with merge commit 3bc54e0a28e1853c23456ee3eb0e3ead27fe7a78. SDK Automation 13.0.17.20200918.2 
          Failed to find any diff after autorest so no changed packages was found.

        @openapi-sdkautomation
        Copy link

        openapi-sdkautomation bot commented Oct 10, 2020 •
        edited
        Loading

        azure-sdk-for-net

        ️✔️ succeeded [Logs] [Expand Details]

        @openapi-sdkautomation
        Copy link

        openapi-sdkautomation bot commented Oct 10, 2020 •
        edited
        Loading

        azure-sdk-for-python-track2

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

        azure-resource-manager-schemas

        ⚠️ warning [Logs] [Expand Details]
        • ⚠️ Generate from fa184e5 with merge commit 3bc54e0a28e1853c23456ee3eb0e3ead27fe7a78. Schema Automation 13.0.17.20200918.2 
          Failed to find any diff after autorest so no changed packages was found.

        @openapi-sdkautomation
        Copy link

        openapi-sdkautomation bot commented Oct 10, 2020 •
        edited
        Loading

        Trenton Generation

        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 Oct 10, 2020
        Open
        Open
        Open
        Open
        Open
        Open
        @akning-ms
        Copy link
        Contributor

        /azp run unifiedPipeline

        @azure-pipelines
        Copy link

        No pipelines are associated with this pull request.

        @ghost
        Copy link

        ghost commented Dec 27, 2020

        Hi, @xccc-msft. Your PR has no update for 14 days and it is marked as stale PR. If no further update for over 14 days, the bot will close the PR. If you want to refresh the PR, please remove no-recent-activity label.

        @ghost ghost added the no-recent-activity label Dec 27, 2020
        @ghost ghost closed this Mar 22, 2021
        @ghost
        Copy link

        ghost commented Mar 22, 2021

        Hi, @xccc-msft. The PR will be closed since the PR has no update for 28 days. If you still need the PR review to proceed, please reopen it and @ mention PR assignee @myronfanqiu, @akning-ms.

        @JackTn
        Copy link
        Member

        JackTn commented Aug 23, 2021

        Hi, @xseeseesee. The PR has be closed for a long time and it's related branch still exist. If no further used for over 14 days, I will delete this related branch. Please tell me if you still need this branch.

        @JackTn
        Copy link
        Member

        JackTn commented Jun 16, 2022

        Hi, @xseeseesee. The PR has be closed for a long time and it's related branch still exist. Please tell me if you still need this branch or i will delete it in 14 days.

        @JackTn JackTn deleted the java-update-loganalytics branch July 5, 2022 01:21
        This pull request was closed.
        Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
        Reviewers

        @hohershk hohershk Awaiting requested review from hohershk

        Assignees

        @mmyyrroonn mmyyrroonn

        @akning-ms akning-ms

        Labels
        None yet
        Projects
        None yet
        Milestone
        No milestone
        Development

        Successfully merging this pull request may close these issues.
        4 participants
        @xseeseesee @akning-ms @JackTn @mmyyrroonn