RFR : 8225345: Provide Cloud IAAS related info on Linux in the hs_err file

[David Holmes]

On 6/06/2019 10:17 pm, Baesken, Matthias wrote:
> Hi  David ,   sure I can  guard this by  "ExtensiveErrorReports" .
> 
>>
>> Sorry but how is it at all relevant to crash diagnosis?
>>
> 
> It  gives a more complete picture where you run , helps to improve the error analysis .
> ( like - involve  the  Cloud escalation team  because ...  )

Sorry not seeing it.

>>
>> And maybe a customer doesn't want you to know whose Cloud services they  use!
>>
> 
> Some people might not want the  number of cores in hs_err, some might not want to OS version  ,
> some people might  not want something else in hs_err .
> That's always hard to say , but I think  the Cloud info is on the same technical level  than what we  currently  already have in the  hs_err  so
>   Nothing special .

Vastly different IMO. Whose Cloud services company X uses could be 
considered commercial-in-confidence. I don't see anything "technical" in 
knowing whose cloud was used.

David
-----

> 
> Best regards, Matthias
> 
> 
>> From: David Holmes <david.holmes at oracle.com>
>> Sent: Donnerstag, 6. Juni 2019 13:51
>> To: Baesken, Matthias <matthias.baesken at sap.com>; 'hotspot-
>> dev at openjdk.java.net' <hotspot-dev at openjdk.java.net>
>> Subject: Re: RFR : 8225345: Provide Cloud IAAS related info on Linux in the
>> hs_err file
>>
>> Hi Matthias,
>>
>> On 6/06/2019 9:29 pm, Baesken, Matthias wrote:
>>> Hello, please review the following change .
>>> It  enhances the hs_error file  output on Linux by  Cloud IAAS detection .
>>>
>>> The DMI information  exposed  by the  /sys virtual filesystem  can be used
>> to detect various Cloud IAAS on Linux.
>>> It is helpful, for instance in support cases, to get some information about
>> the Cloud IAAS in the hs_error file .
>>
>> Sorry but how is it at all relevant to crash diagnosis?
>>
>> And maybe a customer doesn't want you to know whose Cloud services they
>> use!
>>
>> This seems like something that should be guarded by
>> ExtensiveErrorReports - at best.
>>
>> David
>> -----
>>
>>>
>>> Bug/webrev :
>>>
>>> https://bugs.openjdk.java.net/browse/JDK-8225345
>>>
>>> http://cr.openjdk.java.net/~mbaesken/webrevs/8225345.0/
>>>
>>>
>>> Thanks, Matthias
>>>