Skip to content

Commit 31967f3

Browse files
1LA: Runtime Processor Diagnostics Updated (#112)
* Fabric Adapters showing right info in the respective tabs (#75) - Fabric adapters were showing incorrect information when we switch between the tabs. It is fixed here. - Similar change is made to rest of the tables in the Inventory and LEDs page. - Progress bar is added for the switching between the tabs. - BQ defect: https://w3.rchland.ibm.com/projects/bestquest/?defect=PE00F3H9 Signed-off-by: Nikhil Ashoka <[email protected]> * 1LA: Runtime Processor Diagnostics Updated 1. RPD feature -> In toast message added -> Changes made will be available in next reboot. 2. RPD policy helptext -> Enabled (Run on each core to test them on periodic basis), Scheduled (Run sequentially on each core at the scheduled time each day for the specified duration (in minutes)), Disabled (No diagnostics or exercisers will be run). 3. RPD scheduled run time of day -> 1. Removed in seconds from title and description. 2. Updated "Duration" to "Duration (minutes)". 3. Updated "Start time" to "Start time (24-hour time (UTC))". 4. Immediate Test Requested 1. helptext -> On pressing the Run now button the RPD policy will be overriden and diagnostic test run will start immediately and the button will be greyed out untill the test runs. 2. Added Button and named "Stop diagnostic test run" on click of which we are changing the Attribute `pvm_rpd_immediate_test` to 'Disabled'. Signed-off-by: Nikhil Ashoka <[email protected]> --------- Signed-off-by: Nikhil Ashoka <[email protected]>
1 parent 534f06e commit 31967f3

File tree

3 files changed

+35
-15
lines changed

3 files changed

+35
-15
lines changed

src/locales/en-US.json

Lines changed: 9 additions & 8 deletions
Original file line numberDiff line numberDiff line change
@@ -929,20 +929,21 @@
929929
"lateralCastOutDescription": "Only change this value if instructed by your service provider as it might degrade system performance.",
930930
"rpdPolicy": "Runtime processor diagnostic policy",
931931
"updateRpdPolicy": "Update RPD policy",
932-
"rpdPolicyDescription": "Enabled (Run on each core once daily, at an equally spaced interval, to test each core every 24 hours. For example, on a 48-core system, the RPD would test one core every 30 minutes), Scheduled (Run sequentially on each core at the scheduled time each day for the specified duration (in minutes)), Disabled (No diagnostics or exercisers will be run). ",
932+
"rpdPolicyDescription": "Enabled (Run on each core to test them on periodic basis), Scheduled (Run sequentially on each core at the scheduled time each day for the specified duration (in minutes)), Disabled (No diagnostics or exercisers will be run).",
933933
"rpdFeature": "Runtime processor diagnostic feature",
934-
"updateRpdFeature": "Update RPD feature policy",
934+
"updateRpdFeature": "Update RPD feature",
935935
"rpdFeatureDescription": "Controls whether or not the Runtime Processor Diagnostics (RPD) Feature will be configured on the system. Enabled ( The RPD feature will be configured regardless of the amount of installed memory), Automatic (The RPD feature will be configured on systems with at least 128GB of installed memory and not configured on systems with less than 128GB), Disabled (The RPD feature will not be configured).",
936-
"startTime": "Start time",
936+
"startTime": "Start time (24-hour time (UTC))",
937937
"runNow": "Run now",
938-
"duration": "Duration",
938+
"duration": "Duration (minutes)",
939939
"immediateTestRequested": "Immediate Test Requested",
940-
"immediateTestRequestedDescription": "On pressing the Run now button the RPD policy will be overriden and diagnostic test run will start immediately and the button will be greyed out untill the test runs. RPD will set RPD immediate test to “Disabled” when an immediate test run completes",
941-
"rpdScheduledRun": "RPD scheduled run time of day in Seconds",
942-
"rpdScheduledRunDescription": "Only used when RPD policy is set to “Scheduled”. This value represents the time of day in seconds at which to run the processor diagnostics.",
940+
"immediateTestRequestedDescription": "On pressing the Run now button the RPD policy will be overriden and diagnostic test run will start immediately and the button will be greyed out untill the test runs.",
941+
"rpdScheduledRun": "RPD scheduled run time of day",
942+
"rpdScheduledRunDescription": "Only used when RPD policy is set to “Scheduled”. This value represents the time of day at which to run the processor diagnostics.",
943943
"updateRpdScheduledRun": "Update RPD scheduled run",
944944
"gardOnError": "GARD on error",
945945
"gardOnErrorDescription": "Controls whether or not a processor core will be de-configured on error.",
946+
"stopTest": "Stop diagnostic test run",
946947
"toast": {
947948
"errorSavingAggressivePrefetch": "Error saving aggressive prefetch",
948949
"errorSavingFrequencyCap": "Error saving frequency cap",
@@ -959,7 +960,7 @@
959960
"successSavingGardOnError": "Gard on error saved successfully",
960961
"successSavingLateralCastOut": "Lateral cast out mode saved successfully",
961962
"successSavingRpdPolicy": "RPD policy saved successfully",
962-
"successSavingRpdFeature": "RPD feature saved successfully."
963+
"successSavingRpdFeature": "RPD feature saved successfully. Changes made will take effect on next reboot."
963964
}
964965
},
965966
"pageLdap": {

src/store/modules/ResourceManagement/SystemParametersStore.js

Lines changed: 12 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -297,10 +297,14 @@ const systemParametersStore = {
297297
);
298298
});
299299
},
300-
async saveImmediateTestRequested({ commit }) {
301-
commit('setImmediateTestRequested', true);
300+
async saveImmediateTestRequested({ commit }, { value }) {
301+
if (value === 'Enabled') {
302+
commit('setImmediateTestRequested', true);
303+
} else {
304+
commit('setImmediateTestRequested', false);
305+
}
302306
const updatedImmediateTestRequestedValue = {
303-
Attributes: { pvm_rpd_immediate_test: 'Enabled' },
307+
Attributes: { pvm_rpd_immediate_test: value },
304308
};
305309
return api
306310
.patch(
@@ -314,7 +318,11 @@ const systemParametersStore = {
314318
})
315319
.catch((error) => {
316320
console.log(error);
317-
commit('setImmediateTestRequested', false);
321+
if (value === 'Enabled') {
322+
commit('setImmediateTestRequested', false);
323+
} else {
324+
commit('setImmediateTestRequested', true);
325+
}
318326
throw new Error(
319327
i18n.t(
320328
'pageSystemParameters.toast.errorSavingImmediateTestRequested'

src/views/ResourceManagement/SystemParameters/RuntimeProcessorDiagnostic.vue

Lines changed: 14 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -179,10 +179,19 @@
179179
type="submit"
180180
class="ml-3"
181181
:disabled="immediateTestRequestedState || isRpdFeatureCurrentDisabled"
182-
@click="updateImmediateTestRequestedState()"
182+
@click="updateImmediateTestRequestedState(true)"
183183
>
184184
{{ $t('pageSystemParameters.runNow') }}
185185
</b-button>
186+
<b-button
187+
variant="danger"
188+
type="submit"
189+
class="ml-3"
190+
:disabled="!immediateTestRequestedState || isRpdFeatureCurrentDisabled"
191+
@click="updateImmediateTestRequestedState(false)"
192+
>
193+
{{ $t('pageSystemParameters.stopTest') }}
194+
</b-button>
186195
</b-row>
187196
</div>
188197
</template>
@@ -319,10 +328,12 @@ export default {
319328
},
320329
},
321330
methods: {
322-
updateImmediateTestRequestedState() {
331+
updateImmediateTestRequestedState(value) {
323332
this.startLoader();
324333
Promise.all([
325-
this.$store.dispatch('systemParameters/saveImmediateTestRequested'),
334+
this.$store.dispatch('systemParameters/saveImmediateTestRequested', {
335+
value: value ? 'Enabled' : 'Disabled',
336+
}),
326337
this.$store.dispatch('systemParameters/getRpdScheduledRun'),
327338
]).finally(() => this.endLoader());
328339
},

0 commit comments

Comments
 (0)