Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We鈥檒l occasionally send you account related emails.

Already on GitHub? Sign in to your account

support WarmUp Trigger #575

Open
wants to merge 3 commits into
base: master
Choose a base branch
from

Conversation

jdaviderb
Copy link

@jdaviderb jdaviderb commented Oct 11, 2021

What did you implement:

Closes #574

How did you implement it:

in order to support the warm-up trigger in Serverless, I added the binding manually in the bindings.json file

How can we verify it:

add this config in the serverless.yml

functions:
  warmup:
    handler: src/handlers/warmup.warmupHandler
    events:
      - warmup:
          x-azure-settings:
            name: warmupContext
// src/handlers/warmup.warmupHandler

module.exports.warmupHandler = async function (context, warmupContext) {
    context.log('Function App instance is warm 馃尀馃尀馃尀');
};

Is this ready for review?: YES
Is it a breaking change?: NO

Refs
https://github.com/MicrosoftDocs/azure-docs/blob/master/articles/azure-functions/functions-bindings-warmup.md
https://docs.microsoft.com/en-us/azure/azure-functions/functions-bindings-warmup?tabs=javascript

@jdaviderb jdaviderb force-pushed the feature/support-warmup-trigger branch from f62c2ff to c655b4d Compare October 11, 2021 16:50
@jdaviderb jdaviderb force-pushed the feature/support-warmup-trigger branch from d5e1699 to 7669a13 Compare October 12, 2021 13:59
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Support to warmupTrigger
1 participant