search

databricks / spark-pr-dashboard

Dashboard to aid in Spark pull request reviews
spark-prs.appspot.com
Apache License 2.0
54 stars 46 forks source link

Filter out Jenkins success / failure messages from AmplabJenkins #31

Closed JoshRosen closed 9 years ago

JoshRosen commented 10 years ago

Now that @amplabjenkins is also reporting statuses on our PRs, we should filter out its "build succeeded / build failed" messages so that the test outcome messages from our own script are viewable in the PR dashboard's tooltips.

nchammas commented 10 years ago

+1 on this.

nchammas commented 9 years ago

What if we created some kind of barebones API that Jenkins could call directly to set the PR status? Basically, we'd be replicating the Commit Status API that GitHub offers (but which we can't use).

JoshRosen commented 9 years ago

Meh, we might as well just remove "AmplabJenkins" from the list of QA bots and only count messages from SparkQA; this is simpler and avoids introducing any new code in Spark itself.