-
Notifications
You must be signed in to change notification settings - Fork 1.2k
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Monitoring & Tracing capability for multiple Thread pools of Che Server #14727
Conversation
a9d80b1
to
88b35f2
Compare
88b35f2
to
25782db
Compare
E2E tests of Eclipse Che Multiuser on OCP has been successful:
|
…y Signed-off-by: Sergii Kabashniuk <skabashniuk@redhat.com> Signed-off-by: Sergii Kabashniuk <skabashniuk@redhat.com>
…vability Signed-off-by: Sergii Kabashniuk <skabashniuk@redhat.com> Signed-off-by: Sergii Kabashniuk <skabashniuk@redhat.com>
I prefer to crash the whole JVM on guice startup phase. Because logs can be configured in the wrong way or it may be skipped or missed. But JVM crash unlikely can't be ignored
Java Flight Recorder requires a commercial license for use in production. To learn more about commercial features and how to enable them please visit http://www.oracle.com/technetwork/java/javaseproducts/. I think because of that we unlikely can reuse it in Eclipse Che. |
ci-build |
E2E Happy path tests of Eclipse Che Single User on K8S (minikube v1.1.1) has been successful:
|
E2E Happy path tests of Eclipse Che Single User on K8S (minikube v1.1.1) has failed:
|
E2E tests of Eclipse Che Multiuser on OCP has been successful:
|
ci-build |
Yes, fail fast and visibly is good. I'm just afraid of incompatibilities with different java versions and compatibility with java modules (if it will ever be a concern for us :)
JFR was opensourced in java 11 (https://openjdk.java.net/jeps/328). We (Red Hat) even have people working on it. |
Signed-off-by: Sergii Kabashniuk <skabashniuk@redhat.com>
E2E Happy path tests of Eclipse Che Single User on K8S (minikube v1.1.1) has been successful:
|
E2E tests of Eclipse Che Multiuser on OCP has been successful:
|
…lity Signed-off-by: Sergii Kabashniuk <skabashniuk@redhat.com> Signed-off-by: Sergii Kabashniuk <skabashniuk@redhat.com>
Signed-off-by: Sergii Kabashniuk <skabashniuk@redhat.com>
...bility/src/test/java/org/eclipse/che/commons/observability/TimedCronExecutorServiceTest.java
Outdated
Show resolved
Hide resolved
E2E Happy path tests of Eclipse Che Single User on K8S (minikube v1.1.1) has been successful:
|
…lity Signed-off-by: Sergii Kabashniuk <skabashniuk@redhat.com> Signed-off-by: Sergii Kabashniuk <skabashniuk@redhat.com>
E2E Happy path tests of Eclipse Che Single User on K8S (minikube v1.1.1) has been successful:
|
E2E tests of Eclipse Che Multiuser on OCP has failed:
|
…servability Signed-off-by: Sergii Kabashniuk <skabashniuk@redhat.com> Signed-off-by: Sergii Kabashniuk <skabashniuk@redhat.com>
E2E Happy path tests of Eclipse Che Single User on K8S (minikube v1.1.1) has been successful:
|
E2E Happy path tests of Eclipse Che Single User on K8S (minikube v1.1.1) has failed:
|
crw-ci-test |
E2E Happy path tests of Eclipse Che Single User on K8S (minikube v1.1.1) has been successful:
|
E2E tests of Eclipse Che Multiuser on OCP has been successful:
|
What does this PR do?
Add monitoring & Tracing capability for multiple Thread pools of Che Server.
ExecutorServiceWrapper
One of the trickiest thing that was preventing us to add this monitoring earlier was a
To help to handle this kind of situation I've added such interface
implementation of this class has to be injected in all classes where ExecutorService is used to wrap the instance of executor.
Minor enhancement.
Grafana
To help to consume values from monitored executor services I've added some changes on Grafana dashboards.
Executors
Traces
What issues does this PR fix or reference?
#14601
Release Notes
n/a
Docs PR
eclipse-che/che-docs#870