Check services for bindability before creating bindings

openshift · Jun 7, 2017 · b8faec6 · b8faec6
1 parent ad1da6b
commit b8faec6
Show file tree

Hide file tree

Showing 10 changed files with 346 additions and 349 deletions.
diff --git a/app/scripts/controllers/overview.js b/app/scripts/controllers/overview.js
@@ -23,6 +23,7 @@ angular.module('openshiftConsole').controller('OverviewController', [
   'OwnerReferencesService',
   'PodsService',
   'ProjectsService',
+  'BindingService',
   'ResourceAlertsService',
   'RoutesService',
   OverviewController
@@ -50,6 +51,7 @@ function OverviewController($scope,
                             OwnerReferencesService,
                             PodsService,
                             ProjectsService,
+                            BindingService,
                             ResourceAlertsService,
                             RoutesService) {
   var overview = this;
@@ -1132,9 +1134,16 @@ function OverviewController($scope,
   // extract & share
   var sortServiceInstances = function() {
     if(!state.serviceInstances && !state.serviceClasses) {
+      state.bindableServiceInstances = null;
       return;
     }
-    state.orderedServiceInstances = _.toArray(state.serviceInstances).sort(function(left, right) {
+
+    var serviceInstancesArray = _.toArray(state.serviceInstances);
+    state.bindableServiceInstances = _.filter(serviceInstancesArray, function(serviceInstance) {
+      return BindingService.isServiceBindable(serviceInstance, state.serviceClasses);
+    });
+
+    state.orderedServiceInstances = serviceInstancesArray.sort(function(left, right) {
       var leftName = _.get(state.serviceClasses, [left.spec.serviceClassName, 'osbMetadata', 'displayName']) || left.spec.serviceClassName;
       var rightName = _.get(state.serviceClasses, [left.spec.serviceClassName, 'osbMetadata', 'displayName']) || right.spec.serviceClassName;
 

diff --git a/app/scripts/directives/bindService.js b/app/scripts/directives/bindService.js
@@ -1,6 +1,6 @@
 'use strict';
 
-(function() {  
+(function() {
   angular.module('openshiftConsole').component('bindService', {
     controller: [
       '$scope',
@@ -72,6 +72,7 @@
                                           .concat(replicaSets)
                                           .concat(statefulSets);
         ctrl.applications = _.sortByAll(apiObjects, ['metadata.name', 'kind']);
+        ctrl.bindType = ctrl.applications.length ? "application" : "secret-only";
       }
     };
 
@@ -133,7 +134,7 @@
 
       // TODO is it ever realistically possible for target to not be defined at this point
       if (ctrl.target.kind === 'Instance') {
-        ctrl.shouldBindToApp = "true";
+        ctrl.bindType = "secret-only";
         ctrl.appToBind = null;
         ctrl.serviceToBind = ctrl.target.metadata.name;
         // Load all the "application" types
@@ -168,6 +169,8 @@
         });
       }
       else {
+        ctrl.bindType = 'application';
+        ctrl.appToBind = ctrl.target;
         DataService.list({
           group: 'servicecatalog.k8s.io',
           resource: 'instances'
@@ -195,13 +198,13 @@
 
     ctrl.bindService = function() {
       var svcToBind = ctrl.target.kind === 'Instance' ? ctrl.target : ctrl.serviceInstances[ctrl.serviceToBind];
-      var appToBind = ctrl.target.kind !== 'Instance' ? ctrl.target : ctrl.appToBind;
+      var application = ctrl.bindType === 'application' ? _.get(ctrl.appToBind, 'metadata.name') : undefined;
 
       var context = {
         namespace: _.get(svcToBind, 'metadata.namespace')
       };
 
-      BindingService.bindService(context, _.get(svcToBind, 'metadata.name'), _.get(appToBind, 'metadata.name')).then(function(binding){
+      BindingService.bindService(context, _.get(svcToBind, 'metadata.name'), application).then(function(binding){
         ctrl.binding = binding;
         ctrl.error = null;
 

diff --git a/app/scripts/directives/overview/serviceInstanceRow.js b/app/scripts/directives/overview/serviceInstanceRow.js
@@ -1,11 +1,12 @@
 'use strict';
 
-(function() {  
+(function() {
   angular.module('openshiftConsole').component('serviceInstanceRow', {
     controller: [
       '$filter',
       '$uibModal',
       'DataService',
+      'BindingService',
       'ListRowUtils',
       'NotificationsService',
       ServiceInstanceRow
@@ -22,6 +23,7 @@
   function ServiceInstanceRow($filter,
                               $uibModal,
                               DataService,
+                              BindingService,
                               ListRowUtils,
                               NotificationsService) {
     var row = this;
@@ -51,6 +53,8 @@
       return binding && _.get(row, ['state', 'secrets', binding.spec.secretName]);
     };
 
+    row.isBindable = BindingService.isServiceBindable(row.apiObject, row.state.serviceClasses);
+
     row.closeOverlayPanel = function() {
       _.set(row, 'overlay.panelVisible', false);
     };

diff --git a/app/scripts/filters/resources.js b/app/scripts/filters/resources.js
@@ -1310,17 +1310,4 @@ angular.module('openshiftConsole')
       var alternateBackends = _.get(route, 'spec.alternateBackends', []);
       return !_.isEmpty(alternateBackends);
     };
-  })
-  // TODO: these two filters are defined in origin-web-catalog but are not available, need to figure out why or move to common
-  .filter('applicationHasDeployment', function ($filter) {
-    var annotation = $filter('annotation');
-    return function (replicaSet) {
-      return annotation(replicaSet, 'deployment.kubernetes.io/revision');
-    };
-  })
-  .filter('applicationHasDeploymentConfig', function($filter) {
-    var annotationFilter = $filter('annotation');
-    return function (deployment) {
-      return annotationFilter(deployment, 'deploymentConfig');
-    };
   });
diff --git a/app/views/directives/bind-service/bind-service-form.html b/app/views/directives/bind-service/bind-service-form.html
@@ -12,8 +12,7 @@
                      service-class-name="ctrl.serviceClassName"
                      form-name="ctrl.selectionForm"
                      applications="ctrl.applications"
-                     app-to-bind="ctrl.appToBind"
-                     should-bind-to-app="ctrl.shouldBindToApp"
-                     group-by-kind="ctrl.groupByKind">
+                     bind-type="ctrl.bindType"
+                     app-to-bind="ctrl.appToBind">
   </bind-service-form>
 </div>
diff --git a/app/views/directives/bind-service/results.html b/app/views/directives/bind-service/results.html
@@ -1,6 +1,7 @@
 <bind-results error="ctrl.error"
               binding="ctrl.binding"
               service-to-bind="ctrl.serviceToBind"
+              bind-type="{{ctrl.bindType}}"
               application-to-bind="ctrl.appToBind.metadata.name"
               generated-secret-name="ctrl.generatedSecretName"
               show-pod-presets="'pod_presets' | enableTechPreviewFeature"

diff --git a/app/views/overview/_list-row-actions.html b/app/views/overview/_list-row-actions.html
@@ -25,7 +25,7 @@
             </li>
             <!-- FIXME: Can't enable canI checks on svc cat resources until we have aggregation
             <li ng-if="(row.state.serviceInstances | hashSize) > 0 && {resource: 'bindings', group: 'servicecatalog.k8s.io'} | canI : 'create'" role="menuitem"> -->
-            <li ng-if="('pod_presets' | enableTechPreviewFeature) && (row.state.serviceInstances | hashSize) > 0" role="menuitem">
+            <li ng-if="('pod_presets' | enableTechPreviewFeature) && row.state.bindableServiceInstances.length" role="menuitem">
               <a href="" ng-click="row.showOverlayPanel('bindService', {target: row.apiObject})">Create Binding</a>
             </li>
             <li ng-if="row.current && ('deploymentconfigs/log' | canI : 'get')" role="menuitem">

diff --git a/app/views/overview/_service-instance-row.html b/app/views/overview/_service-instance-row.html
@@ -16,7 +16,7 @@ <h3>
       <div class="list-pf-details">
         <div ng-if="!row.expanded">
           <div class="hidden-xs hidden-sm">
-            <span ng-if="!row.bindings.length">
+            <span ng-if="!row.bindings.length && row.isBindable">
               <a href="" ng-click="row.showOverlayPanel('bindService', {target: row.apiObject})">Create Binding</a>
             </span>
             <span ng-if="row.bindings.length" class="component-label">Bindings</span>
@@ -44,7 +44,7 @@ <h3>
         uib-dropdown-toggle
         class="actions-dropdown-kebab"><i class="fa fa-ellipsis-v"></i><span class="sr-only">Actions</span></a>
         <ul class="dropdown-menu dropdown-menu-right" uib-dropdown-menu role="menu">
-          <li role="menuitem">
+          <li role="menuitem" ng-if="row.isBindable">
             <a href="" ng-click="row.showOverlayPanel('bindService', {target: row.apiObject})">Create Binding</a>
           </li>
           <li role="menuitem">
@@ -84,7 +84,7 @@ <h3>
             <p class="pre-wrap" ng-bind-html="row.description | linky"></p>
           </div>
         </div>
-        <div class="section-title">
+        <div class="section-title" ng-if="row.isBindable || row.bindings">
           Bindings
         </div>
         <div
@@ -108,7 +108,7 @@ <h3>
             </a>
           </div>
         </div>
-        <div class="row">
+        <div class="row" ng-if="row.isBindable">
           <div class="col-sm-12">
             <a href="" ng-click="row.showOverlayPanel('bindService', {target: row.apiObject})">Create Binding</a>
           </div>