Project

General

Profile

Actions

Fix #8712

closed

Teuthology-lock should not lock if the downburst failed

Added by Anonymous almost 10 years ago. Updated over 9 years ago.

Status:
Resolved
Priority:
Normal
Assignee:
Category:
-
Target version:
% Done:

0%

Source:
other
Tags:
Backport:
Reviewed:
Affected Versions:
ceph-qa-suite:
Crash signature (v1):
Crash signature (v2):

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.

Actions #1

Updated by Anonymous almost 10 years ago

  • Assignee set to Anonymous
Actions #2

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.

Actions #3

Updated by Anonymous over 9 years ago

  • Status changed from New to In Progress
Actions #4

Updated by Anonymous over 9 years ago

  • Status changed from In Progress to Fix Under Review
Actions #5

Updated by Ian Colle over 9 years ago

  • Tracker changed from Bug to Fix
Actions #6

Updated by Ian Colle over 9 years ago

  • Target version set to sprint11
Actions #7

Updated by Ian Colle over 9 years ago

Warren, what's the status of this?

Actions #8

Updated by Anonymous over 9 years ago

The last review suggest some changes. I will resubmit today.

Actions #9

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.

Actions #10

Updated by Zack Cerza over 9 years ago

  • Target version changed from sprint11 to sprint13
Actions #11

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?

Actions #12

Updated by Anonymous over 9 years ago

  • Assignee changed from Anonymous to Zack Cerza
Actions #13

Updated by Anonymous over 9 years ago

  • Status changed from Fix Under Review to Resolved

Fixed in commit 6f5296d40c8fd73b3cbb7db2875289ddbeebf0bd

Actions

Also available in: Atom PDF