Skip to content

Debian pkg config file updated in snippet #1457

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Open
wants to merge 1 commit into
base: main
Choose a base branch
from

Conversation

eedugon
Copy link
Contributor

@eedugon eedugon commented May 22, 2025

Minor change to fix the package configuration file in the Debian installation guide, which was incorrectly set to the RPM based file.

@shainaraskas : As the content came from a reused snippet we have lost the code formatting. Let me know your preference here. If you want to not use the snippet for this block of content or if we keep using it and we lose the code formatting.

If we use `{{pkg-conf}}` the substitution doesn't work.

I'm going to write a docs-builder issue to see if we can support inline code blocks with substitutions.

Closes #1430

Copy link
Collaborator

@shainaraskas shainaraskas left a comment

Choose a reason for hiding this comment

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

approved with a possible workaround to the code formatting thing you can take or leave

@@ -4,7 +4,7 @@ The `setgid` flag applies group permissions on the `/etc/elasticsearch` director

{{es}} loads its configuration from the `/etc/elasticsearch/elasticsearch.yml` file by default. The format of this config file is explained in [](/deploy-manage/deploy/self-managed/configure-elasticsearch.md).

The {{distro}} package also has a system configuration file (`/etc/sysconfig/elasticsearch`), which allows you to set the following parameters:
The {{distro}} package also has a system configuration file ({{pkg-conf}}), which allows you to set the following parameters:
Copy link
Collaborator

@shainaraskas shainaraskas May 22, 2025

Choose a reason for hiding this comment

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

I don't mind the missing code formatting too much. you could do it like that, or alter it like this:

Suggested change
The {{distro}} package also has a system configuration file ({{pkg-conf}}), which allows you to set the following parameters:
The {{distro}} package also has a system configuration file at the following path:
```txt subs=true
{{pkg-conf}}
```
In this file, you can set the following parameters:

no strong preference.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Team:Admin Issues owned by the Admin Docs Team
Projects
None yet
Development

Successfully merging this pull request may close these issues.

[Issue]: /etc/sysconfig/elasticsearch file referenced in Debian install instructions
2 participants