Opened 10 months ago

Last modified 4 months ago

#16719 new enhancement

[ProcessController] replace Kill button with Restart (easy)

Reported by: diver Owned by: nobody
Priority: normal Milestone: Unscheduled
Component: Applications/ProcessController Version: R1/beta2
Keywords: Cc:
Blocked By: Blocking:
Platform: All

Description (last modified by diver)

Kill this theam button should be replaced with Restart this team for system (launch_daemon controlled) processes.

Change History (4)

comment:1 by X512, 10 months ago

Why? Threads are not restarted, only teams (processes). Also I don't think that knowledge of which team is auto-restarting is ProcessController task.

comment:2 by diver, 10 months ago

Description: modified (diff)

Correct, I was meaning teams of course. It's just that I find the message a bit misleading - it makes one think that the process would be killed while in reality it wouldn't.

comment:3 by X512, 10 months ago

it makes one think that the process would be killed while in reality it wouldn't

Process is actually killed. launch_daemon creates new process. Restart should be done in tools like Services. Each software has its own domain of responsibility.

Also process restarting is emergency behavior that it intended to recover crashed service. It is not regular way to restart service and can cause data loss.

Last edited 10 months ago by X512 (previous) (diff)

comment:4 by zakero, 4 months ago

A commit for this ticket is ready for review.

Does anything need to be done for translations or does the B_TRANSLATE() handle that?

Note: See TracTickets for help on using tickets.