{"action":"created","issue":{"url":"https://api.github.com/repos/cms-sw/cmssw/issues/17852","repository_url":"https://api.github.com/repos/cms-sw/cmssw","labels_url":"https://api.github.com/repos/cms-sw/cmssw/issues/17852/labels{/name}","comments_url":"https://api.github.com/repos/cms-sw/cmssw/issues/17852/comments","events_url":"https://api.github.com/repos/cms-sw/cmssw/issues/17852/events","html_url":"https://github.com/cms-sw/cmssw/pull/17852","id":213027463,"number":17852,"title":"Update HLT STORM validation to Phase1 (91X)","user":{"login":"silviodonato","id":6177433,"avatar_url":"https://avatars0.githubusercontent.com/u/6177433?v=3","gravatar_id":"","url":"https://api.github.com/users/silviodonato","html_url":"https://github.com/silviodonato","followers_url":"https://api.github.com/users/silviodonato/followers","following_url":"https://api.github.com/users/silviodonato/following{/other_user}","gists_url":"https://api.github.com/users/silviodonato/gists{/gist_id}","starred_url":"https://api.github.com/users/silviodonato/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/silviodonato/subscriptions","organizations_url":"https://api.github.com/users/silviodonato/orgs","repos_url":"https://api.github.com/users/silviodonato/repos","events_url":"https://api.github.com/users/silviodonato/events{/privacy}","received_events_url":"https://api.github.com/users/silviodonato/received_events","type":"User","site_admin":false},"labels":[{"id":88653795,"url":"https://api.github.com/repos/cms-sw/cmssw/labels/comparison-pending","name":"comparison-pending","color":"fbca04","default":false},{"id":52822551,"url":"https://api.github.com/repos/cms-sw/cmssw/labels/hlt-pending","name":"hlt-pending","color":"fbca04","default":false},{"id":56953091,"url":"https://api.github.com/repos/cms-sw/cmssw/labels/orp-pending","name":"orp-pending","color":"fbca04","default":false},{"id":53167770,"url":"https://api.github.com/repos/cms-sw/cmssw/labels/pending-signatures","name":"pending-signatures","color":"eb6420","default":false},{"id":56131152,"url":"https://api.github.com/repos/cms-sw/cmssw/labels/tests-pending","name":"tests-pending","color":"fbca04","default":false}],"state":"open","locked":false,"assignee":null,"assignees":[],"milestone":{"url":"https://api.github.com/repos/cms-sw/cmssw/milestones/65","html_url":"https://github.com/cms-sw/cmssw/milestone/65","labels_url":"https://api.github.com/repos/cms-sw/cmssw/milestones/65/labels","id":2358370,"number":65,"title":"Next CMSSW_9_1_X","description":"","creator":{"login":"smuzaffar","id":4115138,"avatar_url":"https://avatars0.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":73,"closed_issues":58,"state":"open","created_at":"2017-03-02T09:15:47Z","updated_at":"2017-03-09T13:45:50Z","due_on":null,"closed_at":null},"comments":6,"created_at":"2017-03-09T12:49:29Z","updated_at":"2017-03-09T14:36:25Z","closed_at":null,"pull_request":{"url":"https://api.github.com/repos/cms-sw/cmssw/pulls/17852","html_url":"https://github.com/cms-sw/cmssw/pull/17852","diff_url":"https://github.com/cms-sw/cmssw/pull/17852.diff","patch_url":"https://github.com/cms-sw/cmssw/pull/17852.patch"},"body":"This PR is a general update of the HLT menu validation to Phase1.\r\nIt includes:\r\n- update of the HLT menus to Phase1 (ie. confDB /dev/CMSSW_9_0_1/)\r\n- clean up of PIon and HIon menus, as accorded with HI people\r\n- clean up of modifyHLTforEras \r\n- add of GRun2016 to test the 2016 menu. Currently it is the /dev/CMSSW_9_0_0/ menu. It will become GRun + reverse customization function, as soon as the function will be ready\r\n- update of the symbolic update to use the 'phase1_2017_realistic'\r\n- update of the input files/eras to run the phase1 menu\r\n- add 'HLT:@relval2017' cmsDriver option to peak the 2017 HLT menu\r\n\r\nOpen issues:\r\n- we are waiting for the reverse customization function to get a 2016-like menu from the current menu\r\n- we need to update relval to use HLT:@relval2017 instead of HLT:@relval2016.\r\n- we need to update the IB tests to test also GRun2016\r\n- one of the test will fail, because of a issue with EcalAlCaRecoProducers. It is caused by this line https://github.com/cms-sw/cmssw/blob/CMSSW_9_1_X/Calibration/EcalAlCaRecoProducers/plugins/SelectedElectronFEDListProducer.cc#L309 . I strongly suspect that this code is not yet compatible with the Phase-1 pixels. I contacted HLT AlCa ( @trtomei ).\r\nCC: @mtosi @ndaci @georgia14\r\n90X PR: #17853 \r\n\r\nBased on CMSSW_9_1_X_2017-03-08-1100."},"comment":{"url":"https://api.github.com/repos/cms-sw/cmssw/issues/comments/285368035","html_url":"https://github.com/cms-sw/cmssw/pull/17852#issuecomment-285368035","issue_url":"https://api.github.com/repos/cms-sw/cmssw/issues/17852","id":285368035,"user":{"login":"silviodonato","id":6177433,"avatar_url":"https://avatars0.githubusercontent.com/u/6177433?v=3","gravatar_id":"","url":"https://api.github.com/users/silviodonato","html_url":"https://github.com/silviodonato","followers_url":"https://api.github.com/users/silviodonato/followers","following_url":"https://api.github.com/users/silviodonato/following{/other_user}","gists_url":"https://api.github.com/users/silviodonato/gists{/gist_id}","starred_url":"https://api.github.com/users/silviodonato/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/silviodonato/subscriptions","organizations_url":"https://api.github.com/users/silviodonato/orgs","repos_url":"https://api.github.com/users/silviodonato/repos","events_url":"https://api.github.com/users/silviodonato/events{/privacy}","received_events_url":"https://api.github.com/users/silviodonato/received_events","type":"User","site_admin":false},"created_at":"2017-03-09T14:36:25Z","updated_at":"2017-03-09T14:36:25Z","body":"We need to modify the jenkin tests. They failed because the symbolic global tag (eg. run2_mc_PRef) is now pointing to the phase-I GT and so doesn't match with \"--era Run2_2016\". We need to move the tests to \"--era Run2_2017\". I'm contacting STEAM and PdmV to solve it."}}
{"actor":{"display_login":"silviodonato"}}