Update membership filter to use MEMBERSHIP_WHITELIST in Constants.js

Author: benjaminapetersen

app/scripts/constants.js

@@ -150,6 +150,16 @@ angular.extend(window.OPENSHIFT_CONSTANTS, {
     {resource: 'services', group: ''},
     {resource: 'statefulsets', group: 'apps'}
   ],
+  MEMBERSHIP_WHITELIST: [
+    "admin",
+    "basic-user",
+    "edit",
+    "system:deployer",
+    "system:image-builder",
+    "system:image-puller",
+    "system:image-pusher",
+    "view",
+  ],
   // TODO:
   // This map can drive both the drawer & toast messages by
   // updating it to the following format:

app/scripts/services/membership/membership.js

@@ -2,7 +2,7 @@
 
 angular
   .module('openshiftConsole')
-  .factory('MembershipService', function($filter) {
+  .factory('MembershipService', function($filter, Constants) {
 
     var annotation = $filter('annotation');
 
@@ -118,13 +118,20 @@ angular
       return _.sortBy(roles, 'metadata.name');
     };
 
-    var filterRoles = function(roles) {
+    var filterRoles_PREV = function(roles) {
       return _.filter(roles, function(item) {
         // system-only must be explicitly <string> 'true' to hide the annotation
         return annotation(item, 'systemOnly') !== 'true';
       });
     };
 
+
+    var filterRoles = function(roles) {
+      return _.filter(roles, function(item) {
+        return _.includes(Constants.MEMBERSHIP_WHITELIST, item.metadata.name);
+      });
+    };
+
     var keyedRoles = function(roles) {
       return _.reduce(
               roles,

dist/scripts/scripts.js

@@ -712,6 +712,7 @@ group: ""
 resource: "statefulsets",
 group: "apps"
 } ],
+MEMBERSHIP_WHITELIST: [ "admin", "basic-user", "edit", "system:deployer", "system:image-builder", "system:image-puller", "system:image-pusher", "view" ],
 EVENTS_TO_SHOW: {
 FailedCreate: !0,
 FailedDelete: !0,
@@ -2343,8 +2344,9 @@ t[e.tag] = t[e.tag] || {}, t[e.tag].name = e.tag, t[e.tag].status = angular.copy
 }), t;
 }
 };
-}), angular.module("openshiftConsole").factory("MembershipService", [ "$filter", function(e) {
-var t = e("annotation"), n = function() {
+}), angular.module("openshiftConsole").factory("MembershipService", [ "$filter", "Constants", function(e, t) {
+e("annotation");
+var n = function() {
 return _.reduce(_.slice(arguments), function(e, t, n) {
 return t ? _.isEqual(n, 0) ? t : e + "-" + t : e;
 }, "");
@@ -2398,7 +2400,7 @@ return _.sortBy(e, "metadata.name");
 },
 filterRoles: function(e) {
 return _.filter(e, function(e) {
-return "true" !== t(e, "systemOnly");
+return _.includes(t.MEMBERSHIP_WHITELIST, e.metadata.name);
 });
 },
 mapRolesForUI: function(e, t) {