Allow the user to select containers when adding a secret to an application

Fixes openshift#2024

Author: jeff-phillips-18

app/scripts/directives/addSecretToApplication.js

@@ -87,13 +87,18 @@
       getApplications();
     };
 
+    var isContainerSelected = function(container) {
+      return ctrl.attachAllContainers || ctrl.attachContainers[container.name];
+    };
+
     ctrl.$postLink = function() {
       $scope.$watch(function() {
         return ctrl.application;
       }, function() {
         // Look at the existing mount paths so that we can warn if the new value is not unique.
         var podTemplate = _.get(ctrl.application, 'spec.template');
         ctrl.existingMountPaths = StorageService.getMountPaths(podTemplate);
+        ctrl.attachAllContainers = true;
       });
     };
 
@@ -113,8 +118,10 @@
 
         // For each container, add the new volume mount.
         _.each(podTemplate.spec.containers, function(container) {
-          container.envFrom = container.envFrom || [];
-          container.envFrom.push(newEnvFrom);
+          if (isContainerSelected(container)) {
+            container.envFrom = container.envFrom || [];
+            container.envFrom.push(newEnvFrom);
+          }
         });
       } else {
         var generateName = $filter('generateName');
@@ -127,8 +134,10 @@
 
         // For each selected container, add the new volume mount.
         _.each(podTemplate.spec.containers, function(container) {
-          container.volumeMounts = container.volumeMounts || [];
-          container.volumeMounts.push(newVolumeMount);
+          if (isContainerSelected(container)) {
+            container.volumeMounts = container.volumeMounts || [];
+            container.volumeMounts.push(newVolumeMount);
+          }
         });
 
         var newVolume = {

app/styles/_secrets.less

@@ -3,6 +3,17 @@
     max-width: 600px;
   }
 
+  .container-options {
+    margin-left: 20px;
+    .select-container {
+      max-height: 200px;
+      overflow-y: auto;
+      .checkbox:first-of-type {
+        margin-top: 0;
+      }
+    }
+  }
+
   .dialog-title {
     border-bottom: 1px solid @color-pf-black-300;
 

app/views/directives/add-secret-to-application.html

@@ -64,6 +64,23 @@ <h3>Add to Application</h3>
             </div>
           </div>
         </div>
+        <legend ng-if-start="ctrl.application.spec.template.spec.containers.length > 1">Containers:</legend>
+        <div ng-if-end class="form-group container-options">
+          <div ng-if="ctrl.attachAllContainers">
+            The secret will be added to all containers. You can
+            <a href="" ng-click="ctrl.attachAllContainers = false">select specific containers</a>
+            instead.
+          </div>
+          <div ng-if="!ctrl.attachAllContainers" class="form-group">
+            <label class="sr-only required">Containers</label>
+            <select-containers
+              ng-model="ctrl.attachContainers"
+              pod-template="ctrl.application.spec.template"
+              ng-required="true"
+              help-text="Add the secret to the selected containers.">
+            </select-containers>
+          </div>
+        </div>
         <div class="button-group pull-right">
           <button
             class="btn btn-default"

dist/scripts/scripts.js

@@ -21395,12 +21395,24 @@ d = _.toArray(e.by("metadata.name")), p();
 };
 m.$onInit = function() {
 m.addType = "env", m.disableInputs = !1, f();
+<<<<<<< a8e13079948a1153dd52ad0ff49228eb68945806
 }, m.$postLink = function() {
+=======
+var e = new RegExp("^[A-Za-z_]{1}[A-Za-z0-9_]*$");
+m.hasInvalidEnvVars = _.some(m.secret.data, function(t, n) {
+return !e.test(n);
+});
+};
+var g = function(e) {
+return m.attachAllContainers || m.attachContainers[e.name];
+};
+m.$postLink = function() {
+>>>>>>> Allow the user to select containers when adding a secret to an application
 t.$watch(function() {
 return m.application;
 }, function() {
 var e = _.get(m.application, "spec.template");
-m.existingMountPaths = i.getMountPaths(e);
+m.existingMountPaths = i.getMountPaths(e), m.attachAllContainers = !0;
 });
 }, m.addToApplication = function() {
 var t = angular.copy(m.application), i = _.get(t, "spec.template");
@@ -21411,7 +21423,7 @@ name: m.secret.metadata.name
 }
 };
 _.each(i.spec.containers, function(e) {
-e.envFrom = e.envFrom || [], e.envFrom.push(s);
+g(e) && (e.envFrom = e.envFrom || [], e.envFrom.push(s));
 });
 } else {
 var c = e("generateName")(m.secret.metadata.name + "-"), l = {
@@ -21420,7 +21432,7 @@ mountPath: m.mountVolume,
 readOnly: !0
 };
 _.each(i.spec.containers, function(e) {
-e.volumeMounts = e.volumeMounts || [], e.volumeMounts.push(l);
+g(e) && (e.volumeMounts = e.volumeMounts || [], e.volumeMounts.push(l));
 });
 var u = {
 name: c,
@@ -21430,10 +21442,10 @@ secretName: m.secret.metadata.name
 };
 i.spec.volumes = i.spec.volumes || [], i.spec.volumes.push(u);
 }
-var d = e("humanizeKind"), p = d(m.secret.kind), f = d(t.kind), g = {
+var d = e("humanizeKind"), p = d(m.secret.kind), f = d(t.kind), h = {
 namespace: m.project.metadata.name
 };
-a.update(n.kindToResource(t.kind), t.metadata.name, t, g).then(function() {
+a.update(n.kindToResource(t.kind), t.metadata.name, t, h).then(function() {
 o.addNotification({
 type: "success",
 message: "Successfully added " + p + " " + m.secret.metadata.name + " to " + f + " " + t.metadata.name + ".",

dist/scripts/templates.js

@@ -6518,6 +6518,19 @@ angular.module('openshiftConsoleTemplates', []).run(['$templateCache', function(
     "</div>\n" +
     "</div>\n" +
     "</div>\n" +
+    "<legend ng-if-start=\"ctrl.application.spec.template.spec.containers.length > 1\">Containers:</legend>\n" +
+    "<div ng-if-end class=\"form-group container-options\">\n" +
+    "<div ng-if=\"ctrl.attachAllContainers\">\n" +
+    "The secret will be added to all containers. You can\n" +
+    "<a href=\"\" ng-click=\"ctrl.attachAllContainers = false\">select specific containers</a>\n" +
+    "instead.\n" +
+    "</div>\n" +
+    "<div ng-if=\"!ctrl.attachAllContainers\" class=\"form-group\">\n" +
+    "<label class=\"sr-only required\">Containers</label>\n" +
+    "<select-containers ng-model=\"ctrl.attachContainers\" pod-template=\"ctrl.application.spec.template\" ng-required=\"true\" help-text=\"Add the secret to the selected containers.\">\n" +
+    "</select-containers>\n" +
+    "</div>\n" +
+    "</div>\n" +
     "<div class=\"button-group pull-right\">\n" +
 <<<<<<< 579cbe4ce2e640c919f9687b5c885f661897db96
     "<button class=\"btn btn-default\" ng-class=\"{'dialog-btn': isDialog}\" ng-click=\"ctrl.onCancel()\" translate>\n" +