ampproject / amphtml

The AMP web component framework.
https://amp.dev
Apache License 2.0
14.89k stars 3.89k forks source link

Visual diff tests frequently failing with `bad_request` errors #18141

Closed rsimha closed 6 years ago

rsimha commented 6 years ago

This has happened on a few PR builds today (I've had to restart all of them)

[16:33:06] StatusCodeError: 400 - {"errors":[{"status":"bad_request"},{"source":{"pointer":"/data/attributes/base"},"detail":"The name of each snapshot must be unique, and this name already exists in the build: 'Amp By Example (prod)' -- You can fix this by passing a 'name' param when creating the snapshot. See the docs for more info on identifying snapshots for your specific client: https://percy.io/docs"}]}
    at new StatusCodeError (/home/travis/build/ampproject/amphtml/node_modules/request-promise-core/lib/errors.js:32:15)
    at Request.plumbing.callback (/home/travis/build/ampproject/amphtml/node_modules/request-promise-core/lib/plumbing.js:104:33)
    at Request.RP$callback [as _callback] (/home/travis/build/ampproject/amphtml/node_modules/request-promise-core/lib/plumbing.js:46:31)
    at Request.self.callback (/home/travis/build/ampproject/amphtml/node_modules/request/request.js:185:22)
    at emitTwo (events.js:126:13)
    at Request.emit (events.js:214:7)
    at Request.<anonymous> (/home/travis/build/ampproject/amphtml/node_modules/request/request.js:1157:10)
    at emitOne (events.js:116:13)
    at Request.emit (events.js:211:7)
    at IncomingMessage.<anonymous> (/home/travis/build/ampproject/amphtml/node_modules/request/request.js:1079:12)
    at Object.onceWrapper (events.js:313:30)
    at emitNone (events.js:111:20)
    at IncomingMessage.emit (events.js:208:7)
    at endReadableNT (_stream_readable.js:1064:12)
    at _combinedTickCallback (internal/process/next_tick.js:139:11)
    at process._tickCallback (internal/process/next_tick.js:181:9)
danielrozenberg commented 6 years ago

Hasn't happened in a while. Closing.