Open Bug 1668766 Opened 5 years ago Updated 3 years ago

mach try auto didn't schedule hazard builds

Categories

(Developer Infrastructure :: Try, defect, P3)

Tracking

(Not tracked)

People

(Reporter: sg, Unassigned)

References

(Blocks 1 open bug)

Details

The mach try auto push https://treeherder.mozilla.org/#/jobs?repo=try&revision=7a7f8f60d33b5382e77a4c8b44f67b61ec85fd08 didn't file any hazard build AFAIU, and the subsequent landing was backed out due to a bustage of that build: https://bugzilla.mozilla.org/show_bug.cgi?id=1665347#c9

Thanks for the report, Marco can the ML schedule hazard builds? Or is this just a case of not enough training?

Severity: -- → S4
Flags: needinfo?(mcastelluccio)
Priority: -- → P3

Clearing stale needinfo

Flags: needinfo?(mcastelluccio)

Sorry I had missed this... I will check.

Flags: needinfo?(mcastelluccio)

I've checked, we are indeed ignoring them. I can add them if we want to.
What are they doing exactly compared to normal builds?

Flags: needinfo?(mcastelluccio)

They are performing a GC rooting hazard analysis. sfink can provide a more substantial answer, if required.

Yes, it's a static analysis that requires a full browser build (with GCC). https://wiki.mozilla.org/Javascript:Hazard_Builds describes them. I'm actually not sure when it makes the most sense to trigger them. Certainly any code change within js/** could cause them to fail, but also any code change anywhere else can cause them to fail in any code that calls the changed code. (Such is the story with whole-program analysis.) But the vast majority of changes won't be relevant. I guess the failures are pretty heavily clustered around changes to certain files, so perhaps ML would do well here?

Flags: needinfo?(mcastelluccio)

Clearing stale needinfos (please re-request if still needed)

Flags: needinfo?(mcastelluccio)
Product: Firefox Build System → Developer Infrastructure
You need to log in before you can comment on or make changes to this bug.