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.

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

Xiaoxu/testreamd13 #2460

Closed
wants to merge 2 commits into from
Closed

Xiaoxu/testreamd13 #2460

wants to merge 2 commits into from

Conversation

xiaoxuqi-ms
Copy link

Choose a PR Template

Switch to "Preview" on this description then select one of the choices below.

Click here to open a PR for a Data Plane API.

Click here to open a PR for a Control Plane (ARM) API.

@openapi-pipeline-app-test
Copy link

openapi-pipeline-app-test bot commented Jul 11, 2022

Swagger Validation Report

️️✔️BreakingChange succeeded [Detail] [Expand]
There are no breaking changes.
compared swaggers (via Oad v0.9.6)] new version base version
adp.json 2021-11-01(d6a025e) 2021-11-01(main)
️️✔️Breaking Change(Cross-Version) succeeded [Detail] [Expand]
There are no breaking changes.
️⚠️LintDiff: 0 Warnings warning [Detail]
compared tags (via openapi-validator v1.13.0) new version base version
package-2021-11-01-preview package-2021-11-01-preview(d6a025e) package-2021-11-01-preview(main)

The following errors/warnings exist before current PR submission:

Rule Message
⚠️ R3010 - TrackedResourceListByImmediateParent The child tracked resource, 'dataPools' with immediate parent 'Account', must have a list by immediate parent operation.
Location: Microsoft.AutonomousDevelopmentPlatform/preview/2021-11-01-preview/adp.json#L806
⚠️ R3018 - EnumInsteadOfBoolean Booleans are not descriptive and make them hard to use. Consider using string enums with allowed set of values defined. Property: nameAvailable
Location: Microsoft.AutonomousDevelopmentPlatform/preview/2021-11-01-preview/adp.json#L686
⚠️ R3018 - EnumInsteadOfBoolean Booleans are not descriptive and make them hard to use. Consider using string enums with allowed set of values defined. Property: isDataAction
Location: Microsoft.AutonomousDevelopmentPlatform/preview/2021-11-01-preview/adp.json#L1012
️️✔️~[Staging] Lint(RPaaS) succeeded [Detail] [Expand]
Validation passes for Lint(RPaaS).
️️✔️ApiReadinessCheck succeeded [Detail] [Expand]
️️✔️~[Staging] ApiReadinessCheck succeeded [Detail] [Expand]
Validation passes for ApiReadinessCheck.
️⚠️~[Staging] TrafficValidation: 1 Warnings warning [Detail]
Rule Message
⚠️ Traffic validation result. "Starting task Test...\nUsing python: 'python3' (Python 3.9.5)\nRequest coverage (successful / total): 2 / 13\nNo bugs were found.\nTask Test succeeded.\nCollecting logs...\n"
️️✔️CredScan succeeded [Detail] [Expand]
There is no credential detected.
️️✔️PoliCheck succeeded [Detail] [Expand]
Validation passed for PoliCheck.
️️✔️Avocado succeeded [Detail] [Expand]
Validation passes for Avocado.
️️✔️ModelValidation succeeded [Detail] [Expand]
Validation passes for ModelValidation.
️️✔️SemanticValidation succeeded [Detail] [Expand]
Validation passes for SemanticValidation.
️️✔️~[Staging] PrettierCheck succeeded [Detail] [Expand]
Validation passes for PrettierCheck.
️️✔️~[Staging] SpellCheck succeeded [Detail] [Expand]
Validation passes for SpellCheck.
️⚠️[Staging] SDK Track2 Validation: 0 Warnings warning [Detail]
  • The following tags are being changed in this PR
    • "https://github.com/test-repo-billy/azure-rest-api-specs/blob/d6a025e0cd78df85e771148cd298656565062f5e/specification/adp/resource-manager/readme.md#tag-package-2021-11-01-preview">adp/resource-manager/readme.md#package-2021-11-01-preview
The following errors/warnings exist before current PR submission:
Rule Message
⚠️ UnkownSecurityScheme "readme":"adp/resource-manager/readme.md",
"tag":"package-2021-11-01-preview",
"details":"Security scheme azure_auth is unknown and will not be processed. Only supported types are AADToken,
AzureKey,
Anonymous"
Posted by Swagger Pipeline | How to fix these errors?

@workflow-bot-test
Copy link

Hi @xiaoxuqi-ms, this service has enabled Shift Left. All spec updates MUST be initiated from service ADO project repo, so that to guarantee it to be the source of truth.

  • You can refer to Shift Left Solution Introduction for details.
  • Please contact service focal @xiaoxuqi-ms contacts or Shift Left Support Channel if need further help.
  • @openapi-pipeline-app-test
    Copy link

    openapi-pipeline-app-test bot commented Jul 11, 2022

    Swagger Generation Artifacts

    ️️✔️~[Staging] ApiDocPreview succeeded [Detail] [Expand]
     Please click here to preview with your @microsoft account. 
    ️⚠️ azure-sdk-for-python-track2 warning [Detail]
    • ⚠️Warning [Logs] Generate from 6080e2fd2544f900ea278d3e9c0bcb3d9cf72f69. 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] config global `--global`, `--local` are deprecated. Use `--location=global` instead.
      cmderr	[automation_generate.sh] npm notice
      cmderr	[automation_generate.sh] npm notice New minor version of npm available! 8.11.0 -> 8.13.2
      cmderr	[automation_generate.sh] npm notice Changelog: <https://github.com/npm/cli/releases/tag/v8.13.2>
      cmderr	[automation_generate.sh] npm notice Run `npm install -g npm@8.13.2` to update!
      cmderr	[automation_generate.sh] npm notice
    • ️✔️track2_azure-mgmt-adp [View full logs]  [Preview SDK Changes]
      info	[Changelog]   - Initial Release
    ️️✔️ azure-sdk-for-java succeeded [Detail] [Expand]
    ️️✔️ azure-sdk-for-go succeeded [Detail] [Expand]
    • ️✔️Succeeded [Logs] Generate from 6080e2fd2544f900ea278d3e9c0bcb3d9cf72f69. SDK Automation 14.0.0
      command	sh ./eng/initScript.sh ../../../../../azure-sdk-for-go_tmp/initInput.json ../../../../../azure-sdk-for-go_tmp/initOutput.json
      command	generator automation ../../../../../azure-sdk-for-go_tmp/generateInput.json ../../../../../azure-sdk-for-go_tmp/generateOutput.json
    • ️✔️preview/adp/mgmt/2021-11-01-preview/adp [View full logs]  [Preview SDK Changes]
      info	[Changelog] This is a new package
      info	[Changelog]
    • ️✔️preview/adp/mgmt/2021-02-01-preview/adp [View full logs
      info	[Changelog] This is a new package
      info	[Changelog]
    • ️✔️preview/adp/mgmt/2020-07-01-preview/adp [View full logs
      info	[Changelog] This is a new package
      info	[Changelog]
    ️⚠️ azure-sdk-for-js warning [Detail]
    • ⚠️Warning [Logs] Generate from 6080e2fd2544f900ea278d3e9c0bcb3d9cf72f69. 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
      cmderr	[automation_init.sh] deprecated read-package-tree@5.1.6: The functionality that this package provided is now in @npmcli/arborist
      cmderr	[automation_init.sh]  uuid@3.4.0: Please upgrade  to version 7 or higher.  Older versions may use Math.random() in certain circumstances, which is known to be problematic.  See https://v8.dev/blog/math-random for details.
      cmderr	[automation_init.sh]  deprecated @opentelemetry/types@0.2.0: Package renamed to @opentelemetry/api, see https://github.com/open-telemetry/opentelemetry-js
      cmderr	[automation_init.sh]  deprecated @azure/storage-blob@10.5.0: This version has been deprecated, please upgrade to the version tagged as latest
      cmderr	[automation_init.sh] npm WARN @octokit/plugin-request-log@1.0.4 requires a peer of @octokit/core@>=3 but none is installed. You must install peer dependencies yourself.
      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
    • info	[Changelog]
      error	breakingChangeTracking is enabled, but version or changelogItem is not found in output.
    ️❌ azure-cli-extensions failed [Detail]
    • Failed [Logs] Generate from 6080e2fd2544f900ea278d3e9c0bcb3d9cf72f69. Azure CLI Automation 14.0.0
      warn	Skip initScript due to not configured
      command	npx n 12
      command	python3 -m venv venv
      command	pip install --upgrade setuptools wheel pip
      command	pip install pathlib jinja2 msrestazure
      command	autorest --az --use=@autorest/az@latest --version=3.0.6271 --clear-output-folder=true --azure-cli-extension-folder=/mnt/vss/_work/1/s/azure-cli-extensions ../azure-rest-api-specs/specification/adp/resource-manager/readme.md
      cmderr	[Autorest] ERROR: Schema violation: Additional properties not allowed: x-ms-identifiers
      cmderr	[Autorest]     - file:///mnt/vss/_work/1/s/azure-rest-api-specs/specification/adp/resource-manager/Microsoft.AutonomousDevelopmentPlatform/preview/2021-11-01-preview/adp.json:1:1
      cmderr	[Autorest] ERROR: Schema violation: Additional properties not allowed: x-ms-identifiers
      cmderr	[Autorest]     - file:///mnt/vss/_work/1/s/azure-rest-api-specs/specification/adp/resource-manager/Microsoft.AutonomousDevelopmentPlatform/preview/2021-11-01-preview/adp.json:1:1
      cmderr	[Autorest] ERROR: Schema violation: Additional properties not allowed: x-ms-identifiers
      cmderr	[Autorest]     - file:///mnt/vss/_work/1/s/azure-rest-api-specs/specification/adp/resource-manager/Microsoft.AutonomousDevelopmentPlatform/preview/2021-11-01-preview/adp.json:1:1
      cmderr	[Autorest] ERROR: Schema violation: Additional properties not allowed: x-ms-identifiers
      cmderr	[Autorest]     - file:///mnt/vss/_work/1/s/azure-rest-api-specs/specification/adp/resource-manager/Microsoft.AutonomousDevelopmentPlatform/preview/2021-11-01-preview/adp.json:1:1
      cmderr	[Autorest] ERROR: Schema violation: Additional properties not allowed: x-ms-identifiers
      cmderr	[Autorest]     - file:///mnt/vss/_work/1/s/azure-rest-api-specs/specification/adp/resource-manager/Microsoft.AutonomousDevelopmentPlatform/preview/2021-11-01-preview/adp.json:1:1
      cmderr	[Autorest] ERROR: Schema violation: Additional properties not allowed: x-ms-identifiers
      cmderr	[Autorest]     - file:///mnt/vss/_work/1/s/azure-rest-api-specs/specification/adp/resource-manager/Microsoft.AutonomousDevelopmentPlatform/preview/2021-11-01-preview/adp.json:1:1
      cmderr	[Autorest] ERROR: Schema violation: Additional properties not allowed: x-ms-identifiers
      cmderr	[Autorest]     - file:///mnt/vss/_work/1/s/azure-rest-api-specs/specification/adp/resource-manager/Microsoft.AutonomousDevelopmentPlatform/preview/2021-11-01-preview/adp.json:1:1
      cmderr	[Autorest] ERROR: Schema violation: Additional properties not allowed: x-ms-identifiers
      cmderr	[Autorest]     - file:///mnt/vss/_work/1/s/azure-rest-api-specs/specification/adp/resource-manager/Microsoft.AutonomousDevelopmentPlatform/preview/2021-11-01-preview/adp.json:1:1
      cmderr	[Autorest] ERROR: Schema violation: Additional properties not allowed: x-ms-identifiers
      cmderr	[Autorest]     - file:///mnt/vss/_work/1/s/azure-rest-api-specs/specification/adp/resource-manager/Microsoft.AutonomousDevelopmentPlatform/preview/2021-11-01-preview/adp.json:1:1
      cmderr	[Autorest] ERROR: Schema violation: Additional properties not allowed: x-ms-identifiers
      cmderr	[Autorest]     - file:///mnt/vss/_work/1/s/azure-rest-api-specs/specification/adp/resource-manager/Microsoft.AutonomousDevelopmentPlatform/preview/2021-11-01-preview/adp.json:1:1
      cmderr	[Autorest] ERROR: Schema violation: Additional properties not allowed: x-ms-identifiers
      cmderr	[Autorest]     - file:///mnt/vss/_work/1/s/azure-rest-api-specs/specification/adp/resource-manager/Microsoft.AutonomousDevelopmentPlatform/preview/2021-11-01-preview/adp.json:1:1
      cmderr	[Autorest] ERROR: Schema violation: Additional properties not allowed: x-ms-identifiers
      cmderr	[Autorest]     - file:///mnt/vss/_work/1/s/azure-rest-api-specs/specification/adp/resource-manager/Microsoft.AutonomousDevelopmentPlatform/preview/2021-11-01-preview/adp.json:1:1
      cmderr	[Autorest] Process() cancelled due to failure
      error	Script return with result [failed] code [1] signal [null] cwd [azure-cli-extensions]: autorest
      warn	Skip package processing as generation is failed
    Posted by Swagger Pipeline | How to fix these errors?

    @workflow-bot-test
    Copy link

    Hi, @xiaoxuqi-ms Thanks for your PR. I am workflow bot for review process. Here are some small tips.

  • Please ensure to do self-check against checklists in first PR comment.
  • PR assignee is the person auto-assigned and responsible for your current PR reviewing and merging.
  • For cross api-version diff. See OpenAPI Hub Diff
  • If there is CI failure(s), to fix CI error(s) is mandatory for PR merging; or you need to provide justification in PR comment for explanation. How to fix?

  • Any feedback about review process or workflow bot, pls contact swagger and tools team. vsswagger@microsoft.com

    @openapi-pipeline-app-test
    Copy link

    Hi @xiaoxuqi-ms, this service has enabled Shift Left. All spec updates MUST be initiated from service ADO project repo, so that to guarantee it to be the source of truth., please review this pull request if it's intentional or reject it if it's not expected. Normally, all the specification or example changes should start with PR created in ADO.

  • You can refer to Shift Left Solution Introduction for details.
  • Please contact service focal contacts or Shift Left Support Channel if need further help.
  • Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
    Projects
    None yet
    Development

    Successfully merging this pull request may close these issues.

    1 participant