From c7178034be928bb9ab713687eb3b9844a4a38d1b Mon Sep 17 00:00:00 2001
From: =?UTF-8?q?Stephan=20K=C3=B6ninger?= <stephan.koeninger@codecentric.de>
Date: Fri, 1 Apr 2022 11:29:46 +0200
Subject: [PATCH] feat(#2009): Update properties through environment manager to
 all instances (#2010)

closes #2009
---
 .../spring-boot-admin-sample-servlet/pom.xml  |  4 +++
 .../src/main/resources/application.yml        |  4 ++-
 .../src/main/frontend/services/application.js |  6 +++++
 .../views/instances/env/env-manager.vue       | 26 ++++++++++++++++---
 .../frontend/views/instances/env/index.vue    | 15 ++++++-----
 5 files changed, 44 insertions(+), 11 deletions(-)

diff --git a/spring-boot-admin-samples/spring-boot-admin-sample-servlet/pom.xml b/spring-boot-admin-samples/spring-boot-admin-sample-servlet/pom.xml
index ae3e00443fc..19bb3f37ac2 100644
--- a/spring-boot-admin-samples/spring-boot-admin-sample-servlet/pom.xml
+++ b/spring-boot-admin-samples/spring-boot-admin-sample-servlet/pom.xml
@@ -52,6 +52,10 @@
             <groupId>de.codecentric</groupId>
             <artifactId>spring-boot-admin-starter-client</artifactId>
         </dependency>
+        <dependency>
+            <groupId>org.springframework.cloud</groupId>
+            <artifactId>spring-cloud-starter</artifactId>
+        </dependency>
         <dependency>
             <groupId>org.springframework.session</groupId>
             <artifactId>spring-session-core</artifactId>
diff --git a/spring-boot-admin-samples/spring-boot-admin-sample-servlet/src/main/resources/application.yml b/spring-boot-admin-samples/spring-boot-admin-sample-servlet/src/main/resources/application.yml
index 199158d72f5..5c28e87810e 100644
--- a/spring-boot-admin-samples/spring-boot-admin-sample-servlet/src/main/resources/application.yml
+++ b/spring-boot-admin-samples/spring-boot-admin-sample-servlet/src/main/resources/application.yml
@@ -15,10 +15,12 @@ management:
       exposure:
         include: "*"
   endpoint:
+    env:
+      post:
+        enabled: true
     health:
       show-details: ALWAYS
 
-
 spring:
   application:
     name: spring-boot-admin-sample-servlet
diff --git a/spring-boot-admin-server-ui/src/main/frontend/services/application.js b/spring-boot-admin-server-ui/src/main/frontend/services/application.js
index 555152ffb43..89a901ff830 100644
--- a/spring-boot-admin-server-ui/src/main/frontend/services/application.js
+++ b/spring-boot-admin-server-ui/src/main/frontend/services/application.js
@@ -112,6 +112,12 @@ class Application {
     return {responses};
   }
 
+  async setEnv(name, value) {
+    return this.axios.post(uri`actuator/env`, {name, value}, {
+      headers: {'Content-Type': 'application/json'}
+    });
+  }
+
   async refreshContext() {
     return this.axios.post(uri`actuator/refresh`);
   }
diff --git a/spring-boot-admin-server-ui/src/main/frontend/views/instances/env/env-manager.vue b/spring-boot-admin-server-ui/src/main/frontend/views/instances/env/env-manager.vue
index 722259778ef..b5a0c07fe57 100644
--- a/spring-boot-admin-server-ui/src/main/frontend/views/instances/env/env-manager.vue
+++ b/spring-boot-admin-server-ui/src/main/frontend/views/instances/env/env-manager.vue
@@ -63,6 +63,9 @@
               <span v-else v-text="$t('instances.env.context_reset')" />
             </button>
           </div>
+          <div class="control" v-if="application.instances.length > 1">
+            <sba-toggle-scope-button :instance-count="application.instances.length" v-model="scope" />
+          </div>
           <div class="control">
             <button class="button is-primary"
                     :class="{'is-loading' : updateStatus === 'executing', 'is-danger' : updateStatus === 'failed', 'is-success' : updateStatus === 'completed'}"
@@ -85,10 +88,16 @@ import Instance from '@/services/instance';
 import {concatMap, filter, from, listen} from '@/utils/rxjs';
 import debounce from 'lodash/debounce';
 import uniq from 'lodash/uniq';
-
+import Application from '@/services/application';
+import SbaToggleScopeButton from '@/components/sba-toggle-scope-button';
 
 export default {
+  components: {SbaToggleScopeButton},
   props: {
+    application: {
+      type: Application,
+      required: true
+    },
     instance: {
       type: Instance,
       required: true
@@ -100,6 +109,7 @@ export default {
   },
   data: () => ({
     error: null,
+    scope: 'instance',
     resetStatus: null,
     updateStatus: null,
     managedProperties: [{
@@ -148,8 +158,18 @@ export default {
           filter(property => !!property.name && property.input !== property.value),
           listen(status => vm.updateStatus = status),
           concatMap(
-            property => from(vm.instance.setEnv(property.name, property.input))
-              .pipe(listen(status => property.status = status))
+            property => {
+              let target;
+
+              if (vm.scope === 'instance') {
+                target = vm.instance;
+              } else {
+                target = vm.application;
+              }
+
+              return from(target.setEnv(property.name, property.input))
+                .pipe(listen(status => property.status = status));
+            }
           )
         )
         .subscribe({
diff --git a/spring-boot-admin-server-ui/src/main/frontend/views/instances/env/index.vue b/spring-boot-admin-server-ui/src/main/frontend/views/instances/env/index.vue
index b3e2001df7f..d609b6d4b70 100644
--- a/spring-boot-admin-server-ui/src/main/frontend/views/instances/env/index.vue
+++ b/spring-boot-admin-server-ui/src/main/frontend/views/instances/env/index.vue
@@ -33,6 +33,7 @@
              @reset="fetchEnv"
     />
     <sba-env-manager v-if="env && hasEnvManagerSupport"
+                     :application="application"
                      :instance="instance" :property-sources="env.propertySources"
                      @refresh="fetchEnv" @update="fetchEnv"
     />
@@ -73,8 +74,8 @@ import Instance from '@/services/instance';
 import pickBy from 'lodash/pickBy';
 import {VIEW_GROUP} from '../../index';
 import sbaEnvManager from './env-manager';
-  import refresh from './refresh';
-  import Application from '@/services/application';
+import refresh from './refresh';
+import Application from '@/services/application';
 
 const filterProperty = (needle) => (property, name) => {
   return name.toString().toLowerCase().includes(needle) || (property.value && property.value.toString().toLowerCase().includes(needle));
@@ -95,13 +96,13 @@ export default {
     instance: {
       type: Instance,
       required: true
-      },
-      application: {
-        type: Application,
-        required: true
+    },
+    application: {
+      type: Application,
+      required: true
     }
   },
-    components: {sbaEnvManager, refresh},
+  components: {sbaEnvManager, refresh},
   data: () => ({
     hasLoaded: false,
     error: null,