{"action":"created","issue":{"url":"https://api.github.com/repos/jenkinsci/gerrit-trigger-plugin/issues/284","repository_url":"https://api.github.com/repos/jenkinsci/gerrit-trigger-plugin","labels_url":"https://api.github.com/repos/jenkinsci/gerrit-trigger-plugin/issues/284/labels{/name}","comments_url":"https://api.github.com/repos/jenkinsci/gerrit-trigger-plugin/issues/284/comments","events_url":"https://api.github.com/repos/jenkinsci/gerrit-trigger-plugin/issues/284/events","html_url":"https://github.com/jenkinsci/gerrit-trigger-plugin/pull/284","id":155286983,"number":284,"title":"Provide GERRIT_MERGED_REVISION after change-merged event","user":{"login":"polygon","id":51489,"avatar_url":"https://avatars0.githubusercontent.com/u/51489?v=4","gravatar_id":"","url":"https://api.github.com/users/polygon","html_url":"https://github.com/polygon","followers_url":"https://api.github.com/users/polygon/followers","following_url":"https://api.github.com/users/polygon/following{/other_user}","gists_url":"https://api.github.com/users/polygon/gists{/gist_id}","starred_url":"https://api.github.com/users/polygon/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/polygon/subscriptions","organizations_url":"https://api.github.com/users/polygon/orgs","repos_url":"https://api.github.com/users/polygon/repos","events_url":"https://api.github.com/users/polygon/events{/privacy}","received_events_url":"https://api.github.com/users/polygon/received_events","type":"User","site_admin":false},"labels":[],"state":"closed","locked":false,"assignee":null,"assignees":[],"milestone":null,"comments":6,"created_at":"2016-05-17T15:19:35Z","updated_at":"2018-04-17T13:59:43Z","closed_at":"2016-05-19T10:51:30Z","author_association":"NONE","pull_request":{"url":"https://api.github.com/repos/jenkinsci/gerrit-trigger-plugin/pulls/284","html_url":"https://github.com/jenkinsci/gerrit-trigger-plugin/pull/284","diff_url":"https://github.com/jenkinsci/gerrit-trigger-plugin/pull/284.diff","patch_url":"https://github.com/jenkinsci/gerrit-trigger-plugin/pull/284.patch"},"body":"The newRev field of a change-merged event was not parsed yet. Since this field holds the hash of a merge commit it can be important in certain build conditions.\n\nThis patch exposes this field as a GERRIT_MERGED_REVISION variable to Jenkins builds. It requires the following PR in gerrit-events to compile successfully: https://github.com/sonyxperiadev/gerrit-events/pull/55 (which is the reason why Jenkins reports a failure below)\n\nIt fixes https://issues.jenkins-ci.org/browse/JENKINS-23873\n\nSome changes to pom.xml to update the gerrit-events version might be required. Since I'm not normally a Java person I leave that to the maintainer.\n"},"comment":{"url":"https://api.github.com/repos/jenkinsci/gerrit-trigger-plugin/issues/comments/382002937","html_url":"https://github.com/jenkinsci/gerrit-trigger-plugin/pull/284#issuecomment-382002937","issue_url":"https://api.github.com/repos/jenkinsci/gerrit-trigger-plugin/issues/284","id":382002937,"user":{"login":"rsandell","id":300688,"avatar_url":"https://avatars2.githubusercontent.com/u/300688?v=4","gravatar_id":"","url":"https://api.github.com/users/rsandell","html_url":"https://github.com/rsandell","followers_url":"https://api.github.com/users/rsandell/followers","following_url":"https://api.github.com/users/rsandell/following{/other_user}","gists_url":"https://api.github.com/users/rsandell/gists{/gist_id}","starred_url":"https://api.github.com/users/rsandell/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/rsandell/subscriptions","organizations_url":"https://api.github.com/users/rsandell/orgs","repos_url":"https://api.github.com/users/rsandell/repos","events_url":"https://api.github.com/users/rsandell/events{/privacy}","received_events_url":"https://api.github.com/users/rsandell/received_events","type":"User","site_admin":false},"created_at":"2018-04-17T13:59:43Z","updated_at":"2018-04-17T13:59:43Z","author_association":"MEMBER","body":"@clabu Yes, if you use the gerrit build chooser (\"choose what revision to build\" I believe it says in the UI) it will pick the revision from the active patchset for all change based events, and for `RefUpdated` it picks the newRev.\r\nSo for change merged events, if you want to use the merge commit you will need a different scm configuration."}}
{"actor":{"display_login":"rsandell"}}