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

chore: don't run system tests with grpc-js #417

Merged
merged 2 commits into from
Feb 9, 2019
Merged

Conversation

alexander-fenster
Copy link
Contributor

@kinwa91 will enable system tests with grpc-js as a separate Kokoro task.
It's a little bit flaky now so will likely bring more harm than good.

But good news is that it works (#400)!

  • Tests and linter pass
  • Code coverage does not decrease (if any source code was changed)
  • Appropriate docs were updated (if necessary)

@kinwa91 will enable system tests with grpc-js as a separate Kokoro task.
@googlebot googlebot added the cla: yes This human has signed the Contributor License Agreement. label Feb 8, 2019
@alexander-fenster alexander-fenster added the kokoro:force-run Add this label to force Kokoro to re-run the tests. label Feb 8, 2019
@kokoro-team kokoro-team removed the kokoro:force-run Add this label to force Kokoro to re-run the tests. label Feb 8, 2019
@alexander-fenster alexander-fenster added the kokoro:force-run Add this label to force Kokoro to re-run the tests. label Feb 8, 2019
@kokoro-team kokoro-team removed the kokoro:force-run Add this label to force Kokoro to re-run the tests. label Feb 8, 2019
@alexander-fenster
Copy link
Contributor Author

@JustinBeckwith @kinwa91 Since #418 is on its way, can you approve&merge this one so that flaky system tests don't interfere with PRs? Thanks!

@jkwlui jkwlui self-requested a review February 9, 2019 00:31
@jkwlui jkwlui merged commit 934b49f into master Feb 9, 2019
alexander-fenster added a commit that referenced this pull request Feb 11, 2019
This was originally merged in #417 but looks like it got lost in merges. We still need to remove this since we now have a separate Kokoro task for grpc-js.
alexander-fenster added a commit that referenced this pull request Feb 11, 2019
This was originally merged in #417 but looks like it got lost in merges. We still need to remove this since we now have a separate Kokoro task for grpc-js.
@alexander-fenster alexander-fenster deleted the no-grpc-js-system-tests branch April 24, 2019 20:50
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
cla: yes This human has signed the Contributor License Agreement.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants