Bug 235427 - [webkitscmpy] Stop killing finished processes
Summary: [webkitscmpy] Stop killing finished processes
Status: RESOLVED FIXED
Alias: None
Product: WebKit
Classification: Unclassified
Component: Tools / Tests (show other bugs)
Version: WebKit Nightly Build
Hardware: Unspecified Unspecified
: P2 Normal
Assignee: Jonathan Bedard
URL:
Keywords: InRadar
Depends on:
Blocks:
 
Reported: 2022-01-20 17:37 PST by Aakash Jain
Modified: 2022-01-21 10:32 PST (History)
2 users (show)

See Also:


Attachments

Note You need to log in before you can comment on or make changes to this bug.
Description Aakash Jain 2022-01-20 17:37:10 PST
show-identifier failed with error "ProcessLookupError: [Errno 3] No such process" in https://build.webkit.org/#/builders/56/builds/8455/steps/4/logs/stdio

Traceback (most recent call last):
  File "Tools/Scripts/git-webkit", line 69, in <module>
    sys.exit(program.main(
  File "/home/buildbot/worker/win10-debug/build/Tools/Scripts/libraries/webkitscmpy/webkitscmpy/program/__init__.py", line 134, in main
    return parsed.main(
  File "/home/buildbot/worker/win10-debug/build/Tools/Scripts/libraries/webkitscmpy/webkitscmpy/program/find.py", line 116, in main
    return Info.main(args, repository=repository, reference=args.argument[0], **kwargs)
  File "/home/buildbot/worker/win10-debug/build/Tools/Scripts/libraries/webkitscmpy/webkitscmpy/program/find.py", line 56, in main
    commit = repository.find(reference, include_log=args.include_log)
  File "/home/buildbot/worker/win10-debug/build/Tools/Scripts/libraries/webkitscmpy/webkitscmpy/scm_base.py", line 169, in find
    return self.commit(
  File "/home/buildbot/worker/win10-debug/build/Tools/Scripts/libraries/webkitscmpy/webkitscmpy/local/svn.py", line 444, in commit
    self.cache.populate(branch=branch)
  File "/home/buildbot/worker/win10-debug/build/Tools/Scripts/libraries/webkitscmpy/webkitscmpy/local/svn.py", line 130, in populate
    log.kill()
  File "/usr/lib/python3.8/subprocess.py", line 1943, in kill
    self.send_signal(signal.SIGKILL)
  File "/usr/lib/python3.8/subprocess.py", line 1933, in send_signal
    os.kill(self.pid, sig)
ProcessLookupError: [Errno 3] No such process
Comment 1 Jonathan Bedard 2022-01-21 09:13:11 PST
Looks like we're failing to check if the process is still running before killing it.
Comment 2 Radar WebKit Bug Importer 2022-01-21 09:13:27 PST
<rdar://problem/87887233>
Comment 3 Jonathan Bedard 2022-01-21 09:16:43 PST
Pull-request: https://github.com/WebKit/WebKit/pull/86
Comment 4 Jonathan Bedard 2022-01-21 10:32:44 PST
246268@main (r288366)