Project

General

Profile

Actions

Bug #10593

closed

s3tests branch defaults to master and cause tests fail

Added by Yuri Weinstein over 9 years ago. Updated over 9 years ago.

Status:
Resolved
Priority:
Urgent
Category:
-
Target version:
% Done:

0%

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

Description

Run http://pulpito.front.sepia.ceph.com/teuthology-2015-01-19_18:08:36-upgrade:dumpling-dumpling-distro-basic-multi/

Jobs ['713401', '713402', '713403', '713404', '713405', '713406', '713407']

Log for one http://qa-proxy.ceph.com/teuthology/teuthology-2015-01-19_18:08:36-upgrade:dumpling-dumpling-distro-basic-multi/713401/

2015-01-20T12:59:42.240 INFO:tasks.s3tests:Downloading s3-tests...
2015-01-20T12:59:42.241 INFO:teuthology.orchestra.run.plana49:Running: 'git clone -b master git://ceph.com/git/s3-tests.git /home/ubuntu/cephtest/s3-tests'

Related issues 1 (0 open1 closed)

Has duplicate teuthology - Bug #10574: s3tests failed in upgrade:dumpling-dumpling-distro-basic-multi runResolved01/19/2015

Actions
Actions #1

Updated by Yuri Weinstein over 9 years ago

  • Status changed from New to In Progress

See code https://github.com/ceph/ceph-qa-suite/blob/master/tasks/s3tests.py#L125

To fix - use force-branch: dumpling in yaml

Actions #2

Updated by Yuri Weinstein over 9 years ago

  • Assignee changed from Yuri Weinstein to Zack Cerza

This is temporarily fixed by https://github.com/ceph/ceph-qa-suite/pull/305

We need to fix it in teuthology code, so s3tests brach defaults correctly to ceph-qa branch when force-branch is not used.

Pls revert https://github.com/ceph/ceph-qa-suite/pull/305 when this is done.

Zack - assigned to you for triaging.

Actions #3

Updated by Yuri Weinstein over 9 years ago

  • Target version set to sprint22
Actions #4

Updated by Yuri Weinstein over 9 years ago

It looks like we can cherrypick this commit 01b556afc148dd70d59260f7f4b0712deddbb844

Pls review https://github.com/ceph/ceph-qa-suite/pull/306

Actions #5

Updated by Yuri Weinstein over 9 years ago

  • Status changed from In Progress to Fix Under Review

It looks like we can cherrypick this commit 01b556afc148dd70d59260f7f4b0712deddbb844

Actions #6

Updated by Zack Cerza over 9 years ago

I would encourage using 'branch:' and not 'force-branch:'.

Also, as Loic mentoined, there is a merge conflict in your cherry-pick commit.

Finally: with the "Smarter s3tests branch selection", the priority is:

force-branch -> branch -> suite_branch -> ceph branch

Are you asking for a different order?

Actions #7

Updated by Zack Cerza over 9 years ago

  • Status changed from Fix Under Review to 4
  • Assignee changed from Zack Cerza to Yuri Weinstein
Actions #8

Updated by Yuri Weinstein over 9 years ago

  • Status changed from 4 to Resolved
Actions

Also available in: Atom PDF