Skip to content

Use variables for installation instructions#5331

Open
yetanothertw wants to merge 5 commits intomainfrom
2524-variables-use
Open

Use variables for installation instructions#5331
yetanothertw wants to merge 5 commits intomainfrom
2524-variables-use

Conversation

@yetanothertw
Copy link
Contributor

@yetanothertw yetanothertw commented Mar 2, 2026

Summary

Fixes #2524 and makes use of version variables.

For the Install with Debian package page it adds tabs for installing from the APT repository step.

For the Install with RPM package page, it adds a note about replacing specific version numbers.

Generative AI disclosure

  1. Did you use a generative AI (GenAI) tool to assist in creating this contribution?
  • Yes
  • No

@github-actions
Copy link
Contributor

github-actions bot commented Mar 2, 2026

Vale Linting Results

Summary: 1 warning found

⚠️ Warnings (1)
File Line Rule Message
deploy-manage/deploy/self-managed/install-elasticsearch-with-debian-package.md 95 Elastic.BritishSpellings Use American English spelling 'among' instead of British English 'amongst'.

The Vale linter checks documentation changes against the Elastic Docs style guide.

To use Vale locally or report issues, refer to Elastic style guide for Vale.

@github-actions
Copy link
Contributor

github-actions bot commented Mar 2, 2026

@yetanothertw yetanothertw marked this pull request as ready for review March 3, 2026 11:17
@yetanothertw yetanothertw requested a review from a team as a code owner March 3, 2026 11:17
@yetanothertw
Copy link
Contributor Author

Hi @eedugon, this PR is a leftover task from when we worked on this PR.

Does this look right to you, wdyt?

Copy link
Contributor

@eedugon eedugon left a comment

Choose a reason for hiding this comment

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

Feels weird. Repository configuration is independent of the version to install. The rpm or deb repo is the same for all 9.x releases.

```

2. Save the repository definition to `/etc/apt/sources.list.d/elastic-9.x.list`:
2. Save the repository definition to `/etc/apt/sources.list.d/elastic-<SPECIFIC.VERSION.NUMBER>.list`:
Copy link
Contributor

Choose a reason for hiding this comment

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

The file 9.x was correct and expected, as the repo is common for all 9.x releases. Users shouldn't substitute anything there (if im not mistaken pf course :) )

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

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Review guidance for the Debian and RPM installation instructions to use variables

2 participants