sermant-io / Sermant

CNCF sandbox project, a Cloud-Native Proxyless Service Mesh based on Java Bytecode Enhancement Technology
https://sermant.io/
Apache License 2.0
1.25k stars 164 forks source link

Add hot plugging configuration listeners and execution services #1612

Closed hanbingleixue closed 1 month ago

hanbingleixue commented 2 months ago

What type of PR is this?

Feature.

What this PR does / why we need it?

1、Add The Sermant hot swap configuration listening service is . 2、Add the plug-in update command execution service. 3、Modify the heartbeat service by adding information such as the application name and instance Id.

Which issue(s) this PR fixes?

Fixes #1606

Does this PR introduce a user-facing change?

No

Checklist

codecov[bot] commented 2 months ago

Codecov Report

Attention: Patch coverage is 53.22581% with 58 lines in your changes missing coverage. Please review.

Files with missing lines Patch % Lines
...ain/java/io/sermant/core/plugin/PluginManager.java 0.00% 15 Missing :warning:
...vice/hotplugging/listener/HotPluggingListener.java 66.66% 6 Missing and 3 partials :warning:
...ant/core/command/PluginsUpdateCommandExecutor.java 33.33% 7 Missing and 1 partial :warning:
...ore/service/heartbeat/common/HeartbeatMessage.java 0.00% 8 Missing :warning:
...in/java/io/sermant/core/service/ServiceConfig.java 0.00% 6 Missing :warning:
...n/java/io/sermant/core/common/BootArgsIndexer.java 60.00% 4 Missing :warning:
...nt/service/hotplugging/HotPluggingServiceImpl.java 76.92% 3 Missing :warning:
...nt/core/command/PluginsInstallCommandExecutor.java 0.00% 2 Missing :warning:
.../core/command/PluginsUnInstallCommandExecutor.java 0.00% 2 Missing :warning:
...c/main/java/io/sermant/core/AgentCoreEntrance.java 0.00% 1 Missing :warning:
Flag Coverage Δ Complexity Δ
unittests 43.78% <53.22%> (+0.94%) 182.00 <0.00> (ø)

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

Files with missing lines Coverage Δ Complexity Δ
...src/main/java/io/sermant/core/command/Command.java 100.00% <100.00%> (+100.00%) 0.00 <0.00> (ø)
...java/io/sermant/core/command/CommandProcessor.java 32.00% <100.00%> (+32.00%) 0.00 <0.00> (ø)
...in/java/io/sermant/core/common/CommonConstant.java 0.00% <ø> (ø) 0.00 <0.00> (ø)
.../core/event/collector/FrameworkEventCollector.java 14.28% <100.00%> (+14.28%) 0.00 <0.00> (ø)
...ore/event/collector/FrameworkEventDefinitions.java 100.00% <100.00%> (+100.00%) 0.00 <0.00> (ø)
...n/java/io/sermant/core/service/ServiceManager.java 9.23% <ø> (ø) 0.00 <0.00> (ø)
.../service/hotplugging/entity/HotPluggingConfig.java 100.00% <100.00%> (ø) 0.00 <0.00> (?)
...c/main/java/io/sermant/core/AgentCoreEntrance.java 0.00% <0.00%> (ø) 0.00 <0.00> (ø)
...nt/core/command/PluginsInstallCommandExecutor.java 20.00% <0.00%> (+20.00%) 0.00 <0.00> (ø)
.../core/command/PluginsUnInstallCommandExecutor.java 20.00% <0.00%> (+20.00%) 0.00 <0.00> (ø)
... and 7 more

... and 48 files with indirect coverage changes

lilai23 commented 2 months ago

UT is required