Skip to content
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

Use min node version to guard injecting settings in logs provider #123005

Merged
merged 5 commits into from
Feb 20, 2025

Conversation

kkrik-es
Copy link
Contributor

The logs index settings provider injects settings that were introduced in 8.17 and 8.18, so it may cause issues during rolling upgrades as some nodes may not be able to process the injected settings.

Fixes #122950

@elasticsearchmachine
Copy link
Collaborator

Hi @kkrik-es, I've created a changelog YAML for you.

@kkrik-es kkrik-es added auto-backport Automatically create backport pull requests when merged v8.18.1 v8.19.0 v9.0.1 v8.17.3 labels Feb 20, 2025
@kkrik-es kkrik-es requested a review from martijnvg February 20, 2025 13:09
@kkrik-es kkrik-es marked this pull request as ready for review February 20, 2025 13:09
@elasticsearchmachine
Copy link
Collaborator

Pinging @elastic/es-storage-engine (Team:StorageEngine)

Copy link
Member

@martijnvg martijnvg left a comment

Choose a reason for hiding this comment

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

LGTM 👍

@elasticsearchmachine
Copy link
Collaborator

💔 Backport failed

Status Branch Result
8.18
8.x
9.0
8.17 Commit could not be cherrypicked due to conflicts

You can use sqren/backport to manually backport by running backport --upstream elastic/elasticsearch --pr 123005

kkrik-es added a commit to kkrik-es/elasticsearch that referenced this pull request Feb 20, 2025
…astic#123005)

* Use min node version to guard injecting settings in logs provider

* Update docs/changelog/123005.yaml

* no random in cluster init
kkrik-es added a commit to kkrik-es/elasticsearch that referenced this pull request Feb 20, 2025
…astic#123005)

* Use min node version to guard injecting settings in logs provider

* Update docs/changelog/123005.yaml

* no random in cluster init
kkrik-es added a commit to kkrik-es/elasticsearch that referenced this pull request Feb 20, 2025
…astic#123005)

* Use min node version to guard injecting settings in logs provider

* Update docs/changelog/123005.yaml

* no random in cluster init
@kkrik-es
Copy link
Contributor Author

💚 All backports created successfully

Status Branch Result
8.17

Questions ?

Please refer to the Backport tool documentation

kkrik-es added a commit to kkrik-es/elasticsearch that referenced this pull request Feb 20, 2025
…astic#123005)

* Use min node version to guard injecting settings in logs provider

* Update docs/changelog/123005.yaml

* no random in cluster init

(cherry picked from commit 5c12978)

# Conflicts:
#	x-pack/plugin/logsdb/src/main/java/org/elasticsearch/xpack/logsdb/LogsDBPlugin.java
#	x-pack/plugin/logsdb/src/main/java/org/elasticsearch/xpack/logsdb/LogsdbIndexModeSettingsProvider.java
#	x-pack/plugin/logsdb/src/test/java/org/elasticsearch/xpack/logsdb/LogsdbIndexModeSettingsProviderTests.java
#	x-pack/plugin/logsdb/src/test/java/org/elasticsearch/xpack/logsdb/SyntheticSourceIndexSettingsProviderLegacyLicenseTests.java
elasticsearchmachine pushed a commit that referenced this pull request Feb 20, 2025
…23005) (#123047)

* Use min node version to guard injecting settings in logs provider

* Update docs/changelog/123005.yaml

* no random in cluster init
elasticsearchmachine pushed a commit that referenced this pull request Feb 20, 2025
…23005) (#123048)

* Use min node version to guard injecting settings in logs provider

* Update docs/changelog/123005.yaml

* no random in cluster init
elasticsearchmachine pushed a commit that referenced this pull request Feb 20, 2025
…23005) (#123049)

* Use min node version to guard injecting settings in logs provider

* Update docs/changelog/123005.yaml

* no random in cluster init
kkrik-es added a commit that referenced this pull request Feb 20, 2025
…der (#123005) (#123051)

* Use min node version to guard injecting settings in logs provider (#123005)

* Use min node version to guard injecting settings in logs provider

* Update docs/changelog/123005.yaml

* no random in cluster init

(cherry picked from commit 5c12978)

# Conflicts:
#	x-pack/plugin/logsdb/src/main/java/org/elasticsearch/xpack/logsdb/LogsDBPlugin.java
#	x-pack/plugin/logsdb/src/main/java/org/elasticsearch/xpack/logsdb/LogsdbIndexModeSettingsProvider.java
#	x-pack/plugin/logsdb/src/test/java/org/elasticsearch/xpack/logsdb/LogsdbIndexModeSettingsProviderTests.java
#	x-pack/plugin/logsdb/src/test/java/org/elasticsearch/xpack/logsdb/SyntheticSourceIndexSettingsProviderLegacyLicenseTests.java

* port fixes
afoucret pushed a commit to afoucret/elasticsearch that referenced this pull request Feb 21, 2025
…astic#123005)

* Use min node version to guard injecting settings in logs provider

* Update docs/changelog/123005.yaml

* no random in cluster init
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Error triggered while upgrading [index.mapping.source.mode] did you mean [index.mapping.coerce]
3 participants