Skip to content
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

Include Runtime diagnostics tool into the server #13402

Merged
merged 1 commit into from
Mar 6, 2024

Conversation

arunans23
Copy link
Member

@arunans23 arunans23 commented Mar 5, 2024

Purpose

Describe the problems, issues, or needs driving this feature/fix and include links to related issues in the following format: Resolves issue1, issue2, etc.

$subject
Resolves: wso2/micro-integrator#2985

Currently, APIM and MI are sharing the synapse runtime for transport and message mediation. In error scenarios, it is time-consuming to gather all the information to investigate the issue. This PR is about implementing a tool that helps to generate the required data for troubleshooting issues quickly.

The Readme guide can be used to get an understanding of the tool. https://github.com/wso2/runtime-diagnostic-tool/blob/main/README.md The configurations are added by default. Once you start the server, the tool will be started as a second process and monitor the server. The diagnostics logs can be found inside /diagnostics-tool/logs. The output zip files are stored inside /diagnostics-tool/data.

High level Architecture can be found below.

Screenshot 2024-03-05 at 17 34 16

There are different watcher threads running in the Diagnostic tool which monitors the server performance and logs. Once an anomaly/error log is detected, the configured Action Executors will be executed and the data will be stored in a Zip file. If a PostAction Executor (FTP, SFTP) is configured, the zip file will be uploaded to the configured location.

Copy link

codecov bot commented Mar 5, 2024

Codecov Report

All modified and coverable lines are covered by tests ✅

Project coverage is 24.31%. Comparing base (7af7612) to head (e7b5c1f).
Report is 53 commits behind head on master.

Additional details and impacted files
@@             Coverage Diff              @@
##             master   #13402      +/-   ##
============================================
+ Coverage     24.30%   24.31%   +0.01%     
- Complexity     1240     1242       +2     
============================================
  Files           254      254              
  Lines         11489    11489              
  Branches       1200     1200              
============================================
+ Hits           2792     2794       +2     
+ Misses         8670     8668       -2     
  Partials         27       27              
Flag Coverage Δ
integration_tests 24.31% <ø> (+0.01%) ⬆️

Flags with carried forward coverage won't be shown. Click here to find out more.

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

@tharikaGitHub tharikaGitHub merged commit de3f26a into wso2:master Mar 6, 2024
8 of 9 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Implement a diagnostic tool to capture necessary details during an error
3 participants