amphtml: `gulp dist` is timing out on Travis

When gulp dist was last instrumented, it used to take ~13 mins to run on Travis during master builds. Since then, the running time went up to ~18 minutes, and as of today, it takes ~22 minutes. This is causing it to frequently time out on Travis, resulting in redness on master.

See https://travis-ci.org/ampproject/amphtml/jobs/352664009#L632 for a recent failure See https://travis-ci.org/ampproject/amphtml/jobs/352471805#L639 for a build where gulp dist took ~27 mins to complete

Related to #10277

About this issue

  • Original URL
  • State: closed
  • Created 6 years ago
  • Comments: 16 (14 by maintainers)

Most upvoted comments

More extensions? Shouldn’t nearly double the running time though. Separately, having our own metrics would be great to see trends over time.

I think @choumx is correct. Some recent extensions (amp-mathml.js, amp-fx-collection.js) are taking nearly a minute a piece.

For temporary relief, can we increase the timeout?

@erwinmombay perhaps we can do a deep dive here with a few folks to debug?

Deep dive done earlier today. @erwinmombay is working on the closure compiler upgrade in #12558, which should set us on the path to a single invocation of the closure compiler. Any other work than can be done on the side will be spun off into separate issues.