Skip to content

Conversation

eedugon
Copy link
Contributor

@eedugon eedugon commented Sep 24, 2025

Moving DNS cache settings content from important system configuration to Important settings configuration section.

The important system configuration section is intended for OS settings that need to be configured before installing Elasticsearch, and the DNS cache settings page is a post-installation config item and not even an OS setting.

The content was originally an indivitual page and we are integrating the content into the "important settings configuration" section.

In a separate PR we should also place the content into Elasticsearch reference docs, same as the rest of that page.

Related issue: #3007

Copy link

github-actions bot commented Sep 24, 2025

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.

Maybe we should move the content to Elasticsearch reference docs, same as GC logging settings?

I think the settings should be duplicated to the JVM setting reference to keep both sets of content complete.

the only reason i'd put it only in the reference is if the relative importance of setting it is reduced

@eedugon eedugon requested a review from a team as a code owner September 24, 2025 19:13
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.

lgtm

@eedugon eedugon merged commit eda4790 into main Sep 24, 2025
7 checks passed
@eedugon eedugon deleted the dns_jvm_move branch September 24, 2025 19:49
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.

2 participants