Okay,

The fix in the previous bug didn't actually fix the problem. Specifically, the following commit being pulled in:

    $ git show 5122e0d51f8d392bbeebff27f5a8fbab4bed8ba4
    commit 5122e0d51f8d392bbeebff27f5a8fbab4bed8ba4
    Author: Jeremy Crafts <gravitypriest.github.com>
    Date:   Thu Jul 16 10:24:15 2020 -0400
    
        use SIGTERM for rpm and yum commands (#2630)
        
        Signed-off-by: Jeremy Crafts <jcrafts>
    
    diff --git a/insights/client/insights_spec.py b/insights/client/insights_spec.py
    index 1e2de068..d0b9037b 100644
    --- a/insights/client/insights_spec.py
    +++ b/insights/client/insights_spec.py
    @@ -47,8 +47,17 @@ class InsightsCommand(InsightsSpec):
     
             # all commands should timeout after a long interval so the client does not hang
             # prepend native nix 'timeout' implementation
    -        timeout_command = 'timeout -s KILL %s %s' % (
    -            self.config.cmd_timeout, self.command)
    +
    +        # use TERM for rpm/yum commands, KILL for everything else
    +        if (self.command.startswith('/bin/rpm') or
    +           self.command.startswith('yum') or
    +           self.command.startswith('/usr/bin/yum')):
    +            signal = 'TERM'
    +        else:
    +            signal = 'KILL'
    +
    +        timeout_command = 'timeout -s %s %s %s' % (
    +            signal, self.config.cmd_timeout, self.command)
     
             # ensure consistent locale for collected command output
             cmd_env = {'LC_ALL': 'C',

The problem here is that there are a number of problematic commands that are in insights/specs/default.py:

$ grep -e 'simple_command.*yum' -e 'simple_command.*/rpm' insights/specs/default.py 
    rpm_V_packages = simple_command("/bin/rpm -V coreutils procps procps-ng shadow-utils passwd sudo chrony", keep_rc=True)
    yum_list_available = simple_command("yum -C --noplugins list available")
    yum_repolist = simple_command("/usr/bin/yum -C --noplugins repolist")
    yum_updateinfo = simple_command("/usr/bin/yum -C updateinfo list")
    installed_rpms = simple_command("/bin/rpm -qa --qf '%s'" % rpm_format, context=HostContext)



That isn't fixed at all by the InsightsCommand() class modifications (AFAICT). Unless I'm badly mistaken, we need to provide a way for the simple_command() definitions to indicate the specific signal to be delivered... Something akin to:

$ grep -e 'simple_command.*yum' -e 'simple_command.*/rpm' insights/specs/default.py 
    rpm_V_packages = simple_command("/bin/rpm -V coreutils procps procps-ng shadow-utils passwd sudo chrony", keep_rc=True, signum=signal.SIGTERM)
    yum_list_available = simple_command("yum -C --noplugins list available", signum=signal.SIGTERM)
    yum_repolist = simple_command("/usr/bin/yum -C --noplugins repolist", signum=signal.SIGTERM)
    yum_updateinfo = simple_command("/usr/bin/yum -C updateinfo list", signum=signal.SIGTERM)
    installed_rpms = simple_command("/bin/rpm -qa --qf '%s'" % rpm_format, context=HostContext, signum=signal.SIGTERM)


This doesn't look impossible, just a matter of plumbing through:

  simple_command()
  -> CommandOutputProvider()
   -> shell_out()

(In reply to Kyle Walker from comment #3)

That's correct. We've since switched the collection code to the insights-core collection lib, and so the previous fix will need to be ported.

The change to insights-core [1] will go out release today for testing.  The release will be generally available to customers on Monday morning if no problems are identified. The change uses SIGTERM instead of SIGKILL for all yum and rpm collection commands.

[1] https://github.com/RedHatInsights/insights-core/pull/2974

@jcastran are you still seeing this problem?