Re: RFR: 8289711: Add container configuration data to mbeans [v10]

2022-09-29 Thread Andrey Turbanov
On Wed, 10 Aug 2022 02:10:25 GMT, xpbob wrote: >> Container configuration information is useful for troubleshooting >> problems,Exposing information in MBeans is ideal for monitoring, jConsole, >> and other scenarios. >> Results the following >> ![图片](https://user-images.githubusercontent.com/7

Re: RFR: 8289711: Add container configuration data to mbeans [v7]

2022-08-10 Thread xpbob
On Wed, 3 Aug 2022 09:18:26 GMT, Alan Bateman wrote: >> xpbob has updated the pull request incrementally with two additional commits >> since the last revision: >> >> - remove line >> - add export > > What happened with the experiment to move ContainerInfoMXBean to > jdk.management? Hi, @Al

Re: RFR: 8289711: Add container configuration data to mbeans [v10]

2022-08-09 Thread xpbob
> Container configuration information is useful for troubleshooting > problems,Exposing information in MBeans is ideal for monitoring, jConsole, > and other scenarios. > Results the following > ![图片](https://user-images.githubusercontent.com/7837910/177248834-50beefe9-4db6-470c-8f15-df5a93892804.

Re: RFR: 8289711: Add container configuration data to mbeans [v9]

2022-08-03 Thread xpbob
On Wed, 3 Aug 2022 10:36:06 GMT, xpbob wrote: >> Container configuration information is useful for troubleshooting >> problems,Exposing information in MBeans is ideal for monitoring, jConsole, >> and other scenarios. >> Results the following >> ![图片](https://user-images.githubusercontent.com/78

Re: RFR: 8289711: Add container configuration data to mbeans [v8]

2022-08-03 Thread xpbob
On Wed, 3 Aug 2022 09:51:06 GMT, xpbob wrote: >> Container configuration information is useful for troubleshooting >> problems,Exposing information in MBeans is ideal for monitoring, jConsole, >> and other scenarios. >> Results the following >> ![图片](https://user-images.githubusercontent.com/78

Re: RFR: 8289711: Add container configuration data to mbeans [v9]

2022-08-03 Thread xpbob
> Container configuration information is useful for troubleshooting > problems,Exposing information in MBeans is ideal for monitoring, jConsole, > and other scenarios. > Results the following > ![图片](https://user-images.githubusercontent.com/7837910/177248834-50beefe9-4db6-470c-8f15-df5a93892804.

Re: RFR: 8289711: Add container configuration data to mbeans [v8]

2022-08-03 Thread xpbob
> Container configuration information is useful for troubleshooting > problems,Exposing information in MBeans is ideal for monitoring, jConsole, > and other scenarios. > Results the following > ![图片](https://user-images.githubusercontent.com/7837910/177248834-50beefe9-4db6-470c-8f15-df5a93892804.

Re: RFR: 8289711: Add container configuration data to mbeans [v7]

2022-08-03 Thread Alan Bateman
On Wed, 3 Aug 2022 09:06:39 GMT, xpbob wrote: >> Container configuration information is useful for troubleshooting >> problems,Exposing information in MBeans is ideal for monitoring, jConsole, >> and other scenarios. >> Results the following >> ![图片](https://user-images.githubusercontent.com/78

Re: RFR: 8289711: Add container configuration data to mbeans [v7]

2022-08-03 Thread Alan Bateman
On Wed, 3 Aug 2022 09:06:39 GMT, xpbob wrote: >> Container configuration information is useful for troubleshooting >> problems,Exposing information in MBeans is ideal for monitoring, jConsole, >> and other scenarios. >> Results the following >> ![图片](https://user-images.githubusercontent.com/78

Re: RFR: 8289711: Add container configuration data to mbeans [v7]

2022-08-03 Thread xpbob
> Container configuration information is useful for troubleshooting > problems,Exposing information in MBeans is ideal for monitoring, jConsole, > and other scenarios. > Results the following > ![图片](https://user-images.githubusercontent.com/7837910/177248834-50beefe9-4db6-470c-8f15-df5a93892804.

Re: RFR: 8289711: Add container configuration data to mbeans [v2]

2022-07-14 Thread Laurence Cable
agree Alan On 7/14/22 1:27 AM, Alan Bateman wrote: On Thu, 14 Jul 2022 07:54:56 GMT, xpbob wrote: Thanks for review. The runtime information is already fetched through the OperatingSystemMXBean, and I'm wrapped through the interface, keeping only the configuration data.Such an interface cou

Re: RFR: 8289711: Add container configuration data to mbeans [v2]

2022-07-14 Thread xpbob
On Thu, 14 Jul 2022 08:23:37 GMT, Alan Bateman wrote: >>> > Thanks for review. >>> > I add mBeans using the registerMBean method. >>> > We can get configuration information through JConsole, JMX exporter >>> >>> This iteration is a bit confusing because it adds a public interface to >>> java.la

Re: RFR: 8289711: Add container configuration data to mbeans [v2]

2022-07-14 Thread Alan Bateman
On Thu, 14 Jul 2022 07:54:56 GMT, xpbob wrote: > Thanks for review. The runtime information is already fetched through the > OperatingSystemMXBean, and I'm wrapped through the interface, keeping only > the configuration data.Such an interface could be part of management. I don't think this fea

Re: RFR: 8289711: Add container configuration data to mbeans [v2]

2022-07-14 Thread xpbob
On Thu, 14 Jul 2022 07:05:25 GMT, Alan Bateman wrote: > > Thanks for review. > > I add mBeans using the registerMBean method. > > We can get configuration information through JConsole, JMX exporter > > This iteration is a bit confusing because it adds a public interface to > java.lang.managemen

Re: RFR: 8289711: Add container configuration data to mbeans [v2]

2022-07-14 Thread Alan Bateman
On Thu, 14 Jul 2022 03:32:35 GMT, xpbob wrote: > Thanks for review. > I add mBeans using the registerMBean method. > We can get configuration information through JConsole, JMX exporter This iteration is a bit confusing because it adds a public interface to java.lang.management. For the register

Re: RFR: 8289711: Add container configuration data to mbeans [v2]

2022-07-13 Thread xpbob
On Tue, 12 Jul 2022 15:48:57 GMT, Alan Bateman wrote: >>> @AlanBateman @jerboaa That's a good idea.Adding a special bean is only >>> available on Linux systems.I do not know the process of creating a CSR, can >>> you help me create a CSR >> >> It's too early to think about a CSR, probably a bi

Re: RFR: 8289711: Add container configuration data to mbeans [v6]

2022-07-13 Thread xpbob
> Container configuration information is useful for troubleshooting > problems,Exposing information in MBeans is ideal for monitoring, jConsole, > and other scenarios. > Results the following > ![图片](https://user-images.githubusercontent.com/7837910/177248834-50beefe9-4db6-470c-8f15-df5a93892804.

Re: RFR: 8289711: Add container configuration data to mbeans [v5]

2022-07-13 Thread xpbob
> Container configuration information is useful for troubleshooting > problems,Exposing information in MBeans is ideal for monitoring, jConsole, > and other scenarios. > Results the following > ![图片](https://user-images.githubusercontent.com/7837910/177248834-50beefe9-4db6-470c-8f15-df5a93892804.

Re: RFR: 8289711: Add container configuration data to mbeans [v4]

2022-07-13 Thread xpbob
> Container configuration information is useful for troubleshooting > problems,Exposing information in MBeans is ideal for monitoring, jConsole, > and other scenarios. > Results the following > ![图片](https://user-images.githubusercontent.com/7837910/177248834-50beefe9-4db6-470c-8f15-df5a93892804.

Re: RFR: 8289711: Add container configuration data to mbeans [v3]

2022-07-13 Thread xpbob
> Container configuration information is useful for troubleshooting > problems,Exposing information in MBeans is ideal for monitoring, jConsole, > and other scenarios. > Results the following > ![图片](https://user-images.githubusercontent.com/7837910/177248834-50beefe9-4db6-470c-8f15-df5a93892804.

Re: RFR: 8289711: Add container configuration data to mbeans [v2]

2022-07-12 Thread Alan Bateman
On Mon, 11 Jul 2022 09:30:59 GMT, Alan Bateman wrote: >> It's not clear that introducing this as a standard API is the right thing to >> do. Are you 100% confident that the concepts of "CPU quota", "CPU shares", >> "CPU period", "soft limit" etc. will last the test of time and that we don't >>

Re: RFR: 8289711: Add container configuration data to mbeans [v2]

2022-07-12 Thread Poonam Bajaj
On Mon, 11 Jul 2022 09:30:59 GMT, Alan Bateman wrote: >> It's not clear that introducing this as a standard API is the right thing to >> do. Are you 100% confident that the concepts of "CPU quota", "CPU shares", >> "CPU period", "soft limit" etc. will last the test of time and that we don't >>

Re: RFR: 8289711: Add container configuration data to mbeans [v2]

2022-07-11 Thread Alan Bateman
On Wed, 6 Jul 2022 05:59:21 GMT, Alan Bateman wrote: >> xpbob has updated the pull request incrementally with one additional commit >> since the last revision: >> >> update header > > It's not clear that introducing this as a standard API is the right thing to > do. Are you 100% confident th

Re: RFR: 8289711: Add container configuration data to mbeans [v2]

2022-07-11 Thread Severin Gehwolf
On Wed, 6 Jul 2022 05:59:21 GMT, Alan Bateman wrote: >> xpbob has updated the pull request incrementally with one additional commit >> since the last revision: >> >> update header > > It's not clear that introducing this as a standard API is the right thing to > do. Are you 100% confident th

Re: RFR: 8289711: Add container configuration data to mbeans [v2]

2022-07-10 Thread xpbob
On Wed, 6 Jul 2022 05:59:21 GMT, Alan Bateman wrote: >> xpbob has updated the pull request incrementally with one additional commit >> since the last revision: >> >> update header > > It's not clear that introducing this as a standard API is the right thing to > do. Are you 100% confident th

Re: RFR: 8289711: Add container configuration data to mbeans [v2]

2022-07-07 Thread Severin Gehwolf
On Wed, 6 Jul 2022 03:52:30 GMT, xpbob wrote: >> Container configuration information is useful for troubleshooting >> problems,Exposing information in MBeans is ideal for monitoring, jConsole, >> and other scenarios. >> Results the following >> ![图片](https://user-images.githubusercontent.com/78

Re: RFR: 8289711: Add container configuration data to mbeans [v2]

2022-07-07 Thread xpbob
> Container configuration information is useful for troubleshooting > problems,Exposing information in MBeans is ideal for monitoring, jConsole, > and other scenarios. > Results the following > ![图片](https://user-images.githubusercontent.com/7837910/177248834-50beefe9-4db6-470c-8f15-df5a93892804.

Re: RFR: 8289711: Add container configuration data to mbeans

2022-07-06 Thread Jie Fu
On Tue, 5 Jul 2022 04:21:55 GMT, xpbob wrote: > Container configuration information is useful for troubleshooting > problems,Exposing information in MBeans is ideal for monitoring, jConsole, > and other scenarios. > Results the following > ![图片](https://user-images.githubusercontent.com/7837910

Re: RFR: 8289711: Add container configuration data to mbeans

2022-07-06 Thread xpbob
On Tue, 5 Jul 2022 04:21:55 GMT, xpbob wrote: > Container configuration information is useful for troubleshooting > problems,Exposing information in MBeans is ideal for monitoring, jConsole, > and other scenarios. > Results the following > ![图片](https://user-images.githubusercontent.com/7837910

Re: RFR: 8289711: Add container configuration data to mbeans [v2]

2022-07-06 Thread Alan Bateman
On Wed, 6 Jul 2022 03:52:30 GMT, xpbob wrote: >> Container configuration information is useful for troubleshooting >> problems,Exposing information in MBeans is ideal for monitoring, jConsole, >> and other scenarios. >> Results the following >> ![图片](https://user-images.githubusercontent.com/78

RFR: 8289711: Add container configuration data to mbeans

2022-07-05 Thread xpbob
Container configuration information is useful for troubleshooting problems,Exposing information in MBeans is ideal for monitoring, jConsole, and other scenarios. Results the following ![图片](https://user-images.githubusercontent.com/7837910/177248834-50beefe9-4db6-470c-8f15-df5a93892804.png)