🐛 fix metrics servce level config bug #13449

songxiaosheng · 2023-12-01T06:03:05Z

What is the purpose of the change

servce level config is init after application config be load

test result is such as:

Brief changelog

Verifying this change

Checklist

Make sure there is a GitHub_issue field for the change (usually before you start working on it). Trivial changes like typos do not require a GitHub issue. Your pull request should address just this issue, without pulling in other changes - one PR resolves one issue.
Each commit in the pull request should have a meaningful subject line and body.
Write a pull request description that is detailed enough to understand what the pull request does, how, and why.
Check if is necessary to patch to Dubbo 3 if you are work on Dubbo 2.7
Write necessary unit-test to verify your logic correction, more mock a little better when cross module dependency exist. If the new feature or significant change is committed, please remember to add sample in dubbo samples project.
Add some description to dubbo-website project if you are requesting to add a feature.
GitHub Actions works fine on your own branch.
If this contribution is large, please follow the Software Donation Guide.

AlbumenJ · 2023-12-01T12:12:48Z

...s/dubbo-metrics-api/src/main/java/org/apache/dubbo/metrics/report/AbstractMetricsExport.java

+
+    private void initServiceLevelConfig() {
+        if (serviceLevel == null) {
+            synchronized (AbstractMetricsExport.class) {


Lock current object is enough

AlbumenJ · 2023-12-01T12:13:08Z

...s/dubbo-metrics-api/src/main/java/org/apache/dubbo/metrics/report/AbstractMetricsExport.java

 import org.apache.dubbo.rpc.model.ApplicationModel;

 /**
 * Store public information such as application
 */
 public abstract class AbstractMetricsExport implements MetricsExport {

+    private Boolean serviceLevel;
