Travis fails on Node.js 10, because it can't install it. CI passes on Node.js 8.

Hey @merlinnot just rerun the job if it fails on stuff like installing Node 10, it usually is a transient error. I just reran your last one and it passed.

Great, thanks! The PR is ready for a first pass, please not the question in the description of the PR.

@thechenky Hey, just a friendly ping for a review :)

Ah, will get to this in a bit - for future reference, please add me as reviewer at the top so that I understand it's ready for review.

I'll extract opts -> options rename and comments fixes to separate PRs once the three new ones I created are merged, it will be easier for me to extract these changes.

Sounds good. Reviewing the others now.

@merlinnot looks like you might need to run prettier on this one more time before we can merge? I merged master into your branch so please pull (or just force-push your own thing)

@samtstern Done!

Thanks @merlinnot !

Any ideas when this is going to be re-merged back in / released?

Good call! @samtstern Is the work in the CLI complete?

I know this sounds pathetic but no matter what I tried I could not get this to work, the GCF API did not seem to be taking my PATCH requests.

@tinaliang I'll bring you into the internal thread.

Just found this PR, I'm happy this is in the works!
Just hoping it'll be merged again soon :)

Hi

I find workaround, you can define your functions like this:

const functions = require('firebase-functions')

const fn = functions.firestore
  .document('/collection/{item_id}')
  .onWrite(async (change, context) => {
    // duno
    return 0
  })

module.exports = new Proxy(fn, {
  get(target, phrase) {
    const obj = target[phrase]
    if (phrase === '__trigger') obj.eventTrigger.failurePolicy = { retry: {} }
    return obj
  }
})

@merlinnot I may be mistaken here, but the current firebase docs do not reflect this change, correct? https://firebase.google.com/docs/reference/functions/function_builder_.functionbuilder#runwith

@cjimmy you're right looks like we need to regenerate the reference docs, thanks for the reminder!

Here are some notes, for my future self and others.

For anyone looking add retries to firebase functions, without having to use GCP console, this PR is it and enabled in firebase-functions 3.10.0. Since it's not entirely obvious, here's a quick example of the syntax:

export const myFirebaseFunc = functions
  .runWith({
    failurePolicy: {
      retry: {},
    },
    memory: '512MB',
    timeoutSeconds: 60,
  })
  .firestore.document('/path/to/some/doc')
  .onCreate(async (snap, context) => {
   /* do stuff */
 })

Right now, it looks like the failure policy is simply on or off. Thus, this is equivalent

export const myFirebaseFunc = functions
  .runWith({
    failurePolicy: true,
    memory: '512MB',
    timeoutSeconds: 60,
  })
  .firestore.document('/path/to/some/doc')
  .onCreate(async (snap, context) => {
   /* do stuff */
 })

Some caveats:

• You'll also have to deploy with --force
• You can enable retries only on triggered functions, not http-called functions.
• It would be idiotic not to build in some safeguards. Retry policy maxes out at 7 days, and bills just like any other function invocation, so if you have some unhandled error, it could repeatedly run for a full week. You can use context.eventTimestamp to know when the first attempt roughly started.

Read this: https://firebase.google.com/docs/functions/retries and make sure your function in idempotent.

It was also difficult to discover what to return to force a retry or avoid a retry.
Triggered Firebase functions must return a Promise.

A Firebase function will retry if:

• it returns a rejected promise
• throws an exception
• or the function times out

That means that if you run into an error that you know won't eventually resolve itself with a retry (i.e. you want to stop the function execution and not retry), you can return Promise.resolve({message: 'some message'});