talos icon indicating copy to clipboard operation
talos copied to clipboard

feat: remove wrapperd and launch processes directly

Open dsseng opened this issue 1 year ago • 1 comments

Fixes #9472

TODO:

  • [x] tests
  • [x] finalize

dsseng avatar Oct 11 '24 10:10 dsseng

Maybe the test should be a separate PR going in before refactoring. If so, will do later today

dsseng avatar Oct 17 '24 09:10 dsseng

https://github.com/siderolabs/talos/actions/runs/11515928408/job/32057560561?pr=9489#step:18:6799

there's a chance this process just completes before cgroup code kicks in. What can we do? Qualify no such process as a warning?

dsseng avatar Oct 25 '24 10:10 dsseng

siderolabs/talos/actions/runs/11515928408/job/32057560561?pr=9489#step:18:6799

there's a chance this process just completes before cgroup code kicks in. What can we do? Qualify no such process as a warning?

I guess so. As we have a test which verifies that processes are in a proper cgroup, we should be good

smira avatar Oct 25 '24 10:10 smira

/m

dsseng avatar Oct 28 '24 17:10 dsseng