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

Acquire sermant enhancement information: plugins, enhancement classes, methods #1355

Merged
merged 1 commit into from
Nov 8, 2023

Conversation

TangLeDaily
Copy link
Collaborator

@TangLeDaily TangLeDaily commented Nov 6, 2023

【Fix issue】#1259

【Modified content】

  1. Added storage enhancement information class for storing enhancement class and method information
  2. Added enhanced information query command executor, used to print enhanced information through instructions

[Example description] No

[Self-test status] 1. Local static check passed; 2. UT passed; 3. Mount command execution passed

[Scope of influence] None

@TangLeDaily TangLeDaily added kind/feature Issue for new feature area/framework Issues or PRs releated to sermant core service labels Nov 6, 2023
@TangLeDaily TangLeDaily added this to the v1.3.0 milestone Nov 6, 2023
@TangLeDaily TangLeDaily self-assigned this Nov 6, 2023
Copy link

codecov bot commented Nov 6, 2023

Codecov Report

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

Comparison is base (a1308ed) 42.83% compared to head (b4764b7) 42.66%.
Report is 4 commits behind head on develop.

Additional details and impacted files
@@              Coverage Diff              @@
##             develop    #1355      +/-   ##
=============================================
- Coverage      42.83%   42.66%   -0.18%     
+ Complexity      3301     3298       -3     
=============================================
  Files            824      826       +2     
  Lines          18311    18373      +62     
  Branches        2018     2028      +10     
=============================================
- Hits            7844     7839       -5     
- Misses          9481     9550      +69     
+ Partials         986      984       -2     
Flag Coverage Δ
unittests 42.66% <0.00%> (-0.18%) ⬇️

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

Files Coverage Δ
...om/huaweicloud/sermant/core/AgentCoreEntrance.java 0.00% <0.00%> (ø)
...eicloud/sermant/core/command/CommandProcessor.java 0.00% <0.00%> (ø)
.../com/huaweicloud/sermant/core/command/Command.java 0.00% <0.00%> (ø)
...huaweicloud/sermant/core/plugin/PluginManager.java 0.00% <0.00%> (ø)
...plugin/agent/transformer/ReentrantTransformer.java 0.00% <0.00%> (ø)
...ant/core/plugin/agent/info/EnhancementManager.java 0.00% <0.00%> (ø)
...core/command/CheckEnhancementsCommandExecutor.java 0.00% <0.00%> (ø)

... and 22 files with indirect coverage changes

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

hanbingleixue
hanbingleixue previously approved these changes Nov 7, 2023
lilai23
lilai23 previously approved these changes Nov 7, 2023
@TangLeDaily TangLeDaily dismissed stale reviews from lilai23 and hanbingleixue via 7a1ee2b November 7, 2023 08:04
@TangLeDaily TangLeDaily force-pushed the issue1259_a branch 2 times, most recently from 7a1ee2b to d768519 Compare November 7, 2023 08:07
hanbingleixue
hanbingleixue previously approved these changes Nov 7, 2023
@TangLeDaily TangLeDaily force-pushed the issue1259_a branch 2 times, most recently from 2c57eb2 to 1316f23 Compare November 7, 2023 12:17
hanbingleixue
hanbingleixue previously approved these changes Nov 7, 2023
@Sherlockhan Sherlockhan merged commit 20d3c4d into sermant-io:develop Nov 8, 2023
150 checks passed
@kkf1 kkf1 changed the title 获取sermant增强信息:插件、增强类、方法 Acquire sermant enhancement information: plugins, enhancement classes, methods Apr 13, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
area/framework Issues or PRs releated to sermant core service kind/feature Issue for new feature
Projects
None yet
Development

Successfully merging this pull request may close these issues.

5 participants