p4d_1.service #7

  • //
  • guest/
  • perforce_software/
  • sdp/
  • dev/
  • Server/
  • Unix/
  • setup/
  • systemd/
  • p4d_1.service
  • View
  • Commits
  • Open Download .zip Download (393 B)
[Unit]
Description=Helix Core Server Instance 1.
Documentation=https://swarm.workshop.perforce.com/projects/perforce-software-sdp/view/main/doc/SDP_Guide.Unix.pdf
Requires=network.target
After=network.target

[Service]
Type=simple
ExecStart=/p4/1/bin/p4d_1_init start
ExecStop=/p4/1/bin/p4d_1_init stop
Environment=OS_INIT_MECHANISM=systemd
User=perforce

[Install]
WantedBy=multi-user.target
# Change User Description Committed
#7 27201 C. Thomas Tyler Removed PIDFile= directory in systemd, as it actually introduces an
issue. We randomly get an error like:

p4d_1.service: Supervising process 16545 which is not our child. We'll most likely not notice when it exits.

The PIDFile directive was added for informational purposes, and is
not needed for processing. Even without it, the Main PID is accurate.
#6 27148 C. Thomas Tyler Added templates for systemd unit files.
#5 27109 C. Thomas Tyler Refined logic so that if a shutdown is attempted and there is no
server.pid file, the 'fallback/2nd pass/assured shutdown' logic
to do a 'p4 admin stop' files only if a 'p4 info' command can
reach the server. Otherwise, the stop command is silently
ignored.

Refined exit code handling and fixed log interaciton issues.
Fixed issue with systemd status not indicating successful startup.

The systemd unit files now specify the Type as simple rather than
forking, and defer to systemd to handle the forking. When systemd is
used, the -d/--daemonsafe flags are removed from service start commands
for p4p/p4broker/p4d, and for p4dtg the & is removed so the process is
not started in the background. For compatibility with non-systemd systems,
the -d/--daemonsafe flags are still applied when systemd is not in use.

With this change, systemd's concept of whether the service is running should
now be reliable.  If you try to start without using systemd on a system for
which a systemd unit file exists, an error is displayed -- a big, hopefully
useful and informative error message.

#review
#4 26888 C. Thomas Tyler Updated doc links to reference SDP rather than core docs in systemd service files.

Added sample systemd file for p4p (Helix Proxy).
#3 24880 C. Thomas Tyler Removed references to NetworkManager, as this is an unnecessary
dependency and hinders portability.
#2 22649 C. Thomas Tyler Added 'Requires' and 'After' targets to sample systemd service files.

#review @giles
#1 15797 C. Thomas Tyler Routine    Merge Down to dev from main for SDP.
//guest/perforce_software/sdp/main/Server/Unix/setup/systemd/p4d_1.service
#1 15793 C. Thomas Tyler Added sample systemd init scripts for the SDP for RHEL/CentOS 7
and other Linux distros that use systemd.

Also updated README.md for firewalld.