openfaas / cron-connector

Invoke functions on a schedule.
MIT License
39 stars 10 forks source link

Add async endpoint support #15

Closed viveksyngh closed 3 years ago

viveksyngh commented 3 years ago

This commit adds async endpoint support to cron connector which can be enabled using async_invocation environment variable

Description

Motivation and Context

11

How Has This Been Tested?

Tested on my local system

k logs cron-connector-79d479b777-bvh7w -n openfaas -f
2021/03/20 17:20:54 Version: 43da735bc8f99800a52718a3e5ac31c4884349b7   Commit: dev
2021/03/20 17:20:54 Gateway URL:  http://gateway.openfaas:8080
2021/03/20 17:20:54 Async Invocation:  true
2021/03/20 17:21:04 added function nodeinfo in openfaas-fn
2021/03/20 17:25:00 Executed function: nodeinfo (ns=openfaas-fn)

Types of changes

Checklist:

derek[bot] commented 3 years ago

Thank you for your contribution. I've just checked and your commit doesn't appear to be signed-off. That's something we need before your Pull Request can be merged. Please see our contributing guide. Tip: if you only have one commit so far then run: git commit --amend --signoff and then git push --force.

alexellis commented 3 years ago

LGTM, next could you:

And we should be able to finally ship this 🙂

viveksyngh commented 3 years ago

@alexellis Added missing parts :)