> On Aug. 14, 2014, 5:47 p.m., Nikita Vetoshkin wrote: > > I don't think that si_uid can give something interesting because it can be > > either 0 or UID of receiving process. How about logging si_pid too? > > Ben Mahler wrote: > si_uid represents the user id of the _sending_ process, not the receiving > process. Am I missing something? > > We could log the pid as well, hopefully that helps us identify long-lived > proceses, like init or monit. > > Nikita Vetoshkin wrote: > >represents the user id of the sending process, not the receiving process. > > I think otherwise you won't receive the signal, sender will get EPERM > upon calling kill(2). > > >that helps us identify long-lived proceses, like init or monit. > > Yes, that's exactly my point. > > Alexandra Sava wrote: > Nikita, please be more specific. Logging or not logging something will > not influence the fact that the signal is delivered or not to the process. > Anyhow, I will log the si_pid since it might be helpful in debugging.
I just noted that si_uid can be either 0 or "UID of receiving process", other values or not possible due to permissions. Thus it is useful, but not very. That's all. - Nikita ----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/24700/#review50607 ----------------------------------------------------------- On Aug. 18, 2014, 11:33 a.m., Alexandra Sava wrote: > > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > https://reviews.apache.org/r/24700/ > ----------------------------------------------------------- > > (Updated Aug. 18, 2014, 11:33 a.m.) > > > Review request for mesos and Ben Mahler. > > > Bugs: MESOS-1567 > https://issues.apache.org/jira/browse/MESOS-1567 > > > Repository: mesos-git > > > Description > ------- > > Log the user id when receiving a SIGTERM. This will make debugging more easy. > > > Diffs > ----- > > 3rdparty/libprocess/3rdparty/stout/include/stout/glog.hpp > 5f763e52a6a3d5a7cb38110ea650283b7a9fa789 > > Diff: https://reviews.apache.org/r/24700/diff/ > > > Testing > ------- > > * make check > * run master binary and send it SIGTERM signal with kill > > > Thanks, > > Alexandra Sava > >
