Fix the problem of not initializing extensions in the plugin properly

[JohnNiang]

What type of PR is this?

/kind bug
/area core
/milestone 2.12.x

What this PR does / why we need it:

This PR waits extensions deleted when stopping plugins and rearrange destruction order between plugin manager and reconcilers.

See #5226 for more.

Which issue(s) this PR fixes:

Fixes #5226

Special notes for your reviewer:

Does this PR introduce a user-facing change?

None

[codecov]

Codecov Report

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

Comparison is base (95878b3) 57.06% compared to head (d52c7d5) 57.03%.
Report is 7 commits behind head on main.

Files	Patch %	Lines
.../halo/app/plugin/PluginBeforeStopSyncListener.java	5.26%	18 Missing ⚠️
...extension/controller/DefaultControllerManager.java	33.33%	6 Missing and 2 partials ⚠️
...ava/run/halo/app/plugin/PluginStartedListener.java	0.00%	3 Missing ⚠️

Additional details and impacted files

@@             Coverage Diff              @@
##               main    #5236      +/-   ##
============================================
- Coverage     57.06%   57.03%   -0.03%     
- Complexity     3333     3338       +5     
============================================
  Files           584      584              
  Lines         19190    19221      +31     
  Branches       1444     1445       +1     
============================================
+ Hits          10950    10963      +13     
- Misses         7664     7680      +16     
- Partials        576      578       +2     

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

[guqing]

/remove-approve

我发现升级 oauth2 插件等待移除 extensions 的时间很长超过一分钟：

2024-01-24T17:16:27.099+08:00
2024-01-24T17:17:27.427+08:00

java.lang.IllegalStateException: Timeout on blocking read for 60000000000 NANOSECONDS

虽然最终会自愈，但是反复升级试了几次总是能复现，可以配合 #5245 一起测试
相关日志：

log.log

[f2c-ci-robot]

[APPROVALNOTIFIER] This PR is NOT APPROVED

This pull-request has been approved by:
Once this PR has been reviewed and has the lgtm label, please ask for approval from johnniang. For more information see the Kubernetes Code Review Process.

The full list of commands accepted by this bot can be found here.

Needs approval from an approver in each of these files:

• OWNERS

Approvers can indicate their approval by writing /approve in a comment
Approvers can cancel approval by writing /approve cancel in a comment

[JohnNiang]

@guqing

/remove-approve

我发现升级 oauth2 插件等待移除 extensions 的时间很长：

2024-01-24T17:16:27.099+08:00 2024-01-24T17:17:27.427+08:00

java.lang.IllegalStateException: Timeout on blocking read for 60000000000 NANOSECONDS

虽然最终会自愈，但是反复升级试了几次总是能复现，可以配合 #5245 一起测试 相关日志：

log.log

应该是删除的资源中包含了 finalizers，此时负责删除 finalizers 的 reconciler 已经停止了。目前对 core 中的 reconciler 的启动顺序做了处理，可能是插件中的 reconciler 提前停止导致的。稍后我排查一下。

[guqing]

@guqing

/remove-approve
我发现升级 oauth2 插件等待移除 extensions 的时间很长：
2024-01-24T17:16:27.099+08:00 2024-01-24T17:17:27.427+08:00
java.lang.IllegalStateException: Timeout on blocking read for 60000000000 NANOSECONDS
虽然最终会自愈，但是反复升级试了几次总是能复现，可以配合 #5245 一起测试 相关日志：
log.log

应该是删除的资源中包含了 finalizers，此时负责删除 finalizers 的 reconciler 已经停止了。目前对 core 中的 reconciler 的启动顺序做了处理，可能是插件中的 reconciler 提前停止导致的。稍后我排查一下。

好的 ❤️‍🔥

[JohnNiang]

目前我更倾向系统停止的时候不执行 pluginManager#stopPlugins 方法， 在系统再次启动的时候，会自动覆盖 extension 数据。这样是否可行呢？@guqing

[guqing]

目前我更倾向系统停止的时候不执行 pluginManager#stopPlugins 方法， 在系统再次启动的时候，会自动覆盖 extension 数据。这样是否可行呢？@guqing

Halo 停止时，是否可以通过不发出插件停止事件 HaloPluginStoppedEvent 来实现呢

[JohnNiang]

目前我更倾向系统停止的时候不执行 pluginManager#stopPlugins 方法， 在系统再次启动的时候，会自动覆盖 extension 数据。这样是否可行呢？@guqing

Halo 停止时，是否可以通过不发出插件停止事件 HaloPluginStoppedEvent 来实现呢

我们只需要在 destroy PluginManager 的时候不调用 #stopPlugins 方法即可。

halo/application/src/main/java/run/halo/app/plugin/HaloPluginManager.java

Line 354 in 3846525

stopPlugins();

[guqing]

目前我更倾向系统停止的时候不执行 pluginManager#stopPlugins 方法， 在系统再次启动的时候，会自动覆盖 extension 数据。这样是否可行呢？@guqing

Halo 停止时，是否可以通过不发出插件停止事件 HaloPluginStoppedEvent 来实现呢

我们只需要在 destroy PluginManager 的时候不调用 #stopPlugins 方法即可。

halo/application/src/main/java/run/halo/app/plugin/HaloPluginManager.java

Line 354 in 3846525

stopPlugins();

那需要测试一下停止 Halo 时插件的 ApplicationContext 能否被正确关闭如果可以的话应该没什么问题

[JohnNiang]

/hold

[JohnNiang]

Close in favor of #5251.

/close