Adopt more service catalog API changes

* Update to v1beta1 instead of v1alpha1
* Cluster prefix for service classes and plans
* ServiceInstanceCredential -> ServiceBinding
* Check removedFromBrokerCatalog

Bump origin-web-common to 0.0.66 and origin-web-catalog to 0.0.54

Author: spadgett

app/scripts/controllers/overview.js

@@ -325,7 +325,7 @@ function OverviewController($scope,
   };
 
   // Updated on viewBy changes to include the app label when appropriate.
-  var filterFields = ['metadata.name', 'spec.externalServiceClassName'];
+  var filterFields = ['metadata.name', 'spec.externalClusterServiceClassName'];
   var filterByName = function(items) {
     return KeywordService.filterForKeywords(items, filterFields, state.filterKeywords);
   };

app/scripts/controllers/serviceInstance.js

@@ -29,7 +29,7 @@ angular.module('openshiftConsole')
     var serviceInstanceDisplayName = $filter('serviceInstanceDisplayName');
 
     // API Versions
-    var serviceInstancesVersion = APIService.getPreferredVersion('serviceinstances');
+    $scope.serviceInstancesVersion = APIService.getPreferredVersion('serviceinstances');
 
     var updateBreadcrumbs = function() {
       $scope.breadcrumbs.push({
@@ -81,10 +81,10 @@ angular.module('openshiftConsole')
         $scope.projectContext = context;
 
         DataService
-          .get(serviceInstancesVersion, $routeParams.instance, context, { errorNotification: false })
+          .get($scope.serviceInstancesVersion, $routeParams.instance, context, { errorNotification: false })
           .then(function(serviceInstance) {
             serviceResolved(serviceInstance);
-            watches.push(DataService.watchObject(serviceInstancesVersion, $routeParams.instance, context, serviceResolved));
+            watches.push(DataService.watchObject($scope.serviceInstancesVersion, $routeParams.instance, context, serviceResolved));
           }, function(error) {
             $scope.loaded = true;
             $scope.alerts["load"] = {

app/scripts/directives/overview/listRow.js

@@ -44,6 +44,8 @@
     var isBinaryBuild = $filter('isBinaryBuild');
     var enableTechPreviewFeature = $filter('enableTechPreviewFeature');
 
+    row.serviceBindingsVersion = APIService.getPreferredVersion('servicebindings');
+
     var updateTriggers = function(apiObject) {
       var triggers = _.get(apiObject, 'spec.triggers');
       if (_.isEmpty(triggers)) {
@@ -174,13 +176,13 @@
         // Create Binding is displayed.
         if (enableTechPreviewFeature('pod_presets') &&
             !_.isEmpty(row.state.bindableServiceInstances) &&
-            canI({resource: 'serviceinstancecredentials', group: 'servicecatalog.k8s.io'}, 'create')) {
+            canI(row.serviceBindingsVersion, 'create')) {
           return true;
         }
         // Delete Binding is displayed.
         if (enableTechPreviewFeature('pod_presets') &&
             !_.isEmpty(deleteableBindings) &&
-            canI({resource: 'serviceinstancecredentials', group: 'servicecatalog.k8s.io'}, 'delete')) {
+            canI(row.serviceBindingsVersion, 'delete')) {
           return true;
         }
         // Check if one of the start build actions is displayed
@@ -210,13 +212,13 @@
         // Create Binding is displayed.
         if (enableTechPreviewFeature('pod_presets') &&
             !_.isEmpty(row.state.bindableServiceInstances) &&
-            canI({resource: 'serviceinstancecredentials', group: 'servicecatalog.k8s.io'}, 'create')) {
+            canI(row.serviceBindingsVersion, 'create')) {
           return true;
         }
         // Delete Binding is displayed.
         if (enableTechPreviewFeature('pod_presets') &&
             !_.isEmpty(deleteableBindings) &&
-            canI({resource: 'serviceinstancecredentials', group: 'servicecatalog.k8s.io'}, 'delete')) {
+            canI(row.serviceBindingsVersion, 'delete')) {
           return true;
         }
         return false;

app/scripts/directives/overview/serviceInstanceRow.js

@@ -4,6 +4,7 @@
   angular.module('openshiftConsole').component('serviceInstanceRow', {
     controller: [
       '$filter',
+      'APIService',
       'AuthorizationService',
       'BindingService',
       'ListRowUtils',
@@ -20,6 +21,7 @@
   });
 
   function ServiceInstanceRow($filter,
+                              APIService,
                               AuthorizationService,
                               BindingService,
                               ListRowUtils,
@@ -32,6 +34,9 @@
 
     var serviceInstanceDisplayName = $filter('serviceInstanceDisplayName');
 
+    row.serviceBindingsVersion = APIService.getPreferredVersion('servicebindings');
+    row.serviceInstancesVersion = APIService.getPreferredVersion('serviceinstances');
+
     var getServiceClass = function() {
       var serviceClassName = ServiceInstancesService.getServiceClassNameForInstance(row.apiObject);
       return _.get(row, ['state','serviceClasses', serviceClassName]);
@@ -81,15 +86,15 @@
       }
 
       // We can create bindings
-      if (row.isBindable && AuthorizationService.canI({resource: 'serviceinstancecredentials', group: 'servicecatalog.k8s.io'}, 'create')) {
+      if (row.isBindable && AuthorizationService.canI(row.serviceBindingsVersion, 'create')) {
         return true;
       }
       // We can delete bindings
-      if (!_.isEmpty(row.deleteableBindings) && AuthorizationService.canI({resource: 'serviceinstancecredentials', group: 'servicecatalog.k8s.io'}, 'delete')) {
+      if (!_.isEmpty(row.deleteableBindings) && AuthorizationService.canI(row.serviceBindingsVersion, 'delete')) {
         return true;
       }
       // We can delete instances
-      if (AuthorizationService.canI({resource: 'serviceinstances', group: 'servicecatalog.k8s.io'}, 'delete')) {
+      if (AuthorizationService.canI(row.serviceInstancesVersion, 'delete')) {
         return true;
       }
 

app/scripts/directives/serviceBinding.js

@@ -3,6 +3,8 @@
 (function() {
   angular.module('openshiftConsole').component('serviceBinding', {
     controller: [
+      "APIService",
+      "ServiceInstancesService",
       ServiceBinding
     ],
     controllerAs: '$ctrl',
@@ -17,14 +19,16 @@
     templateUrl: 'views/directives/_service-binding.html'
   });
 
-  function ServiceBinding() {
+  function ServiceBinding(APIService,
+                          ServiceInstancesService) {
     var ctrl = this;
+    ctrl.serviceBindingsVersion = APIService.getPreferredVersion('servicebindings');
 
     var updateServiceClass = function() {
       if (_.get(ctrl.refApiObject, 'kind') !== 'ServiceInstance') {
         var instanceName = _.get(ctrl.binding, 'spec.instanceRef.name');
         var instance = _.get(ctrl.serviceInstances, [instanceName]);
-        var serviceClassName = _.get(instance, 'spec.serviceClassRef.name');
+        var serviceClassName = ServiceInstancesService.getServiceClassNameForInstance(instance);
         ctrl.serviceClass = _.get(ctrl.serviceClasses, [serviceClassName]);
       }
     };

app/scripts/filters/canI.js

@@ -34,7 +34,7 @@ angular
         _.assign({}, APIService.getPreferredVersion('imagestreams'), {verbs: ['update', 'delete']})
       ],
       'serviceInstances': [
-        {group: 'servicecatalog.k8s.io', resource: 'serviceinstances', verbs: ['update', 'delete']}
+        _.assign({}, APIService.getPreferredVersion('serviceinstances'), {verbs: ['update', 'delete']})
       ],
       'persistentVolumeClaims': [
         {group: '', resource: 'persistentvolumeclaims', verbs: ['update', 'delete']}

app/scripts/filters/resources.js

@@ -1316,45 +1316,6 @@ angular.module('openshiftConsole')
       return !_.isEmpty(alternateBackends);
     };
   })
-  .filter('serviceClassDisplayName', function() {
-    return function(serviceClass) {
-      var serviceClassDisplayName = _.get(serviceClass, 'spec.externalMetadata.displayName');
-      if (serviceClassDisplayName) {
-        return serviceClassDisplayName;
-      }
-
-      var serviceClassExternalName = _.get(serviceClass, 'spec.externalName');
-      if (serviceClassExternalName) {
-        return serviceClassExternalName;
-      }
-
-      return _.get(serviceClass, 'metadata.name');
-    };
-  })
-  .filter('serviceInstanceDisplayName', function(serviceClassDisplayNameFilter) {
-    return function(instance, serviceClass) {
-      if (serviceClass) {
-        return serviceClassDisplayNameFilter(serviceClass);
-      }
-
-      return _.get(instance, 'metadata.name');
-    };
-  })
-  .filter('serviceInstanceStatus', function(isServiceInstanceReadyFilter) {
-    return function(instance) {
-      var status = 'Pending';
-      var conditions = _.get(instance, 'status.conditions');
-      var instanceError = _.find(conditions, {type: 'Failed', status: 'True'});
-
-      if (instanceError) {
-        status = 'Failed';
-      } else if (isServiceInstanceReadyFilter(instance)) {
-        status = 'Ready';
-      }
-
-      return status;
-    };
-  })
   .filter('readyConditionMessage', function(statusConditionFilter) {
     return function(instance) {
       return _.get(statusConditionFilter(instance, 'Ready'), 'message');

app/scripts/services/serviceInstances.js

@@ -14,7 +14,7 @@ angular.module("openshiftConsole")
     var servicePlansVersion = APIService.getPreferredVersion('clusterserviceplans');
 
     var getServiceClassNameForInstance = function(serviceInstance) {
-      return _.get(serviceInstance, 'spec.serviceClassRef.name');
+      return _.get(serviceInstance, 'spec.clusterServiceClassRef.name');
     };
 
     var fetchServiceClassForInstance = function(serviceInstance) {
@@ -23,7 +23,7 @@ angular.module("openshiftConsole")
     };
 
     var getServicePlanNameForInstance = function(serviceInstance) {
-      return _.get(serviceInstance, 'spec.servicePlanRef.name');
+      return _.get(serviceInstance, 'spec.clusterServicePlanRef.name');
     };
 
     var fetchServicePlanForInstance = function(serviceInstance) {

app/views/browse/service-instance.html

@@ -20,10 +20,10 @@ <h1 class="contains-actions">
                    class="dropdown-toggle actions-dropdown-kebab visible-xs-inline"
                    data-toggle="dropdown"><i class="fa fa-ellipsis-v" aria-hidden="true"></i><span class="sr-only">Actions</span></a>
                 <ul class="dropdown-menu dropdown-menu-right actions action-button">
-                  <li ng-if="{resource: 'serviceinstances', group: 'servicecatalog.k8s.io'} | canI : 'update'">
+                  <li ng-if="serviceInstancesVersion | canI : 'update'">
                     <a ng-href="{{serviceInstance | editYamlURL}}" role="button">Edit YAML</a>
                   </li>
-                  <li ng-if="{resource: 'serviceinstances', group: 'servicecatalog.k8s.io'} | canI : 'delete'">
+                  <li ng-if="serviceInstancesVersion | canI : 'delete'">
                     <a href="" ng-click="deprovision()" role="button">Delete</a>
                   </li>
                 </ul>

app/views/directives/_service-binding.html

@@ -35,8 +35,8 @@ <h3>
   </div>
   <div class="service-binding-actions" ng-if="!$ctrl.binding.metadata.deletionTimestamp">
     <delete-link
-      ng-if="({resource: 'serviceinstancecredentials', group: 'servicecatalog.k8s.io'} | canI : 'delete')"
-      kind="ServiceInstanceCredential"
+      ng-if="$ctrl.serviceBindingsVersion | canI : 'delete'"
+      kind="ServiceBinding"
       group="servicecatalog.k8s.io"
       type-display-name="binding"
       resource-name="{{$ctrl.binding.metadata.name}}"