{"action":"created","issue":{"url":"https://api.github.com/repos/ashwanthkumar/gocd-build-github-pull-requests/issues/76","repository_url":"https://api.github.com/repos/ashwanthkumar/gocd-build-github-pull-requests","labels_url":"https://api.github.com/repos/ashwanthkumar/gocd-build-github-pull-requests/issues/76/labels{/name}","comments_url":"https://api.github.com/repos/ashwanthkumar/gocd-build-github-pull-requests/issues/76/comments","events_url":"https://api.github.com/repos/ashwanthkumar/gocd-build-github-pull-requests/issues/76/events","html_url":"https://github.com/ashwanthkumar/gocd-build-github-pull-requests/pull/76","id":181911724,"number":76,"title":"Force GoCD to build every commit (WIP)","user":{"login":"juhasipo","id":317264,"avatar_url":"https://avatars.githubusercontent.com/u/317264?v=3","gravatar_id":"","url":"https://api.github.com/users/juhasipo","html_url":"https://github.com/juhasipo","followers_url":"https://api.github.com/users/juhasipo/followers","following_url":"https://api.github.com/users/juhasipo/following{/other_user}","gists_url":"https://api.github.com/users/juhasipo/gists{/gist_id}","starred_url":"https://api.github.com/users/juhasipo/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/juhasipo/subscriptions","organizations_url":"https://api.github.com/users/juhasipo/orgs","repos_url":"https://api.github.com/users/juhasipo/repos","events_url":"https://api.github.com/users/juhasipo/events{/privacy}","received_events_url":"https://api.github.com/users/juhasipo/received_events","type":"User","site_admin":false},"labels":[],"state":"open","locked":false,"assignee":null,"assignees":[],"milestone":null,"comments":1,"created_at":"2016-10-09T21:06:49Z","updated_at":"2016-10-10T04:51:48Z","closed_at":null,"pull_request":{"url":"https://api.github.com/repos/ashwanthkumar/gocd-build-github-pull-requests/pulls/76","html_url":"https://github.com/ashwanthkumar/gocd-build-github-pull-requests/pull/76","diff_url":"https://github.com/ashwanthkumar/gocd-build-github-pull-requests/pull/76.diff","patch_url":"https://github.com/ashwanthkumar/gocd-build-github-pull-requests/pull/76.patch"},"body":"I've been working on a fix for a major issue with Gerrit plugin. At this phase I wanted know what other people are thinking about this feature since it is a workaround to a problem that I think should be solved in GoCD itself. This feature is still WIP. It has been tested in production but it still needs couple of finishing touches:\r\n\r\n- Commits should be built in chronological order. Now the order is still pseudo-random order of the hash map.\r\n- Better instructions and labels (and add tooltips?)\r\n\r\nAt the moment GoCD merges commits if it sees that there are multiple related incoming commits. This causes GoCD to skip building and testing some of the commits pushed to Gerrit. This makes the Gerrit plugin quite useless because most of the time developers push more than one commit/change to Gerrit for review so many of of these changes won't ever get the pass/failed status from GoCD.\r\n\r\nThis fix isn't really an ideal way to fix the issue but it should work well enough. The fix changes the plugin behavior so that it doesn't give any new commits to GoCD until the pipeline has completed all the stages. This is checked via the pipeline status and history APIs.\r\n\r\nIn order to do this, the SCM material has to know the name of the pipeline that is building the changes so that the plugin can search pipeline activity for the correct pipeline. The pipeline name is given in the SCM material settings and it is not part of the identity so that there can be multiple related SCM materials in different pipelines.\r\n\r\nThis PR is built on top of the white/blacklist feature's setting changes so that PR should be merged first."},"comment":{"url":"https://api.github.com/repos/ashwanthkumar/gocd-build-github-pull-requests/issues/comments/252537842","html_url":"https://github.com/ashwanthkumar/gocd-build-github-pull-requests/pull/76#issuecomment-252537842","issue_url":"https://api.github.com/repos/ashwanthkumar/gocd-build-github-pull-requests/issues/76","id":252537842,"user":{"login":"ashwanthkumar","id":600279,"avatar_url":"https://avatars.githubusercontent.com/u/600279?v=3","gravatar_id":"","url":"https://api.github.com/users/ashwanthkumar","html_url":"https://github.com/ashwanthkumar","followers_url":"https://api.github.com/users/ashwanthkumar/followers","following_url":"https://api.github.com/users/ashwanthkumar/following{/other_user}","gists_url":"https://api.github.com/users/ashwanthkumar/gists{/gist_id}","starred_url":"https://api.github.com/users/ashwanthkumar/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/ashwanthkumar/subscriptions","organizations_url":"https://api.github.com/users/ashwanthkumar/orgs","repos_url":"https://api.github.com/users/ashwanthkumar/repos","events_url":"https://api.github.com/users/ashwanthkumar/events{/privacy}","received_events_url":"https://api.github.com/users/ashwanthkumar/received_events","type":"User","site_admin":false},"created_at":"2016-10-10T04:51:48Z","updated_at":"2016-10-10T04:51:48Z","body":"There're too many changes to view in this PR. May be we should merge the other one first so we know the exact changes that's needed for this. I have never used Gerrit before so I'm not familiar with the workflow, but my only request would be to make this an opt-in feature (may be at a material / plugin level) than to have it enabled by default for all plugins.\r\n\r\nThis is great work! 👍 "}}
{"actor":{"display_login":"ashwanthkumar"}}