apache / openwhisk-package-alarms

Apache OpenWhisk package that can be used to create periodic, time-based alarms.
https://openwhisk.apache.org/
Apache License 2.0
24 stars 49 forks source link

No alarm when openwhisk runs on different port #147

Closed fmaschler closed 6 years ago

fmaschler commented 6 years ago

When the openwhisk API runs on a different port than 443 it seems that periodic alarms don't work anymore. From the log you can see that the alarm action connects to the default port:

[ERROR] [??] [alarmsTrigger] [postTrigger] there was an error invoking xxxxxxxx/guest/test_alarm {"message":"connect ECONNREFUSED 172.17.0.1:443","stack":"Error: connect ECONNREFUSED 172.17.0.1:443\n at Object._errnoException (util.js:1022:11)\n at _exceptionWithHostPort (util.js:1044:20)\n at TCPConnectWrap.afterConnect [as oncomplete] (net.js:1198:14)","code":"ECONNREFUSED","errno":"ECONNREFUSED","syscall":"connect","address":"172.17.0.1","port":443}

To adjust the port on install I changed the following:

The installation finished without errors and except from the different port openwhisk runs in default local deployment. Firing a trigger manually works but the cron jobs don't. Any ideas which configuration is missing?

jasonpet commented 6 years ago

looks like you found a bug. it appears ":443" is being appended to the value of ROUTER_HOST. I will create a PR to fix this.

fmaschler commented 6 years ago

As said before I adjusted the ROUTER_HOST variable for the alarmtrigger container. When I attach to it the variable seems correct (172.17.0.1:4433). Let me know if I can provide any more details.

jasonpet commented 6 years ago

this issue was resolved by PR #148