<div><br></div><div><br><div class="gmail_quote"><div dir="ltr" class="gmail_attr">On Thu, Jan 20, 2022 at 21:14 Michal Prívozník <<a href="mailto:mprivozn@redhat.com">mprivozn@redhat.com</a>> wrote:<br></div><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left-width:1px;border-left-style:solid;padding-left:1ex;border-left-color:rgb(204,204,204)">On 1/20/22 16:31, Ani Sinha wrote:<br>
> <br>
> <br>
> On Thu, 20 Jan 2022, Andrea Bolognani wrote:<br>
> <br>
>> On Tue, Jan 18, 2022 at 12:44:51PM +0100, Michal Privoznik wrote:<br>
>>> +++ b/src/util/virprocess.c<br>
>>> @@ -1784,10 +1784,7 @@ virProcessGetStatInfo(unsigned long long *cpuTime,<br>
>>>          virStrToLong_ullp(proc_stat[VIR_PROCESS_STAT_STIME], NULL, 10, &systime) < 0 ||<br>
>>>          virStrToLong_l(proc_stat[VIR_PROCESS_STAT_RSS], NULL, 10, &rss) < 0 ||<br>
>>>          virStrToLong_i(proc_stat[VIR_PROCESS_STAT_PROCESSOR], NULL, 10, &cpu) < 0) {<br>
>>> -        virReportError(VIR_ERR_INTERNAL_ERROR,<br>
>>> -                       _("cannot parse process status data for pid '%d/%d'"),<br>
>>> -                       (int) pid, (int) tid);<br>
>>> -        return -1;<br>
>>> +        VIR_WARN("cannot parse process status data");<br>
>><br>
>> Shame to lose the improved error/warning message. Perhaps it could be<br>
>> reintroduced separately.<br>
>><br>
> <br>
> Functions in general are not coded around success path only. Most<br>
> well written functions have a success path and an error path. In case of<br>
> error, they should be able to report warning/errors. If raising an error<br>
> from a function causes a breakage of an external api, then that is an<br>
> architectural problem imho. Instead of reverting the error/warning, we<br>
> should try to fix the larger problem at hand in the longer term.<br>
> <br>
<br>
Well, until we get there we should fix the upstream so that we don't<br>
have another broken release.</blockquote><div dir="auto"><br></div><div dir="auto">AKA kicking the can one more time 🙃</div></div></div>