From: Kamalesh Babulal Date: Tue, 15 Feb 2022 07:19:24 +0000 (+0530) Subject: test-018: make it PEP8 compliant X-Git-Tag: v3.1.0~308^2~2^2~85 X-Git-Url: http://git.ipfire.org/cgi-bin/gitweb.cgi?a=commitdiff_plain;h=da9a40ba1e1fde48ecdf4d4fcb2f5a5066d51efa;p=thirdparty%2Flibcgroup.git test-018: make it PEP8 compliant Make test 018-cgconfig-load_dir.py PEP-8 compliant. Signed-off-by: Kamalesh Babulal Signed-off-by: Tom Hromatka --- diff --git a/ftests/018-cgconfig-load_dir.py b/ftests/018-cgconfig-load_dir.py index 6c57c665..6245788e 100755 --- a/ftests/018-cgconfig-load_dir.py +++ b/ftests/018-cgconfig-load_dir.py @@ -23,8 +23,8 @@ from cgroup import Cgroup, CgroupVersion import consts import ftests -import os import sys +import os CPU_CTRL = 'cpu' MEMORY_CTRL = 'memory' @@ -53,22 +53,24 @@ CONFIG_FILE = '''group CONFIG_FILE_DIR = os.path.join(os.getcwd(), '018cgconfig') CONFIG_FILE_NAME = os.path.join(CONFIG_FILE_DIR, 'cgconfig.conf') + def prereqs(config): result = consts.TEST_PASSED cause = None if CgroupVersion.get_version('cpu') != CgroupVersion.CGROUP_V1: result = consts.TEST_SKIPPED - cause = "This test requires the cgroup v1 cpu controller" + cause = 'This test requires the cgroup v1 cpu controller' return result, cause if CgroupVersion.get_version('memory') != CgroupVersion.CGROUP_V1: result = consts.TEST_SKIPPED - cause = "This test requires the cgroup v1 memory controller" + cause = 'This test requires the cgroup v1 memory controller' return result, cause return result, cause + def setup(config): os.mkdir(CONFIG_FILE_DIR) @@ -76,24 +78,28 @@ def setup(config): f.write(CONFIG_FILE) f.close() + def test(config): Cgroup.configparser(config, load_dir=CONFIG_FILE_DIR) Cgroup.get_and_validate(config, CGNAME, 'cpu.cfs_period_us', CFS_PERIOD) Cgroup.get_and_validate(config, CGNAME, 'cpu.cfs_quota_us', CFS_QUOTA) Cgroup.get_and_validate(config, CGNAME, 'cpu.shares', SHARES) - Cgroup.get_and_validate(config, CGNAME, 'memory.limit_in_bytes', LIMIT_IN_BYTES) + Cgroup.get_and_validate(config, CGNAME, 'memory.limit_in_bytes', + LIMIT_IN_BYTES) Cgroup.get_and_validate(config, CGNAME, 'memory.soft_limit_in_bytes', SOFT_LIMIT_IN_BYTES) return consts.TEST_PASSED, None + def teardown(config): Cgroup.delete(config, CPU_CTRL, CGNAME) Cgroup.delete(config, MEMORY_CTRL, CGNAME) os.remove(CONFIG_FILE_NAME) os.rmdir(CONFIG_FILE_DIR) + def main(config): [result, cause] = prereqs(config) if result != consts.TEST_PASSED: @@ -107,6 +113,7 @@ def main(config): return [result, cause] + if __name__ == '__main__': config = ftests.parse_args() # this test was invoked directly. run only it