600474 - Please run Talos suite for try [e-mail]-e23ad8a2d9dc

Status: RESOLVED FIXED

(Release Engineering :: General, defect, P2)

Description

[Justin Lebar (not reading bugmail)]

I'm not sure why, but try push [e-mail]-e23ad8a2d9dc does not appear to have resulted in Talos runs.

Unless Talos is currently running on this build (I don't see it on [1] or [2]), please trigger a set of Talos runs for this build.

[1] http://build.mozilla.org/builds/pending.html
[2] http://build.mozilla.org/builds/running.html

Comment 1

[Rail Aliiev [:rail]]

(In reply to comment #0)
> I'm not sure why, but try push [e-mail]-e23ad8a2d9dc does not appear
> to have resulted in Talos runs.

Try doesn't run Talos by default. You should explicitly ask for it. See https://wiki.mozilla.org/Build:TryServer#How_to_get_a_talos_run for the details.

I triggered one run for each talos test for fedora64. Feel free to reopen if you need more than one run.

Comment 2

[Justin Lebar (not reading bugmail)]

I pushed with

  try: -b do -p linux,linux64 -m none -u all -t all

in the topmost commit.  Is this not sufficient to get Talos?

Please trigger a set of Linux-32 runs as well.

Comment 3

[Rail Aliiev [:rail]]

(In reply to comment #2)
> I pushed with
> 
>   try: -b do -p linux,linux64 -m none -u all -t all
> 
> in the topmost commit.  Is this not sufficient to get Talos?

Should be. Lukas, any ideas?

> Please trigger a set of Linux-32 runs as well.

Done.

Comment 4

[Justin Lebar (not reading bugmail)]

Got 'em.  Thanks.

Comment 5

[Lukas Blakk [:lsblakk] use ?needinfo]

So this push had three commits in it:

[e-mail]
Tue Sep 28 14:01:01 2010 -0700	e23ad8a2d9dc	Justin Lebar — Bug 590181 - Use -O3 on Linux. r=ted try: -b do -p linux,linux64 -m none -u all -t all
	4d034699f6b0	Justin Lebar — Use gcc 4.5, take two. try: -b o -p linux,linux64 -u all -t none
	3163dac58eb9	Justin Lebar — try: -b do -p linux,linux64 -u all -m none -t none

And since the one with -t none got picked first (try_chooser goes through json looking for 'try:' and grabs first instance) this push got no talos.

Comment 6

[Justin Lebar (not reading bugmail)]

Ah, so it *doesn't* get try: from the topmost commit.  I'll update the docs.

Comment 7

[Lukas Blakk [:lsblakk] use ?needinfo]

only for now - see bug 600626 - maybe not worth updating docs only to change again