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

devops: run less tests on each PR #6518

Conversation

aslushnikov
Copy link
Contributor

With this patch, we have the following behaviors:

  • For PR's, we will run only Linux tests for Chromium, Firefox and
    Webkit
  • For commits to master and release-*, we'll run all the tests

To run all tests on PR, assign a "CQ1" label to the PR.

With this patch, we have the following behaviors:
- For PR's, we will run only Linux tests for Chromium, Firefox and
  Webkit
- For commits to master and release-*, we'll run all the tests

To run all tests on PR, assign a "CQ1" label to the PR.

on:
push:
branches:
- master
- release-*
pull_request:
types: [ labeled ]
Copy link
Member

Choose a reason for hiding this comment

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

Is it any label or cq1 specifically?

Copy link
Contributor Author

Choose a reason for hiding this comment

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

any label. It's quite messy to whitelist a specific one: you have to add an if condition in every job

Copy link
Member

@yury-s yury-s left a comment

Choose a reason for hiding this comment

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

Primary and secondary is somewhat misleading as they make "secondary" tests appear less important. How about 'minimal test suite' and 'all tests' or something like that?

@aslushnikov aslushnikov merged commit bf81a28 into microsoft:master May 12, 2021
@aslushnikov aslushnikov deleted the devops-split-tests-into-primary-and-secondary branch May 12, 2021 00:01
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants