teuthology-suite: add default machine type(smithi)

right now the default_machine_type fallback is set to None via some internal mangaling, this makes users always have to pass --machine_type which mostly we prefer to smithi's . Hence, making it a default unless otherwise, should be helpful for users. Signed-off-by: Deepika Upadhyay <[email protected]>
ceph · Mar 5, 2021 · 1aae78d · 1aae78d
1 parent 17c0996
commit 1aae78d
Show file tree

Hide file tree

Showing 3 changed files with 6 additions and 8 deletions.
diff --git a/scripts/suite.py b/scripts/suite.py
@@ -7,8 +7,8 @@
 
 doc = """
 usage: teuthology-suite --help
-       teuthology-suite [-v | -vv ] --machine-type <type> --suite <suite> [options] [<config_yaml>...]
-       teuthology-suite [-v | -vv ] --machine-type <type> --rerun <name>  [options] [<config_yaml>...]
+       teuthology-suite [-v | -vv ] --suite <suite> [options] [<config_yaml>...]
+       teuthology-suite [-v | -vv ] --rerun <name>  [options] [<config_yaml>...]
 
 Run a suite of ceph integration tests. A suite is a directory containing
 facets. A facet is a directory containing config snippets. Running a suite
@@ -62,7 +62,7 @@
                               the user or system `teuthology.yaml` configuration
                               files respectively, otherwise use `master`.
   -m <type>, --machine-type <type>
-                              Machine type [default: {default_machine_type}]
+                              Machine type [default: {teuth_machine_type}]
   -d <distro>, --distro <distro>
                               Distribution to run against
   -D <distroversion>, --distro-version <distroversion>
@@ -174,7 +174,7 @@
  --force-priority             Skip the priority check.
 
 """.format(
-    default_machine_type=config.default_machine_type,
+    teuth_machine_type=config.teuth_machine_type,
     default_results_timeout=config.results_timeout,
     default_ceph_repo=defaults('--ceph-repo',
                             config.get_ceph_git_url()),

diff --git a/teuthology/config.py b/teuthology/config.py
@@ -140,6 +140,7 @@ class TeuthologyConfig(YamlConfig):
         'archive_upload_key': None,
         'archive_upload_url': None,
         'automated_scheduling': False,
+        'teuth_machine_type': 'smithi',
         'reserve_machines': 5,
         'ceph_git_base_url': 'https://github.com/ceph/',
         'ceph_git_url': None,

diff --git a/teuthology/suite/__init__.py b/teuthology/suite/__init__.py
@@ -112,10 +112,7 @@ def main(args):
         teuthology.log.setLevel(logging.DEBUG)
 
     if not conf.machine_type or conf.machine_type == 'None':
-        schedule_fail("Must specify a machine_type")
-    elif 'multi' in conf.machine_type:
-        schedule_fail("'multi' is not a valid machine_type. " +
-                      "Maybe you want 'plana,mira,burnupi' or similar")
+        conf.machine_type = config.teuth_machine_type
 
     if conf.email:
         config.results_email = conf.email