Fix get_hash() arg ordering

Signed-off-by: Zack Cerza <zack.cerza@inktank.com>

diff --git a/teuthology/suite.py b/teuthology/suite.py
index 3a28ea791b7d646afdef5cffb66eed5edb3adf90..e1672dabb60bdec692ebc8510e9135a57a926432 100644 (file)
--- 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 14869a372512cc43f39baef234124f71081d55c8..27558b6dbc892200c2b5b3024a37d9b8c4786d74 100644 (file)
--- 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