Fix #8712
closedTeuthology-lock should not lock if the downburst failed
0%
Description
Teuthology-lock --lock-many 1 --machine-type vps --os-type aardvark --os-version vampirebat successfully locks a machine, but the downburst create failed (Image not found). The lock should definitely not happen, and an appropriate error message other than the downburst dump should be displayed.
Updated by Anonymous almost 10 years ago
The following cases all end up locking a vps machine that is improperly downbursted:
teuthology-lock --lock-many 1 --machine-type vps --os-type aardvark
teuthology-lock --lock-many 1 --machine-type vps --os-type aardvark --os-version vampirebat
teuthology-lock --lock-many 1 --machine-type vps --os-type rhel --os-version vampirebat
teuthology-lock --lock-many 1 --machine-type vps --os-version vampirebat
The last 3 examples lock a machine and say that they lock a machine, the first example throws a KeyError exception and still locks a machine.
I am sending in a pull request that displays error messages and does not lock a machine in the above examples.
Updated by Anonymous over 9 years ago
- Status changed from In Progress to Fix Under Review
Updated by Anonymous over 9 years ago
The last review suggest some changes. I will resubmit today.
Updated by Anonymous over 9 years ago
New changes have been pushed to https://github.com/ceph/teuthology/pull/297
The changes are failing jenkins testing. I believe that this is because the unit tests that
I have added are failing because the downburst changes in wip-8925-wusui are not yet checked in.
Updated by Zack Cerza over 9 years ago
- Target version changed from sprint11 to sprint13
Updated by Anonymous over 9 years ago
Urgh. I have implemented this change to be able to handle checking without downburst (using the downburst data if it's available). To test, I removed the local downburst executable ('which downburst' now shows nothing), and tried to create a vm system. This failed in another part of teuthology/lock.py which tried to start with a fresh vm using downburst. How does this ever work normally when downburst is not installed? I suspect it does not, in which case my checking to make sure vm parameters are valid is moot because this code would never work in the first place.
Excuse me while I go chase a squirrel.
Why are there teeth marks on my tail?
Updated by Anonymous over 9 years ago
- Assignee changed from Anonymous to Zack Cerza
Updated by Anonymous over 9 years ago
- Status changed from Fix Under Review to Resolved
Fixed in commit 6f5296d40c8fd73b3cbb7db2875289ddbeebf0bd