Closed Bug 1233829 Opened 5 years ago Closed 5 years ago

thunderbird builds can't generate info from application.ini because of invocation


(Release Engineering :: Release Automation: Other, defect)

Not set


(Not tracked)



(Reporter: jlund, Unassigned)




(1 file)

history time!

1. build sys got rid of config/ from gecko recently: Bug 957911 - Remove config/

2. which meant that we had to change how we invoked in our mozharness automation: Bug 957951 - Use mach to invoke

3. we forgot to change buildbot automation however which was still the primary place for building TB everywhere and for FF on beta/release/esr. So TB fixed this first: Bug 1213715 - TB needs changes to how it gets the app version and buildid via bug 957911)

4. but then we had to fix FF once (1) landed on beta: Bug 1232466 - beta CI builds fail trying to use and running prettyname targets for package and update when GTK+3 is enabled

5. that fix in (4) was supposed to be a unified way to fix it for all the things, TB included but it's not quite right:

job (platform wide):

/tools/buildbot/bin/python /builds/slave/tb-rel-c-beta-lx_bld-000000000/build/mach python /builds/slave/tb-rel-c-beta-lx_bld-000000000/build/mozilla/config/ /builds/slave/tb-rel-c-beta-lx_bld-000000000/build/objdir-tb/dist/bin/application.ini App BuildID
 in dir /builds/slave/tb-rel-c-beta-lx_bld-000000000/build/objdir-tb (timeout 1200 secs)
/tools/buildbot/bin/python: can't open file '/builds/slave/tb-rel-c-beta-lx_bld-000000000/build/mach': [Errno 2] No such file or directory

My guess is we are looking for mach in the comm repo and not gecko repo. Need a special case for TB in our automation...
I suppose as an option, we could just re-open

but I'll leave this open in case we want to clearly highlight the history from comment 0
Summary: thunderbird builds can't generate info form application.ini because of invocation → thunderbird builds can't generate info from application.ini because of invocation
I have no idea if this patch is correct, or if it works. It is *completely* untested. And before landing we should try and make sure that even calling mach for TB here "works" as well as make sure this patch passes checkconfig in buildbot.

Still flagging for review though.

Additionally a dump of the config would be useful to verify we don't change the string used for non TB builds here.
Attachment #8700263 - Flags: review?(jlund)
Thanks all.

I haven't made this critical or blocker so that people don't get paged - but it is that important to get resolved quickly, so that we get our builds done in time for SHA2, etc
Severity: normal → major
Looks like a duplicate of bug 1233577?
Comment on attachment 8700263 [details] [diff] [review]
[custom] untested fix to call mach at the right place

Obsoleted by Bug 1233577
Attachment #8700263 - Flags: review?(jlund)
Closed: 5 years ago
Resolution: --- → DUPLICATE
Duplicate of bug: 1233577
You need to log in before you can comment on or make changes to this bug.