Closed Bug 1145487 Opened 10 years ago Closed 10 years ago

provide systemd service files for Socorro

Categories

(Socorro :: Infra, task)

task
Not set
normal

Tracking

(Not tracked)

RESOLVED FIXED

People

(Reporter: rhelmer, Assigned: rhelmer)

References

Details

uWSGI has recommendations for how to run under systemd: http://uwsgi-docs.readthedocs.org/en/latest/Systemd.html
(In reply to Robert Helmer [:rhelmer] from comment #0) > uWSGI has recommendations for how to run under systemd: > > http://uwsgi-docs.readthedocs.org/en/latest/Systemd.html We could either provide a systemd service file for each app (socorro-collector, socorro-middleware, socorro-processor) or we could have a single generic service and control what's started via config (ini files) I think the first option would probably be simplest, any thoughts?
Flags: needinfo?(dmaher)
Blocks: 1123833
Hm one thing to consider - we need to make sure this plays nice with consul. The way I am doing it now is invoking a separate uwsgi in "master" mode for each app, running under envconsul. We could just do this in our systemd service files. "emperor" mode (which the doc in comment 0 suggests) watches a directory of uWSGI config files, and spawns instances ("vassals") for each.
Both approaches are fine from a config management perspective. Given uWSGI's architecture it makes sense that they'd advocate this approach (it fits well into systemd, too). That said, this seems like a decision we could make later - if the current methodology (per-app) works and we're comfortable with it, why rock the boat? Instead, I suggest we file a post-AWS bug to review this and move forward. Sound good?
Flags: needinfo?(dmaher)
Summary: provide uwsgi systemd service file for Socorro → provide systemd service files for Socorro
Commits pushed to master at https://github.com/mozilla/socorro https://github.com/mozilla/socorro/commit/d9b56f8d222c38e9f5cde2e958422200cd7ae36e fix bug 1145487 - add systemd service files https://github.com/mozilla/socorro/commit/e3b20778d03831dc932e9900a7aef38f22f409cb Merge pull request #2685 from rhelmer/centos7-fpm-update fixes bug 1106992, 1145487, 1146984 - Centos7 major packaging / infra update
Status: NEW → RESOLVED
Closed: 10 years ago
Resolution: --- → FIXED
You need to log in before you can comment on or make changes to this bug.