[GitHub] [openwhisk-package-alarms] upgle opened a new pull request #223: Bump up cron to 1.8.2

[GitBox <gi...@apache.org>]

upgle opened a new pull request #223:
URL: https://github.com/apache/openwhisk-package-alarms/pull/223


   ## Description
   The cron expression `5/10 * * * *` doesn't work in the alarm package using the cron library version 1.7.0. 
   I expect it to run at 5, 15, 25, 35, 45, and 55 minutes every hour, but it only runs once at 5 minutes.
   
   This can be solved by upgrading the latest version of the cron library.
   
   - cron@ 1.7.0 (current version)
   - cron@ 1.8.2 (latest)
   
   ### Sample code
   ```javascript
   var CronJob = require('cron').CronJob;
   var job = new CronJob('0 5/10 * * * *', function() {
     console.log('You will see this message every second', new Date());
   }, null, true, 'America/Los_Angeles');
   job.start();
   ```
   
   And also I confirmed that the alarms package using the latest cron library works well in our cluster.


----------------------------------------------------------------
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

For queries about this service, please contact Infrastructure at:
users@infra.apache.org

[GitHub] [openwhisk-package-alarms] dgrove-oss merged pull request #223: Bump up cron to 1.8.2

[GitBox <gi...@apache.org>]

dgrove-oss merged pull request #223:
URL: https://github.com/apache/openwhisk-package-alarms/pull/223


   


----------------------------------------------------------------
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

For queries about this service, please contact Infrastructure at:
users@infra.apache.org