Skip to content

feat: add --pid-file option to write PID files#25499

Merged
gwossum merged 2 commits into2.7from
gw/25498/pid_file
Oct 29, 2024
Merged

feat: add --pid-file option to write PID files#25499
gwossum merged 2 commits into2.7from
gw/25498/pid_file

Conversation

@gwossum
Copy link
Copy Markdown
Member

@gwossum gwossum commented Oct 29, 2024

Add --pid-file option to write PID files on startup. The PID filename is specified by the argument after --pid-file. If the PID file already exists, influxd will exit unless the --overwrite-pid-file flag is also used.

Example: influxd --pid-file /var/lib/influxd/influxd.pid

PID files are automatically removed when the influxd process is shutdown.

Closes: #25498

(cherry picked from commit c35321b)
(cherry picked from commit 48f7600)

Add `--pid-file` option to write PID files on startup. The PID filename
is specified by the argument after `--pid-file`. If the PID file already exists, influxd will exit unless the `--overwrite-pid-file` flag is also used.

Example: `influxd --pid-file /var/lib/influxd/influxd.pid`

PID files are automatically removed when the influxd process is shutdown.

Closes: 25498
(cherry picked from commit c35321b)
@gwossum gwossum added area/configuration kind/feature area/2.x OSS 2.0 related issues and PRs labels Oct 29, 2024
@gwossum gwossum self-assigned this Oct 29, 2024
@gwossum gwossum marked this pull request as ready for review October 29, 2024 21:04
@davidby-influx davidby-influx self-requested a review October 29, 2024 21:07
Copy link
Copy Markdown
Contributor

@davidby-influx davidby-influx left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM; clean pick

@gwossum gwossum merged commit e9ea8cf into 2.7 Oct 29, 2024
@gwossum gwossum deleted the gw/25498/pid_file branch October 29, 2024 21:10
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants