Skip to content

Suggest systemd override file instead of unit file for tmpdir#93211

Merged
rjernst merged 1 commit intoelastic:mainfrom
rjernst:systemd/tmpdir_override
Jan 24, 2023
Merged

Suggest systemd override file instead of unit file for tmpdir#93211
rjernst merged 1 commit intoelastic:mainfrom
rjernst:systemd/tmpdir_override

Conversation

@rjernst
Copy link
Copy Markdown
Member

@rjernst rjernst commented Jan 24, 2023

The systemd unit file is part of the Elasticsearch package and should not be edited. Instead, we recommend creating a service override file. This commit tweaks the docs for setting tmp dir with systemd to use the override file instead of editing the unit file.

relates #93121

The systemd unit file is part of the Elasticsearch package and should
not be edited. Instead, we recommend creating a service override file.
This commit tweaks the docs for setting tmp dir with systemd to use the
override file instead of editing the unit file.

relates elastic#93121
@rjernst rjernst added >docs General docs changes :Core/Infra/Core Core issues without another label v8.6.1 v8.7.0 v7.17.9 labels Jan 24, 2023
@rjernst rjernst requested a review from DaveCTurner January 24, 2023 18:47
@github-actions
Copy link
Copy Markdown
Contributor

Documentation preview:

@elasticsearchmachine elasticsearchmachine added Team:Core/Infra Meta label for core/infra team Team:Docs Meta label for docs team labels Jan 24, 2023
@elasticsearchmachine
Copy link
Copy Markdown
Collaborator

Pinging @elastic/es-docs (Team:Docs)

@elasticsearchmachine
Copy link
Copy Markdown
Collaborator

Pinging @elastic/es-core-infra (Team:Core/Infra)

@rjernst
Copy link
Copy Markdown
Member Author

rjernst commented Jan 24, 2023

Note the docs section I intend to link here is https://www.elastic.co/guide/en/elasticsearch/reference/current/setting-system-settings.html#systemd which describes the override file.

Copy link
Copy Markdown
Member

@DaveCTurner DaveCTurner left a comment

Choose a reason for hiding this comment

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

TIL, thanks Ryan.

@elasticsearchmachine
Copy link
Copy Markdown
Collaborator

💚 Backport successful

Status Branch Result
8.6
7.17

rjernst added a commit to rjernst/elasticsearch that referenced this pull request Jan 24, 2023
…c#93211)

The systemd unit file is part of the Elasticsearch package and should
not be edited. Instead, we recommend creating a service override file.
This commit tweaks the docs for setting tmp dir with systemd to use the
override file instead of editing the unit file.

relates elastic#93121
rjernst added a commit to rjernst/elasticsearch that referenced this pull request Jan 24, 2023
…c#93211)

The systemd unit file is part of the Elasticsearch package and should
not be edited. Instead, we recommend creating a service override file.
This commit tweaks the docs for setting tmp dir with systemd to use the
override file instead of editing the unit file.

relates elastic#93121
elasticsearchmachine pushed a commit that referenced this pull request Jan 24, 2023
#93214)

The systemd unit file is part of the Elasticsearch package and should
not be edited. Instead, we recommend creating a service override file.
This commit tweaks the docs for setting tmp dir with systemd to use the
override file instead of editing the unit file.

relates #93121
elasticsearchmachine pushed a commit that referenced this pull request Jan 24, 2023
#93215)

The systemd unit file is part of the Elasticsearch package and should
not be edited. Instead, we recommend creating a service override file.
This commit tweaks the docs for setting tmp dir with systemd to use the
override file instead of editing the unit file.

relates #93121
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

:Core/Infra/Core Core issues without another label >docs General docs changes Team:Core/Infra Meta label for core/infra team Team:Docs Meta label for docs team v7.17.9 v8.6.1 v8.7.0

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants