[Unit] Description=P4 Server Instance 1. Documentation=https://workshop.perforce.com/view/p4-sdp/main/doc/SDP_Guide.Unix.html 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 | |
|---|---|---|---|---|---|
| #2 | 33409 | Claude (AI Agent by Anthropic) |
Copy Up from //p4-sdp/dev_rebrand into //p4-sdp/dev. This is the first promotion of dev_rebrand's work into dev since dev_rebrand was created (2025-05-24) -- 303 files, covering the entire 2026.1 rebranding effort (SDP-1379), the Secure By Default adaptation (SDP-1350), OrgName-aware auth.id/ServerID (SDP-1286), RCS-keyword version identification (SDP-1161/SDP-799), and the Streams-native release process redesign (Task 5) done this session, plus everything else accumulated in dev_rebrand's history before this session. Per the Merge Down/Copy Up flow, this is intentionally a full, unconditional blast-replace of dev's content from dev_rebrand -- all selectivity/care happened in the preceding Merge Down (dev -> dev_rebrand, changes 33407-33408), which absorbed Robert Cowham's independent dev-side work first so nothing of his is lost by this Copy Up. Two files are worth calling out since they might look alarming in isolation: - tools/mdcu.sh is deleted -- intentional, retired this session in favor of the two direct Streams commands now documented in doc/ReleaseProcessOverview.md. - tools/ReleaseProcessOverview.md is deleted -- this is a stale relic of a file move dev_rebrand made back in 2025-05-24 (tools/ -> doc/) that was never previously propagated to dev; the current, fully-rewritten doc/ReleaseProcessOverview.md is added/updated correctly by this same changelist. |
||
| #1 | 31397 | C. Thomas Tyler | Populate -b SDP_Classic_to_Streams -s //guest/perforce_software/sdp/...@31368. | ||
| //guest/perforce_software/sdp/dev/Server/Unix/setup/systemd/p4d_1.service | |||||
| #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. |
||