Skip to content

Commit

Permalink
8297959: Provide better descriptions for some Operating System JFR ev…
Browse files Browse the repository at this point in the history 
…ents

Reviewed-by: stuefe
  • Loading branch information
@MBaesken
MBaesken committed Dec 5, 2022
1 parent d523d9d commit dea2161
Showing 1 changed file with 9 additions and 3 deletions.
12 changes: 9 additions & 3 deletions src/hotspot/share/jfr/metadata/metadata.xml
View file
Original file line number Diff line number Diff line change
Expand Up @@ -732,11 +732,15 @@
<Field type="long" name="pid" label="Process Identifier" />
</Event>

<Event name="OSInformation" category="Operating System" label="OS Information" period="endChunk">
<Event name="OSInformation" category="Operating System" label="OS Information"
description="Description of the OS the JVM runs on, for example, a uname-like output"
period="endChunk">
<Field type="string" name="osVersion" label="OS Version" />
</Event>

<Event name="VirtualizationInformation" category="Operating System" label="Virtualization Information" period="endChunk">
<Event name="VirtualizationInformation" category="Operating System" label="Virtualization Information"
description="Description of the virtualization technology the JVM runs on"
period="endChunk">
<Field type="string" name="name" label="Name" />
</Event>

Expand All @@ -745,7 +749,9 @@
<Field type="string" name="value" label="Value" />
</Event>

<Event name="InitialEnvironmentVariable" category="Operating System" label="Initial Environment Variable" period="endChunk">
<Event name="InitialEnvironmentVariable" category="Operating System" label="Initial Environment Variable"
description="Key-value pairs for environment variables at JVM startup"
period="endChunk">
<Field type="string" name="key" label="Key" />
<Field type="string" name="value" label="Value" />
</Event>
Expand Down

3 comments on commit dea2161

@openjdk-notifier
Copy link

Choose a reason for hiding this comment

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

Review

Issues

@MBaesken
Copy link
Member Author

Choose a reason for hiding this comment

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

/backport jdk17u-dev

@openjdk
Copy link

@openjdk openjdk bot commented on dea2161 Dec 22, 2022

Choose a reason for hiding this comment

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

@MBaesken the backport was successfully created on the branch MBaesken-backport-dea2161f in my personal fork of openjdk/jdk17u-dev. To create a pull request with this backport targeting openjdk/jdk17u-dev:master, just click the following link:

➡️ Create pull request

The title of the pull request is automatically filled in correctly and below you find a suggestion for the pull request body:

Hi all,

This pull request contains a backport of commit dea2161f from the openjdk/jdk repository.

The commit being backported was authored by Matthias Baesken on 5 Dec 2022 and was reviewed by Thomas Stuefe.

Thanks!

If you need to update the source branch of the pull then run the following commands in a local clone of your personal fork of openjdk/jdk17u-dev:

$ git fetch https://github.com/openjdk-bots/jdk17u-dev MBaesken-backport-dea2161f:MBaesken-backport-dea2161f
$ git checkout MBaesken-backport-dea2161f
# make changes
$ git add paths/to/changed/files
$ git commit --message 'Describe additional changes made'
$ git push https://github.com/openjdk-bots/jdk17u-dev MBaesken-backport-dea2161f

Please sign in to comment.