search

openfaas / cron-connector

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

Log message confusing without namespaces #19

Closed alexellis closed 3 years ago

alexellis commented 3 years ago

The log message added function nodeinfo in is confusing because it's presuambly trying to print a namespace which isn't present or specified.

Also see in #18

Expected Behaviour

Print something similar to:

Registered nodeinfo with schedule:  "*/5 * * * *"

And when a namespace is given:

Registered nodeinfo.openfaas-fn with schedule:  "*/5 * * * *"

Current Behaviour

ubuntu@faasd:~$ sudo journalctl -t openfaas:cron-connector  -f
-- Logs begin at Wed 2021-07-14 12:01:26 BST. --
Jul 14 12:09:59 faasd openfaas:cron-connector[10324]: 2021/07/14 11:09:59 Version: 63646e5f5c917160ea4185d17dcdfeca2511169d        Commit: 0.4.2
Jul 14 12:09:59 faasd openfaas:cron-connector[10324]: 2021/07/14 11:09:59 Gateway URL:  http://gateway:8080
Jul 14 12:09:59 faasd openfaas:cron-connector[10324]: 2021/07/14 11:09:59 Async Invocation:  false
Jul 14 12:10:09 faasd openfaas:cron-connector[10324]: 2021/07/14 11:10:09 added function nodeinfo in

Your Environment

Latest faasd version and cron-connector version ghcr.io/openfaas/cron-connector:latest running on multipass with the example from my eBook.

alexellis commented 3 years ago

Related to: #20