minibison / javamelody

Automatically exported from code.google.com/p/javamelody
1 stars 0 forks source link

JavaMelody reports OS as Linux unknown #439

Closed GoogleCodeExporter closed 9 years ago

GoogleCodeExporter commented 9 years ago
What steps will reproduce the problem?
1. Run JIRA or Confluence on CentOS
2. Browse to JavaMelody and expand "System Information" -> Details
3. See what OS JavaMelody reports

What is the expected output? What do you see instead?

I expect to see something like:

Linux 2.6.18-371.12.1.el5 amd64/64 (4 cores)

or:

CentOS release 5.10 (Final) (x86_64)

JavaMelody is actually displaying:  "Linux unknown, amd64/64 (4 cores)"

What version of the product are you using? On what application server, JDK, 
operating system?

JavaMelody v1.52.0 (also affects 1.53 snapshot)
Application server: Apache Tomcat/7.0.55 (also 7.0.52).
JDK:  1.8.0_20-b26 (also 1.7.0_60-b19)
Operating System: Linux 2.6.18-371.12.1.el5  (also Linux 2.6.18-194.26.1.el5)

Please provide any additional information below.

Original issue reported on code.google.com by msymo...@gmail.com on 28 Sep 2014 at 3:27

GoogleCodeExporter commented 9 years ago
"Linux unknown, amd64/64" are the java system properties "os.name 
sun.os.patch.level, os.arch/sun.arch.data.model"

You are right, I should add "os.version" and remove "sun.os.patch.level" on 
Linux.
This would give something like:
- "Linux, 2.6.32-358.23.2.el6.x86_64, amd64/64" on Linux
- "Windows 7, Service Pack 1, x86/32" on Windows

Original comment by evernat@free.fr on 28 Sep 2014 at 4:35

GoogleCodeExporter commented 9 years ago
It's done in trunk as said above (revision 3927) and ready for the next release 
(1.53).

Thanks!

Original comment by evernat@free.fr on 28 Sep 2014 at 4:37

GoogleCodeExporter commented 9 years ago
[deleted comment]
GoogleCodeExporter commented 9 years ago
Everything look great.

Tested with JavaMelody 1.53.0-SNAPSHOT (build #304 from Sept 28th 2014) on 
Confluence

Original comment by msymo...@gmail.com on 30 Sep 2014 at 1:44

Attachments: