Added 30 secs delay for Updating Network IPMI (out-of-band IPMI) protocol (#127)

* added 30 secs delay for network protocol

* updated success message

* added 30 secs delay for ssh protocol

* reverted the changes for ssh BMC shell protocol

Author: Renuka9527

src/locales/en-US.json

@@ -1274,8 +1274,9 @@
     "toast": {
       "errorNetworkPolicyUpdate": "Error updating %{policy}.",
       "successDisableBmcShell": "Successfully disabled BMC shell (via SSH), Please allow 15 seconds for the disable to become effective.",
-      "successEnableBmcShell": "Successfully enabled BMC shell (via SSH)",
+      "successEnableBmcShell": "Successfully enabled BMC shell (via SSH).",
       "successNetworkPolicyUpdate": "Successfully updated %{policy}.",
+      "successIpmiNetworkPolicyUpdate": "Successfully updated %{policy}. The BMC normally takes about 30 seconds to change the state.",
       "successNextBootToast": "Applying changes to %{policy}. Changes made to %{policy} will take effect on next reboot."
     },
     "acfUploadEnablementConfirmText": "Caution: Enabling this setting allows unauthenticated users to upload Access Control Files (ACFs) without logging in and without pressing Function 74 on the Operator Panel. Do you want to proceed?",

src/store/modules/SecurityAndAccess/PoliciesStore.js

@@ -189,7 +189,12 @@ const PoliciesStore = {
           );
         });
     },
-    async saveIpmiProtocolState({ commit }, protocolEnabled) {
+    async getNetworkProtocolStatusAfterDelay({ dispatch }) {
+      setTimeout(() => {
+        dispatch('getNetworkProtocolStatus');
+      }, 30000);
+    },
+    async saveIpmiProtocolState({ commit, dispatch }, protocolEnabled) {
       commit('setIpmiProtocolEnabled', protocolEnabled);
       const ipmi = {
         IPMI: {
@@ -199,7 +204,11 @@ const PoliciesStore = {
       return await api
         .patch('/redfish/v1/Managers/bmc/NetworkProtocol', ipmi)
         .then(() => {
-          return i18n.t('pagePolicies.toast.successNetworkPolicyUpdate', {
+          // Getting protocol data here so that the Ipmi gets enabled/disabled
+          dispatch('getNetworkProtocolStatusAfterDelay');
+        })
+        .then(() => {
+          return i18n.t('pagePolicies.toast.successIpmiNetworkPolicyUpdate', {
             policy: i18n.t('pagePolicies.ipmi'),
           });
         })

src/views/SecurityAndAccess/Policies/Policies.vue

@@ -349,7 +349,9 @@ export default {
     this.startLoader();
     Promise.all([
       this.$store.dispatch('policies/getBiosStatus'),
-      this.$store.dispatch('policies/getNetworkProtocolStatus'),
+      setTimeout(() => {
+        this.$store.dispatch('policies/getNetworkProtocolStatus');
+      }, 30000),
       this.$store.dispatch('policies/getUsbFirmwareUpdatePolicyEnabled'),
       this.$store.dispatch('policies/getUnauthenticatedACFUploadEnablement'),
       this.$store.dispatch('policies/getTpmPolicy'),
@@ -359,7 +361,9 @@ export default {
       this.unauthenticatedACFUploadEnablementState = this.$store.getters[
         'policies/acfUploadEnablement'
       ];
-      this.endLoader();
+      setTimeout(() => {
+        this.endLoader();
+      }, 30000);
     });
   },
   methods: {
@@ -378,13 +382,21 @@ export default {
     changeIpmiProtocolState(state) {
       this.$store
         .dispatch('policies/saveIpmiProtocolState', state)
-        .then((message) => this.successToast(message))
+        .then((message) => {
+          this.startLoader();
+          setTimeout(() => {
+            this.endLoader();
+          }, 30000);
+          this.successToast(message);
+        })
         .catch(({ message }) => this.errorToast(message));
     },
     changeSshProtocolState(state) {
       this.$store
         .dispatch('policies/saveSshProtocolState', state)
-        .then((message) => this.successToast(message))
+        .then((message) => {
+          this.successToast(message);
+        })
         .catch(({ message }) => this.errorToast(message));
     },
     changeRtadState(state) {