aspnetcore icon indicating copy to clipboard operation
aspnetcore copied to clipboard

[release/6.0] Backport deletion of playwright tests

Open HaoK opened this issue 3 years ago • 9 comments

These tests were removed in https://github.com/dotnet/aspnetcore/commit/5247048126a2bdfac17d44d63d497055a53d08a2 for 7.0, and due to pw having some infrastructure costs, better to remove these in 6.0 rather than update pw infrastructure

HaoK avatar Jan 03 '23 21:01 HaoK

Hi @HaoK. If this is not a tell-mode PR, please make sure to follow the instructions laid out in the servicing process document. Otherwise, please add tell-mode label.

ghost avatar Jan 03 '23 21:01 ghost

Per discussion with @captainsafia and @TanayParikh these playwright tests were removed in 7.0 so this just backports their removal to 6.0 so we avoid any extra playwright issues

HaoK avatar Jan 04 '23 17:01 HaoK

/azp help

TanayParikh avatar Jan 04 '23 21:01 TanayParikh

Supported commands

  • help:
    • Get descriptions, examples and documentation about supported commands
    • Example: help "command_name"
  • list:
    • List all pipelines for this repository using a comment.
    • Example: "list"
  • run:
    • Run all pipelines or specific pipelines for this repository using a comment. Use this command by itself to trigger all related pipelines, or specify specific pipelines to run.
    • Example: "run" or "run pipeline_name, pipeline_name, pipeline_name"
  • where:
    • Report back the Azure DevOps orgs that are related to this repository and org
    • Example: "where"

See additional documentation.

azure-pipelines[bot] avatar Jan 04 '23 21:01 azure-pipelines[bot]

/azp list

TanayParikh avatar Jan 04 '23 21:01 TanayParikh

/azp run aspnetcore-ci

TanayParikh avatar Jan 04 '23 21:01 TanayParikh

Azure Pipelines successfully started running 1 pipeline(s).

azure-pipelines[bot] avatar Jan 04 '23 21:01 azure-pipelines[bot]

@wtgodbe this PR is green, is there any other process we need to follow to get this in, or is this sufficient for the build team to merge during the window?

HaoK avatar Jan 05 '23 00:01 HaoK

This PR is test-only. Any reason not to merge it @HaoK❔

dougbu avatar Jan 11 '23 18:01 dougbu

Nope, should be good to merge!

HaoK avatar Jan 11 '23 19:01 HaoK