screwdriver-queue-service 3.0.6 → 3.0.8

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "screwdriver-queue-service",
-  "version": "3.0.6",
+  "version": "3.0.8",
   "description": "Screwdriver Queue Service API",
   "main": "app.js",
   "directories": {
@@ -28,7 +28,7 @@
     "npm-auto-version": "^1.0.0",
     "redlock": "^4.2.0",
     "screwdriver-aws-producer-service": "^2.0.0",
-    "screwdriver-data-schema": "^22.0.1",
+    "screwdriver-data-schema": "^23.0.4",
     "screwdriver-executor-docker": "^6.0.0",
     "screwdriver-executor-jenkins": "^6.0.0",
     "screwdriver-executor-k8s": "^15.0.0",

package/plugins/queue/scheduler.js

@@ -221,6 +221,9 @@ async function startPeriodic(executor, config) {
 
         // Note: arguments to enqueueAt are [timestamp, queue name, job name, array of args]
         let shouldRetry = false;
+        const nextDate = new Date(next);
+
+        logger.info(`Enqueued periodic job ${job.id} to be executed at ${nextDate}`);
 
         try {
             await executor.queue.enqueueAt(next, executor.periodicBuildQueue, 'startDelayed', [{ jobId: job.id }]);
@@ -229,9 +232,9 @@ async function startPeriodic(executor, config) {
             // eslint-disable-next-line max-len
             if (err && err.message !== 'Job already enqueued at this time with same arguments') {
                 shouldRetry = true;
-                logger.warn(`duplicate build: failed to enqueue for job ${job.id}: ${err}`);
-            } else {
                 logger.error(`failed to enqueue for job ${job.id}: ${err}`);
+            } else {
+                logger.warn(`duplicate build: failed to enqueue for job ${job.id}: ${err}`);
             }
         }
 

package/plugins/queue/utils/cron.js

@@ -107,7 +107,15 @@ const transformCron = (cronExp, jobId) => {
  * @return {Number}         Epoch timestamp (time of next execution).
  */
 const nextExecution = cronExp => {
-    const interval = parseExpression(cronExp);
+    // Scheduled jobs may run a little ahead of schedule.
+    // The next job to be executed is delayed so that it will be at the next timing even in that case.
+    const date = new Date();
+
+    date.setMinutes(date.getMinutes() + 5);
+    const options = {
+        currentDate: date
+    };
+    const interval = parseExpression(cronExp, options);
 
     return interval.next().getTime();
 };