Merge lp:~vila/bzr/837926-log-make-check into lp:bzr/2.0

Proposed by Vincent Ladeuil
Status: Rejected
Rejected by: Vincent Ladeuil
Proposed branch: lp:~vila/bzr/837926-log-make-check
Merge into: lp:bzr/2.0
Diff against target: 30 lines (+5/-1)
2 files modified
Makefile (+3/-0)
NEWS (+2/-1)
To merge this branch: bzr merge lp:~vila/bzr/837926-log-make-check
Reviewer Review Type Date Requested Status
Martin Pool Approve
Review via email: mp+73492@code.launchpad.net

Description of the change

I'd like to have more precise timings of our landings on pqm.

I've asked our losas for the log files and learned they were
already mirrored on chinstrap.

But their content still don't give us enough data: at best we get
the end date which is only useful if several jobs have been
queued.

This patch add some echo statements redirected to stderr so they
end up in the log file.

The plan is to apply this patch to all our branches there (2.0 ->
trunk) with the nedeed tweaks (the makefile have evolved for
almost all series so merging up will conflict).

To post a comment you must log in.
Revision history for this message
Martin Pool (mbp) wrote :

+ echo `date` selftest [ascii] starts 1>&2

for portability across shells, i think you should quote the string (specifically the square brackets.)

if that's enough to help with this, it's ok with me.

i would think about changing it to run ./bzr underneath 'time'.

review: Approve
Revision history for this message
Vincent Ladeuil (vila) wrote :
Revision history for this message
John A Meinel (jameinel) wrote :

-----BEGIN PGP SIGNED MESSAGE-----
Hash: SHA1

On 8/31/2011 11:09 AM, Vincent Ladeuil wrote:
> Vincent Ladeuil has proposed merging
> lp:~vila/bzr/837926-log-make-check into lp:bzr/2.0.
>
> Requested reviews: bzr-core (bzr-core)
>
> For more details, see:
> https://code.launchpad.net/~vila/bzr/837926-log-make-check/+merge/73492
>
>
>
I'd like to have more precise timings of our landings on pqm.
>
> I've asked our losas for the log files and learned they were
> already mirrored on chinstrap.
>
> But their content still don't give us enough data: at best we get
> the end date which is only useful if several jobs have been
> queued.
>
> This patch add some echo statements redirected to stderr so they
> end up in the log file.
>
> The plan is to apply this patch to all our branches there (2.0 ->
> trunk) with the nedeed tweaks (the makefile have evolved for
> almost all series so merging up will conflict).

So we don't keep the selftest output itself? (when using --subunit
we record the absolute times as it runs.) Which means failing tests
give you pretty accurate timing, which is one of the reasons I'm doing
it. (That and because it allows me to run things without adding them
to a real branch.)

John
=:->

-----BEGIN PGP SIGNATURE-----
Version: GnuPG v1.4.9 (Cygwin)
Comment: Using GnuPG with Mozilla - http://enigmail.mozdev.org/

iEYEARECAAYFAk5eBGsACgkQJdeBCYSNAANG9ACglHQcxV5ageGNSU9erSr564ie
hx8AnRfdSD79b4oBsTDP0DzUAlpp5+XE
=Z/Wa
-----END PGP SIGNATURE-----

Revision history for this message
Vincent Ladeuil (vila) wrote :

>>>>> John Arbash Meinel <email address hidden> writes:

<snip/>

    > So we don't keep the selftest output itself?

This doesn't interfere with the existing behavior to avoid fallouts.

    > (when using --subunit we record the absolute times as it runs.)
    > Which means failing tests give you pretty accurate timing, which
    > is one of the reasons I'm doing it. (That and because it allows me
    > to run things without adding them to a real branch.)

Right, I want to keep that behavior.

But we don't get the subunit output at all for successful landings nor
do we get subunit outputs for others submissions. So I went a different
route.

Unmerged revisions

4771. By Vincent Ladeuil

Output time stamps while running ``make check`` to get better timings from pqm.

Preview Diff

[H/L] Next/Prev Comment, [J/K] Next/Prev File, [N/P] Next/Prev Hunk
1=== modified file 'Makefile'
2--- Makefile 2009-11-18 04:29:56 +0000
3+++ Makefile 2011-08-31 09:09:17 +0000
4@@ -39,9 +39,12 @@
5 check: docs check-nodocs
6
7 check-nodocs: extensions
8+ echo `date` selftest starts 1>&2
9 $(PYTHON) -Werror -O ./bzr selftest -1v $(tests)
10 @echo "Running all tests with no locale."
11+ echo `date` selftest [ascii] starts 1>&2
12 LC_CTYPE= LANG=C LC_ALL= ./bzr selftest -1v $(tests) 2>&1 | sed -e 's/^/[ascii] /'
13+ echo `date` selftest ends 1>&2
14
15 # Run Python style checker (apt-get install pyflakes)
16 #
17
18=== modified file 'NEWS'
19--- NEWS 2011-08-19 11:14:17 +0000
20+++ NEWS 2011-08-31 09:09:17 +0000
21@@ -50,7 +50,8 @@
22 Testing
23 *******
24
25-
26+* Output time stamps while running ``make check`` to get better timings from
27+ pqm. (Vincent Ladeuil, #837926)
28
29 bzr 2.0.6
30 #########

Subscribers

People subscribed via source and target branches