{"action":"created","issue":{"url":"https://api.github.com/repos/dmwm/CRABServer/issues/5081","labels_url":"https://api.github.com/repos/dmwm/CRABServer/issues/5081/labels{/name}","comments_url":"https://api.github.com/repos/dmwm/CRABServer/issues/5081/comments","events_url":"https://api.github.com/repos/dmwm/CRABServer/issues/5081/events","html_url":"https://github.com/dmwm/CRABServer/pull/5081","id":117240518,"number":5081,"title":"Implement resubmission of publications. Also fixes #5070.","user":{"login":"AndresTanasijczuk","id":6575844,"avatar_url":"https://avatars.githubusercontent.com/u/6575844?v=3","gravatar_id":"","url":"https://api.github.com/users/AndresTanasijczuk","html_url":"https://github.com/AndresTanasijczuk","followers_url":"https://api.github.com/users/AndresTanasijczuk/followers","following_url":"https://api.github.com/users/AndresTanasijczuk/following{/other_user}","gists_url":"https://api.github.com/users/AndresTanasijczuk/gists{/gist_id}","starred_url":"https://api.github.com/users/AndresTanasijczuk/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/AndresTanasijczuk/subscriptions","organizations_url":"https://api.github.com/users/AndresTanasijczuk/orgs","repos_url":"https://api.github.com/users/AndresTanasijczuk/repos","events_url":"https://api.github.com/users/AndresTanasijczuk/events{/privacy}","received_events_url":"https://api.github.com/users/AndresTanasijczuk/received_events","type":"User","site_admin":false},"labels":[{"url":"https://api.github.com/repos/dmwm/CRABServer/labels/enhancement","name":"enhancement","color":"84b6eb"}],"state":"open","locked":false,"assignee":{"login":"dciangot","id":4144326,"avatar_url":"https://avatars.githubusercontent.com/u/4144326?v=3","gravatar_id":"","url":"https://api.github.com/users/dciangot","html_url":"https://github.com/dciangot","followers_url":"https://api.github.com/users/dciangot/followers","following_url":"https://api.github.com/users/dciangot/following{/other_user}","gists_url":"https://api.github.com/users/dciangot/gists{/gist_id}","starred_url":"https://api.github.com/users/dciangot/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/dciangot/subscriptions","organizations_url":"https://api.github.com/users/dciangot/orgs","repos_url":"https://api.github.com/users/dciangot/repos","events_url":"https://api.github.com/users/dciangot/events{/privacy}","received_events_url":"https://api.github.com/users/dciangot/received_events","type":"User","site_admin":false},"milestone":{"url":"https://api.github.com/repos/dmwm/CRABServer/milestones/75","html_url":"https://github.com/dmwm/CRABServer/milestones/CRAB3%20December%202015","labels_url":"https://api.github.com/repos/dmwm/CRABServer/milestones/75/labels","id":1251881,"number":75,"title":"CRAB3 December 2015","description":"","creator":{"login":"mmascher","id":1586246,"avatar_url":"https://avatars.githubusercontent.com/u/1586246?v=3","gravatar_id":"","url":"https://api.github.com/users/mmascher","html_url":"https://github.com/mmascher","followers_url":"https://api.github.com/users/mmascher/followers","following_url":"https://api.github.com/users/mmascher/following{/other_user}","gists_url":"https://api.github.com/users/mmascher/gists{/gist_id}","starred_url":"https://api.github.com/users/mmascher/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/mmascher/subscriptions","organizations_url":"https://api.github.com/users/mmascher/orgs","repos_url":"https://api.github.com/users/mmascher/repos","events_url":"https://api.github.com/users/mmascher/events{/privacy}","received_events_url":"https://api.github.com/users/mmascher/received_events","type":"User","site_admin":false},"open_issues":31,"closed_issues":11,"state":"open","created_at":"2015-08-12T14:52:53Z","updated_at":"2015-11-17T08:15:40Z","due_on":null,"closed_at":null},"comments":6,"created_at":"2015-11-16T23:06:56Z","updated_at":"2015-11-17T09:34:08Z","closed_at":null,"pull_request":{"url":"https://api.github.com/repos/dmwm/CRABServer/pulls/5081","html_url":"https://github.com/dmwm/CRABServer/pull/5081","diff_url":"https://github.com/dmwm/CRABServer/pull/5081.diff","patch_url":"https://github.com/dmwm/CRABServer/pull/5081.patch"},"body":"This is resubmission via the 'crab resubmit' command. \r\nI tested this PR using ASO/Couch in testbed and it worked as I expected, but I would like @HassenRiahi to check the query of the view that I do in DagmanResubmitter, because I tried first using the same query as in the retry-publish script\r\n```\r\nfailedPublications = database.loadView('DBSPublisher','PublicationFailedByWorkflow',{'reduce':False,'key':taskname})['rows']\r\n```\r\nbut it didn't work; it returned an empty list. So I changed to\r\n```\r\nfailedPublications = database.loadView('DBSPublisher', 'PublicationFailedByWorkflow', {'reduce': False})['rows']\r\n```"},"comment":{"url":"https://api.github.com/repos/dmwm/CRABServer/issues/comments/157319626","html_url":"https://github.com/dmwm/CRABServer/pull/5081#issuecomment-157319626","issue_url":"https://api.github.com/repos/dmwm/CRABServer/issues/5081","id":157319626,"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":"2015-11-17T09:34:08Z","updated_at":"2015-11-17T09:34:08Z","body":"Summary of pylint changes for pull request 5081:\n* Score for src/python/CRABInterface/DataWorkflow.py changed from 6.62 to 6.92 with 9 (12) total errors (warnings)\n* Score for src/python/CRABInterface/RESTUserWorkflow.py changed from 8.59 to 8.47 with 1 (42) total errors (warnings)\n* Score for src/python/TaskWorker/Actions/DagmanResubmitter.py changed from 7.81 to 8.26 with 4 (9) total errors (warnings)\n\nsrc/python/CRABInterface/DataWorkflow.py fails the pylint check. Report follows:\n* Line 174 W0311 Bad indentation. Found 14 spaces, expected 16\n* Line 6 W0403 Relative import 'Regexps', should be 'CRABInterface.Regexps'\n* Line 103 in DataWorkflow.submit R0913 Too many arguments (56/7)\n* Line 103 in DataWorkflow.submit R0914 Too many local variables (64/30)\n* Line 176 in DataWorkflow.submit W1201 Specify string format arguments as logging function parameters\n* Line 107 in DataWorkflow.submit W0613 Unused argument 'userproxy'\n* Line 105 in DataWorkflow.submit W0613 Unused argument 'publishname'\n* Line 264 in DataWorkflow.resubmit R0913 Too many arguments (13/7)\n* Line 275 in DataWorkflow.resubmit W1201 Specify string format arguments as logging function parameters\n* Line 313 in DataWorkflow.resubmit E1101 Instance of 'DataWorkflow' has no 'failedList' member\n* Line 314 in DataWorkflow.resubmit E1101 Instance of 'DataWorkflow' has no 'successList' member\n* Line 319 in DataWorkflow.resubmit E1101 Instance of 'DataWorkflow' has no 'successList' member\n* Line 321 in DataWorkflow.resubmit E1101 Instance of 'DataWorkflow' has no 'failedList' member\n* Line 322 in DataWorkflow.resubmit E1101 Instance of 'DataWorkflow' has no 'successList' member\n* Line 354 in DataWorkflow.resubmit E1101 Instance of 'DataWorkflow' has no 'failedList' member\n* Line 355 in DataWorkflow.resubmit E1101 Instance of 'DataWorkflow' has no 'successList' member\n* Line 359 in DataWorkflow.resubmit W1201 Specify string format arguments as logging function parameters\n* Line 361 in DataWorkflow.resubmit W1201 Specify string format arguments as logging function parameters\n* Line 264 in DataWorkflow.resubmit R0912 Too many branches (30/12)\n* Line 420 in DataWorkflow.kill W1201 Specify string format arguments as logging function parameters\n* Line 428 in DataWorkflow.kill E1101 Instance of 'DataWorkflow' has no 'successList' member\n* Line 436 in DataWorkflow.kill E1101 Instance of 'DataWorkflow' has no 'successList' member\n* Line 437 in DataWorkflow.kill W1201 Specify string format arguments as logging function parameters\n* Line 414 in DataWorkflow.kill W0613 Unused argument 'force'\n* Line 5 W0611 Unused import cherrypy\n* Line 16 in DataWorkflow R0921 Abstract class not referenced\n* plus 151 comments on code style\n\nsrc/python/CRABInterface/RESTUserWorkflow.py got worse in pylint. Report follows:\n* Line 268 W0511 TODO this should be changed to be non-optional\n* Line 10 E0611 No name 'DataWorkflow' in module 'CRABInterface.DataUserWorkflow'\n* Line 12 W0401 Wildcard import CRABInterface.Regexps\n* Line 44 in RESTUserWorkflow._expandSites W0141 Used builtin function 'map'\n* Line 44 in RESTUserWorkflow._expandSites W0141 Used builtin function 'filter'\n* Line 46 in RESTUserWorkflow._expandSites W1201 Specify string format arguments as logging function parameters\n* Line 58 in RESTUserWorkflow._checkOutLFN R0201 Method could be a function\n* Line 135 in RESTUserWorkflow._checkPublishDataName2 R0201 Method could be a function\n* Line 185 in RESTUserWorkflow._checkPrimaryDataset R0201 Method could be a function\n* Line 261 in RESTUserWorkflow.validate W0613 Unused argument 'apiobj'\n* Line 261 in RESTUserWorkflow.validate W0613 Unused argument 'api'\n* Line 261 in RESTUserWorkflow.validate R0912 Too many branches (24/12)\n* Line 261 in RESTUserWorkflow.validate R0915 Too many statements (128/100)\n* Line 457 in RESTUserWorkflow.put R0913 Too many arguments (55/7)\n* Line 457 in RESTUserWorkflow.put R0914 Too many local variables (55/30)\n* Line 462 in RESTUserWorkflow.put W0613 Unused argument 'blacklistT1'\n* Line 462 in RESTUserWorkflow.put W0613 Unused argument 'nonprodsw'\n* Line 538 in RESTUserWorkflow.post R0913 Too many arguments (12/7)\n* Line 562 in RESTUserWorkflow.get R0913 Too many arguments (10/7)\n* Line 12 W0614 Unused import pNameRE from wildcard import\n* Line 12 W0614 Unused import RX_TEXT_FAIL from wildcard import\n* Line 12 W0614 Unused import RX_OUTTYPES from wildcard import\n* Line 12 W0614 Unused import RX_PUBLISH from wildcard import\n* Line 12 W0614 Unused import RX_PARENTLFN from wildcard import\n* Line 12 W0614 Unused import RX_PUBLICATION from wildcard import\n* Line 12 W0614 Unused import RX_CHECKSUM from wildcard import\n* Line 12 W0614 Unused import RX_CAMPAIGN from wildcard import\n* Line 12 W0614 Unused import wfBase from wildcard import\n* Line 12 W0614 Unused import RX_OUT_DATASET from wildcard import\n* Line 12 W0614 Unused import lfnParts from wildcard import\n* Line 12 W0614 Unused import RX_SUBGETWORKER from wildcard import\n* Line 12 W0614 Unused import RX_LUMILIST from wildcard import\n* Line 12 W0614 Unused import RX_HOURS from wildcard import\n* Line 12 W0614 Unused import RX_LFNPATH from wildcard import\n* Line 12 W0614 Unused import RX_DN from wildcard import\n* Line 12 W0614 Unused import RX_SCHEDULER from wildcard import\n* Line 12 W0614 Unused import RX_WORKFLOW_LEN from wildcard import\n* Line 12 W0614 Unused import RX_URL from wildcard import\n* Line 12 W0614 Unused import RX_CERT from wildcard import\n* Line 12 W0614 Unused import RX_STATUS from wildcard import\n* Line 12 W0614 Unused import RX_BLOCK from wildcard import\n* Line 12 W0614 Unused import RX_SUBPOSTWORKER from wildcard import\n* Line 11 W0611 Unused import authz_login_valid\n* Line 12 W0614 Unused import RX_GLOBALTAG from wildcard import\n* Line 12 W0614 Unused import RX_WORKER_NAME from wildcard import\n* Line 12 W0614 Unused import DATASET_RE from wildcard import\n* Line 12 W0614 Unused import RX_SUBRES_TASK from wildcard import\n* Line 10 W0611 Unused import DataWorkflow\n* Line 12 W0614 Unused import RX_SUBRES_SI from wildcard import\n* Line 12 W0614 Unused import RX_OUTDSLFN from wildcard import\n* Line 18 W0611 Unused import time\n* Line 12 W0614 Unused import RX_FILESTATE from wildcard import\n* plus 53 comments on code style\n\nAbbreviated pylint report for src/python/CRABInterface/RESTWorkerWorkflow.py follows:\n* Line 25 in RESTWorkerWorkflow.validate W0613 Unused argument 'apiobj'\n* Line 25 in RESTWorkerWorkflow.validate W0613 Unused argument 'api'\n* Line 12 W0611 Unused import cherrypy\n* plus 4 total warnings\n* plus 21 comments on code style\n\nsrc/python/TaskWorker/Actions/DagmanResubmitter.py got worse in pylint. Report follows:\n* Line 44 in DagmanResubmitter.executeInternal W1201 Specify string format arguments as logging function parameters\n* Line 45 in DagmanResubmitter.executeInternal W1201 Specify string format arguments as logging function parameters\n* Line 68 in DagmanResubmitter.executeInternal W1201 Specify string format arguments as logging function parameters\n* Line 132 in DagmanResubmitter.executeInternal W1201 Specify string format arguments as logging function parameters\n* Line 27 in DagmanResubmitter.executeInternal W0613 Unused argument 'args'\n* Line 60 in DagmanResubmitter.executeInternal W0612 Unused variable 'address'\n* Line 27 in DagmanResubmitter.executeInternal R0912 Too many branches (30/12)\n* Line 171 in DagmanResubmitter.execute W1201 Specify string format arguments as logging function parameters\n* Line 186 in DagmanResubmitter.resubmitPublication W0621 Redefining name 'server' from outer scope (line 231)\n* Line 211 in DagmanResubmitter.resubmitPublication W0703 Catching too general exception Exception\n* Line 228 E1101 Instance of 'Configuration' has no 'TaskWorker' member\n* Line 229 E1101 Instance of 'Configuration' has no 'TaskWorker' member\n* Line 231 E1101 Instance of 'Configuration' has no 'TaskWorker' member\n* Line 231 E1101 Instance of 'Configuration' has no 'TaskWorker' member\n* plus 12 comments on code style\n"}}