If you think a bug might affect users in the 57 release, please set the correct tracking and status flags for Release Management.

Enable Gbn & Gbun for Node.js build script

RESOLVED WONTFIX

Status

Release Engineering
General Automation
P2
normal
RESOLVED WONTFIX
3 years ago
2 years ago

People

(Reporter: rickychien, Assigned: rickychien)

Tracking

Firefox Tracking Flags

(Not tracked)

Details

(Assignee)

Description

3 years ago
Enable Gbn & Gbun for all platform including Linux/Mac/Windows in Try-server (gaia-try / try / b2g-inbound) and set invisible until M1 complete
(Assignee)

Comment 1

3 years ago
See also build system plan Milestone 0: https://wiki.mozilla.org/Gaia/Build/RefactoringToNodejs#Milestone_0

In this bug, we want to create two test-suites Gbn (Gaia Build Node.js Test) and Gbun (Gaia Build Unit Node.js Test) on tryserver in order to protect Node.js build script and set it invisible during development stage until Milestone 1 finish. Notice both test-suites would be red for a while until Milestone 1 finish.

Additionally, tryserver has to pass RUN_ON_NODE=1 in their scripts when launching Gbn or Gbun.
(Assignee)

Updated

3 years ago
Assignee: ricky060709 → nobody
Component: Gaia::Build → Treeherder
Product: Firefox OS → Tree Management
Version: unspecified → ---
(Assignee)

Comment 2

3 years ago
Hi Ryan,

In addition to enable Gbn & Gbun, is it possible to enable both test suites on:

B2G Desktop Linux opt
B2G Desktop OS X opt
B2G Desktop Windows opt

thanks!
Flags: needinfo?(ryanvm)

Comment 3

3 years ago
If these are taskcluster jobs:
- enabling will need to be done by the taskcluster team (Testing::Taskcluster)
- support for the job names _don't_ need to be added to treeherder
- the jobs will be shown by default, and the sheriffs can only hide them once they start running (file a "hide job foo" bug in Tree Management::Visibility Requests).

If these are buildbot jobs:
- enabling will need to be done by release engineering (Release Engineering::General Automation)
- support for the job name _will_ need to be added to treeherder (file a bug in Tree Management::Data Ingestion)
- the jobs will be shown by default, and the sheriffs can only hide them once they start running (file a "hide job foo" bug in Tree Management::Visibility Requests).

Looking at the comments in this bug and the wiki, this seems like it's still at the "enable jobs" stage, so this part doesn't belong in a Treeherder component (file new bugs for the other stages above when ready :-)).
Component: Treeherder → Gaia::Build
Flags: needinfo?(ryanvm)
Product: Tree Management → Firefox OS
Version: --- → unspecified
(Assignee)

Updated

3 years ago
Component: Gaia::Build → General Automation
Product: Firefox OS → Release Engineering
QA Contact: catlee
(Assignee)

Comment 4

3 years ago
I would like to take this bug as meta bug regarding node.js build system and automation, then file other bugs mentioned on comment 3 depend on this for addressing follow-up issues.
(Assignee)

Updated

3 years ago
Flags: needinfo?(coop)

Comment 5

3 years ago
Not sure what I'm being asked here, so I'll defer to catlee as the component owner.

However...

If these are new tests you want added, i.e. tests that aren't currently running elsewhere, at the very least we'll need:

* pointers to the scripts
* instructions about how to invoke them
* any assumptions the scripts make re: setup and environment
Flags: needinfo?(coop) → needinfo?(catlee)
Are these to be incorporated into existing test suites, or is this a new test suite?
Flags: needinfo?(catlee)
(Assignee)

Comment 7

3 years ago
(In reply to Chris AtLee [:catlee] from comment #6)
> Are these to be incorporated into existing test suites, or is this a new
> test suite?

Not sure what's your test suite mean, I think our treeherder would be:

B2G Desktop Linux opt    B Gb Gbu Gbn Gbun ...
B2G Desktop OS X opt     B Gbn Gbun ...
B2G Desktop Windows opt  B Gbn Gbun ...

I would be great if we could run tests on OS X and Windows.
(Assignee)

Updated

3 years ago
Flags: needinfo?(catlee)
(Assignee)

Updated

3 years ago
Flags: needinfo?(coop)

Comment 8

3 years ago
(In reply to Ricky Chien [:rickychien] from comment #7)
> (In reply to Chris AtLee [:catlee] from comment #6)
> > Are these to be incorporated into existing test suites, or is this a new
> > test suite?
> 
> Not sure what's your test suite mean, I think our treeherder would be:
> 
> B2G Desktop Linux opt    B Gb Gbu Gbn Gbun ...
> B2G Desktop OS X opt     B Gbn Gbun ...
> B2G Desktop Windows opt  B Gbn Gbun ...

That's the last step. We're at still at step #1 here.

What catlee and I are asking is *how* are these tests run. AFAICT, we don't current run Gbn & Gbun on any branch, which means that these are new tests.

* Are they variants of the existing Gb & Gbu test run with different params?
* If they are completely new tests, where do the scripts to run these tests live?
** Are they mozharness-based, or something else?
* How do we run these tests, i.e. what does a default command-line invocation look like?
Flags: needinfo?(coop)
Flags: needinfo?(catlee)
(Assignee)

Comment 9

3 years ago
(In reply to Chris Cooper [:coop] from comment #8)
> * Are they variants of the existing Gb & Gbu test run with different params?

Yes, just clone Gb / Gbu script and add an additional flag RUN_ON_NODE=1 behind [1] to enable node.js tests.

> * How do we run these tests, i.e. what does a default command-line
> invocation look like?

same as above

I'll file a bug for enabling node Gbu/Gb testing when RUN_ON_NODE=1

[1] https://dxr.mozilla.org/build:mozharness/source/scripts/gaia_build_integration.py#39
(Assignee)

Updated

3 years ago
Blocks: 1141469
(Assignee)

Updated

3 years ago
No longer blocks: 1141469
Depends on: 1141469
(Assignee)

Comment 10

2 years ago
Our plan is going to do migration and merge it directly soon so it won't enable another node version Gb/Gbn.
Assignee: nobody → rchien
Status: ASSIGNED → RESOLVED
Last Resolved: 2 years ago
Resolution: --- → WONTFIX
(Assignee)

Updated

2 years ago
Priority: -- → P2
You need to log in before you can comment on or make changes to this bug.