Skip to content

Switch continuous tests to only test changed files. #3339

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
Merged
kurtisvg merged 4 commits into from
Apr 11, 2020
Merged

Switch continuous tests to only test changed files. #3339

kurtisvg merged 4 commits into from
Apr 11, 2020

Conversation

@kurtisvg
Copy link
Contributor

@kurtisvg kurtisvg commented Apr 10, 2020 •
edited
Loading

  • Switches continuous tests to only test files changed from the last commit
  • Switches buildcop to only report fails from the periodic build

Fixes #3314

@kurtisvg kurtisvg requested a review from a team as a code owner April 10, 2020 18:41
@googlebot googlebot added the cla: yes This human has signed the Contributor License Agreement. label Apr 10, 2020
tmatsuo
tmatsuo reviewed Apr 10, 2020
View reviewed changes
leahecole
leahecole approved these changes Apr 10, 2020
View reviewed changes
Copy link
Collaborator

@leahecole leahecole left a comment

Choose a reason for hiding this comment

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

I think it's fine but I'm generally not comfortable being the authority on kokoro

@kurtisvg
Copy link
Contributor Author

kurtisvg commented Apr 10, 2020

Presubmit check still workin

@kurtisvg
Copy link
Contributor Author

kurtisvg commented Apr 11, 2020

Verified continuous tests are working

@kurtisvg kurtisvg merged commit d5afc7c into master Apr 11, 2020
@kurtisvg kurtisvg deleted the test-tweak branch April 11, 2020 01:48
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@leahecole leahecole leahecole approved these changes

+2 more reviewers

@tmatsuo tmatsuo tmatsuo approved these changes

@busunkim96 busunkim96 busunkim96 approved these changes

Reviewers whose approvals may not affect merge requirements

Assignees

@gguuss gguuss

Labels

cla: yes This human has signed the Contributor License Agreement.

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

testing: only run tests which has code changes in continuous builds

6 participants

@kurtisvg @tmatsuo @leahecole @busunkim96 @gguuss @googlebot