[pylint] Enable no-member check (#5034)

.pylintrc

@@ -141,7 +141,6 @@ disable=print-statement,
         comprehension-escape,
         fixme,
         import-error,
-        no-member
 
 # Enable the message, report, category or checker with the given id(s). You can
 # either give multiple identifier separated by comma (,) or put this option

infra/build/functions/project_sync_test.py

@@ -32,6 +32,8 @@ from project_sync import ProjectMetadata
 from project_sync import sync_projects
 import test_utils
 
+# pylint: disable=no-member
+
 
 # pylint: disable=too-few-public-methods
 class Repository:

infra/build/functions/request_build_test.py

@@ -33,6 +33,8 @@ from request_build import get_project_data
 from request_build import update_build_history
 import test_utils
 
+# pylint: disable=no-member
+
 
 class TestRequestBuilds(unittest.TestCase):
   """Unit tests for sync."""

infra/build/functions/request_coverage_build_test.py

@@ -30,6 +30,8 @@ from datastore_entities import Project
 from build_and_run_coverage import get_build_steps
 import test_utils
 
+# pylint: disable=no-member
+
 
 class TestRequestCoverageBuilds(unittest.TestCase):
   """Unit tests for sync."""

infra/build/functions/update_build_status_test.py

@@ -30,6 +30,8 @@ from datastore_entities import LastSuccessfulBuild
 import test_utils
 import update_build_status
 
+# pylint: disable=no-member
+
 
 # pylint: disable=too-few-public-methods
 class MockGetBuild:

infra/retry.py

@@ -62,9 +62,8 @@ def wrap(retries,
         sleep(get_delay(num_try, delay, backoff))
         return True
 
-      logging.log_error('Retrying on %s failed with %s. Raise.' %
+      logging.error('Retrying on %s failed with %s. Raise.', function_with_type,
-                        (function_with_type, sys.exc_info()[1]),
+                    sys.exc_info()[1])
-                        total=tries)
       return False
 
     @functools.wraps(func)