Defaulting function invocation to async

[jdewinne]

Signed-off-by: Joris De Winne joris.dewinne@gmail.com

Description

This makes the default function invocation async. It can be changed by setting the environment variable async.

Motivation and Context

• [x] I have raised an issue to propose this change (required)
• [ ] My issue has received approval from the maintainers or lead with the design/approved label

How Has This Been Tested?

Did a manual test with the nodeinfo function.

Types of changes

• [ ] Bug fix (non-breaking change which fixes an issue)
• [ ] New feature (non-breaking change which adds functionality)
• [x] Breaking change (fix or feature that would cause existing functionality to change) **It changes the default invocation from sync to async (which is what it probably should be).

Checklist:

• [x] My code follows the code style of this project.
• [ ] My change requires a change to the documentation.
• [] I have updated the documentation accordingly.
• [ ] I've read the CONTRIBUTION guide
• [x] I have signed-off my commits with git commit -s
• [ ] I have added tests to cover my changes.
• [x] All new and existing tests passed.

[noam09]

Hey @jdewinne, regarding the environment variable, just a reminder that Alex requested that:

The environment variable should be called async_invocation as per the other connectors.

[jdewinne]

Hey @jdewinne, regarding the environment variable, just a reminder that Alex requested that:

The environment variable should be called async_invocation as per the other connectors.

Thanks. Made the change.

[alexellis]

Let us know if you need anything.

We've invested a significant amount in OpenFaaS Pro recently, which may be of interest to you.