{"action":"created","issue":{"url":"https://api.github.com/repos/dmwm/WMCore/issues/6127","labels_url":"https://api.github.com/repos/dmwm/WMCore/issues/6127/labels{/name}","comments_url":"https://api.github.com/repos/dmwm/WMCore/issues/6127/comments","events_url":"https://api.github.com/repos/dmwm/WMCore/issues/6127/events","html_url":"https://github.com/dmwm/WMCore/pull/6127","id":100541993,"number":6127,"title":"Apply known breaking changes to types,","user":{"login":"ericvaandering","id":1672366,"avatar_url":"https://avatars.githubusercontent.com/u/1672366?v=3","gravatar_id":"","url":"https://api.github.com/users/ericvaandering","html_url":"https://github.com/ericvaandering","followers_url":"https://api.github.com/users/ericvaandering/followers","following_url":"https://api.github.com/users/ericvaandering/following{/other_user}","gists_url":"https://api.github.com/users/ericvaandering/gists{/gist_id}","starred_url":"https://api.github.com/users/ericvaandering/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/ericvaandering/subscriptions","organizations_url":"https://api.github.com/users/ericvaandering/orgs","repos_url":"https://api.github.com/users/ericvaandering/repos","events_url":"https://api.github.com/users/ericvaandering/events{/privacy}","received_events_url":"https://api.github.com/users/ericvaandering/received_events","type":"User","site_admin":false},"labels":[],"state":"open","locked":false,"assignee":null,"milestone":null,"comments":5,"created_at":"2015-08-12T13:23:03Z","updated_at":"2015-08-12T16:49:12Z","closed_at":null,"pull_request":{"url":"https://api.github.com/repos/dmwm/WMCore/pulls/6127","html_url":"https://github.com/dmwm/WMCore/pull/6127","diff_url":"https://github.com/dmwm/WMCore/pull/6127.diff","patch_url":"https://github.com/dmwm/WMCore/pull/6127.patch"},"body":"Do not merge this! It will break things, but I want Jenkins to have a look at it for me."},"comment":{"url":"https://api.github.com/repos/dmwm/WMCore/issues/comments/130367365","html_url":"https://github.com/dmwm/WMCore/pull/6127#issuecomment-130367365","issue_url":"https://api.github.com/repos/dmwm/WMCore/issues/6127","id":130367365,"user":{"login":"DMWMBot","id":1124170,"avatar_url":"https://avatars.githubusercontent.com/u/1124170?v=3","gravatar_id":"","url":"https://api.github.com/users/DMWMBot","html_url":"https://github.com/DMWMBot","followers_url":"https://api.github.com/users/DMWMBot/followers","following_url":"https://api.github.com/users/DMWMBot/following{/other_user}","gists_url":"https://api.github.com/users/DMWMBot/gists{/gist_id}","starred_url":"https://api.github.com/users/DMWMBot/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/DMWMBot/subscriptions","organizations_url":"https://api.github.com/users/DMWMBot/orgs","repos_url":"https://api.github.com/users/DMWMBot/repos","events_url":"https://api.github.com/users/DMWMBot/events{/privacy}","received_events_url":"https://api.github.com/users/DMWMBot/received_events","type":"User","site_admin":false},"created_at":"2015-08-12T16:49:12Z","updated_at":"2015-08-12T16:49:12Z","body":"Summary of pylint changes for pull request 6127:\n* Score for src/python/WMCore/WMRuntime/Tools/Plugins/ApMonLite/ApMonDestination.py decreased from 9.66 to 9.55\n* Warnings for src/python/WMCore/WMRuntime/Tools/Plugins/ApMonLite/ApMonDestination.py increased from 3 to 4\n* Score for src/python/WMCore/Configuration.py decreased from 9.54 to 9.50\n* Warnings for src/python/WMCore/Configuration.py increased from 13 to 14\n* Score for src/python/WMCore/Database/ipy_profile_couch.py decreased from -2.77 to -2.82\n* Warnings for src/python/WMCore/Database/ipy_profile_couch.py increased from 10 to 11\n* Score for src/python/WMCore/WMSpec/ConfigSectionTree.py decreased from 9.89 to 9.83\n* Warnings for src/python/WMCore/WMSpec/ConfigSectionTree.py increased from 2 to 3\n* Score for src/python/WMCore/WebTools/Page.py decreased from 9.66 to 9.62\n* Warnings for src/python/WMCore/WebTools/Page.py increased from 6 to 7\n* Score for src/python/WMCore/SiteScreening/BlackWhiteListParser.py decreased from 9.25 to 9.17\n* Warnings for src/python/WMCore/SiteScreening/BlackWhiteListParser.py increased from 9 to 10\n* Score for src/python/WMCore/WebTools/WebAPI.py decreased from 7.97 to 7.84\n* Warnings for src/python/WMCore/WebTools/WebAPI.py increased from 14 to 15\n\nsrc/python/WMCore/WMRuntime/Tools/Plugins/ApMonLite/ApMonDestination.py got worse in pylint. Report follows:\n* Line 35 in ApMonDestination.__init__ W0231 __init__ method from base class 'dict' is not called\n* Line 149 in ApMonDestination.packParameter R0201 Method could be a function\n* Line 197 in ApMonDestination.udpBroadcast W0703 Catching too general exception Exception\n* Line 197 in ApMonDestination.udpBroadcast W0612 Unused variable 'ex'\n* Line 4 W0611 Unused import types\n* plus 16 comments on code style\n\nsrc/python/WMCore/Configuration.py got worse in pylint. Report follows:\n* Line 40 in format W0622 Redefining built-in 'format'\n* Line 89 in ConfigSection.__eq__ W0212 Access to a protected member _internal_documentation of a client class\n* Line 90 in ConfigSection.__eq__ W0212 Access to a protected member _internal_name of a client class\n* Line 91 in ConfigSection.__eq__ W0212 Access to a protected member _internal_settings of a client class\n* Line 92 in ConfigSection.__eq__ W0212 Access to a protected member _internal_docstrings of a client class\n* Line 93 in ConfigSection.__eq__ W0212 Access to a protected member _internal_children of a client class\n* Line 94 in ConfigSection.__eq__ W0212 Access to a protected member _internal_parent_ref of a client class\n* Line 126 in ConfigSection.__setattr__ W0212 Access to a protected member _internal_parent_ref of a client class\n* Line 164 in ConfigSection.__add__ W0212 Access to a protected member _internal_settings of a client class\n* Line 172 in ConfigSection.__add__ W0212 Access to a protected member _internal_name of a client class\n* Line 267 in ConfigSection.dictionary_ W0106 Expression \"[result.__setitem__(x, getattr(self, x)) for x in self._internal_settings]\" is assigned to nothing\n* Line 374 in Configuration.__add__ W0212 Access to a protected member _internal_sections of a client class\n* Line 377 in Configuration.__add__ W0106 Expression \"(getattr(self, configSect)) + (getattr(otherConfig, configSect))\" is assigned to nothing\n* Line 16 W0611 Unused import types\n* plus 33 comments on code style\n\nsrc/python/WMCore/Database/ipy_profile_couch.py fails the pylint check. Report follows:\n* Line 66 in load_config W0102 Dangerous default value [] as argument\n* Line 106 in httplib_request E0602 Undefined variable 'httplib'\n* Line 108 in httplib_request E0602 Undefined variable 'urllib'\n* Line 115 in httplib_request E0602 Undefined variable 'httplib'\n* Line 118 in httplib_request E0602 Undefined variable 'httplib'\n* Line 138 in print_data E0602 Undefined variable 'json'\n* Line 139 in print_data E0602 Undefined variable 'PM'\n* Line 151 in print_data E0602 Undefined variable 'PM'\n* Line 170 in couch_help W0602 Using global for 'magic_list' but no assignment is done\n* Line 165 in couch_help W0613 Unused argument 'arg'\n* Line 165 in couch_help W0613 Unused argument 'self'\n* Line 185 in db_info E0602 Undefined variable 'URI'\n* Line 186 in db_info E0602 Undefined variable 'DB'\n* Line 187 in db_info E0602 Undefined variable 'DEBUG'\n* Line 190 in db_info E0602 Undefined variable 'json'\n* Line 197 in couch_views E0602 Undefined variable 'URI'\n* Line 198 in couch_views E0602 Undefined variable 'DB'\n* Line 199 in couch_views E0602 Undefined variable 'DEBUG'\n* Line 200 in couch_views E0602 Undefined variable 'json'\n* Line 204 in couch_views E0602 Undefined variable 'PM'\n* Line 205 in couch_views E0602 Undefined variable 'DB'\n* Line 206 in couch_views E0602 Undefined variable 'DEBUG'\n* Line 207 in couch_views E0602 Undefined variable 'json'\n* Line 209 in couch_views E0602 Undefined variable 'PM'\n* Line 210 in couch_views E0602 Undefined variable 'PM'\n* Line 211 in couch_views E0602 Undefined variable 'PM'\n* Line 213 in couch_views E0602 Undefined variable 'PM'\n* Line 214 in couch_views E0602 Undefined variable 'PM'\n* Line 225 in create_view E0602 Undefined variable 'URI'\n* Line 226 in create_view E0602 Undefined variable 'DB'\n* Line 226 in create_view E0602 Undefined variable 'DESIGN'\n* Line 227 in create_view E0602 Undefined variable 'DEBUG'\n* Line 228 in create_view E0602 Undefined variable 'json'\n* Line 233 in create_view E0602 Undefined variable 'JSONEncoder'\n* Line 236 in create_view E0602 Undefined variable 'DEBUG'\n* Line 240 in create_view E0602 Undefined variable 'json'\n* Line 249 in delete_view E0602 Undefined variable 'URI'\n* Line 250 in delete_view E0602 Undefined variable 'DB'\n* Line 250 in delete_view E0602 Undefined variable 'DESIGN'\n* Line 251 in delete_view E0602 Undefined variable 'DEBUG'\n* Line 254 in delete_view E0602 Undefined variable 'json'\n* Line 265 in delete_view W0702 No exception type(s) specified\n* Line 260 in delete_view E0602 Undefined variable 'JSONEncoder'\n* Line 263 in delete_view E0602 Undefined variable 'DEBUG'\n* Line 275 in delete_all_views E0602 Undefined variable 'URI'\n* Line 276 in delete_all_views E0602 Undefined variable 'DB'\n* Line 277 in delete_all_views E0602 Undefined variable 'DEBUG'\n* Line 280 in delete_all_views E0602 Undefined variable 'json'\n* Line 287 in create_db E0602 Undefined variable 'URI'\n* Line 289 in create_db E0602 Undefined variable 'DEBUG'\n* Line 292 in create_db E0602 Undefined variable 'json'\n* Line 299 in delete_db E0602 Undefined variable 'URI'\n* Line 301 in delete_db E0602 Undefined variable 'DEBUG'\n* Line 304 in delete_db E0602 Undefined variable 'json'\n* Line 311 in get_all_docs E0602 Undefined variable 'URI'\n* Line 312 in get_all_docs E0602 Undefined variable 'DB'\n* Line 318 in get_all_docs E0602 Undefined variable 'DEBUG'\n* Line 324 in get_all_docs E0602 Undefined variable 'json'\n* Line 326 in get_doc W0622 Redefining built-in 'id'\n* Line 331 in get_doc E0602 Undefined variable 'URI'\n* Line 332 in get_doc E0602 Undefined variable 'DB'\n* Line 334 in get_doc E0602 Undefined variable 'DEBUG'\n* Line 337 in get_doc E0602 Undefined variable 'json'\n* Line 362 in load_module W0702 No exception type(s) specified\n* Line 357 in load_module W0123 Use of eval\n* Line 359 in load_module E0602 Undefined variable 'PM'\n* Line 364 in load_module W0107 Unnecessary pass statement\n* Line 22 W0611 Unused import types\n* Line 20 W0611 Unused import re\n* plus 66 comments on code style\n\nsrc/python/WMCore/WMSpec/ConfigSectionTree.py got worse in pylint. Report follows:\n* Line 223 in format W0622 Redefining built-in 'format'\n* Line 313 in TreeHelper.deleteNode W0621 Redefining name 'nodeName' from outer scope (line 18)\n* Line 13 W0611 Unused import types\n* plus 7 comments on code style\n\nsrc/python/WMCore/WebTools/Page.py got worse in pylint. Report follows:\n* Line 65 in Page.log R0201 Method could be a function\n* Line 81 in TemplatedPage.__init__ W0102 Dangerous default value {} as argument\n* Line 99 in TemplatedPage.templatepage W0622 Redefining built-in 'file'\n* Line 341 in runDas W0622 Redefining built-in 'object'\n* Line 343 in runDas W0702 No exception type(s) specified\n* Line 15 W0611 Unused import traceback\n* Line 6 W0611 Unused import urllib\n* Line 14 W0611 Unused import types\n* plus 70 comments on code style\n\nsrc/python/WMCore/SiteScreening/BlackWhiteListParser.py got worse in pylint. Report follows:\n* Line 29 in BlackWhiteListParser.__init__ W0622 Redefining built-in 'dict'\n* Line 29 in BlackWhiteListParser.__init__ W0102 Dangerous default value {} as argument\n* Line 93 in BlackWhiteListParser.checkBlackList W0102 Dangerous default value [] as argument\n* Line 123 in BlackWhiteListParser.checkWhiteList W0102 Dangerous default value [] as argument\n* Line 219 in BlackWhiteListParser.match W0702 No exception type(s) specified\n* Line 203 in BlackWhiteListParser.match R0201 Method could be a function\n* Line 241 in SEBlackWhiteListParser.__init__ W0622 Redefining built-in 'dict'\n* Line 241 in SEBlackWhiteListParser.__init__ W0102 Dangerous default value {} as argument\n* Line 254 in CEBlackWhiteListParser.__init__ W0622 Redefining built-in 'dict'\n* Line 254 in CEBlackWhiteListParser.__init__ W0102 Dangerous default value {} as argument\n* Line 15 W0611 Unused import types\n* plus 16 comments on code style\n\nsrc/python/WMCore/WebTools/WebAPI.py fails the pylint check. Report follows:\n* Line 13 in WebAPI.__init__ W0102 Dangerous default value {} as argument\n* Line 50 in WebAPI.index W0613 Unused argument 'kwargs'\n* Line 58 in WebAPI.default W0613 Unused argument 'args'\n* Line 58 in WebAPI.default W0613 Unused argument 'kwargs'\n* Line 77 in WebAPI.json W0123 Use of eval\n* Line 86 in WebAPI.dasOutput W0622 Redefining built-in 'dict'\n* Line 89 in WebAPI.dasOutput W0702 No exception type(s) specified\n* Line 126 in WebAPI.plist W0123 Use of eval\n* Line 142 in WebAPI.xml W0123 Use of eval\n* Line 152 in WebAPI.runMethod W0622 Redefining built-in 'dict'\n* Line 147 in WebAPI.runMethod W0622 Redefining built-in 'input'\n* Line 159 in WebAPI.runMethod W0703 Catching too general exception Exception\n* Line 11 in WebAPI R0904 Too many public methods (31/20)\n* Line 5 W0611 Unused import sitetier\n* Line 5 W0611 Unused import countrycode\n* Line 8 W0611 Unused import types\n* plus 11 comments on code style\n"}}