Amphtml: Github app authentication intermittently fails on Travis

Created on 26 Jun 2019  路  3Comments  路  Source: ampproject/amphtml

I've seen quite a few instances over the past day. Maybe something was up with the auth token? Filing this issue in case it becomes a persistent problem.

https://travis-ci.org/ampproject/amphtml/jobs/550854287#L387-L388

dist-bundle-size.js: Running gulp bundle-size --on_pr_build...
[15:04:02] Using gulpfile ~/build/ampproject/amphtml/gulpfile.js
[15:04:02] Starting 'bundle-size'...
[15:04:02] Running npx bundlesize -f "./dist/v0.js"...
[15:04:10] Bundle size is 86.63KB
[15:04:10] Could not report the bundle size of this pull request
[15:04:10] Error: 403 Forbidden: You are not Travis!
[15:04:10] Finished 'bundle-size' after 7.88 s

/cc @ampproject/wg-infra

Bug infra

Most helpful comment

@estherkim reported this in https://github.com/ampproject/amp-github-apps/issues/198, related to recently changed IP address ranges from Travis. I've updated both the test-status and bundle-size apps about an hour ago with the new ranges - let me know if you see any new cases of this happening this week! Keeping open for a day or two until we're confident

All 3 comments

It turns out this is more widespread than I initially thought. Here's an example from the test status app:

https://travis-ci.org/ampproject/amphtml/jobs/550558693#L2042-L2043

[00:05:49] WARNING: failed to report unit/saucelabs/report/7780/0 to the test-status GitHub App:
 403 - "You are not Travis!"

@estherkim reported this in https://github.com/ampproject/amp-github-apps/issues/198, related to recently changed IP address ranges from Travis. I've updated both the test-status and bundle-size apps about an hour ago with the new ranges - let me know if you see any new cases of this happening this week! Keeping open for a day or two until we're confident

This hasn't reared its head recently.

Was this page helpful?
0 / 5 - 0 ratings