Project

General

Profile

Bug #35951

Recently merged "$" feature broke --filter

Added by Nathan Cutler about 1 year ago. Updated about 1 year ago.

Status:
New
Priority:
Urgent
Assignee:
Category:
-
Target version:
-
Start date:
09/12/2018
Due date:
% Done:

0%

Source:
Tags:
Backport:
Regression:
No
Severity:
3 - minor
Reviewed:
Affected Versions:
ceph-qa-suite:

Description

teuthology-suite --ceph wip-sage3-testing-2018-09-10-1637 --machine-type smithi --dry-run --suite rados/thrash --filter="{0-size-min-size-overrides/2-size-2-min-size.yaml 1-pg-log-overrides/normal_pg_log.yaml 2-recovery-overrides/{default.yaml} backoff/normal.yaml ceph.yaml clusters/{fixed-2.yaml openstack.yaml} d-balancer/crush-compat.yaml msgr-failures/osd-delay.yaml msgr/random.yaml objectstore/bluestore-bitmap.yaml rados.yaml rocksdb.yaml supported-random-distro$/{ubuntu_16.04.yaml} thrashers/none.yaml thrashosds-health.yaml workloads/rados_api_tests.yaml}"

The same command, with newlines for readability:

teuthology-suite --ceph wip-sage3-testing-2018-09-10-1637 --machine-type smithi --dry-run 
--suite rados/thrash --filter="{0-size-min-size-overrides/2-size-2-min-size.yaml 
1-pg-log-overrides/normal_pg_log.yaml 2-recovery-overrides/{default.yaml} backoff/normal.yaml 
ceph.yaml clusters/{fixed-2.yaml openstack.yaml} d-balancer/crush-compat.yaml
msgr-failures/osd-delay.yaml msgr/random.yaml objectstore/bluestore-bitmap.yaml rados.yaml
rocksdb.yaml supported-random-distro$/{ubuntu_16.04.yaml} thrashers/none.yaml thrashosds-health.yaml
workloads/rados_api_tests.yaml}"

The above combination of --suite and --filter should match this job: http://pulpito.ceph.com/sage-2018-09-11_18:22:38-rados-wip-sage3-testing-2018-09-10-1637-distro-basic-smithi/3005949/

But it matches 0 jobs due to presence of the magic "$"

Presumably the fix is to disable the "magicness" of "$" when it appears in the filter string.

History

#1 Updated by Nathan Cutler about 1 year ago

  • Description updated (diff)

#2 Updated by Zack Cerza about 1 year ago

  • Assignee set to Warren Usui

#3 Updated by Yuri Weinstein about 1 year ago

  • Priority changed from Normal to Urgent

@Warren this seems pretty annoying pls take a look

#4 Updated by Warren Usui about 1 year ago

Yuri: I pushed a change in wip-wusui-35951. Could you test this to make sure that $ functionality still works? I think that it does but I could be wrong. Thanks.

Also available in: Atom PDF