Page MenuHomeDevCentral

Init OpenDKIM service
AcceptedPublic

Authored by DorianWinty on Thu, Nov 14, 19:43.
Tags
None
Referenced Files
F3758356: D3591.id9269.diff
Tue, Nov 19, 11:01
F3758355: D3591.id9255.diff
Tue, Nov 19, 11:01
F3758354: D3591.id9254.diff
Tue, Nov 19, 11:01
F3758341: D3591.id9269.diff
Tue, Nov 19, 11:00
F3758302: D3591.id.diff
Tue, Nov 19, 10:25
F3758278: D3591.diff
Tue, Nov 19, 10:10
Unknown Object (File)
Tue, Nov 19, 05:10
Unknown Object (File)
Tue, Nov 19, 03:59
Subscribers
None

Details

Reviewers
dereckson

Diff Detail

Repository
rOPS Nasqueron Operations
Lint
Lint Passed
Unit
No Test Coverage
Branch
opendkimService
Build Status
Buildable 5715
Build 5997: arc lint + arc unit

Event Timeline

DorianWinty created this revision.

add provisionning of the files

dereckson retitled this revision from init opendkim service to Init OpenDKIM service.Thu, Nov 14, 20:58
roles/mailserver/dkim/files/rc/opendkim
10
26
55

Normal signal should work too.

roles/mailserver/dkim/service.sls
17
DorianWinty marked 4 inline comments as done.

edit following commands

create a folder for sock and pid file

edit for making it work (work in prod now)

dereckson requested changes to this revision.Tue, Nov 19, 18:32
dereckson added inline comments.
roles/mailserver/dkim/files/rc/opendkim
60

With the new approach, we need to create the directory if it doesn't exist, mkdir -p works fine for that use: -p is to create parent directories, but that also allows to silently ignore error if directory already exists.

To create pid isn't needed anymore if the directory has the correct permissions, opendkim can create it itself.

This revision now requires changes to proceed.Tue, Nov 19, 18:32
DorianWinty marked an inline comment as done.

make more usefull for a service for everyone

This revision is now accepted and ready to land.Tue, Nov 19, 20:14