{"action":"created","issue":{"url":"https://api.github.com/repos/cms-sw/cmssw/issues/15810","repository_url":"https://api.github.com/repos/cms-sw/cmssw","labels_url":"https://api.github.com/repos/cms-sw/cmssw/issues/15810/labels{/name}","comments_url":"https://api.github.com/repos/cms-sw/cmssw/issues/15810/comments","events_url":"https://api.github.com/repos/cms-sw/cmssw/issues/15810/events","html_url":"https://github.com/cms-sw/cmssw/pull/15810","id":176270056,"number":15810,"title":"Fix all DQM paths to properly handle TriggerResults","user":{"login":"Dr15Jones","id":3757616,"avatar_url":"https://avatars.githubusercontent.com/u/3757616?v=3","gravatar_id":"","url":"https://api.github.com/users/Dr15Jones","html_url":"https://github.com/Dr15Jones","followers_url":"https://api.github.com/users/Dr15Jones/followers","following_url":"https://api.github.com/users/Dr15Jones/following{/other_user}","gists_url":"https://api.github.com/users/Dr15Jones/gists{/gist_id}","starred_url":"https://api.github.com/users/Dr15Jones/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/Dr15Jones/subscriptions","organizations_url":"https://api.github.com/users/Dr15Jones/orgs","repos_url":"https://api.github.com/users/Dr15Jones/repos","events_url":"https://api.github.com/users/Dr15Jones/events{/privacy}","received_events_url":"https://api.github.com/users/Dr15Jones/received_events","type":"User","site_admin":false},"labels":[{"url":"https://api.github.com/repos/cms-sw/cmssw/labels/comparison-pending","name":"comparison-pending","color":"fbca04"},{"url":"https://api.github.com/repos/cms-sw/cmssw/labels/dqm-pending","name":"dqm-pending","color":"fbca04"},{"url":"https://api.github.com/repos/cms-sw/cmssw/labels/operations-pending","name":"operations-pending","color":"fbca04"},{"url":"https://api.github.com/repos/cms-sw/cmssw/labels/orp-pending","name":"orp-pending","color":"fbca04"},{"url":"https://api.github.com/repos/cms-sw/cmssw/labels/pending-signatures","name":"pending-signatures","color":"eb6420"},{"url":"https://api.github.com/repos/cms-sw/cmssw/labels/tests-pending","name":"tests-pending","color":"fbca04"}],"state":"open","locked":false,"assignee":null,"assignees":[],"milestone":{"url":"https://api.github.com/repos/cms-sw/cmssw/milestones/59","html_url":"https://github.com/cms-sw/cmssw/milestones/Next%20CMSSW_8_1_X","labels_url":"https://api.github.com/repos/cms-sw/cmssw/milestones/59/labels","id":1590252,"number":59,"title":"Next CMSSW_8_1_X","description":null,"creator":{"login":"smuzaffar","id":4115138,"avatar_url":"https://avatars.githubusercontent.com/u/4115138?v=3","gravatar_id":"","url":"https://api.github.com/users/smuzaffar","html_url":"https://github.com/smuzaffar","followers_url":"https://api.github.com/users/smuzaffar/followers","following_url":"https://api.github.com/users/smuzaffar/following{/other_user}","gists_url":"https://api.github.com/users/smuzaffar/gists{/gist_id}","starred_url":"https://api.github.com/users/smuzaffar/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/smuzaffar/subscriptions","organizations_url":"https://api.github.com/users/smuzaffar/orgs","repos_url":"https://api.github.com/users/smuzaffar/repos","events_url":"https://api.github.com/users/smuzaffar/events{/privacy}","received_events_url":"https://api.github.com/users/smuzaffar/received_events","type":"User","site_admin":false},"open_issues":68,"closed_issues":1402,"state":"open","created_at":"2016-02-18T09:12:12Z","updated_at":"2016-09-11T22:48:18Z","due_on":null,"closed_at":null},"comments":7,"created_at":"2016-09-11T22:47:44Z","updated_at":"2016-09-12T02:49:04Z","closed_at":null,"pull_request":{"url":"https://api.github.com/repos/cms-sw/cmssw/pulls/15810","html_url":"https://github.com/cms-sw/cmssw/pull/15810","diff_url":"https://github.com/cms-sw/cmssw/pull/15810.diff","patch_url":"https://github.com/cms-sw/cmssw/pull/15810.patch"},"body":"This pull request fixes all problems from issue #15769 and includes the changes in #15809. The most important additional change on top #15809 changes ConfigBuilder.py and therefore could have a larger impact on the evaluation of this pull request as opposed to #15809.\r\n\r\nThe ConfigBuilder.py change always makes the dqmoffline\\*_step path be an EndPath. Previously, based on which processing step is being configured the dqmoffline*_step might sometimes be a Path and other times be an EndPath. The reason appeared to be because sometimes the TriggerResults was being requested by some modules using an empty processName and the hope was one would pick up the HLT results. This change has all modules explicitly state what process name to use when getting the TriggerResults. \r\n\r\nThis is needed since the next threaded framework change requires that the consumes of a module and its place on a Path or EndPath guarantee that it can get all the data it requests (or will fail since there is no source or module producing the data it wants). Asking for a TriggerResults for the present process from a Path is therefore not allowed (and was not a good idea before either)."},"comment":{"url":"https://api.github.com/repos/cms-sw/cmssw/issues/comments/246231838","html_url":"https://github.com/cms-sw/cmssw/pull/15810#issuecomment-246231838","issue_url":"https://api.github.com/repos/cms-sw/cmssw/issues/15810","id":246231838,"user":{"login":"cmsbuild","id":4170616,"avatar_url":"https://avatars.githubusercontent.com/u/4170616?v=3","gravatar_id":"","url":"https://api.github.com/users/cmsbuild","html_url":"https://github.com/cmsbuild","followers_url":"https://api.github.com/users/cmsbuild/followers","following_url":"https://api.github.com/users/cmsbuild/following{/other_user}","gists_url":"https://api.github.com/users/cmsbuild/gists{/gist_id}","starred_url":"https://api.github.com/users/cmsbuild/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/cmsbuild/subscriptions","organizations_url":"https://api.github.com/users/cmsbuild/orgs","repos_url":"https://api.github.com/users/cmsbuild/repos","events_url":"https://api.github.com/users/cmsbuild/events{/privacy}","received_events_url":"https://api.github.com/users/cmsbuild/received_events","type":"User","site_admin":false},"created_at":"2016-09-12T02:49:04Z","updated_at":"2016-09-12T02:49:04Z","body":"The tests are being triggered in jenkins."}}
{"actor":{"display_login":"cmsbuild"}}