<div dir="ltr"><div>In addition, code change in widget/{android,cocoa,gonk,gtk,windows,...} should only need build and test in the corresponding platforms.<br><br></div>- Xidorn<br><div><div class="gmail_extra"><br><div class="gmail_quote">On Sun, Feb 22, 2015 at 4:11 AM, Gijs Kruitbosch <span dir="ltr"><<a href="mailto:gijskruitbosch@gmail.com" target="_blank">gijskruitbosch@gmail.com</a>></span> wrote:<br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">Hi,<br>
<br>
I've been noticing that the number of builds + tests we run on certain changesets could still do with improving. I realize that we try to be conservative because we don't want "magically appearing" orange, but I still think that it should be possible to do better here. That seems like it might also help with recent capacity problems (maybe?).<br>
<br>
I don't know who's responsible for improving this, so I'm just posting this here. The list is biased towards frontend desktop work because that is what I do. Perhaps other people can provide further examples or insight from android, b2g, platform, releng or sheriff perspectives into what could/couldn't be improved here.<br>
<br>
<br>
Some examples:<br>
<br>
(1)<br>
<a href="https://treeherder.mozilla.org/#/jobs?repo=fx-team&revision=0fd6c4c24dc9" target="_blank">https://treeherder.mozilla.<u></u>org/#/jobs?repo=fx-team&<u></u>revision=0fd6c4c24dc9</a><br>
<br>
This checkin only touched a single test file in browser/components/<u></u>preferences/in-content/tests/<br>
<br>
It triggered browser builds on all platforms, but also all kinds of B2G builds and tests, which are clearly useless, as well as running all kinds of unrelated test suites for the browser builds.<br>
<br>
Ideally, commits that only touch things in directories matching **test** should only run the relevant test suite, if possible even with builds from the previous rev.<br>
<br>
(2)<br>
<a href="https://treeherder.mozilla.org/#/jobs?repo=fx-team&revision=2b1d7ebb8325" target="_blank">https://treeherder.mozilla.<u></u>org/#/jobs?repo=fx-team&<u></u>revision=2b1d7ebb8325</a><br>
<br>
This touched files in:<br>
browser/base<br>
browser/components<br>
browser/locales<br>
toolkit/obsolete/content<br>
<br>
I don't think any of these are used outside Firefox for Desktop, and yet android and b2g were built, too. The same happened with:<br>
<br>
<a href="https://treeherder.mozilla.org/#/jobs?repo=fx-team&revision=80bd1ae9dd0a" target="_blank">https://treeherder.mozilla.<u></u>org/#/jobs?repo=fx-team&<u></u>revision=80bd1ae9dd0a</a><br>
which touched toolkit/content/widgets/ which AIUI is also desktop-only.<br>
<br>
(3)<br>
Checkins (couldn't find one offhand because summaries are little help in identifying these) that only touch windows/linux/osx styles in browser/ and/or toolkit should only run those builds<br>
<br>
<br>
~ Gijs<br>
______________________________<u></u>_________________<br>
dev-platform mailing list<br>
<a href="mailto:dev-platform@lists.mozilla.org" target="_blank">dev-platform@lists.mozilla.org</a><br>
<a href="https://lists.mozilla.org/listinfo/dev-platform" target="_blank">https://lists.mozilla.org/<u></u>listinfo/dev-platform</a><br>
</blockquote></div><br></div></div></div>