Add logic to dedupe kinds by ignoring "openshift" namespace in APIService.calculateAvailableKinds()

Author: benjaminapetersen

dist/origin-web-common-services.js

@@ -471,8 +471,10 @@ angular.module('openshiftCommonServices')
     var kinds = [];
     var rejectedKinds = Constants.AVAILABLE_KINDS_BLACKLIST;
 
-    // Legacy openshift and k8s kinds
-    _.each(API_CFG, function(api) {
+    // ignore the legacy openshift kinds, these have been migrated to api groups
+    _.each(_.pick(API_CFG, function(value, key) {
+      return key !== 'openshift';
+    }), function(api) {
       _.each(api.resources.v1, function(resource) {
         if (resource.namespaced || includeClusterScoped) {
           // Exclude subresources and any rejected kinds

dist/origin-web-common.js

@@ -1764,8 +1764,10 @@ angular.module('openshiftCommonServices')
     var kinds = [];
     var rejectedKinds = Constants.AVAILABLE_KINDS_BLACKLIST;
 
-    // Legacy openshift and k8s kinds
-    _.each(API_CFG, function(api) {
+    // ignore the legacy openshift kinds, these have been migrated to api groups
+    _.each(_.pick(API_CFG, function(value, key) {
+      return key !== 'openshift';
+    }), function(api) {
       _.each(api.resources.v1, function(resource) {
         if (resource.namespaced || includeClusterScoped) {
           // Exclude subresources and any rejected kinds

dist/origin-web-common.min.js

@@ -725,7 +725,9 @@ var kind = "<none>", version = "<none>";
 return apiObject && apiObject.kind && (kind = apiObject.kind), apiObject && apiObject.apiVersion && (version = apiObject.apiVersion), "The API version " + version + " for kind " + kind + " is not supported by this server";
 }, calculateAvailableKinds = function(includeClusterScoped) {
 var kinds = [], rejectedKinds = Constants.AVAILABLE_KINDS_BLACKLIST;
-return _.each(API_CFG, function(api) {
+return _.each(_.pick(API_CFG, function(value, key) {
+return "openshift" !== key;
+}), function(api) {
 _.each(api.resources.v1, function(resource) {
 if (resource.namespaced || includeClusterScoped) {
 if (resource.name.indexOf("/") >= 0 || _.contains(rejectedKinds, resource.kind)) return;

src/services/apiService.js

@@ -268,8 +268,10 @@ angular.module('openshiftCommonServices')
     var kinds = [];
     var rejectedKinds = Constants.AVAILABLE_KINDS_BLACKLIST;
 
-    // Legacy openshift and k8s kinds
-    _.each(API_CFG, function(api) {
+    // ignore the legacy openshift kinds, these have been migrated to api groups
+    _.each(_.pick(API_CFG, function(value, key) {
+      return key !== 'openshift';
+    }), function(api) {
       _.each(api.resources.v1, function(resource) {
         if (resource.namespaced || includeClusterScoped) {
           // Exclude subresources and any rejected kinds

test/.jshintrc

@@ -0,0 +1,54 @@
+{
+  "node": true,
+  "browser": true,
+  "esversion": 6,
+  "bitwise": true,
+  "camelcase": false,
+  "curly": true,
+  "eqeqeq": true,
+  "immed": true,
+  "indent": 2,
+  "latedef": true,
+  "newcap": false,
+  "noarg": true,
+  "quotmark": false,
+  "regexp": true,
+  "smarttabs": true,
+  "strict": true,
+  "sub" : true,
+  "undef": true,
+  "unused": true,
+  "expr" : true,
+  "globals": {
+    "ace": false,
+    "after": false,
+    "afterEach": false,
+    "afterAll": false,
+    "angular": false,
+    "by": false,
+    "before": false,
+    "beforeAll": false,
+    "beforeEach": false,
+    "browser": false,
+    "describe": false,
+    "element" : false,
+    "expect": false,
+    "inject": false,
+    "it": false,
+    "xit": false,
+    "jasmine": false,
+    "spyOn": false,
+    "hawtioPluginLoader": false,
+    "HawtioCore": false,
+    "Logger" : false,
+    "LabelSelector": false,
+    "moment": false,
+    "Messenger" : false,
+    "protractor" : false,
+    "URI": false,
+    "ZeroClipboard": false,
+    "$": false,
+    "_" : false
+
+  }
+}