openwhisk-commits mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From berst...@apache.org
Subject [incubator-openwhisk-package-alarms] branch master updated: DB view should filter out disabled triggers (#84)
Date Mon, 14 Aug 2017 17:19:27 GMT
This is an automated email from the ASF dual-hosted git repository.

berstler pushed a commit to branch master
in repository https://gitbox.apache.org/repos/asf/incubator-openwhisk-package-alarms.git


The following commit(s) were added to refs/heads/master by this push:
     new 1450692  DB view should filter out disabled triggers (#84)
1450692 is described below

commit 14506927ce8a6b610a6cf7873622d597e59dc460
Author: Jason Peterson <jasonpet@us.ibm.com>
AuthorDate: Mon Aug 14 13:19:25 2017 -0400

    DB view should filter out disabled triggers (#84)
---
 provider/app.js       | 2 +-
 provider/lib/utils.js | 2 +-
 2 files changed, 2 insertions(+), 2 deletions(-)

diff --git a/provider/app.js b/provider/app.js
index 2746a54..3d90fd3 100755
--- a/provider/app.js
+++ b/provider/app.js
@@ -63,7 +63,7 @@ function createDatabase() {
                     views: {
                         triggers_by_worker: {
                             map: function (doc) {
-                                if (doc.maxTriggers) {
+                                if (doc.maxTriggers && (!doc.status || doc.status.active
=== true)) {
                                     emit(doc.worker || 'worker0', 1);
                                 }
                             }.toString(),
diff --git a/provider/lib/utils.js b/provider/lib/utils.js
index 02b3718..25935d0 100644
--- a/provider/lib/utils.js
+++ b/provider/lib/utils.js
@@ -225,7 +225,7 @@ module.exports = function(
                     var triggerIdentifier = trigger.id;
                     var doc = trigger.doc;
 
-                    if ((!doc.status || doc.status.active === true) && !(triggerIdentifier
in utils.triggers)) {
+                    if (!(triggerIdentifier in utils.triggers)) {
                         //check if trigger still exists in whisk db
                         var namespace = doc.namespace;
                         var name = doc.name;

-- 
To stop receiving notification emails like this one, please contact
['"commits@openwhisk.apache.org" <commits@openwhisk.apache.org>'].

Mime
View raw message