Skip to content

Conversation

@rjernst
Copy link
Member

@rjernst rjernst commented Mar 14, 2025

Information about the system, and in particular the running JVM, is logged relatively late in startup. When problems occur in low level initialization, it is often useful to have this information. This commit moves logging of system information to right after logging has been initialized. Note that it no longer utilizes the JvmInfo class as most was already easily available and the static init of JvmInfo does a lot.

Information about the system, and in particular the running JVM, is
logged relatively late in startup. When problems occur in low level
initialization, it is often useful to have this information. This commit
moves logging of system information to right after logging has been
initialized. Note that it no longer utilizes the JvmInfo class as most
was already easily available and the static init of JvmInfo does a lot.
@rjernst rjernst added :Core/Infra/Core Core issues without another label >refactoring auto-backport Automatically create backport pull requests when merged v8.18.1 v8.19.0 v9.0.1 v9.1.0 labels Mar 14, 2025
@rjernst rjernst requested review from a team as code owners March 14, 2025 21:32
@elasticsearchmachine elasticsearchmachine added the Team:Core/Infra Meta label for core/infra team label Mar 14, 2025
@elasticsearchmachine
Copy link
Collaborator

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

Copy link
Contributor

@ldematte ldematte left a comment

Choose a reason for hiding this comment

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

LGTM

@rjernst
Copy link
Member Author

rjernst commented Mar 15, 2025

@elasticmachine update branch

@rjernst rjernst enabled auto-merge (squash) March 15, 2025 13:51
@rjernst rjernst merged commit 789eb2f into elastic:main Mar 15, 2025
17 checks passed
rjernst added a commit to rjernst/elasticsearch that referenced this pull request Mar 15, 2025
Information about the system, and in particular the running JVM, is
logged relatively late in startup. When problems occur in low level
initialization, it is often useful to have this information. This commit
moves logging of system information to right after logging has been
initialized. Note that it no longer utilizes the JvmInfo class as most
was already easily available and the static init of JvmInfo does a lot.
@elasticsearchmachine
Copy link
Collaborator

💔 Backport failed

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

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

elasticsearchmachine pushed a commit that referenced this pull request Mar 15, 2025
Information about the system, and in particular the running JVM, is
logged relatively late in startup. When problems occur in low level
initialization, it is often useful to have this information. This commit
moves logging of system information to right after logging has been
initialized. Note that it no longer utilizes the JvmInfo class as most
was already easily available and the static init of JvmInfo does a lot.
mosche pushed a commit to mosche/elasticsearch that referenced this pull request Jul 24, 2025
Information about the system, and in particular the running JVM, is
logged relatively late in startup. When problems occur in low level
initialization, it is often useful to have this information. This commit
moves logging of system information to right after logging has been
initialized. Note that it no longer utilizes the JvmInfo class as most
was already easily available and the static init of JvmInfo does a lot.

(cherry picked from commit 789eb2f)

# Conflicts:
#	server/src/main/java/org/elasticsearch/bootstrap/Elasticsearch.java
#	server/src/main/java/org/elasticsearch/node/NodeConstruction.java
mosche pushed a commit to mosche/elasticsearch that referenced this pull request Jul 24, 2025
Information about the system, and in particular the running JVM, is
logged relatively late in startup. When problems occur in low level
initialization, it is often useful to have this information. This commit
moves logging of system information to right after logging has been
initialized. Note that it no longer utilizes the JvmInfo class as most
was already easily available and the static init of JvmInfo does a lot.

(cherry picked from commit 789eb2f)

# Conflicts:
#	server/src/main/java/org/elasticsearch/bootstrap/Elasticsearch.java
#	server/src/main/java/org/elasticsearch/node/NodeConstruction.java
@mosche
Copy link
Contributor

mosche commented Jul 24, 2025

💚 All backports created successfully

Status Branch Result
8.19
8.18

Questions ?

Please refer to the Backport tool documentation

elasticsearchmachine pushed a commit that referenced this pull request Jul 28, 2025
Information about the system, and in particular the running JVM, is
logged relatively late in startup. When problems occur in low level
initialization, it is often useful to have this information. This commit
moves logging of system information to right after logging has been
initialized. Note that it no longer utilizes the JvmInfo class as most
was already easily available and the static init of JvmInfo does a lot.

(cherry picked from commit 789eb2f)

# Conflicts:
#	server/src/main/java/org/elasticsearch/bootstrap/Elasticsearch.java
#	server/src/main/java/org/elasticsearch/node/NodeConstruction.java

Co-authored-by: Ryan Ernst <[email protected]>
elasticsearchmachine pushed a commit that referenced this pull request Jul 28, 2025
Information about the system, and in particular the running JVM, is
logged relatively late in startup. When problems occur in low level
initialization, it is often useful to have this information. This commit
moves logging of system information to right after logging has been
initialized. Note that it no longer utilizes the JvmInfo class as most
was already easily available and the static init of JvmInfo does a lot.

(cherry picked from commit 789eb2f)

# Conflicts:
#	server/src/main/java/org/elasticsearch/bootstrap/Elasticsearch.java
#	server/src/main/java/org/elasticsearch/node/NodeConstruction.java

Co-authored-by: Ryan Ernst <[email protected]>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

auto-backport Automatically create backport pull requests when merged backport pending :Core/Infra/Core Core issues without another label >refactoring Team:Core/Infra Meta label for core/infra team v8.18.1 v8.19.0 v9.0.1 v9.1.0

Projects

None yet

Development

Successfully merging this pull request may close these issues.

5 participants