{"action":"created","issue":{"url":"https://api.github.com/repos/dmwm/WMCore/issues/7391","repository_url":"https://api.github.com/repos/dmwm/WMCore","labels_url":"https://api.github.com/repos/dmwm/WMCore/issues/7391/labels{/name}","comments_url":"https://api.github.com/repos/dmwm/WMCore/issues/7391/comments","events_url":"https://api.github.com/repos/dmwm/WMCore/issues/7391/events","html_url":"https://github.com/dmwm/WMCore/pull/7391","id":189376914,"number":7391,"title":"Change PU file list behavior during WMRuntime/TrustPUSitelists","user":{"login":"amaltaro","id":1798520,"avatar_url":"https://avatars.githubusercontent.com/u/1798520?v=3","gravatar_id":"","url":"https://api.github.com/users/amaltaro","html_url":"https://github.com/amaltaro","followers_url":"https://api.github.com/users/amaltaro/followers","following_url":"https://api.github.com/users/amaltaro/following{/other_user}","gists_url":"https://api.github.com/users/amaltaro/gists{/gist_id}","starred_url":"https://api.github.com/users/amaltaro/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/amaltaro/subscriptions","organizations_url":"https://api.github.com/users/amaltaro/orgs","repos_url":"https://api.github.com/users/amaltaro/repos","events_url":"https://api.github.com/users/amaltaro/events{/privacy}","received_events_url":"https://api.github.com/users/amaltaro/received_events","type":"User","site_admin":false},"labels":[{"id":311378297,"url":"https://api.github.com/repos/dmwm/WMCore/labels/Do%20not%20merge%20yet","name":"Do not merge yet","color":"fbca04","default":false}],"state":"open","locked":false,"assignee":null,"assignees":[],"milestone":null,"comments":39,"created_at":"2016-11-15T12:17:43Z","updated_at":"2016-11-18T14:09:45Z","closed_at":null,"pull_request":{"url":"https://api.github.com/repos/dmwm/WMCore/pulls/7391","html_url":"https://github.com/dmwm/WMCore/pull/7391","diff_url":"https://github.com/dmwm/WMCore/pull/7391.diff","patch_url":"https://github.com/dmwm/WMCore/pull/7391.patch"},"body":"Fixes #7387\r\n\r\nSummary of the changes and the expected behaviour is as follows:\r\n* pileup json file contains only real data/location, no more fake pnns.\r\n* `TrustPUSitelists` changes the behaviour of two things, when set to `True`.\r\n  - used for bypassing GQ to LQ replication.\r\n  - makes sure **all** the PU files are added to the PSet as secondary input, no matter the site and/or data location\r\n* if job lands on a site that contains only 1 PU block out of 100, the PSet will have only files that belong to that block (unles Trust flag was set to True) \r\n* Needless to say, if the PU dataset is wiped out from the storage by DDM (or anything else), ALL the jobs will read the PU data via AAA (depending on the workflow, it can cause a very bad impact on the grid resources (AAA, storage)) \r\n\r\nI'm not testing it before I get a confirmation whether this is the desired behaviour or not.\r\nSo @ticoann @hufnagel @vlimant @ericvaandering please share your thoughts (Dirk, only short comments please :), otherwise it's impossible to have a productive follow up)"},"comment":{"url":"https://api.github.com/repos/dmwm/WMCore/issues/comments/261539893","html_url":"https://github.com/dmwm/WMCore/pull/7391#issuecomment-261539893","issue_url":"https://api.github.com/repos/dmwm/WMCore/issues/7391","id":261539893,"user":{"login":"cmsdmwmbot","id":15169205,"avatar_url":"https://avatars.githubusercontent.com/u/15169205?v=3","gravatar_id":"","url":"https://api.github.com/users/cmsdmwmbot","html_url":"https://github.com/cmsdmwmbot","followers_url":"https://api.github.com/users/cmsdmwmbot/followers","following_url":"https://api.github.com/users/cmsdmwmbot/following{/other_user}","gists_url":"https://api.github.com/users/cmsdmwmbot/gists{/gist_id}","starred_url":"https://api.github.com/users/cmsdmwmbot/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/cmsdmwmbot/subscriptions","organizations_url":"https://api.github.com/users/cmsdmwmbot/orgs","repos_url":"https://api.github.com/users/cmsdmwmbot/repos","events_url":"https://api.github.com/users/cmsdmwmbot/events{/privacy}","received_events_url":"https://api.github.com/users/cmsdmwmbot/received_events","type":"User","site_admin":false},"created_at":"2016-11-18T14:09:45Z","updated_at":"2016-11-18T14:09:45Z","body":"Unit test changes for pull request 7391:\n* WMCore_t.WMSpec_t.StdSpecs_t.DQMHarvest_t.DQMHarvestTests:testDQMHarvest changed from error to success\n* WMCore_t.WMSpec_t.StdSpecs_t.StepChain_t.StepChainTests:testStepChainMC changed from error to success\n* WMCore_t.WMSpec_t.StdSpecs_t.StepChain_t.StepChainTests:testStepMapping changed from error to success\n* WMCore_t.WMSpec_t.WMTask_t.WMTaskTest:testJobSplittingArgs changed from success to failure\n* WMCore_t.WMSpec_t.WMWorkload_t.WMWorkloadTest:testListJobSplittingParametersByTask changed from success to failure\n* WMCore_t.WMSpec_t.WMWorkload_t.WMWorkloadTest:testUpdatingMergeParameters changed from success to failure\n* WMCore_t.WMSpec_t.WMWorkload_t.WMWorkloadTest:testUpdatingSplitParameters changed from success to failure\n* WMCore_t.WMSpec_t.WMWorkload_t.WMWorkloadTest:testUpdatingSubSplitParameters changed from success to failure\n* WMCore_t.WorkQueue_t.WorkQueue_t.WorkQueueTest:testGlobalBlockSplitting changed from success to failure\n* WMCore_t.WorkQueue_t.WorkQueue_t.WorkQueueTest:testResetWork (unstable) changed from failure to success\n\n\nPreviously working unit tests have failed!\n"}}
{"actor":{"display_login":"cmsdmwmbot"}}