Review: Approve
+1
--
https://code.launchpad.net/~bjornt/maas/+git/maas/+merge/433441
Your team MAAS Committers is subscribed to branch maas:master.
--
Mailing list: https://launchpad.net/~sts-sponsors
Post to : sts-sponsors@lists.launchpad.net
Unsubscribe : https://launchpad.net/~sts-spon
The proposal to merge ~bjornt/maas:perftest-websocket-machine-list-all into
maas:master has been updated.
Status: Approved => Merged
For more details, see:
https://code.launchpad.net/~bjornt/maas/+git/maas/+merge/433442
--
Your team MAAS Committers is subscribed to branch maas:master.
--
The proposal to merge ~bjornt/maas:perftest-no-implicit-path into maas:master
has been updated.
Status: Needs review => Approved
For more details, see:
https://code.launchpad.net/~bjornt/maas/+git/maas/+merge/433441
--
Your team MAAS Committers is subscribed to branch maas:master.
--
Mai
LANDING
-b perftest-no-implicit-path lp:~bjornt/maas/+git/maas into -b master
lp:~maas-committers/maas
STATUS: FAILED BUILD
LOG: http://maas-ci.internal:8080/job/maas-tester/1462/consoleText
--
https://code.launchpad.net/~bjornt/maas/+git/maas/+merge/433441
Your team MAAS Committers is subscribe
The proposal to merge ~bjornt/maas:perftest-no-implicit-path into maas:master
has been updated.
Status: Approved => Needs review
For more details, see:
https://code.launchpad.net/~bjornt/maas/+git/maas/+merge/433441
--
Your team MAAS Committers is subscribed to branch maas:master.
--
Mai
Review: Needs Fixing
UNIT TESTS
-b tests-use-make-dir lp:~ack/maas/+git/maas into -b master
lp:~maas-committers/maas
STATUS: FAILED
LOG: http://maas-ci.internal:8080/job/maas-tester/1463/consoleText
COMMIT: 121ec0d7d123c1c455e7685ba168e75eae6eee82
--
https://code.launchpad.net/~ack/maas/+git/ma
The proposal to merge ~bjornt/maas:perftest-no-implicit-path into maas:master
has been updated.
Status: Needs review => Approved
For more details, see:
https://code.launchpad.net/~bjornt/maas/+git/maas/+merge/433441
--
Your team MAAS Committers is subscribed to branch maas:master.
--
Mai
LANDING
-b perftest-no-implicit-path lp:~bjornt/maas/+git/maas into -b master
lp:~maas-committers/maas
STATUS: FAILED BUILD
LOG: http://maas-ci.internal:8080/job/maas-tester/1464/consoleText
--
https://code.launchpad.net/~bjornt/maas/+git/maas/+merge/433441
Your team MAAS Committers is subscribe
The proposal to merge ~bjornt/maas:perftest-no-implicit-path into maas:master
has been updated.
Status: Approved => Needs review
For more details, see:
https://code.launchpad.net/~bjornt/maas/+git/maas/+merge/433441
--
Your team MAAS Committers is subscribed to branch maas:master.
--
Mai
The proposal to merge ~bjornt/maas:perftest-no-implicit-path into maas:master
has been updated.
Status: Needs review => Approved
For more details, see:
https://code.launchpad.net/~bjornt/maas/+git/maas/+merge/433441
--
Your team MAAS Committers is subscribed to branch maas:master.
--
Mai
jenkins: !test
--
https://code.launchpad.net/~troyanov/maas/+git/maas/+merge/433439
Your team MAAS Committers is subscribed to branch maas:master.
--
Mailing list: https://launchpad.net/~sts-sponsors
Post to : sts-sponsors@lists.launchpad.net
Unsubscribe : https://launchpad.net/~sts-sponsor
Review: Approve
lgtm, +1
--
https://code.launchpad.net/~troyanov/maas/+git/maas/+merge/433439
Your team MAAS Committers is subscribed to branch maas:master.
--
Mailing list: https://launchpad.net/~sts-sponsors
Post to : sts-sponsors@lists.launchpad.net
Unsubscribe : https://launchpad.net/~
The proposal to merge ~bjornt/maas:perftest-no-implicit-path into maas:master
has been updated.
Status: Approved => Merged
For more details, see:
https://code.launchpad.net/~bjornt/maas/+git/maas/+merge/433441
--
Your team MAAS Committers is subscribed to branch maas:master.
--
Mailing l
Björn Tillenius has proposed merging ~bjornt/maas:perf-test-dont-measure-setup
into maas:master.
Commit message:
Make it move explicit what the 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
Review: Needs Fixing
UNIT TESTS
-b vmhost-power-params lp:~troyanov/maas/+git/maas into -b master
lp:~maas-committers/maas
STATUS: FAILED
LOG: http://maas-ci.internal:8080/job/maas-tester/1466/consoleText
COMMIT: dc058f8c66b5298a2334ee36d0039061acbc6b93
--
https://code.launchpad.net/~troyanov/m
Review: Approve
UNIT TESTS
-b perf-test-dont-measure-setup lp:~bjornt/maas/+git/maas into -b master
lp:~maas-committers/maas
STATUS: SUCCESS
COMMIT: 96600367f4dfa6793f91b5628cb9a4aeb5e46dc2
--
https://code.launchpad.net/~bjornt/maas/+git/maas/+merge/433506
Your team MAAS Committers is subscribe
Review: Approve
+1
--
https://code.launchpad.net/~alexsander-souza/maas/+git/maas/+merge/433484
Your team MAAS Committers is subscribed to branch maas:master.
--
Mailing list: https://launchpad.net/~sts-sponsors
Post to : sts-sponsors@lists.launchpad.net
Unsubscribe : https://launchpad.net
The proposal to merge ~bjornt/maas:perf-test-no-custom-runner into maas:master
has been updated.
Status: Needs review => Work in progress
For more details, see:
https://code.launchpad.net/~bjornt/maas/+git/maas/+merge/433518
--
Your team MAAS Committers is subscribed to branch maas:master.
The proposal to merge ~bjornt/maas:perf-test-no-custom-runner into maas:master
has been updated.
Status: Work in progress => Superseded
For more details, see:
https://code.launchpad.net/~bjornt/maas/+git/maas/+merge/433518
--
Your team MAAS Committers is subscribed to branch maas:master.
Björn Tillenius has proposed merging ~bjornt/maas:perf-test-no-custom-runner
into maas:master.
Commit message:
Remove the custom bin/test.perf runner.
There's no reason to have a custom test runner. It's better to
use pytest directly, so that it's clear how to use it.
I also improved the way we
Björn Tillenius has proposed merging ~bjornt/maas:perf-test-no-custom-runner
into maas:master with ~bjornt/maas:perf-test-dont-measure-setup as a
prerequisite.
Commit message:
Remove the custom bin/test.perf runner.
There's no reason to have a custom test runner. It's better to
use pytest direc
Björn Tillenius has proposed merging ~bjornt/maas:perf-test-no-custom-decorator
into maas:master with ~bjornt/maas:perf-test-dont-measure-setup as a
prerequisite.
Commit message:
Requested reviews:
MAAS Maintainers (maas-maintainers)
For more details, see:
https://code.launchpad.net/~bjorn
The proposal to merge ~bjornt/maas:perf-test-no-custom-decorator into
maas:master has been updated.
Description changed to:
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 r
The proposal to merge ~bjornt/maas:perf-test-no-custom-decorator into
maas:master has been updated.
Status: Needs review => Superseded
For more details, see:
https://code.launchpad.net/~bjornt/maas/+git/maas/+merge/433520
--
Your team MAAS Committers is subscribed to branch maas:master.
-
The proposal to merge ~bjornt/maas:perf-test-dont-measure-setup into
maas:master has been updated.
Commit message changed to:
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,
Björn Tillenius has proposed merging ~bjornt/maas:perf-test-no-custom-decorator
into maas:master with ~bjornt/maas:perf-test-dont-measure-setup as a
prerequisite.
Commit message:
Remove the @perf_test decorator.
It's not needed, and it was somewhat broken. It reverted back to
a savepoint, unles
Review: Needs Fixing
UNIT TESTS
-b perf-test-no-custom-runner lp:~bjornt/maas/+git/maas into -b master
lp:~maas-committers/maas
STATUS: FAILED
LOG: http://maas-ci.internal:8080/job/maas-tester/1468/consoleText
COMMIT: 98bfd82808af8217788d3c928d8abca27ea833e3
--
https://code.launchpad.net/~bjorn
Review: Approve
UNIT TESTS
-b perf-test-no-custom-decorator lp:~bjornt/maas/+git/maas into -b master
lp:~maas-committers/maas
STATUS: SUCCESS
COMMIT: 3334e5ea13472b9b4cf35750598efddac43d123a
--
https://code.launchpad.net/~bjornt/maas/+git/maas/+merge/433521
Your team MAAS Committers is subscrib
The proposal to merge ~alexsander-souza/maas:lp1995624_suppress_script_results
into maas:master has been updated.
Status: Needs review => Approved
For more details, see:
https://code.launchpad.net/~alexsander-souza/maas/+git/maas/+merge/433484
--
Your team MAAS Committers is subscribed to b
The proposal to merge ~alexsander-souza/maas:lp1995624_suppress_script_results
into maas:master has been updated.
Status: Approved => Merged
For more details, see:
https://code.launchpad.net/~alexsander-souza/maas/+git/maas/+merge/433484
--
Your team MAAS Committers is subscribed to branch
Alberto Donato has proposed merging ~ack/maas:region-refresh-no-retry into
maas:master.
Commit message:
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 con
Review: Approve
+1
--
https://code.launchpad.net/~ack/maas/+git/maas/+merge/433526
Your team MAAS Committers is subscribed to branch maas:master.
--
Mailing list: https://launchpad.net/~sts-sponsors
Post to : sts-sponsors@lists.launchpad.net
Unsubscribe : https://launchpad.net/~sts-sponsor
Review: Needs Fixing
UNIT TESTS
-b region-refresh-no-retry lp:~ack/maas/+git/maas into -b master
lp:~maas-committers/maas
STATUS: FAILED
LOG: http://maas-ci.internal:8080/job/maas-tester/1471/consoleText
COMMIT: c52c6b192d2d9305d2874389383b1970afcae784
--
https://code.launchpad.net/~ack/maas/+g
The proposal to merge ~ack/maas:region-refresh-no-retry into maas:master has
been updated.
Status: Needs review => Approved
For more details, see:
https://code.launchpad.net/~ack/maas/+git/maas/+merge/433526
--
Your team MAAS Committers is subscribed to branch maas:master.
--
Mailing lis
Review: Approve
UNIT TESTS
-b perf-test-no-custom-runner lp:~bjornt/maas/+git/maas into -b master
lp:~maas-committers/maas
STATUS: SUCCESS
COMMIT: ef1801262983fbe58792cee0b6ca7b2998ecd632
--
https://code.launchpad.net/~bjornt/maas/+git/maas/+merge/433519
Your team MAAS Committers is subscribed
The proposal to merge ~bjornt/maas:perf-test-no-custom-runner into maas:master
has been updated.
Status: Needs review => Work in progress
For more details, see:
https://code.launchpad.net/~bjornt/maas/+git/maas/+merge/433519
--
Your team MAAS Committers is subscribed to branch maas:master.
The proposal to merge ~ack/maas:region-refresh-no-retry into maas:master has
been updated.
Status: Approved => Merged
For more details, see:
https://code.launchpad.net/~ack/maas/+git/maas/+merge/433526
--
Your team MAAS Committers is subscribed to branch maas:master.
--
Mailing list: htt
Review: Approve
+1
--
https://code.launchpad.net/~bjornt/maas/+git/maas/+merge/433521
Your team MAAS Committers is subscribed to branch maas:master.
--
Mailing list: https://launchpad.net/~sts-sponsors
Post to : sts-sponsors@lists.launchpad.net
Unsubscribe : https://launchpad.net/~sts-spon
Björn Tillenius has proposed merging ~bjornt/maas:perf-test-no-custom-runner
into maas:master with ~bjornt/maas:perf-test-dont-measure-setup as a
prerequisite.
Commit message:
Remove the custom bin/test.perf runner.
There's no reason to have a custom test runner. It's better to
use pytest direc
Review: Approve
+1 with a couple nits
Diff comments:
> diff --git a/Makefile b/Makefile
> index 6ddf017..6f826b7 100644
> --- a/Makefile
> +++ b/Makefile
> @@ -62,7 +61,8 @@ build: \
>.run \
>$(VENV) \
>$(BIN_SCRIPTS) \
> - bin/py
> + bin/py \
> + bin/test.perf
I would drop this
jenkins: !test
--
https://code.launchpad.net/~troyanov/maas/+git/maas/+merge/433439
Your team MAAS Committers is subscribed to branch maas:master.
--
Mailing list: https://launchpad.net/~sts-sponsors
Post to : sts-sponsors@lists.launchpad.net
Unsubscribe : https://launchpad.net/~sts-sponsor
Review: Approve
UNIT TESTS
-b vmhost-power-params lp:~troyanov/maas/+git/maas into -b master
lp:~maas-committers/maas
STATUS: SUCCESS
COMMIT: dc058f8c66b5298a2334ee36d0039061acbc6b93
--
https://code.launchpad.net/~troyanov/maas/+git/maas/+merge/433439
Your team MAAS Committers is subscribed to
The proposal to merge ~troyanov/maas:vmhost-power-params into maas:master has
been updated.
Status: Needs review => Approved
For more details, see:
https://code.launchpad.net/~troyanov/maas/+git/maas/+merge/433439
--
Your team MAAS Committers is subscribed to branch maas:master.
--
Maili
The proposal to merge ~troyanov/maas:vmhost-power-params into maas:master has
been updated.
Status: Approved => Merged
For more details, see:
https://code.launchpad.net/~troyanov/maas/+git/maas/+merge/433439
--
Your team MAAS Committers is subscribed to branch maas:master.
--
Mailing lis
Diego Mascialino has proposed merging
~dmascialino/maas-ci/+git/system-tests:write_version_under_test into
~maas-committers/maas-ci/+git/system-tests:master.
Commit message:
Write version_under_test file
Requested reviews:
MAAS Committers (maas-committers)
For more details, see:
https://code
Review: Approve
LGTM
--
https://code.launchpad.net/~dmascialino/maas-ci/+git/system-tests/+merge/433550
Your team MAAS Committers is subscribed to branch
~maas-committers/maas-ci/+git/system-tests:master.
--
Mailing list: https://launchpad.net/~sts-sponsors
Post to : sts-sponsors@lists.la
Review: Approve
UNIT TESTS
-b write_version_under_test lp:~dmascialino/maas-ci/+git/system-tests into -b
master lp:~maas-committers/maas-ci/+git/system-tests
STATUS: SUCCESS
COMMIT: f919a573a8a443203762cc325ae993b6eb6c864e
--
https://code.launchpad.net/~dmascialino/maas-ci/+git/system-tests/+me
The proposal to merge
~dmascialino/maas-ci/+git/system-tests:write_version_under_test into
~maas-committers/maas-ci/+git/system-tests:master has been updated.
Status: Needs review => Approved
For more details, see:
https://code.launchpad.net/~dmascialino/maas-ci/+git/system-tests/+merge/433
The proposal to merge
~dmascialino/maas-ci/+git/system-tests:write_version_under_test into
~maas-committers/maas-ci/+git/system-tests:master has been updated.
Status: Approved => Merged
For more details, see:
https://code.launchpad.net/~dmascialino/maas-ci/+git/system-tests/+merge/433550
--
49 matches
Mail list logo