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

Add Prometheus alerting functionality #1424

Merged
merged 105 commits into from
Nov 16, 2023
Merged

Conversation

cubxxw
Copy link
Contributor

@cubxxw cubxxw commented Nov 16, 2023

Fixes #1417

  • Code adaptation k8s: service discovery and registration adaptation, configuration adaptation
  • Initial submission of the help charts script for openim API
  • change the help charts script
  • change the help charts script
  • change helm chart codes
  • change dockerfiles script
  • change chart script:add configmap mounts
  • change chart script:change repository
  • change chart script:msggateway add one service
  • change config.yaml
  • roll back some config values
  • change chart script:change Ingress rule with a rewrite annotation
  • add mysql charts scrible
  • change chart script:add mysql.config.yaml
  • add nfs provisioner charts
  • change chart script:add nfs.config.yaml
  • add ingress-nginx charts
  • change chart script:add ingress-nginx.config.yaml
  • add redis &mongodb charts
  • add kafka&minio charts
  • change chart script:change redis.values.yaml
  • change chart script:add redis.config.yaml
  • change chart script:change redis.config.yaml
  • change chart script:change mongodb.value.yaml
  • change chart script:change mongodb.value.yaml
  • change chart script:add mongodb.config.yaml
  • change chart script:change minio.values.yaml
  • change chart script:add minio.config.yaml
  • change chart script:change kafka.values.yaml
  • change chart script:add kafka.config.yaml
  • change chart script:change services.config.yaml
  • bug fix:Delete websocket's Port restrictions
  • bug fix:change port value
  • change chart script:Submit a stable version script
  • fix bug:Implement option interface
  • fix bug:change K8sDR.Register
  • change config.yaml
  • change chats script:minio service add ingress
  • change chats script:minio service add ingress
  • change chats script:kafka.replicaCount=3& change minio.api ingress
  • delete change chats script
  • change config.yaml
  • change openim.yaml
  • merge go.sum
  • Add monitoring function and struct for Prometheus on gin and GRPC
  • Add GRPC and gin server monitoring logic
  • Add GRPC and gin server monitoring logic2
  • Add GRPC and gin server monitoring logic3
  • Add GRPC and gin server monitoring logic4
  • Add GRPC and gin server monitoring logic5
  • Add GRPC and gin server monitoring logic6
  • Add GRPC and gin server monitoring logic7
  • delete:old monitoring code
  • add for test
  • fix bug:change packname
  • fix bug:delete getPromPort funciton
  • fix bug:delete getPromPort funciton
  • fix bug:change logs
  • fix bug:change registerName logic in GetGrpcCusMetrics function
  • add getPrometheus url api
  • fix:config path logic
  • fix:prometheus enable function
  • fix:prometheus enable function
  • fix:transfer Multi process monitoring logic
  • del:del not using manifest
  • fix:openim-msgtransfer.sh
  • fix:openim-msgtransfer.sh
  • cicd: robot automated Change
  • delete not using files
  • add prometheus docker-compose for monitor
  • fix prometheus.yaml
  • fix environment.sh
  • fix init-config.sh
  • fix init-config.sh
  • fix env_template.yaml
  • fix docker-compose.yml
  • fix docker-compose.yml
  • add openim_admin_front service
  • change openim-admin-front
  • del not using files
  • add node-exporter-dashaboard.yaml
  • cicd: robot automated Change
  • cicd: robot automated Change
  • feature: add alertmanager function
  • feature: add alertmanager function
  • feature: add alertmanager function
  • feature: add alertmanager function
  • feature: add alertmanager function
  • del:delete not using files
  • del:delete not using files
  • change:change to personal email info
  • feat: deployment and design of management backend and monitoring
  • feat: deployment and design of management backend and monitoring

lin.huang and others added 30 commits September 24, 2023 06:17
@cubxxw cubxxw requested review from withchao and a team as code owners November 16, 2023 07:21
@pull-request-size pull-request-size bot added the size/L Denotes a PR that changes 100-499 lines, ignoring generated files. label Nov 16, 2023
@kubbot
Copy link
Member

kubbot commented Nov 16, 2023

CLA Assistant Lite bot: 💕 Thank you for your contribution and please kindly read and sign our 🎯https://github.com/openim-sigs/cla/blob/main/README.md.
If you wish to sign the CRA, Please copy and comment on the following sentence:


I have read the CLA Document and I hereby sign the CLA


2 out of 3 committers have signed the CLA.
✅ (xuexihuang)[https://github.com/xuexihuang]
✅ (cubxxw)[https://github.com/cubxxw]
❌ @lin.huang
lin.huang seems not to be a GitHub user. You need a GitHub account to be able to sign the CLA. If you have already a GitHub account, please add the email address used for this commit to your account.
You can retrigger this bot by commenting recheck in this Pull Request

Copy link
Contributor

sweep-ai bot commented Nov 16, 2023

Apply Sweep Rules to your PR?

  • Apply: All new business logic should have corresponding unit tests.
  • Apply: Refactor large functions to be more modular.
  • Apply: Add docstrings to all functions and file headers.

Signed-off-by: Xinwei Xiong(cubxxw) <3293172751nss@gmail.com>
Copy link

codecov bot commented Nov 16, 2023

Codecov Report

Attention: 2 lines in your changes are missing coverage. Please review.

Comparison is base (ae04841) 4.35% compared to head (0a0897f) 4.54%.
Report is 7 commits behind head on main.

Files Patch % Lines
test/e2e/framework/config/config.go 0.00% 2 Missing ⚠️
Additional details and impacted files
@@           Coverage Diff            @@
##            main   #1424      +/-   ##
========================================
+ Coverage   4.35%   4.54%   +0.19%     
========================================
  Files         49      49              
  Lines       4114    4140      +26     
========================================
+ Hits         179     188       +9     
- Misses      3923    3940      +17     
  Partials      12      12              

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

@cubxxw cubxxw added this pull request to the merge queue Nov 16, 2023
Merged via the queue into openimsdk:main with commit fb74453 Nov 16, 2023
19 of 20 checks passed
@cubxxw cubxxw deleted the xuexihuang/main branch November 16, 2023 10:02
@github-actions github-actions bot added this to the v3.1 milestone Nov 16, 2023
@openimsdk openimsdk locked and limited conversation to collaborators Nov 16, 2023
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
size/L Denotes a PR that changes 100-499 lines, ignoring generated files.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Feature: Add Prometheus alerting functionality.
5 participants