Closed Bug 1480392 Opened 4 years ago Closed 4 years ago

Bustages on central with ERROR: Cannot find nodejs


(Firefox :: Messaging System, defect, P1)




Firefox 63
63.3 - Aug 6
Tracking Status
firefox63 --- fixed


(Reporter: CosminS, Assigned: dmosedale)


(Blocks 1 open bug)



(1 file)

The tests on central:

Failure log:

[task 2018-08-02T10:35:26.920Z] 10:35:26     INFO -  ERROR: Cannot find nodejs
[task 2018-08-02T10:35:26.947Z] 10:35:26     INFO -  *** Fix above errors and then restart with\
[task 2018-08-02T10:35:26.947Z] 10:35:26     INFO -                 "/usr/bin/make -f build"
[task 2018-08-02T10:35:26.948Z] 10:35:26     INFO - recipe for target 'configure' failed
[task 2018-08-02T10:35:26.948Z] 10:35:26     INFO -  make: *** [configure] Error 1
[task 2018-08-02T10:35:27.022Z] 10:35:27    ERROR - Return code: 2
[task 2018-08-02T10:35:27.023Z] 10:35:27  WARNING - setting return code to 2
[task 2018-08-02T10:35:27.023Z] 10:35:27    FATAL - 'mach build -v' did not run successfully. Please check log for errors.
[task 2018-08-02T10:35:27.023Z] 10:35:27    FATAL - Running post_fatal callback...
[task 2018-08-02T10:35:27.023Z] 10:35:27    FATAL - Exiting -1
[task 2018-08-02T10:35:27.023Z] 10:35:27     INFO - [mozharness: 2018-08-02 10:35:27.023792Z] Finished build step (failed)
[task 2018-08-02T10:35:27.023Z] 10:35:27     INFO - Running post-run listener: _summarize
[task 2018-08-02T10:35:27.024Z] 10:35:27    ERROR - # TBPL FAILURE #
[task 2018-08-02T10:35:27.024Z] 10:35:27     INFO - [mozharness: 2018-08-02 10:35:27.024239Z] FxDesktopBuild summary:
[task 2018-08-02T10:35:27.024Z] 10:35:27    ERROR - # TBPL FAILURE #

This looks like smthg caused by
Flags: needinfo?(dmose)
Should be a straightforward fix; I'll have a patch in a bit.
Assignee: nobody → dmose
Flags: needinfo?(dmose)
Note that the reason I added node to the build toolchain here rather than disabling node in mozconfig is because I'm assuming that searchfox is likely to want access to NodeJS JavaScript parsing stuff (eg JSX parsers, so that it can parse non-transpiled code correctly).
Blocks: 1478995
Iteration: --- → 63.3 - Aug 6
Priority: -- → P1
gps, while these patches seem extremely likely to fix the problem, one thing that's not clear to me is why these jobs are failing at all, since node.configure still has allow_missing=True.  Any ideas?
Oh, perhaps it's that when NODEJS is set in the environment, that effective overrides allow_missing to be False.
Comment on attachment 8997035 [details]
Bug 1480392 - fix searchfox build failures caused by missing node toolchain,

Yes, the presence of the environment variable always results in that value being used. It is up to downstream checks to verify the thing defined in the environment variable is actually used. Our policy (for better or worse) is to never ignore environment variables.
Attachment #8997035 - Flags: review?(gps) → review+
Pushed by
fix searchfox build failures caused by missing node toolchain, r=gps
Closed: 4 years ago
Resolution: --- → FIXED
Target Milestone: --- → Firefox 63
Component: Activity Streams: Newtab → Messaging System
You need to log in before you can comment on or make changes to this bug.