<html><body><p><font size="2">We are getting -1 back, what is the expected exit status from systemd stop ?<br></font><font size="2"><br></font><font size="2">Sent from my iPhone<br></font><font size="2"><br></font><font size="2">> On Apr 7, 2018, at 9:23 AM, Vasant Hegde <hegdevasant@linux.vnet.ibm.com> wrote:<br></font><font size="2">> <br></font><font size="2">>> On 04/07/2018 07:40 PM, Deb McLemore wrote:<br></font><font size="2">>> When the opal-prd.service is running and attempt to stop is<br></font><font size="2">>> performed, ignore the exit status and continue.<br></font><font size="2">> <br></font><font size="2">> Deb,<br></font><font size="2">> <br></font><font size="2">> Can you please explain why do you want to ignore exit status here?<br></font><font size="2">> Is there any issues?<br></font><font size="2">> <br></font><font size="2">> -Vasant<br></font><font size="2">> <br></font><font size="2">> <br></font><font size="2">> <br></font><font size="2">>> <br></font><font size="2">>> Signed-off-by: Deb McLemore <debmc@linux.vnet.ibm.com><br></font><font size="2">>> ---<br></font><font size="2">>> src/opal/prd_info.c | 20 ++++----------------<br></font><font size="2">>> 1 file changed, 4 insertions(+), 16 deletions(-)<br></font><font size="2">>> <br></font><font size="2">>> diff --git a/src/opal/prd_info.c b/src/opal/prd_info.c<br></font><font size="2">>> index 4082a18..2db9413 100644<br></font><font size="2">>> --- a/src/opal/prd_info.c<br></font><font size="2">>> +++ b/src/opal/prd_info.c<br></font><font size="2">>> @@ -73,7 +73,7 @@ static int prd_dev_query(fwts_framework *fw)<br></font><font size="2">>> <br></font><font size="2">>> static int prd_service_check(fwts_framework *fw, int *restart)<br></font><font size="2">>> {<br></font><font size="2">>> - int rc = FWTS_OK, status = 0, stop_status = 0;<br></font><font size="2">>> + int rc = FWTS_OK, status = 0;<br></font><font size="2">>> char *command;<br></font><font size="2">>> char *output = NULL;<br></font><font size="2">>> <br></font><font size="2">>> @@ -97,25 +97,13 @@ static int prd_service_check(fwts_framework *fw, int *restart)<br></font><font size="2">>> goto out;<br></font><font size="2">>> case 0: /* "running" */<br></font><font size="2">>> command = "systemctl stop opal-prd.service 2>&1";<br></font><font size="2">>> - stop_status = fwts_exec2(command, &output);<br></font><font size="2">>> + fwts_exec2(command, &output);<br></font><font size="2">>> <br></font><font size="2">>> if (output)<br></font><font size="2">>> free(output);<br></font><font size="2">>> <br></font><font size="2">>> - switch (stop_status) {<br></font><font size="2">>> - case 0:<br></font><font size="2">>> - *restart = 1;<br></font><font size="2">>> - break;<br></font><font size="2">>> - default:<br></font><font size="2">>> - fwts_failed(fw, LOG_LEVEL_HIGH, "OPAL PRD Info",<br></font><font size="2">>> - "Attempt was made to stop the "<br></font><font size="2">>> - "opal-prd.service but was not "<br></font><font size="2">>> - "successful. Try to "<br></font><font size="2">>> - "\"sudo systemctl stop "<br></font><font size="2">>> - "opal-prd.service\" and retry.");<br></font><font size="2">>> - rc = FWTS_ERROR;<br></font><font size="2">>> - goto out;<br></font><font size="2">>> - }<br></font><font size="2">>> + *restart = 1;<br></font><font size="2">>> + break;<br></font><font size="2">>> default:<br></font><font size="2">>> break;<br></font><font size="2">>> }<br></font><font size="2">>> <br></font><font size="2">> <br></font><font size="2">> <br></font><font size="2">> -- <br></font><font size="2">> fwts-devel mailing list<br></font><font size="2">> fwts-devel@lists.ubuntu.com<br></font><font size="2">> Modify settings or unsubscribe at: <a href="https://urldefense.proofpoint.com/v2/url?u=https-3A__lists.ubuntu.com_mailman_listinfo_fwts-2Ddevel&d=DwIGaQ&c=jf_iaSHvJObTbx-siA1ZOg&r=V3KRDPsp3yMosW9R4elWYg&m=Sy-O20yWd_N3piZoJOEzigB1XzmLV4OUCfEyl3ENAcc&s=oPh1ACx1NGTgif-0V5BIQffXXqjymI8QC_bagI2jZsA&e=">https://urldefense.proofpoint.com/v2/url?u=https-3A__lists.ubuntu.com_mailman_listinfo_fwts-2Ddevel&d=DwIGaQ&c=jf_iaSHvJObTbx-siA1ZOg&r=V3KRDPsp3yMosW9R4elWYg&m=Sy-O20yWd_N3piZoJOEzigB1XzmLV4OUCfEyl3ENAcc&s=oPh1ACx1NGTgif-0V5BIQffXXqjymI8QC_bagI2jZsA&e=</a><br></font><font size="2">> <br></font><BR>
</body></html>