> This is a merge request to fix the issue where a running jstatd does not see > JVMs in docker containers started since the jstatd itself started. > > jstatd was using the set of temp directories gathered on startup to scan for > VMs, by reading their perfdata file. That includes existing running VMs in > docker instances using tmp dirs such as /proc/PID/root/tmp/. But it needs to > re-evaluate that set of temp dirs continually to see new VMs in docker > containers. Rescanning also means it "forgets" about tmp dirs which have > disappeared, for containers that have shutdown, and it will now not be > scanning those directories forever. > > Have been testing this manually, watching jps connecting to jstatd and show > newly created JVMs in docker containers come and go. Running existing tests > OK so far.
Kevin Walls has updated the pull request incrementally with one additional commit since the last revision: Remove tmpdirs member, call getTempDirectories() directly as needed. ------------- Changes: - all: https://git.openjdk.java.net/jdk/pull/3908/files - new: https://git.openjdk.java.net/jdk/pull/3908/files/658a9e27..3654447b Webrevs: - full: https://webrevs.openjdk.java.net/?repo=jdk&pr=3908&range=01 - incr: https://webrevs.openjdk.java.net/?repo=jdk&pr=3908&range=00-01 Stats: 7 lines in 1 file changed: 0 ins; 6 del; 1 mod Patch: https://git.openjdk.java.net/jdk/pull/3908.diff Fetch: git fetch https://git.openjdk.java.net/jdk pull/3908/head:pull/3908 PR: https://git.openjdk.java.net/jdk/pull/3908
