Ping!! Re: RFR: 8203357 Container Metrics

[Roger Riggs]

Hi,

Just a thought on the timeslice units,  would nanoseconds be more future 
proof?
In a variety of cases, milliseconds are starting to seem too coarse.

$.02, Roger


On 6/11/18 10:12 AM, Bob Vandette wrote:
>
>> On Jun 11, 2018, at 4:32 AM, David Holmes <david.holmes at oracle.com 
>> <mailto:david.holmes at oracle.com>> wrote:
>>
>> Sorry Bob I haven't had a chance to look at this detail.
>>
>> For the Java code ... methods that return arrays should return 
>> zero-length arrays when something is not available rather than null.
>
> All methods do return zero length arrays except I missed the 
> getPerCpuUsage.  I’ll fix that one and
> correct the javadoc.
>
>>
>> For getCpuPeriod() the term "operating system time slice" can be 
>> misconstrued as being related to the scheduler timeslice that may, or 
>> may not, exist, depending on the scheduler and scheduling policy etc. 
>> This "timeslice" is something specific to cgroups - no?
>
> The comments reads:
>
>        * Returns the length of the operating system time slice, in
>        * milliseconds, for processes within the Isolation Group.
> The comment does infer that it’s process and cgroup (Isolation group) 
> specific and not the generic os timeslice.
> Isn’t this sufficient?
>
> Thanks,
> Bob.
>>
>> David
>>
>> On 8/06/2018 3:43 AM, Bob Vandette wrote:
>>> Can I get one more reviewer for this RFE so I can integrate it?
>>>> http://cr.openjdk.java.net/~bobv/8203357/webrev.01 
>>>> <http://cr.openjdk.java.net/%7Ebobv/8203357/webrev.01>
>>> Mandy Chung has reviewed this change.
>>> I’ve run Mach5 hotspot and core lib tests.
>>> I’ve reviewed the tests which were written by Harsha Wardhana
>>> I filed a CSR for the command line change and it’s now approved and 
>>> closed.
>>> Thanks,
>>> Bob.
>>>> On May 30, 2018, at 3:45 PM, Bob Vandette <bob.vandette at oracle.com 
>>>> <mailto:bob.vandette at oracle.com>> wrote:
>>>>
>>>> Please review the following RFE which adds an internal API, along 
>>>> with jtreg tests that provide
>>>> access to Docker container configuration data and metrics.  In 
>>>> addition to the API which we hope to
>>>> take advantage of in the future with Java Flight Recorder and a JMX 
>>>> Mbean, I’ve added an additional
>>>> option to -XshowSettings:system than dumps out the container or 
>>>> host cgroup confguration
>>>> information.  See the sample output below:
>>>>
>>>> RFE: Container Metrics
>>>>
>>>> https://bugs.openjdk.java.net/browse/JDK-8203357
>>>>
>>>> WEBREV:
>>>>
>>>> http://cr.openjdk.java.net/~bobv/8203357/webrev.01
>>>>
>>>>
>>>> This commit will also include a fix for the following bug.
>>>>
>>>> BUG: [TESTBUG] Test /runtime/containers/cgroup/PlainRead.java fails
>>>>
>>>> https://bugs.openjdk.java.net/browse/JDK-8203691
>>>>
>>>> WEBREV:
>>>>
>>>> http://cr.openjdk.java.net/~bobv/8203357/webrev.00/test/hotspot/jtreg/runtime/containers/cgroup/PlainRead.java.sdiff.html
>>>>
>>>> SAMPLE USAGE and OUTPUT:
>>>>
>>>> docker run —memory=256m --cpuset-cpus 4-7 -it ubuntu bash
>>>> ./java -XshowSettings:system
>>>> Operating System Metrics:
>>>>    Provider: cgroupv1
>>>>    Effective CPU Count: 4
>>>>    CPU Period: 100000
>>>>    CPU Quota: -1
>>>>    CPU Shares: -1
>>>>    List of Processors, 4 total:
>>>>    4 5 6 7
>>>>    List of Effective Processors, 4 total:
>>>>    4 5 6 7
>>>>    List of Memory Nodes, 2 total:
>>>>    0 1
>>>>    List of Available Memory Nodes, 2 total:
>>>>    0 1
>>>>    CPUSet Memory Pressure Enabled: false
>>>>    Memory Limit: 256.00M
>>>>    Memory Soft Limit: Unlimited
>>>>    Memory & Swap Limit: 512.00M
>>>>    Kernel Memory Limit: Unlimited
>>>>    TCP Memory Limit: Unlimited
>>>>    Out Of Memory Killer Enabled: true
>>>>
>>>> TEST RESULTS:
>>>>
>>>> testing runtime container APIs
>>>> Directory "JTwork" not found: creating
>>>> Passed: runtime/containers/cgroup/PlainRead.java
>>>> Passed: runtime/containers/docker/DockerBasicTest.java
>>>> Passed: runtime/containers/docker/TestCPUAwareness.java
>>>> Passed: runtime/containers/docker/TestCPUSets.java
>>>> Passed: runtime/containers/docker/TestMemoryAwareness.java
>>>> Passed: runtime/containers/docker/TestMisc.java
>>>> Test results: passed: 6
>>>> Results written to /export/users/bobv/jdk11/build/jtreg/JTwork
>>>>
>>>> testing jdk.internal.platform APIs
>>>> Passed: jdk/internal/platform/cgroup/TestCgroupMetrics.java
>>>> Passed: jdk/internal/platform/docker/TestDockerCpuMetrics.java
>>>> Passed: jdk/internal/platform/docker/TestDockerMemoryMetrics.java
>>>> Passed: jdk/internal/platform/docker/TestSystemMetrics.java
>>>> Test results: passed: 4
>>>> Results written to /export/users/bobv/jdk11/build/jtreg/JTwork
>>>>
>>>> testing -XshowSettings:system launcher option
>>>> Passed: tools/launcher/Settings.java
>>>> Test results: passed: 1
>>>>
>>>>
>>>> Bob.
>>>>
>>>>
>

-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://mail.openjdk.java.net/pipermail/serviceability-dev/attachments/20180611/44369368/attachment-0001.html>