From: Zack Cerza Date: Wed, 16 Jul 2014 18:07:55 +0000 (-0600) Subject: Fix get_hash() arg ordering X-Git-Tag: v0.94.10~27^2^2~364^2~24 X-Git-Url: http://git-server-git.apps.pok.os.sepia.ceph.com/?a=commitdiff_plain;h=b8c443b44fcad29325ec33ac2b3932c59eb6a711;p=ceph.git Fix get_hash() arg ordering Signed-off-by: Zack Cerza --- diff --git a/teuthology/suite.py b/teuthology/suite.py index 3a28ea791b7d..e1672dabb60b 100644 --- a/teuthology/suite.py +++ b/teuthology/suite.py @@ -331,7 +331,7 @@ def get_worker(machine_type): def get_hash(project='ceph', branch='master', flavor='basic', - distro='ubuntu', machine_type='plana'): + machine_type='plana', distro='ubuntu'): """ Find the hash representing the head of the project's repository via querying a gitbuilder repo. diff --git a/teuthology/test/test_suite.py b/teuthology/test/test_suite.py index 14869a372512..27558b6dbc89 100644 --- a/teuthology/test/test_suite.py +++ b/teuthology/test/test_suite.py @@ -64,12 +64,16 @@ class TestSuiteOnline(object): if 'TEST_ONLINE' not in os.environ: skip("To run these sets, set the environment variable TEST_ONLINE") - def test_ceph_hash(self): + def test_ceph_hash_simple(self): resp = requests.get( 'https://api.github.com/repos/ceph/ceph/git/refs/heads/master') ref_hash = resp.json()['object']['sha'] assert suite.get_hash('ceph') == ref_hash + def test_kernel_hash_saya(self): + # We don't currently have these packages. + assert suite.get_hash('kernel', 'master', 'basic', 'saya') is None + def test_all_master_branches(self): # Don't attempt to send email config.results_email = None