Re: Automatically deleting pull request comments left by AmplabJenkins
On Fri, Aug 14, 2015 at 4:21 AM, Josh Rosen rosenvi...@gmail.com wrote: Prototype is at https://github.com/databricks/spark-pr-dashboard/pull/59 On Wed, Aug 12, 2015 at 7:51 PM, Josh Rosen rosenvi...@gmail.com wrote: *TL;DR*: would anyone object if I wrote a script to auto-delete pull request comments from AmplabJenkins? Currently there are two bots which post Jenkins test result comments to GitHub, AmplabJenkins and SparkQA. SparkQA is the account which post the detailed Jenkins start and finish messages that contain information on which commit is being tested and which tests have failed. This bot is controlled via the dev/run-tests-jenkins script. AmplabJenkins is controlled by the Jenkins GitHub Pull Request Builder plugin. This bot posts relatively uninformative comments (Merge build triggered, Merge build started, Merge build failed) that do not contain any links or details specific to the tests being run. Some of these can be configured. For instance, make sure to disable Use comments to report intermediate phases: triggered et al, and if you add a publicly accessible URL in Published Jenkins URL, you will get a link to the test result in the test result comment. I know these are global settings, but the Jenkins URL is unique anyway, and intermediate phases are probably equally annoying to everyone. You can see the only comment posted for a successful PR build here: https://github.com/scala-ide/scala-ide/pull/991#issuecomment-128016214 I'd avoid more custom code if possible. my 2c, iulian It is technically non-trivial prevent these AmplabJenkins comments from being posted in the first place (see https://issues.apache.org/jira/browse/SPARK-4216). However, as a short-term hack I'd like to deploy a script which automatically deletes these comments as soon as they're posted, with an exemption carved out for the Can an admin approve this patch for testing? messages. This will help to significantly de-clutter pull request discussions in the GitHub UI. If nobody objects, I'd like to deploy this script sometime in the next few days. (From a technical perspective, my script uses the GitHub REST API and AmplabJenkins' own OAuth token to delete the comments. The final deployment environment will most likely be the backend of http://spark-prs.appspot.com). - Josh -- -- Iulian Dragos -- Reactive Apps on the JVM www.typesafe.com
Re: Automatically deleting pull request comments left by AmplabJenkins
I think that I'm still going to want some custom code to remove the build start messages from SparkQA and it's hardly any code, so I'm going to stick with the custom approach for now. The problem is that I don't want _any_ posts from AmplabJenkins, even if they're improved to be more informative, since our custom SparkQA provides nicer output. On Fri, Aug 14, 2015 at 1:57 AM, Iulian Dragoș iulian.dra...@typesafe.com wrote: On Fri, Aug 14, 2015 at 4:21 AM, Josh Rosen rosenvi...@gmail.com wrote: Prototype is at https://github.com/databricks/spark-pr-dashboard/pull/59 On Wed, Aug 12, 2015 at 7:51 PM, Josh Rosen rosenvi...@gmail.com wrote: *TL;DR*: would anyone object if I wrote a script to auto-delete pull request comments from AmplabJenkins? Currently there are two bots which post Jenkins test result comments to GitHub, AmplabJenkins and SparkQA. SparkQA is the account which post the detailed Jenkins start and finish messages that contain information on which commit is being tested and which tests have failed. This bot is controlled via the dev/run-tests-jenkins script. AmplabJenkins is controlled by the Jenkins GitHub Pull Request Builder plugin. This bot posts relatively uninformative comments (Merge build triggered, Merge build started, Merge build failed) that do not contain any links or details specific to the tests being run. Some of these can be configured. For instance, make sure to disable Use comments to report intermediate phases: triggered et al, and if you add a publicly accessible URL in Published Jenkins URL, you will get a link to the test result in the test result comment. I know these are global settings, but the Jenkins URL is unique anyway, and intermediate phases are probably equally annoying to everyone. You can see the only comment posted for a successful PR build here: https://github.com/scala-ide/scala-ide/pull/991#issuecomment-128016214 I'd avoid more custom code if possible. my 2c, iulian It is technically non-trivial prevent these AmplabJenkins comments from being posted in the first place (see https://issues.apache.org/jira/browse/SPARK-4216). However, as a short-term hack I'd like to deploy a script which automatically deletes these comments as soon as they're posted, with an exemption carved out for the Can an admin approve this patch for testing? messages. This will help to significantly de-clutter pull request discussions in the GitHub UI. If nobody objects, I'd like to deploy this script sometime in the next few days. (From a technical perspective, my script uses the GitHub REST API and AmplabJenkins' own OAuth token to delete the comments. The final deployment environment will most likely be the backend of http://spark-prs.appspot.com). - Josh -- -- Iulian Dragos -- Reactive Apps on the JVM www.typesafe.com
Re: Automatically deleting pull request comments left by AmplabJenkins
The updated prototype listed in https://github.com/databricks/spark-pr-dashboard/pull/59 is now running live on spark-prs as part of its PR comment update task. On Fri, Aug 14, 2015 at 10:51 AM, Josh Rosen rosenvi...@gmail.com wrote: I think that I'm still going to want some custom code to remove the build start messages from SparkQA and it's hardly any code, so I'm going to stick with the custom approach for now. The problem is that I don't want _any_ posts from AmplabJenkins, even if they're improved to be more informative, since our custom SparkQA provides nicer output. On Fri, Aug 14, 2015 at 1:57 AM, Iulian Dragoș iulian.dra...@typesafe.com wrote: On Fri, Aug 14, 2015 at 4:21 AM, Josh Rosen rosenvi...@gmail.com wrote: Prototype is at https://github.com/databricks/spark-pr-dashboard/pull/59 On Wed, Aug 12, 2015 at 7:51 PM, Josh Rosen rosenvi...@gmail.com wrote: *TL;DR*: would anyone object if I wrote a script to auto-delete pull request comments from AmplabJenkins? Currently there are two bots which post Jenkins test result comments to GitHub, AmplabJenkins and SparkQA. SparkQA is the account which post the detailed Jenkins start and finish messages that contain information on which commit is being tested and which tests have failed. This bot is controlled via the dev/run-tests-jenkins script. AmplabJenkins is controlled by the Jenkins GitHub Pull Request Builder plugin. This bot posts relatively uninformative comments (Merge build triggered, Merge build started, Merge build failed) that do not contain any links or details specific to the tests being run. Some of these can be configured. For instance, make sure to disable Use comments to report intermediate phases: triggered et al, and if you add a publicly accessible URL in Published Jenkins URL, you will get a link to the test result in the test result comment. I know these are global settings, but the Jenkins URL is unique anyway, and intermediate phases are probably equally annoying to everyone. You can see the only comment posted for a successful PR build here: https://github.com/scala-ide/scala-ide/pull/991#issuecomment-128016214 I'd avoid more custom code if possible. my 2c, iulian It is technically non-trivial prevent these AmplabJenkins comments from being posted in the first place (see https://issues.apache.org/jira/browse/SPARK-4216). However, as a short-term hack I'd like to deploy a script which automatically deletes these comments as soon as they're posted, with an exemption carved out for the Can an admin approve this patch for testing? messages. This will help to significantly de-clutter pull request discussions in the GitHub UI. If nobody objects, I'd like to deploy this script sometime in the next few days. (From a technical perspective, my script uses the GitHub REST API and AmplabJenkins' own OAuth token to delete the comments. The final deployment environment will most likely be the backend of http://spark-prs.appspot.com). - Josh -- -- Iulian Dragos -- Reactive Apps on the JVM www.typesafe.com
Re: Automatically deleting pull request comments left by AmplabJenkins
Thanks Josh for the initiative. I think reducing the redundancy in QA bot posts would make discussion on GitHub UI more focused. Cheers On Thu, Aug 13, 2015 at 7:21 PM, Josh Rosen rosenvi...@gmail.com wrote: Prototype is at https://github.com/databricks/spark-pr-dashboard/pull/59 On Wed, Aug 12, 2015 at 7:51 PM, Josh Rosen rosenvi...@gmail.com wrote: *TL;DR*: would anyone object if I wrote a script to auto-delete pull request comments from AmplabJenkins? Currently there are two bots which post Jenkins test result comments to GitHub, AmplabJenkins and SparkQA. SparkQA is the account which post the detailed Jenkins start and finish messages that contain information on which commit is being tested and which tests have failed. This bot is controlled via the dev/run-tests-jenkins script. AmplabJenkins is controlled by the Jenkins GitHub Pull Request Builder plugin. This bot posts relatively uninformative comments (Merge build triggered, Merge build started, Merge build failed) that do not contain any links or details specific to the tests being run. It is technically non-trivial prevent these AmplabJenkins comments from being posted in the first place (see https://issues.apache.org/jira/browse/SPARK-4216). However, as a short-term hack I'd like to deploy a script which automatically deletes these comments as soon as they're posted, with an exemption carved out for the Can an admin approve this patch for testing? messages. This will help to significantly de-clutter pull request discussions in the GitHub UI. If nobody objects, I'd like to deploy this script sometime in the next few days. (From a technical perspective, my script uses the GitHub REST API and AmplabJenkins' own OAuth token to delete the comments. The final deployment environment will most likely be the backend of http://spark-prs.appspot.com). - Josh
Re: Automatically deleting pull request comments left by AmplabJenkins
I tried accessing just now. It took several seconds before the page showed up. FYI On Thu, Aug 13, 2015 at 7:56 PM, Cheng, Hao hao.ch...@intel.com wrote: I found the https://spark-prs.appspot.com/ is super slow while open it in a new window recently, not sure just myself or everybody experience the same, is there anyways to speed up? *From:* Josh Rosen [mailto:rosenvi...@gmail.com] *Sent:* Friday, August 14, 2015 10:21 AM *To:* dev *Subject:* Re: Automatically deleting pull request comments left by AmplabJenkins Prototype is at https://github.com/databricks/spark-pr-dashboard/pull/59 On Wed, Aug 12, 2015 at 7:51 PM, Josh Rosen rosenvi...@gmail.com wrote: *TL;DR*: would anyone object if I wrote a script to auto-delete pull request comments from AmplabJenkins? Currently there are two bots which post Jenkins test result comments to GitHub, AmplabJenkins and SparkQA. SparkQA is the account which post the detailed Jenkins start and finish messages that contain information on which commit is being tested and which tests have failed. This bot is controlled via the dev/run-tests-jenkins script. AmplabJenkins is controlled by the Jenkins GitHub Pull Request Builder plugin. This bot posts relatively uninformative comments (Merge build triggered, Merge build started, Merge build failed) that do not contain any links or details specific to the tests being run. It is technically non-trivial prevent these AmplabJenkins comments from being posted in the first place (see https://issues.apache.org/jira/browse/SPARK-4216). However, as a short-term hack I'd like to deploy a script which automatically deletes these comments as soon as they're posted, with an exemption carved out for the Can an admin approve this patch for testing? messages. This will help to significantly de-clutter pull request discussions in the GitHub UI. If nobody objects, I'd like to deploy this script sometime in the next few days. (From a technical perspective, my script uses the GitHub REST API and AmplabJenkins' own OAuth token to delete the comments. The final deployment environment will most likely be the backend of http://spark-prs.appspot.com). - Josh
RE: Automatically deleting pull request comments left by AmplabJenkins
I found the https://spark-prs.appspot.com/ is super slow while open it in a new window recently, not sure just myself or everybody experience the same, is there anyways to speed up? From: Josh Rosen [mailto:rosenvi...@gmail.com] Sent: Friday, August 14, 2015 10:21 AM To: dev Subject: Re: Automatically deleting pull request comments left by AmplabJenkins Prototype is at https://github.com/databricks/spark-pr-dashboard/pull/59 On Wed, Aug 12, 2015 at 7:51 PM, Josh Rosen rosenvi...@gmail.commailto:rosenvi...@gmail.com wrote: TL;DR: would anyone object if I wrote a script to auto-delete pull request comments from AmplabJenkins? Currently there are two bots which post Jenkins test result comments to GitHub, AmplabJenkins and SparkQA. SparkQA is the account which post the detailed Jenkins start and finish messages that contain information on which commit is being tested and which tests have failed. This bot is controlled via the dev/run-tests-jenkins script. AmplabJenkins is controlled by the Jenkins GitHub Pull Request Builder plugin. This bot posts relatively uninformative comments (Merge build triggered, Merge build started, Merge build failed) that do not contain any links or details specific to the tests being run. It is technically non-trivial prevent these AmplabJenkins comments from being posted in the first place (see https://issues.apache.org/jira/browse/SPARK-4216). However, as a short-term hack I'd like to deploy a script which automatically deletes these comments as soon as they're posted, with an exemption carved out for the Can an admin approve this patch for testing? messages. This will help to significantly de-clutter pull request discussions in the GitHub UI. If nobody objects, I'd like to deploy this script sometime in the next few days. (From a technical perspective, my script uses the GitHub REST API and AmplabJenkins' own OAuth token to delete the comments. The final deployment environment will most likely be the backend of http://spark-prs.appspot.com). - Josh
RE: Automatically deleting pull request comments left by AmplabJenkins
OK, thanks, probably just myself… From: Ted Yu [mailto:yuzhih...@gmail.com] Sent: Friday, August 14, 2015 11:04 AM To: Cheng, Hao Cc: Josh Rosen; dev Subject: Re: Automatically deleting pull request comments left by AmplabJenkins I tried accessing just now. It took several seconds before the page showed up. FYI On Thu, Aug 13, 2015 at 7:56 PM, Cheng, Hao hao.ch...@intel.commailto:hao.ch...@intel.com wrote: I found the https://spark-prs.appspot.com/ is super slow while open it in a new window recently, not sure just myself or everybody experience the same, is there anyways to speed up? From: Josh Rosen [mailto:rosenvi...@gmail.commailto:rosenvi...@gmail.com] Sent: Friday, August 14, 2015 10:21 AM To: dev Subject: Re: Automatically deleting pull request comments left by AmplabJenkins Prototype is at https://github.com/databricks/spark-pr-dashboard/pull/59 On Wed, Aug 12, 2015 at 7:51 PM, Josh Rosen rosenvi...@gmail.commailto:rosenvi...@gmail.com wrote: TL;DR: would anyone object if I wrote a script to auto-delete pull request comments from AmplabJenkins? Currently there are two bots which post Jenkins test result comments to GitHub, AmplabJenkins and SparkQA. SparkQA is the account which post the detailed Jenkins start and finish messages that contain information on which commit is being tested and which tests have failed. This bot is controlled via the dev/run-tests-jenkins script. AmplabJenkins is controlled by the Jenkins GitHub Pull Request Builder plugin. This bot posts relatively uninformative comments (Merge build triggered, Merge build started, Merge build failed) that do not contain any links or details specific to the tests being run. It is technically non-trivial prevent these AmplabJenkins comments from being posted in the first place (see https://issues.apache.org/jira/browse/SPARK-4216). However, as a short-term hack I'd like to deploy a script which automatically deletes these comments as soon as they're posted, with an exemption carved out for the Can an admin approve this patch for testing? messages. This will help to significantly de-clutter pull request discussions in the GitHub UI. If nobody objects, I'd like to deploy this script sometime in the next few days. (From a technical perspective, my script uses the GitHub REST API and AmplabJenkins' own OAuth token to delete the comments. The final deployment environment will most likely be the backend of http://spark-prs.appspot.com). - Josh
Re: Automatically deleting pull request comments left by AmplabJenkins
Prototype is at https://github.com/databricks/spark-pr-dashboard/pull/59 On Wed, Aug 12, 2015 at 7:51 PM, Josh Rosen rosenvi...@gmail.com wrote: *TL;DR*: would anyone object if I wrote a script to auto-delete pull request comments from AmplabJenkins? Currently there are two bots which post Jenkins test result comments to GitHub, AmplabJenkins and SparkQA. SparkQA is the account which post the detailed Jenkins start and finish messages that contain information on which commit is being tested and which tests have failed. This bot is controlled via the dev/run-tests-jenkins script. AmplabJenkins is controlled by the Jenkins GitHub Pull Request Builder plugin. This bot posts relatively uninformative comments (Merge build triggered, Merge build started, Merge build failed) that do not contain any links or details specific to the tests being run. It is technically non-trivial prevent these AmplabJenkins comments from being posted in the first place (see https://issues.apache.org/jira/browse/SPARK-4216). However, as a short-term hack I'd like to deploy a script which automatically deletes these comments as soon as they're posted, with an exemption carved out for the Can an admin approve this patch for testing? messages. This will help to significantly de-clutter pull request discussions in the GitHub UI. If nobody objects, I'd like to deploy this script sometime in the next few days. (From a technical perspective, my script uses the GitHub REST API and AmplabJenkins' own OAuth token to delete the comments. The final deployment environment will most likely be the backend of http://spark-prs.appspot.com). - Josh