Project

General

Profile

Actions

Refactor #27874

closed

Optimize process tree in tfm-* wrappers

Added by Ewoud Kohl van Wijngaarden over 4 years ago. Updated over 4 years ago.

Status:
Closed
Priority:
Normal
Category:
RPMs
Target version:
-
Fixed in Releases:
Found in Releases:

Description

Currently we execute bash and within that bash we execute scl enable tfm "bash wrapper". This results in a complex process tree:

/bin/bash /usr/bin/tfm-rake -- console
 \_ scl enable tfm bash /tmp/tmp.4vH4dRd7aM
     \_ /bin/bash /var/tmp/scliO1x5z
         \_ bash /tmp/tmp.4vH4dRd7aM
             \_ /opt/rh/rh-ruby25/root/usr/bin/ruby /opt/rh/rh-ruby25/root/usr/bin/rake -- console

It can also use source scl_enable tfm && exec ruby "$@" and end up with a much smaller process tree since it replaces the parent process:

/opt/rh/rh-ruby25/root/usr/bin/ruby /opt/rh/rh-ruby25/root/usr/bin/rake console

This still avoids https://bugzilla.redhat.com/show_bug.cgi?id=1248418 in a different way.

Actions #1

Updated by The Foreman Bot over 4 years ago

  • Status changed from New to Ready For Testing
  • Assignee set to Ewoud Kohl van Wijngaarden
  • Pull request https://github.com/theforeman/foreman-packaging/pull/4115 added
Actions #2

Updated by The Foreman Bot over 4 years ago

  • Fixed in Releases 1.24.0 added
Actions #3

Updated by Ewoud Kohl van Wijngaarden over 4 years ago

  • Status changed from Ready For Testing to Closed
Actions #4

Updated by The Foreman Bot over 4 years ago

  • Pull request https://github.com/theforeman/foreman-packaging/pull/4129 added
Actions

Also available in: Atom PDF