Some tests are skipped due to the "perf" filtering
Bug #1977820 reported by
Alberto Donato
This bug affects 1 person
Affects | Status | Importance | Assigned to | Milestone | |
---|---|---|---|---|---|
MAAS |
Fix Released
|
Medium
|
Alberto Donato | ||
3.2 |
Fix Released
|
Medium
|
Alberto Donato |
Bug Description
The way we're currently filtering performance tests out with `--exclude=perf` in the runner command line is causing more tests than intended to be skipped.
AFAICT the exclude is matched against the full test name, so a test which contains the "perf" string in the name is skipped.
There are a few tests that contains strings like "performs" or "performed" in the name which are also skipped.
I think we should move the performance tests to a separate top-level package (possibly with different subpackages to match code structure) so that we can just skip/select the package in each runner.
Related branches
~cgrabowski/maas:backport-perftests-split-to-3.2
- MAAS Lander: Approve
- Alexsander de Souza: Approve
-
Diff: 258 lines (+44/-49)8 files modifiedMakefile (+2/-2)
dev/null (+0/-9)
src/maasperf/tests/conftest.py (+41/-0)
src/maasperf/tests/maasserver/models/__init__.py (+0/-0)
src/maasperf/tests/maasserver/websockets/__init__.py (+0/-0)
src/maasserver/conftest.py (+0/-30)
src/maastesting/perftest.py (+1/-2)
src/maastesting/scripts.py (+0/-6)
~ack/maas:perftests-split
Merged
into
maas:master
- MAAS Lander: Approve
- Christian Grabowski: Approve
-
Diff: 258 lines (+44/-49)8 files modifiedMakefile (+2/-2)
dev/null (+0/-9)
src/maasperf/tests/conftest.py (+41/-0)
src/maasperf/tests/maasserver/models/__init__.py (+0/-0)
src/maasperf/tests/maasserver/websockets/__init__.py (+0/-0)
src/maasserver/conftest.py (+0/-30)
src/maastesting/perftest.py (+1/-2)
src/maastesting/scripts.py (+0/-6)
Changed in maas: | |
status: | Triaged → In Progress |
assignee: | nobody → Alberto Donato (ack) |
Changed in maas: | |
status: | In Progress → Fix Committed |
Changed in maas: | |
milestone: | 3.3.0 → 3.3.0-beta1 |
Changed in maas: | |
status: | Fix Committed → Fix Released |
To post a comment you must log in.