From T89142 , we should use systemd for NodePool since we are installing it on Debian Jessie.
Description
Details
Subject | Repo | Branch | Lines +/- | |
---|---|---|---|---|
0.1.1-wmf3: statsd and systemd support | operations/debs/nodepool | debian | +64 -1 | |
nodepool: systemd wrapper | operations/puppet | production | +58 -0 |
Status | Subtype | Assigned | Task | ||
---|---|---|---|---|---|
Resolved | hashar | T89143 Puppetize Nodepool configuration | |||
Resolved | hashar | T89142 Create a Debian package for NodePool on Debian Jessie | |||
Resolved | hashar | T97402 Make services manageable by systemd | |||
Resolved | hashar | T95045 install/deploy labnodepool1001 | |||
Resolved | hashar | T96867 Use systemd for Nodepool |
Event Timeline
From T95003 @Joe pointed to our puppet define base::service_unit which maybe a good template to add systemd to nodepool: https://github.com/wikimedia/operations-puppet/blob/21c72942dd7bf25dbe0759d2f867082e966bfb45/modules/base/manifests/service_unit.pp
Lets give this a try. Will do it in operations/puppet.git for now then "upstream" it in the .deb package.
Change 224102 had a related patch set uploaded (by Hashar):
nodepool: systemd wrapper
Change 224390 had a related patch set uploaded (by Hashar):
(WIP) systemd support (WIP)
Status update
I have finished the systemd patch: https://gerrit.wikimedia.org/r/#/c/224102/6
Encountered a bug in the way nodepool handles graceful stop which caused me to patch Nodepool ( https://gerrit.wikimedia.org/r/#/c/225475/ ). I have bumped our Debian package to include the fix.
The logging configuration is at https://gerrit.wikimedia.org/r/#/c/224106/
Will check with European ops to get those merged then chat with them about how to add systemd in a .deb package.
Thanks to @Muehlenhoff for the final review of the systemd integration. We now have a working systemd setup on labnodepool1001.eqiad.wmnet :-)
Change 224390 abandoned by Hashar:
0.1.1-wmf3: statsd and systemd support
Reason:
Will be redone later (pending E_TOOMANYTASKS)