Skip to content

Commit

Permalink
Resolve syslog obsolete warnings (#478)
Browse files Browse the repository at this point in the history
Standard output type syslog is obsolete, automatically updating to
journal. Please update your unit file, and consider removing the setting
altogether.

As discussed here =>
mattbrictson/tomo-plugin-sidekiq#193

---------

Co-authored-by: Matt Brictson <matt@mattbrictson.com>
  • Loading branch information
uxxman and mattbrictson authored Jan 6, 2025
1 parent 5e370d4 commit e92cc3a
Show file tree
Hide file tree
Showing 2 changed files with 1 addition and 3 deletions.
2 changes: 1 addition & 1 deletion docs/plugins/puma.md
Original file line number Diff line number Diff line change
Expand Up @@ -9,7 +9,7 @@ Tomo's implementation installs puma as a _user-level_ service using `systemctl -
$ loginctl enable-linger <DEPLOY_USER>
```

Stdout and stderr of the puma process will be routed to syslog, as is the convention for systemd services. For Rails, it is recommended that you set `RAILS_LOG_TO_STDOUT=1` so that all Rails logs are handled this way (`tomo init` configures this by default).
Stdout and stderr of the puma process will be routed to the systemd journal. For Rails, it is recommended that you set `RAILS_LOG_TO_STDOUT=1` so that all Rails logs are handled this way (`tomo init` configures this by default).

The tomo puma plugin assumes that your puma server will listen on a single TCP port for HTTP (not HTTPS) traffic. In other words, HTTPS termination will be handled by e.g. Nginx or a separate load balancer.

Expand Down
2 changes: 0 additions & 2 deletions lib/tomo/plugin/puma/systemd/service.erb
Original file line number Diff line number Diff line change
Expand Up @@ -8,9 +8,7 @@ ConditionPathExists=<%= paths.current %>
ExecStart=/bin/bash -lc 'exec bundle exec --keep-file-descriptors puma -C config/puma.rb -b tcp://<%= settings[:puma_host] %>:<%= settings[:puma_port] %>'
KillMode=mixed
Restart=always
StandardError=syslog
StandardInput=null
StandardOutput=syslog
SyslogIdentifier=%n
TimeoutStopSec=5

Expand Down

0 comments on commit e92cc3a

Please sign in to comment.