Project

General

Profile

Actions

Bug #18688

closed

teuthology-suite rerun option won't work with -S arg

Added by Yuri Weinstein about 7 years ago. Updated about 7 years ago.

Status:
Closed
Priority:
Normal
Assignee:
-
Category:
-
% Done:

0%

Source:
Q/A
Tags:
Backport:
Regression:
No
Severity:
3 - minor
Reviewed:
Affected Versions:
ceph-qa-suite:
Crash signature (v1):
Crash signature (v2):

Description

compare

CEPH_BRANCH=83af8cdaaa6d94404e6146b68e532a784e3cc99c; MACHINE_NAME=mira; teuthology-suite -v -S $CEPH_BRANCH -m $MACHINE_NAME -r teuthology-2017-01-25_16:42:38-powercycle-master-testing-basic-mira -R fail,dead -e $CEPH_QA_EMAIL --suite-repo https://github.com/ceph/ceph.git --ceph-repo https://github.com/ceph/ceph.git -p 90 --dry-run

=> 0 jobs

vs

CEPH_BRANCH=hammer; MACHINE_NAME=mira; teuthology-suite -v -c $CEPH_BRANCH -m $MACHINE_NAME -r teuthology-2017-01-25_16:42:38-powercycle-master-testing-basic-mira -R fail,dead -e $CEPH_QA_EMAIL --suite-repo https://github.com/ceph/ceph.git --ceph-repo https://github.com/ceph/ceph.git -p 90 --dry-run

=> 1 job as expected

Actions #1

Updated by Yuri Weinstein about 7 years ago

  • Project changed from rbd to teuthology
Actions #2

Updated by Zack Cerza about 7 years ago

  • Status changed from New to Closed

-S is --sha1 - you appear to be passing a branch...

Actions #3

Updated by Zack Cerza about 7 years ago

My last comment was incorrect; your variable names are pretty confusing.

The real issue is in the first invocation, you're not using -c hammer.

Actions #4

Updated by Yuri Weinstein about 7 years ago

Are you saying it's work-as-designed and -r won't work with -S? And only work with -c ?

Actions #5

Updated by Nathan Cutler about 7 years ago

Zack, he's passing -S 83af8cdaaa6d94404e6146b68e532a784e3cc99c

Actions #6

Updated by Nathan Cutler about 7 years ago

and that SHA1 matches the one in teuthology-2017-01-25_16:42:38-powercycle-master-testing-basic-mira

Yuri, what happens if you omit -S (and -c)? Just -r teuthology-2017-01-25_16:42:38-powercycle-master-testing-basic-mira should be enough.

Actions #7

Updated by Yuri Weinstein about 7 years ago

Nathan, AFAIK you must have -c and I think the reason is that we query yaml configurations from the repo as oppose to from tests' results.

Actions

Also available in: Atom PDF