Bug #10279
closed
--os-version and --os-type is not respected
Added by Andrew Schoen over 9 years ago.
Updated over 9 years ago.
Description
When running teuthology the --os-version and --os-type flags aren't respected. If present, they are ignored and the values from the yaml config is still used.
- Status changed from New to Resolved
- Status changed from Resolved to In Progress
This patch for this was reverted after discovering a bug related to the default value of --os-type. From what I can gather that default value could safely be removed. The default seems to be set in teuthology.misc.get_distro.
It'll be difficult to use --os-type as an override for config values when it's always defaulted to 'ubuntu'. How would we know when it's actually being used and when it's just the default?
As I recall it, the breakage was caused in a run whose job config contained:
os_type: rhel
os_version: '7.0'
Since the job was scheduled, teuthology
was called without --os-type
or --os-version
. Since scripts/run.py
contains a default for --os-type
('ubuntu') and not for --os-version
, the job ended up trying to provision a vps with os_type: ubuntu
and os_version: '7.0'
, which is impossible.
It does seem that removing the --os-type
default from scripts/run.py
and making teutholog.run.main()
respect the values properly would work well. We'd also probably want to move get_distro()
and get_distro_version()
from teuthology.misc
to teuthology.provision
, since that module is the only user of those functions.
- Status changed from In Progress to Resolved
Also available in: Atom
PDF