[STORM-4017] isAnyWindowsProcessAlive does not work with multiple pids

• Type: Bug
• Status: Closed
• Priority: Major
• Resolution: Fixed
• Affects Version/s: 2.3.0, 2.4.0, 2.5.0, 2.6.0
• Fix Version/s: 2.6.1
• Component/s: storm-server
• Environment: Windows
• Assignee: Scott Moore
• Reporter: Scott Moore
• Created: 05/Jan/24 20:45
• Updated: 16/Jan/24 07:14
• Resolved: 16/Jan/24 07:14

Description

The method isAnyWindowsProcessAlive in ServerUtils.java does not work with 2 or more pids in the supplied pid collection

The refactor done as part of STORM-3638 has made an incorrect assumption that the 'tasklist' Windows tool can accept multiple arguments of "/fi pid eq <id>" and treat this list as an 'or' list when, in fact, the /fi arguments should be considered an 'and' list.

e.g. tasklist /fi "pid eq 123" /fi "pid eq 456" means 'give me information on processes where each process has a pid of 123 and 456'

The effect of this means that isAnyWindowsProcessAlive will always return false when given 2 or more (different) pids.

Issue Links

• links to: Github pull request #3615