Revision 9355a604 snf-tools/synnefo_tools/burnin/common.py

b/snf-tools/synnefo_tools/burnin/common.py
54 54

  
55 55
# --------------------------------------------------------------------
56 56
# Global variables
57
logger = None   # Invalid constant name. pylint: disable-msg=C0103
58
success = None  # Invalid constant name. pylint: disable-msg=C0103
57
logger = None   # pylint: disable=invalid-name
58
success = None  # pylint: disable=invalid-name
59 59
SNF_TEST_PREFIX = "snf-test-"
60 60
CONNECTION_RETRY_LIMIT = 2
61 61
SYSTEM_USERS = ["images@okeanos.grnet.gr", "images@demo.synnefo.org"]
......
79 79
        super(BurninTestResult, self).startTest(test)
80 80
        logger.log(test.__class__.__name__, test.shortDescription())
81 81

  
82
    # Method could be a function. pylint: disable-msg=R0201
82
    # pylint: disable=no-self-use
83 83
    def _test_failed(self, test, err):
84 84
        """Test failed"""
85 85
        # Get class name
......
108 108

  
109 109
# --------------------------------------------------------------------
110 110
# Helper Classes
111
# Too few public methods. pylint: disable-msg=R0903
112
# Too many instance attributes. pylint: disable-msg=R0902
111
# pylint: disable=too-few-public-methods
112
# pylint: disable=too-many-instance-attributes
113 113
class Clients(object):
114 114
    """Our kamaki clients"""
115 115
    auth_url = None
......
186 186

  
187 187
# --------------------------------------------------------------------
188 188
# BurninTests class
189
# Too many public methods (45/20). pylint: disable-msg=R0904
189
# pylint: disable=too-many-public-methods
190 190
class BurninTests(unittest.TestCase):
191 191
    """Common class that all burnin tests should implement"""
192 192
    clients = Clients()
......
245 245

  
246 246
    def _run_tests(self, tcases):
247 247
        """Run some generated testcases"""
248
        global success  # Using global. pylint: disable-msg=C0103,W0603,W0602
248
        global success  # pylint: disable=invalid-name, global-statement
249 249

  
250 250
        for tcase in tcases:
251 251
            self.info("Running testsuite %s", tcase.__name__)
......
514 514
        self.info("Getting quotas")
515 515
        return dict(self.clients.astakos.get_quotas())
516 516

  
517
    # Invalid argument name. pylint: disable-msg=C0103
518
    # Too many arguments. pylint: disable-msg=R0913
517
    # pylint: disable=invalid-name
518
    # pylint: disable=too-many-arguments
519 519
    def _check_quotas(self, puuid=None, disk=None, vm=None, diskspace=None,
520 520
                      ram=None, ip=None, cpu=None, network=None):
521 521
        """Check that quotas' changes are consistent
......
605 605

  
606 606
    """
607 607
    # Initialize logger
608
    global logger  # Using global statement. pylint: disable-msg=C0103,W0603
608
    global logger  # pylint: disable=invalid-name, global-statement
609 609
    curr_time = datetime.datetime.now()
610 610
    logger = Log(opts.log_folder, verbose=opts.verbose,
611 611
                 use_colors=opts.use_colors, in_parallel=False,
......
647 647
# Run Burnin
648 648
def run_burnin(testsuites, failfast=False):
649 649
    """Run burnin testsuites"""
650
    # Using global. pylint: disable-msg=C0103,W0603,W0602
650
    # pylint: disable=invalid-name,global-statement
651
    # pylint: disable=global-variable-not-assigned
651 652
    global logger, success
652 653

  
653 654
    success = True
......
662 663

  
663 664
def run_tests(tcases, failfast=False):
664 665
    """Run some testcases"""
665
    global success  # Using global. pylint: disable-msg=C0103,W0603,W0602
666
    # pylint: disable=invalid-name,global-statement
667
    # pylint: disable=global-variable-not-assigned
668
    global success
666 669

  
667 670
    for tcase in tcases:
668 671
        was_success = run_test(tcase)

Also available in: Unified diff