488262 - Tryserver, linux: fix nits to get same "config" on all slaves

Status: RESOLVED WONTFIX

(Release Engineering :: General, defect)

Description

[Serge Gautherie (:sgautherie)]

I compared logs for a build I did,
and noticed some nits between
sm-try-linux-slave04.mozilla.org <-> try-linux-slave03.mozilla.org:

*/builds/slave/sendchange-linux-unittest/... <-> /builds/buildbot/sendchange-slave/sendchange-linux-unittest/...

nthomas said "don't care".
But I do care because I need to replace the string in the whole logs to get a readable diff.

*the hostname 'sm-' <-> does not start with this

nthomas said "don't care".
I'd say I agree, though I wonder why not have the same names/pattern ?

*ends steps with 'elapsedTime=...' <-> does not have this line

nthomas said "buildbot 0.7.7 vs 0.7.9, apparently".
Probably better to use the same buildbot everywhere.

Comment 1

[[e-mail] (:bhearsum)]

(In reply to comment #0)
> I compared logs for a build I did,
> and noticed some nits between
> sm-try-linux-slave04.mozilla.org <-> try-linux-slave03.mozilla.org:
> 
> */builds/slave/sendchange-linux-unittest/... <->
> /builds/buildbot/sendchange-slave/sendchange-linux-unittest/...
> 
> nthomas said "don't care".
> But I do care because I need to replace the string in the whole logs to get a
> readable diff.
> 

I second nthomas' statement. Use a regex to make your replacement work:
|/builds/.*/.*| matches both in my testing. Replace the last .* with the dir name (which is consistent across a given builder+platform combination) if you need to.

> *the hostname 'sm-' <-> does not start with this
> 
> nthomas said "don't care".
> I'd say I agree, though I wonder why not have the same names/pattern ?
> 

Again, seconded.

> *ends steps with 'elapsedTime=...' <-> does not have this line
> 
> nthomas said "buildbot 0.7.7 vs 0.7.9, apparently".
> Probably better to use the same buildbot everywhere.

This will be taken care of as part of bug 481886.

The rest of this bug is WONTFIX.

Comment 2

[Serge Gautherie (:sgautherie)]

(In reply to comment #1)
> I second nthomas' statement. Use a regex to make your replacement work:
> |/builds/.*/.*| matches both in my testing. Replace the last .* with the dir
> name (which is consistent across a given builder+platform combination) if you
> need to.

Actually, a straight replacement did the job, even without a regex.
But my point was not to have to do this!

In my example, I needed this to compare |make check| logs.

Comment 3

[[e-mail] (:bhearsum)]

This bug has nothing to do with the 0.7.10p1 upgrade.

Comment 4

[Serge Gautherie (:sgautherie)]

(In reply to comment #1)
> This will be taken care of as part of bug 481886.

(In reply to comment #3)
> This bug has nothing to do with the 0.7.10p1 upgrade.

Doesn't the two sentences say opposite things ?

Comment 5

[[e-mail] (:bhearsum)]

(In reply to comment #4)
> (In reply to comment #1)
> > This will be taken care of as part of bug 481886.
> 
> (In reply to comment #3)
> > This bug has nothing to do with the 0.7.10p1 upgrade.
> 
> Doesn't the two sentences say opposite things ?

That says it's a dupe. This bug is WONTFIX because of the parts that we're not going to do anything about. Feel free to dupe it if you think that's more clear...