PR#472: update list of copr services - copr/copr

copr / copr

#472 update list of copr services

Merged 5 years ago by msuchy. Opened 5 years ago by msuchy.

copr/ msuchy/copr services into master

update list of copr services

Miroslav Suchý • 5 years ago

6ac9c59

backend/copr-backend-service

file modified

+1 -1

		`@@ -8,7 +8,7 @@`


		`copr_target_services() {`
		`- echo copr-backend copr-backend-vmm copr-backend-log`
		`+ echo copr-backend copr-backend-build copr-backend-log copr-backend-action`
		`}`

		`turn_on() {`

no initial comment

frostyx commented 5 years ago

No problem with this PR @msuchy, +1 from me.
However, I think, that the backend/copr-backend-service should not be needed anymore. See, in every backend/units/copr-backend-*.service, there is

Before=copr-backend.service
PartOf=copr-backend.service

PartOf=
Configures dependencies similar to Requires=, but limited to stopping and restarting of units. When systemd stops or restarts the units listed here, the action is propagated to this unit. Note that this is a one-way dependency — changes to this unit do not affect the listed units.

So the systemctl restart copr-backend should be enough, to restart everything. Which was AFAIK the only purpose of backend/copr-backend-service.

Edited 5 years ago by frostyx