Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

RDX: Implement process groups on Linux #7972

Open
mook-as opened this issue Dec 18, 2024 · 0 comments
Open

RDX: Implement process groups on Linux #7972

mook-as opened this issue Dec 18, 2024 · 0 comments

Comments

@mook-as
Copy link
Contributor

mook-as commented Dec 18, 2024

Actual Behavior

In #7965 we disabled the use of process groups on Linux (for #7945) because Electron doesn't start its own process group. We should make this happen correctly (or do equivalent things, possibly via cgroups) so that we can have a better chance of shutting down processes started by extensions.

Steps to Reproduce

(Via code inspection from the PR)

Result

We are not terminating processes by process groups, nor any other means of ensuring orphaned child processes are removed.

Expected Behavior

Do something to ensure descendant processes from extensions are terminated on factory reset and application exit.

Additional Information

No response

Rancher Desktop Version

1.17.0-hackweek-release-254-g77398647d (1.17.0-RC1)

Rancher Desktop K8s Version

N/A

Which container engine are you using?

containerd (nerdctl)

What operating system are you using?

Other Linux

Operating System / Build Version

openSUSE Leap 15.6

What CPU architecture are you using?

x64

Linux only: what package format did you use to install Rancher Desktop?

None

Windows User Only

No response

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

1 participant