crash-reporter: remove need for crash_sender.pid |
||
Issue descriptionautotest/files/client/cros/crash/crash_test.py has logic to read crash_sender.pid so it can forcibly kill any instances of crash_sender that are running on the system. that shouldn't be necessary at all as autotest should be able to just do `killall crash_sender` (or whatever equiv autotest might already have). once autotest stops relying on that file, we can stop creating it in crash_sender.
,
Aug 15
The following revision refers to this bug: https://chromium.googlesource.com/chromiumos/platform2/+/ae8c010fa63f7d4ba1fab53ebc1a67ad7050104d commit ae8c010fa63f7d4ba1fab53ebc1a67ad7050104d Author: Mike Frysinger <vapier@chromium.org> Date: Wed Aug 15 11:42:48 2018 crash: drop crash_sender.pid creation Now that the autotest no longer uses it, we can stop creating it. BUG=chromium:391887, chromium:868166 TEST=precq passes Change-Id: I9b28efa483af9392cfbbdfa9e66535e347f1c383 Reviewed-on: https://chromium-review.googlesource.com/1168261 Commit-Ready: Mike Frysinger <vapier@chromium.org> Tested-by: Mike Frysinger <vapier@chromium.org> Reviewed-by: Satoru Takabayashi <satorux@google.com> Reviewed-by: Mike Frysinger <vapier@chromium.org> [modify] https://crrev.com/ae8c010fa63f7d4ba1fab53ebc1a67ad7050104d/crash-reporter/crash_sender.cc [modify] https://crrev.com/ae8c010fa63f7d4ba1fab53ebc1a67ad7050104d/crash-reporter/crash_sender_paths.h
,
Aug 17
|
||
►
Sign in to add a comment |
||
Comment 1 by vapier@chromium.org
, Aug 8Status: Started (was: Available)