openfaas / cron-connector

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

Fix invocations for faasd and #18 #21

Closed alexellis closed 3 years ago

alexellis commented 3 years ago

Signed-off-by: Alex Ellis (OpenFaaS Ltd) alexellis2@gmail.com

Description

Fix invocations for faasd and #18

Motivation and Context

When no namespace was set, invocations were failing due to the way strings were being built.

Also fixes: #19 Also fixes: #20

Also updates to Go 1.16 from 1.13

How Has This Been Tested?

Tested e2e with faasd and nodeinfo.

Before faas-cli list showed 0 invocations, after they were shown.

Jul 14 13:06:40 faasd openfaas:cron-connector[16664]: 2021/07/14 12:06:40 Version: 8bf11517b761e5969d4569e63e47bf6f12a04480        Commit: dev
Jul 14 13:06:40 faasd openfaas:cron-connector[16664]: 2021/07/14 12:06:40 Gateway URL: http://gateway:8080
Jul 14 13:06:40 faasd openfaas:cron-connector[16664]: 2021/07/14 12:06:40 Async Invocation: false
Jul 14 13:06:50 faasd openfaas:cron-connector[16664]: 2021/07/14 12:06:50 Registered: nodeinfo [* * * * *]
Jul 14 13:07:00 faasd openfaas:cron-connector[16664]: 2021/07/14 12:07:00 Executing function: nodeinfo
Jul 14 13:07:00 faasd openfaas:cron-connector[16664]: 2021/07/14 12:07:00 Response: nodeinfo [200]

Types of changes