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

【feature】事件上报entity和dao模块 #1121

Merged
merged 1 commit into from
Mar 30, 2023

Conversation

xuezechao1
Copy link
Collaborator

@xuezechao1 xuezechao1 commented Mar 13, 2023

【修复issue】#1099

【修改内容】

  • backend支持事件写入redis数据库/内存
  • agent事件支持webook推送

【用例描述】增加事件写入,查询用例

【自测情况】1、本地静态检查通过

@xuezechao1 xuezechao1 force-pushed the issue_1099_1 branch 7 times, most recently from 6305eb8 to c1cec37 Compare March 14, 2023 01:05
@codecov
Copy link

codecov bot commented Mar 14, 2023

Codecov Report

Patch coverage has no change and project coverage change: -1.99 ⚠️

Comparison is base (e56e893) 43.62% compared to head (97aa084) 41.63%.

Additional details and impacted files
@@              Coverage Diff              @@
##             develop    #1121      +/-   ##
=============================================
- Coverage      43.62%   41.63%   -1.99%     
+ Complexity      3187     3183       -4     
=============================================
  Files            779      808      +29     
  Lines          17001    17791     +790     
  Branches        1896     1943      +47     
=============================================
- Hits            7416     7408       -8     
- Misses          8654     9443     +789     
- Partials         931      940       +9     
Flag Coverage Δ
unittests 41.63% <0.00%> (-1.99%) ⬇️

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

Impacted Files Coverage Δ
.../java/com/huaweicloud/sermant/backend/Backend.java 0.00% <ø> (ø)
...oud/sermant/backend/common/conf/BackendConfig.java 0.00% <0.00%> (ø)
...cloud/sermant/backend/common/conf/CommonConst.java 0.00% <0.00%> (ø)
...ud/sermant/backend/controller/EventController.java 0.00% <0.00%> (ø)
.../huaweicloud/sermant/backend/dao/DatabaseType.java 0.00% <0.00%> (ø)
...d/sermant/backend/dao/memory/MemoryClientImpl.java 0.00% <0.00%> (ø)
...ud/sermant/backend/dao/redis/EventDaoForRedis.java 0.00% <0.00%> (ø)
...oud/sermant/backend/dao/redis/RedisClientImpl.java 0.00% <0.00%> (ø)
...aweicloud/sermant/backend/entity/InstanceMeta.java 0.00% <0.00%> (ø)
...d/sermant/backend/entity/event/DingDingEntity.java 0.00% <0.00%> (ø)
... and 24 more

... and 54 files with indirect coverage changes

Help us with your feedback. Take ten seconds to tell us how you rate us. Have a feature suggestion? Share it here.

☔ View full report in Codecov by Sentry.
📢 Do you have feedback about the report comment? Let us know in this issue.

@xuezechao1 xuezechao1 changed the title 事件上报entity和dao模块 【feature】事件上报entity和dao模块 Mar 15, 2023
@xuezechao1 xuezechao1 self-assigned this Mar 15, 2023
@xuezechao1 xuezechao1 added the area/backend Issues or PRs releated to sermant backend service label Mar 15, 2023
@xuezechao1 xuezechao1 force-pushed the issue_1099_1 branch 4 times, most recently from ffd2806 to b3eba60 Compare March 20, 2023 07:55
@xuezechao1 xuezechao1 force-pushed the issue_1099_1 branch 8 times, most recently from e60f356 to e6b3054 Compare March 28, 2023 02:22
@xuezechao1 xuezechao1 force-pushed the issue_1099_1 branch 4 times, most recently from ece9625 to 35bcc4f Compare March 29, 2023 07:44
@robotLJW robotLJW added the status/lgtm Indicates that a PR is ready to be merged. label Mar 29, 2023
@robotLJW robotLJW added this to the v1.1.0 milestone Mar 29, 2023
robotLJW
robotLJW previously approved these changes Mar 29, 2023
Copy link
Collaborator

@robotLJW robotLJW left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

/lgtm

@robotLJW robotLJW merged commit b586ff2 into sermant-io:develop Mar 30, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
area/backend Issues or PRs releated to sermant backend service status/lgtm Indicates that a PR is ready to be merged.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants