Differential D994

Configure systemd unit to restart Odderon on non 0 exit
ClosedPublic
Actions

Authored by dereckson on Jun 3 2017, 19:05.

Tags

Referenced Files

	F2844173: D994.id2539.diff
	Tue, Apr 23, 13:01

	F2843800: D994.diff
	Tue, Apr 23, 09:06

	Unknown Object (File)
	Mon, Apr 22, 15:59

	Unknown Object (File)
	Sun, Apr 21, 10:00

	Unknown Object (File)
	Thu, Apr 18, 17:34

	Unknown Object (File)
	Tue, Apr 16, 23:16

	Unknown Object (File)
	Tue, Apr 16, 21:56

	Unknown Object (File)
	Sun, Apr 14, 19:22

Subscribers

None

Details

Reviewers

Sandlayth
dereckson

Maniphest Tasks

T1213: Odderon doesn't restart after a non 0 exit code

Commits

rOPS2ecd36350302: Configure systemd unit to restart Odderon on non 0 exit

Summary

The darkbot software implements a feature to catch exceptions like
a segmentation fault, and then exits properly the software with
a non zero code.

The quit message on IRC explains the intercepted signal:
Quit: Caught SIG_SEGV! Aborting connection. Uptime: 893 hours, 10 mins

In such case, we want systemd to immediately restart the service,
to improve service availability. This is not the default behavior,
as systemd prefers to mark unit status as failed.

Reference: https://www.freedesktop.org/software/systemd/man/systemd.service.html#Restart=

Fixes T1213.

Test Plan

Deploy on Eglide
- Function service.force_reload from odderon_unit took care to reload systemd configuration
- Function service.running from odderon_running state took care to restart the service
Unit status switched from failed to an active (running).

Diff Detail

Repository

rOPS Nasqueron Operations

Lint

Lint Not Applicable

Unit

Tests Not Applicable

Event Timeline

dereckson created this revision.Jun 3 2017, 19:05

dereckson edited the test plan for this revision. (Show Details)Jun 3 2017, 19:11

dereckson added a project: systemd.

dereckson edited the summary of this revision. (Show Details)Jun 5 2017, 12:24

dereckson edited the summary of this revision. (Show Details)

Sandlayth accepted this revision.Jun 5 2017, 21:18

This revision is now accepted and ready to land.Jun 5 2017, 21:18

dereckson accepted this revision.Jun 12 2017, 21:23

Closed by commit rOPS2ecd36350302: Configure systemd unit to restart Odderon on non 0 exit (authored by dereckson, committed by dereckson). · Explain WhyJun 12 2017, 21:24

This revision was automatically updated to reflect the committed changes.

Revision Contents
Changeset List

Path

Size

roles/

shellserver/

odderon/

files/

odderon.service

1 line

Diff 2539

roles/shellserver/odderon/files/odderon.service

Loading...