536188 - Create a project branch for Firefox "Lorentz"

Status: RESOLVED FIXED

(Release Engineering :: General, defect)

Description

[Benjamin Smedberg]

As discussed at the workweek and documented at https://wiki.mozilla.org/Talk:Firefox/Roadmap we would like a new project branch for landing and beta testing changes for Firefox "Lorentz", the stable release which is tentatively scheduled for Q1 2010.

This will branch from mozilla-1.9.2 and will contain fixes including out-of-process plugins and other features. We will want to do weekly betas from this branch starting the middle of January until release, which will, fingers crossed, occur in late March.

Project branch questionnaire:
* Do you want builds? Yes
** Which OS? All the OSes associated with 1.9.2, including WinMo and Maemo
** Incremental build-on-checkin: yes
** Nightlies: yes, *with nightly updates*... specific details need to be worked out, but bhearsum/nthomas said this might be possible if we called this Firefox 3.7a2 and bumped trunk to Firefox 4. The current plan is *not* to call this Firefox 3.7, though, so we'll need to figure something out.
** Are en-US builds enough: eventually, probably not, but for the first round yes. I'll file a followup to discuss how we need to deal with l10n.

* Do you want unittests: yes, to match the existing Firefox 3.6/1.9.2 tree.
* Do you want Talos: yes, to match the existing Firefox 3.6/1.9.2 tree.
* Name of branch owner: bsmedberg
** will do periodic merges from 1.9.2

* Timeline: start ASAP, expect the branch to be finished before April
** "Finish" means the branch will merge into 1.9.2

* Misc
** Any toolchain changes: we may need to use GCC 4.3 instead of GCC 4.1 on Linux. This is a simple mozconfig change which I can make at a later date. No other changes are expected.
** Tinderbox waterfall name: tinderbox.mozilla.org/Firefox-Lorentz
** Where to put builds on ftp.m.o: Firefox/latest-lorentz
** hg branch name: projects/firefox-lorentz
** unofficial project name: Lorentz

Comment 1

[Armen [:armenzg]]

Assigning to John for prioritization.

Comment 2

[John O'Duinn [:joduinn] (please use "needinfo?" flag)]

(In reply to comment #2)
> ** Nightlies: yes, *with nightly updates*... specific details need to be worked
> out, but bhearsum/nthomas said this might be possible if we called this Firefox
> 3.7a2 and bumped trunk to Firefox 4. The current plan is *not* to call this
> Firefox 3.7, though, so we'll need to figure something out.
Lets hold off on this for now. If possible, I dont want to do anything here which complicates future real releases. Nightly updates on project branches is a project in itself, not yet possible for any project branch. That work is being tracked in bug#534954. 

> ** Are en-US builds enough: eventually, probably not, but for the first round
> yes. I'll file a followup to discuss how we need to deal with l10n.
ok, so for now, no l10n.

> * Misc
> ** Any toolchain changes: we may need to use GCC 4.3 instead of GCC 4.1 on
> Linux. This is a simple mozconfig change which I can make at a later date. No
> other changes are expected.
aiui, nothing to do for now, but thanks for the headsup.

Comment 3

[Lukas Blakk [:lsblakk] use ?needinfo]

Attachment: Add "lorentz" to staging configs

Comment 4

[Lukas Blakk [:lsblakk] use ?needinfo]

Attachment: Add "lorentz" to production configs

Comment 5

[Lukas Blakk [:lsblakk] use ?needinfo]

We need the repo set up and the Firefox-Lorentz tinderbox page still. If no one has filed bugs for these, I can.

Comment 6

[Lukas Blakk [:lsblakk] use ?needinfo]

Tinderbox page created: http://tinderbox.mozilla.org/showbuilds.cgi?tree=Firefox-Lorentz

Comment 7

[[e-mail] (:bhearsum)]

A couple questions:
* Do we need l10n? (eg, are we taking string changes on this branch?)
* What's the state of nightly updates? Are we keeping them off to start with?

Comment 8

[[e-mail] (:bhearsum)]

Comment on attachment 420657 [details] [diff] [review]
Add "lorentz" to production configs

>+BRANCHES['lorentz']['repo_path'] = 'projects/firefox-lorentz'
>+BRANCHES['lorentz']['major_version'] = '1.9.2'

I don't think this is right - and it'll confuse 1.9.2 updates if set like this. Benjamin, do we know what the Gecko version is going to be yet?

>+BRANCHES['lorentz']['tinderbox_tree'] = 'Firefox-Lorentz'
>+BRANCHES['lorentz']['packaged_unittest_tinderbox_tree'] = 'Firefox-Lorentz'

This is staging, so I think you want MozillaTest

>+# Enable XULRunner / SDK builds
>+BRANCHES['lorentz']['enable_xulrunner'] = False

AIUI we're going to ship things from this branch, so let's flip it on.

>+BRANCHES['lorentz']['enable_l10n'] = False
>+BRANCHES['lorentz']['l10nNightlyUpdate'] = False 
>+BRANCHES['lorentz']['l10nDatedDirs'] = False 

Dunno if we're accepting string changes on this branch, but if we are we'll need l10n. Waiting on a response from Benjamin about this.

>+BRANCHES['lorentz']['aus2_base_upload_dir'] = 'fake'

We'll need something real for this eventually, but probably not today.


>diff --git a/mozilla2/mobile_config.py b/mozilla2/mobile_config.py

I'll look at this file, but I'd like Aki's review on it too.



>+### mobile-lorentz
>+MOBILE_BRANCHES['mobile-lorentz']['main_config'] = config.BRANCHES['lorentz']
>+MOBILE_BRANCHES['mobile-lorentz']['repo_path'] = 'projects/lorentz'

I think this should be firefox-lorentz.

>+MOBILE_BRANCHES['mobile-lorentz']['l10n_repo_path'] = 'l10n-central'

I don't think l10n-central is right - maybe l10n-mozilla-1.9.2?

>+MOBILE_BRANCHES['mobile-lorentz']['mobile_repo_path'] = 'mobile-browser'

Are we branching mobile-browser for this release, too? If we're not, I don't think we need another mobile branch at all!



Let's wait for Aki's review, and Benjamin to answer the followup questions I have, and give these another go.

Comment 9

[Benjamin Smedberg]

The current plan is for the Gecko and Firefox version to be exactly the same as 1.9.2, because eventually this will just be 1.9.2.x/3.6.x. That plan is subject to change based on releng needs... we could certainly use a hacked-up version string on the branch such as 1.9.2.1plugins1 and 3.6.1plugins1 if that somehow makes releasing things easier during the branch lifetime.

There will be l10n changes on this branch, but it's not clear to me how that will work yet... we definitely don't want another set of l10n repositories, and the current l10n repositories may not gain the new strings for a bit yet. In any case, l10n-mozilla-1.9.2 is correct, not l10n-central.

Definitely not branching mobile-browser.

Comment 10

[[e-mail] (:bhearsum)]

(In reply to comment #9)
> The current plan is for the Gecko and Firefox version to be exactly the same as
> 1.9.2, because eventually this will just be 1.9.2.x/3.6.x. That plan is subject
> to change based on releng needs... we could certainly use a hacked-up version
> string on the branch such as 1.9.2.1plugins1 and 3.6.1plugins1 if that somehow
> makes releasing things easier during the branch lifetime.

Okay. I haven't been able to dive into this enough to know what the best route is, so I'm going to have to let someone else (nthomas?) weigh in on that.

> There will be l10n changes on this branch, but it's not clear to me how that
> will work yet... we definitely don't want another set of l10n repositories, and
> the current l10n repositories may not gain the new strings for a bit yet. In
> any case, l10n-mozilla-1.9.2 is correct, not l10n-central.

Complicated....if we can guarantee we won't change any strings on the 3.6 builds that ship while Lorentz is branched we could just use the existing ones without issue. Dunno if that's possible though. (Also, if we do that we should turn off 1.9.2 nightly l10n and move it to the lorentz branch builders, since we don't need two sets of builds from those repos every night.

Comment 11

[Aki Sasaki (not active)]

Comment on attachment 420657 [details] [diff] [review]
Add "lorentz" to production configs

Looks good.

Nit: trailing spaces at EOL after start_hour and start_minute.

Looks like these exist on other branches too... copy&paste at work.

Not important, but they sometimes bug me for no real good reason.

Comment 12

[tymerkaev]

Maybe not latest-lorentz but latest-1.9.3?

Comment 13

[Benjamin Smedberg]

lorentz is not 1.9.3, please don't confuse things.

Comment 14

[tymerkaev]

Why not 1.9.3? Current 3.7 nightly builds use 1.9.3a1pre platform

Comment 15

[Dave Garrett]

Azat, please read the bug and associated materials before commenting. (there are half a dozen "1.9.2"s in comment 0) The whole point of this thing is to port some features to 1.9.2 (roadmap is currently aiming for 1.9.2.2), namely the miracle that will be OOPP. 1.9.3 is a whole other thing.

Comment 16

[Nick Thomas [:nthomas] (UTC+12)]

We should put a tag in mozilla-1.9.2 at the clone point. How about FIREFOX-LORENTZ_BASE ?

Comment 17

[Benjamin Smedberg]

Why? We're going to merge regularly, so the base tag is probably not helpful.

Comment 18

[Nick Thomas [:nthomas] (UTC+12)]

I guess that was a knee-jerk response based on what we've done before, and with regular merging and a planned short lifetime it does seem less useful. We can always add it after the fact if necessary (on revision 05a4cc93f14b).

Comment 19

[Lukas Blakk [:lsblakk] use ?needinfo]

Attachment: Add "lorentz" to production configs

Took out mobile since we're not branching that.  Turned the l10n on, took out extra space nit on start_hour and start_minute.

Comment 20

[Axel Hecht [:Pike]]

IMHO we should change
BRANCHES['lorentz']['l10n_tree'] = 'fx36x'
to be something distinct. 'lorentz' would work for me, but we'll have to copy the fx36x section over.

Is lorentz running on the same master as fx36x? Didn't check. gotta make sure that we're picking the right l10nbuilds.ini.

Comment 21

[Lukas Blakk [:lsblakk] use ?needinfo]

it can be run on the same master as fx36x

Comment 22

[[e-mail] (:bhearsum)]

(In reply to comment #21)
> it can be run on the same master as fx36x

I don't think we should - pm01 is already crapping out from time to time. I don't think Axel is requesting that we run it in the same place, ftr, just making sure that the correct l10nbuilds.ini file is modified.

Comment 23

[[e-mail] (:bhearsum)]

(In reply to comment #20)
> IMHO we should change
> BRANCHES['lorentz']['l10n_tree'] = 'fx36x'
> to be something distinct. 'lorentz' would work for me, but we'll have to copy
> the fx36x section over.

Do we know what the story is for l10n on Lorentz yet? As mentioned in comment #10, if we're using the existing 1.9.2 l10n repos for Lorentz we turn off the mozilla-1.9.2 branch l10n.

Comment 24

[[e-mail] (:bhearsum)]

Comment on attachment 421361 [details] [diff] [review]
Add "lorentz" to production configs

This looks fine, but we still need to figure out the l10n story. Do you have a staging patch for this, too?

Comment 25

[Benjamin Smedberg]

If we have l10n builds, we'll be using the existing http://hg.mozilla.org/releases/l10n-mozilla-1.9.2. We will definitely not be branching the l10n repositories.

Comment 26

[[e-mail] (:bhearsum)]

(In reply to comment #25)
> If we have l10n builds, we'll be using the existing
> http://hg.mozilla.org/releases/l10n-mozilla-1.9.2. We will definitely not be
> branching the l10n repositories.

So, these repositories will now (or soon) track Lorentz, rather than Firefox 3.6?

Comment 27

[Axel Hecht [:Pike]]

I don't agree with comment 10. We'll need l10n builds, including nightlies, to test the new strings. The basic idea is to require the strings for lorentz builds and expose them on the dashboard as such, but not require them for 1.9.2, and build against the 1.9.2 l10n repos. The Lorentz gecko code will have fallback strings to handle the case of not-updated localizations and language packs.

Might be best to track this in a follow-up bug, though.

Comment 28

[Axel Hecht [:Pike]]

(In reply to comment #26)
> So, these repositories will now (or soon) track Lorentz, rather than Firefox
> 3.6?

Both. Lorentz is Firefox 3.6.x, after all.

Comment 29

[[e-mail] (:bhearsum)]

Comment on attachment 421361 [details] [diff] [review]
Add "lorentz" to production configs

I just talked through the l10n bits with Axel.

This is obvious to some, but just so it doesn't get lost:
We're going to be sharing the l10n-mozilla-1.9.2 repositories between firefox-lorentz and mozilla-1.9.2 for now. Lorentz is only making additive string changes, so those will not interfere with the existing mozilla-1.9.2 builds.

So, this patch is basically done, except you need to change 'l10n_tree' to be 'lorentz'.

Still need the buildbot-configs patch.

Comment 31

[Lukas Blakk [:lsblakk] use ?needinfo]

Attachment: Add "lorentz" to production configs

this one has the mozconfigs in it (sorry I had forgotten to do hg add on the last patch) as well as the l10n_tree being "lorentz".

Comment 32

[Axel Hecht [:Pike]]

Comment on attachment 422376 [details] [diff] [review]
Add "lorentz" to production configs

Didn't do a full review, but this lacks the change to l10nbuilds.ini, which of the two depends on the master, I guess. Not sure if this is actually hooked up to a master yet.

Comment 33

[Lukas Blakk [:lsblakk] use ?needinfo]

Attachment: Add "lorentz" to production configs

just added lorentz to master2.cfg for pm02

Comment 34

[Lukas Blakk [:lsblakk] use ?needinfo]

Attachment: Add "lorentz" to production configs

now with l10nbuilds2.ini changes as well.

Comment 35

[Lukas Blakk [:lsblakk] use ?needinfo]

Attachment: Add "lorentz" to production configs

put the l10n and l10nbase back to 1.9.2 in l10nbuilds2.ini - my mistake there in attempting to create a new lorentz l10n repo.

Comment 36

[Nick Thomas [:nthomas] (UTC+12)]

Comment on attachment 422463 [details] [diff] [review]
Add "lorentz" to production configs

I'm confused that there is no staging config here, what testing is being done on these changes ?

>diff --git a/mozilla2/config.py b/mozilla2/config.py

>+BRANCHES['lorentz']['major_version'] = '1.9.2'
>+BRANCHES['lorentz']['brand_name'] = 'Lorentz'

Random aside: these two don't even seem to be used anymore, which is good because we have m-c claiming to be 1.9.2 elsewhere. Lets ignore this.

>+BRANCHES['lorentz']['create_snippet'] = False
...
>+BRANCHES['lorentz']['l10nNightlyUpdate'] = True 
>+BRANCHES['lorentz']['l10nDatedDirs'] = True 

No point disabling en-US updates but enabling l10n ones, so flip the two l10n flags to False. We'll revisit this after we have en-US nightlies updating.

>+BRANCHES['lorentz']['platforms']['linux']['env'] = {
>+BRANCHES['lorentz']['platforms']['win32']['env'] = {
>+BRANCHES['lorentz']['platforms']['macosx']['env'] = {
These three env blocks need this added:
    'MOZ_SYMBOLS_EXTRA_BUILDID': 'lorentz',
to keep the nightly/release symbols separate from 1.9.2.

>+BRANCHES['lorentz']['platforms']['linux64']['env'] = {
Similarly this one should use
    'MOZ_SYMBOLS_EXTRA_BUILDID': 'linux64-lorentz',

>diff --git a/mozilla2/l10nbuilds2.ini b/mozilla2/l10nbuilds2.ini
>+mozilla = releases/lorentz

This should be 'projects/firefox-lorentz' as it's pointing to the repo.

>diff --git a/mozilla2/linux/lorentz b/mozilla2/linux/lorentz
>new file mode 100644
>--- /dev/null
>+++ b/mozilla2/linux/lorentz
>@@ -0,0 +1,1 @@
>+mozilla-1.9.2
>\ No newline at end of file
>diff --git a/mozilla2/linux64/lorentz b/mozilla2/linux64/lorentz
>diff --git a/mozilla2/macosx/lorentz b/mozilla2/macosx/lorentz

Are these proper symlinks ? When I tried to hg import this patch I just got a text file. For comparision, the file mode is 120000 when I create a symlink.

>diff --git a/mozilla2/winmo/mobile-lorentz/nightly/mozconfig b/mozilla2/winmo/mobile-lorentz/nightly/mozconfig
Should there be a mobile master config to go with this to provide WinMo and Maemo builds ?

Comment 37

[Lukas Blakk [:lsblakk] use ?needinfo]

Attachment: Add "lorentz" to production configs

don't know what was up with the symlinks, redid them.  also am attaching the staging patch i used to test this configuration (i used my own repo for mozconfigs and got builds going too). thanks for pointing out the mobile_config being missing, I had it in staging and totally spaced on doing the change for production as well.

Comment 38

[Lukas Blakk [:lsblakk] use ?needinfo]

Attachment: Add "lorentz" to staging configs

Comment 39

[Lukas Blakk [:lsblakk] use ?needinfo]

Attachment: add "lorentz" to talos graphs db

Comment 40

[Nick Thomas [:nthomas] (UTC+12)]

Comment on attachment 422588 [details] [diff] [review]
Add "lorentz" to production configs

>diff --git a/mozilla2/config.py b/mozilla2/config.py

>+BRANCHES['lorentz']['create_snippet'] = False

FYI, even though this disables creation of mar files for nightlies, users will still find updates - back to the main mozilla-1.9.2 nightlies (need bug 534954).

>diff --git a/mozilla2/mobile_config.py b/mozilla2/mobile_config.py

The changes compared to the one aki r+'d (attachment 420657 [details] [diff] [review]) mostly seem fine, except for the builder names ...

>+MOBILE_BRANCHES['mobile-lorentz']['platforms']['linux-gnueabi-arm']['base_name'] = 'Maemo mozilla-lorentz'
>+MOBILE_BRANCHES['mobile-lorentz']['platforms']['linux-i686']['base_name'] = 'Linux Fennec Desktop mozilla-lorentz'
>+MOBILE_BRANCHES['mobile-lorentz']['platforms']['macosx-i686']['base_name'] = 'OS X Fennec Desktop mozilla-lorentz'
>+MOBILE_BRANCHES['mobile-lorentz']['platforms']['win32-i686']['base_name'] = 'Win32 Fennec Desktop mozilla-lorentz'
>+MOBILE_BRANCHES['mobile-lorentz']['platforms']['winmo-arm']['base_name'] = 'WinMo mozilla-lorentz'

Why 'mozilla-lorentz' here ? 'lorentz' would match up the other usage AIUI. r+ to fix that on checkin.

Comment 41

[Nick Thomas [:nthomas] (UTC+12)]

Comment on attachment 422589 [details] [diff] [review]
Add "lorentz" to staging configs

A few nits to fix on checkin for your r+.

>diff --git a/mozilla2-staging/config.py b/mozilla2-staging/config.py
>+BRANCHES['lorentz']['start_hour'] = [3] 
>+BRANCHES['lorentz']['start_minute'] = [32] 

Please remove trailing spaces to minimize difference between staging and prod.

>+BRANCHES['lorentz']['enable_xulrunner'] = False

This is True on prod. Since is staging is completely overwhelmed already lets keep staging and prod in sync by flipping it to True.

>+BRANCHES['lorentz']['enable_l10n'] = False
...
>+BRANCHES['lorentz']['enUS_binaryURL'] = \
>+    BRANCH_LEVEL_VARS['download_base_url'] + '/nightly/latest-lorentz'

L10n is enabled on production and there are some vars defined there but not here, so copy the equivalent section over.

>+BRANCHES['lorentz']['tinderbox_tree'] = 'MozillaTest'

Please add
BRANCHES['lorentz']['packaged_unittest_tinderbox_tree'] = 'MozillaTest'

Comment 42

[Chris AtLee [:catlee]]

Do we need talos master configs for this too?

Comment 43

[alice nodelman [:alice] [:anode]]

Comment on attachment 422624 [details] [diff] [review]
add "lorentz" to talos graphs db

Incorrect os numbers associated with new builders.

Comment 44

[Lukas Blakk [:lsblakk] use ?needinfo]

Attachment: add "lorentz" to talos graphs db

fixed machine types

Comment 45

[Lukas Blakk [:lsblakk] use ?needinfo]

Attachment: Add "lorentz" to production configs (final)

nits addressed in this patch

Comment 46

[Lukas Blakk [:lsblakk] use ?needinfo]

Attachment: Add "lorentz" to staging configs (final)

nits addressed in this patch.

Comment 47

[Lukas Blakk [:lsblakk] use ?needinfo]

Attachment: add "lorentz" to talos buildbot-configs

Comment 48

[Lukas Blakk [:lsblakk] use ?needinfo]

Attachment: add "lorentz" to talos buildbot-configs

took out release_tests and fetching release symbols

Comment 49

[Lukas Blakk [:lsblakk] use ?needinfo]

Comment on attachment 423069 [details] [diff] [review]
Add "lorentz" to production configs (final)

http://hg.mozilla.org/build/buildbot-configs/rev/5d237ae06285

Comment 50

[Lukas Blakk [:lsblakk] use ?needinfo]

Comment on attachment 423070 [details] [diff] [review]
Add "lorentz" to staging configs (final)

http://hg.mozilla.org/build/buildbot-configs/rev/6e8bb076aad2

Comment 51

[Lukas Blakk [:lsblakk] use ?needinfo]

Comment on attachment 423085 [details] [diff] [review]
add "lorentz" to talos buildbot-configs

http://hg.mozilla.org/build/buildbot-configs/rev/38a428dbec41

Comment 52

[Lukas Blakk [:lsblakk] use ?needinfo]

Comment on attachment 423061 [details] [diff] [review]
add "lorentz" to talos graphs db

http://hg.mozilla.org/graphs/rev/c3e116abae05

alice/justdave added Lorentz to stage/prod graphserver

Comment 53

[Lukas Blakk [:lsblakk] use ?needinfo]

bsmedberg: do we need wince builds for Lorentz?

Comment 54

[Benjamin Smedberg]

WinCE or winmo? AFAIK I do need winmo builds but probably not wince.

Comment 55

[(not currently active) Ted Mielczarek]

WinCE are "Firefox on WinCE" for tegra devices, WinMo is "Fennec on winmo".

Comment 56

[Nick Thomas [:nthomas] (UTC+12)]

'Linux lorentz build' and 'OS X 10.5.2 lorentz build' are red with:
  ======== BuildStep started ========
  Error: failed graph server post
  === Output ===
  results not added, response: 
  No branch_id for a branch_name 'Firefox-Lorentz' can be found

Looks like the branch added to the graphs db was 'Lorentz'.

Comment 57

[[e-mail] (:bhearsum)]

I had to land a tiny bustage fix for mobile_config.py in staging: http://hg.mozilla.org/build/buildbot-configs/rev/0f468404a955

Comment 58

[Lukas Blakk [:lsblakk] use ?needinfo]

(In reply to comment #56)
> 'Linux lorentz build' and 'OS X 10.5.2 lorentz build' are red with:
>   ======== BuildStep started ========
>   Error: failed graph server post
>   === Output ===
>   results not added, response: 
>   No branch_id for a branch_name 'Firefox-Lorentz' can be found
> 
> Looks like the branch added to the graphs db was 'Lorentz'.

This has been fixed now and builds are posting to graph server properly.

Unless there are further issues, I'll be closing this bug at the end of the day.

Comment 59

[Benjamin Smedberg]

I pushed something at 13:36 today and builds haven't kicked off yet:
http://hg.mozilla.org/projects/firefox-lorentz/pushloghtml

Comment 60

[Nick Thomas [:nthomas] (UTC+12)]

That could be an HgPoller issue with new repos. After the repo was cloned pushloghtml was empty, and so presumably was the pushlog RSS that buildbot was polling. Now it's querying http://hg.mozilla.org//projects/firefox-lorentz/pushlog?fromchange=e8f5e9c35f9ad7c70e95f716bd36bc0677a49507, so it knows there was a change pushed but for some reason didn't trigger builds on that. 

Could you try merging mrbkap's latest landing on m-1.9.2 to firefox-lorentz ?

Comment 61

[Benjamin Smedberg]

yep, new builds appeared with the second push

Comment 62

[Lukas Blakk [:lsblakk] use ?needinfo]

Attachment: Fix for mozconfig paths and locations (staging and production)

we use $repo to build the path to mozconfigs in the MercurialBuildFactory so the mozconfig path has to be "firefox-lorentz" not "lorentz". This patch adjusts both staging and production configs & file locations.

Comment 63

[Lukas Blakk [:lsblakk] use ?needinfo]

Attachment: Fix for mozconfig paths and locations (staging and production)

This time with the old and no longer usable "lorentz" mozconfigs removed.

Comment 64

[Nick Thomas [:nthomas] (UTC+12)]

Comment on attachment 423441 [details] [diff] [review]
Fix for mozconfig paths and locations (staging and production)

>diff --git a/mozilla2-staging/config.py b/mozilla2-staging/config.py
>--- a/mozilla2-staging/config.py
>+# MercurialBuildFactory uses the repo_path in creating path to mozconfig so make sure they match 

As we discussed on IRC, please change this to (something like)
# UnittestBuildFactory uses repo_path to compute path the mozconfig for its old-style unit test
# builds, which forces the path for these opt and debug builds

Please also make sure that mozilla2*/win{32,ce}/lorentz are completely removed (either I'm importing the patch wrong or there's an empty lorentz/ tree still present).

Comment 65

[Lukas Blakk [:lsblakk] use ?needinfo]

http://hg.mozilla.org/build/buildbot-configs/rev/c6f68b110d44

with comment changed, and yes the lorentz folders were completely removed.

Comment 66

[Lukas Blakk [:lsblakk] use ?needinfo]

It looks like any related Lorentz requests have their own bugs so I'm closing this one.

Comment 67

[Armen [:armenzg]]

Attachment: [mobile_config.py] download_base_url was missing (I also did some clean up)

Are we to upload to /home/ftp/pub/*mobile*?

Including coop in reviews for ['winmo-arm']['create_snippet']

Comment 68

[Armen [:armenzg]]

Attachment: [mobile_config.py] download_base_url was missing (I also did some clean up)

I attached an incorrect patch (missing some download_base_url)

Comment 69

[Armen [:armenzg]]

Attachment: [mobile_config.py] download_base_url was missing (I also did some clean up) (v1)

Comment 70

[Aki Sasaki (not active)]

Comment on attachment 423577 [details] [diff] [review]
[mobile_config.py] download_base_url was missing (I also did some clean up) (v1)

we haven't moved from firefox/ -> mobile/ yet.

Comment 71

[Armen [:armenzg]]

Attachment: [mobile_config.py] download_base_url was missing (I also did some clean up) (v2)

Fixed aki's comment.

Comment 72

[Lukas Blakk [:lsblakk] use ?needinfo]

We need to reconfig Talos to pick up the new Lorentz builds. Downtime to be scheduled asap.

Comment 73

[Lukas Blakk [:lsblakk] use ?needinfo]

Attachment: L10n path fix

Comment 74

[Lukas Blakk [:lsblakk] use ?needinfo]

Attachment: Rename of BRANCHES['lorentz'] to BRANCHES['firefox-lorentz'] in config.py (prod/staging)

So the shark builds were failing because the path to mozconfig is based on the branch name, and we moved all the mozconfigs from 'lorentz' to 'firefox-lorentz' because of unittest mozconfig path building which is based on repo name.

From now on, for all new branches, the repo_name must be the same as the BRANCHES['$name'] and the same as the mozconfig folder name.

Comment 75

[Lukas Blakk [:lsblakk] use ?needinfo]

Attachment: Rename of BRANCHES['lorentz'] to BRANCHES['firefox-lorentz'] in config.py (prod/staging)

now with the Branches dictionary corrected to firefox-lorentz also.

Comment 76

[Lukas Blakk [:lsblakk] use ?needinfo]

Attachment: Rename of BRANCHES['lorentz'] to BRANCHES['firefox-lorentz'] in config.py (prod/staging)

added correction to Branch reference in master2.cfg as well as mobile_config.py

Comment 77

[Lukas Blakk [:lsblakk] use ?needinfo]

Attachment: Rename of BRANCHES['lorentz'] to BRANCHES['firefox-lorentz'] in config.py (prod/staging)

Comment 78

[Lukas Blakk [:lsblakk] use ?needinfo]

Attachment: Rename of BRANCHES['lorentz'] to BRANCHES['firefox-lorentz'] in config.py (prod/staging)

This patch applies cleanly and passes checkconfig on both staging and pm02 - it rolls in the clean up that Armen's patch did in order to do so.

Comment 79

[Nick Thomas [:nthomas] (UTC+12)]

Comment on attachment 423700 [details] [diff] [review]
Rename of BRANCHES['lorentz'] to BRANCHES['firefox-lorentz'] in config.py (prod/staging)

>diff --git a/mozilla2/mobile_config.py b/mozilla2/mobile_config.py

>+MOBILE_BRANCHES['mobile-trunk']['download_base_url'] = 'http://staging-stage.build.mozilla.org/pub/mozilla.org/firefox'
>+MOBILE_BRANCHES['mobile-1.9.2']['download_base_url'] = 'http://staging-stage.build.mozilla.org/pub/mozilla.org/firefox'
>+MOBILE_BRANCHES['mobile-tracemonkey']['download_base_url'] = 'http://staging-stage.build.mozilla.org/pub/mozilla.org/firefox'
>+MOBILE_BRANCHES['mobile-lorentz']['download_base_url'] = 'http://staging-stage.build.mozilla.org/pub/mozilla.org/firefox'

These should be http://stage.mozilla.org/... for production, I think. r+ to fix that on checkin if coop r+'s too

Comment 80

[Lukas Blakk [:lsblakk] use ?needinfo]

Attachment: Talos configs - lorentz -> firefox-lorentz

Comment 81

[Lukas Blakk [:lsblakk] use ?needinfo]

Attachment: Talos graphs/data.sql - lorentz -> firefox-lorentz

Comment 82

[Lukas Blakk [:lsblakk] use ?needinfo]

Attachment: Talos graphs/data.sql - lorentz -> firefox-lorentz

re-did patch with proper platform codes.

Comment 83

[Lukas Blakk [:lsblakk] use ?needinfo]

Comment on attachment 424147 [details] [diff] [review]
Talos graphs/data.sql - lorentz -> firefox-lorentz

http://hg.mozilla.org/graphs/rev/4bc13bf26989

Comment 84

[Lukas Blakk [:lsblakk] use ?needinfo]

Comment on attachment 424144 [details] [diff] [review]
Talos configs - lorentz -> firefox-lorentz

http://hg.mozilla.org/build/buildbot-configs/rev/8725aa3e2683

Comment 85

[Lukas Blakk [:lsblakk] use ?needinfo]

Well the naming conflicts between lorentz & firefox-lorentz have all been resolved. We have talos turned on now.  I believe this branch is really really on - closing.