~bjornt/maas:perf-test-no-custom-decorator

Branch merges

Propose for merging

Merged into maas:master

Alberto Donato (community): Approve on 2022-11-23

MAAS Lander: Approve on 2022-11-23

Diff: 182 lines (+18/-46)

5 files modified

src/maasperf/tests/cli/test_machines.py (+2/-2)
src/maasperf/tests/maasserver/api/test_machines.py (+4/-4)
src/maasperf/tests/maasserver/models/test_machine.py (+8/-5)
src/maasperf/tests/maasserver/websockets/test_machines.py (+4/-3)
src/maastesting/perftest.py (+0/-32)

api

Superseded for merging into maas:master

MAAS Maintainers: Pending requested 2022-11-23

Diff: 182 lines (+18/-46)

5 files modified

src/maasperf/tests/cli/test_machines.py (+2/-2)
src/maasperf/tests/maasserver/api/test_machines.py (+4/-4)
src/maasperf/tests/maasserver/models/test_machine.py (+8/-5)
src/maasperf/tests/maasserver/websockets/test_machines.py (+4/-3)
src/maastesting/perftest.py (+0/-32)

api

Related source package recipes

No recipes using this branch. (?)

Create packaging recipe

Related snap packages

Related charm recipes

15b5b60... by Björn Tillenius on 2022-11-23

Remove the @perf_test decorator.

It's not needed, and it was somewhat broken. It reverted back to
a savepoint, unless commit_transaction was true. In that case
it simply removed the savepoint. I'm not quite sure what the intent
was there. If it was to simulate a commit, it was in the wrong
place, since it was outside the block that measures the duration.

The transaction is rolled back elsewhere for each tests, so no
need for a custom decorator. The decorator also had logic for
skipping the tests if django wasn't loaded, but I don't see
a use case for that.

5003b42... by Björn Tillenius on 2022-11-24

Make it more explicit what to measure in the performance tests

Before, we measured how long the whole test took to run. That includes
any local setup you do in the test, but also the setup of any fixtures
the test uses.

I also combined measuring the time and, optionally, do profiling
using cProfile. It's more logical that you want to profile the
thing that you are actually measuring.

30a43bf... by Anton Troyanov on 2022-11-23

fix(vault): vmhost should use get_power_parameters

02524b2... by Alberto Donato on 2022-11-23

disable request retries for controller script results reporting to region

The logic used for the refresh is the same used by scrips reporting for
machines, but there's no need to retry for controllers refresh, since the
service runs every 30s anyway

963f588... by Peter Makowski on 2022-11-23

Update maas-ui to 44625e63e
integrate suppress_failed_script_results action (#4578)

4941c41... by Nick De Villiers on 2022-11-23

Update maas-ui to 2681e0951
fix(vault): show icon in controller listing if all controllers are configured but secrets are not migrated (#4579)

96b919d... by Alexsander de Souza on 2022-11-23

add suppress_failed_script_results() to machine WS

fixes LP#1995624

560cdd4... by Björn Tillenius on 2022-11-23

No implicit, hardcoded, path for bin/test.perf

bin/test.perf always passed src/maasperf to pyttest. This made it
harder to run only a subset of tests. For example by supplying the
test file to run.

CI uses utilities/run-perf-tests-ci, which already explicitly passes
src/maasperf as the path.

241f3e1... by Björn Tillenius on 2022-11-23

Add a performance test for listing all machines with the websocket.

With this it's easier to track what the default page size should be.

03eec39... by "renovate[bot]" <29139614+renovate[bot]@users.noreply.github.com> on 2022-11-23

Update maas-ui to 0d44ada26
chore(deps): update dependency prettier to v2.8.0 (#4577)

Co-authored-by: renovate[bot] <29139614+renovate[bot]@users.noreply.github.com>