Closed Bug 1145487 Opened 9 years ago Closed 9 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: 9 years ago
Resolution: --- → FIXED
You need to log in before you can comment on or make changes to this bug.