fix: #1558: Set root config to global ptr in Init() function. #1564
Conversation
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Codecov Report
@@ Coverage Diff @@
## 3.0 #1564 +/- ##
==========================================
- Coverage 41.77% 41.75% -0.02%
==========================================
Files 258 258
Lines 15106 15115 +9
==========================================
+ Hits 6310 6311 +1
- Misses 8055 8065 +10
+ Partials 741 739 -2
Continue to review full report at Codecov.
|
AlexStocks
approved these changes
Nov 6, 2021
justxuewei
approved these changes
Nov 7, 2021
cjphaha
approved these changes
Nov 7, 2021
AlexStocks
added a commit
that referenced
this pull request
Nov 22, 2021
* build(deps): bump github.com/hashicorp/vault/sdk from 0.2.1 to 0.3.0 (#1552) Bumps [github.com/hashicorp/vault/sdk](https://github.com/hashicorp/vault) from 0.2.1 to 0.3.0. - [Release notes](https://github.com/hashicorp/vault/releases) - [Changelog](https://github.com/hashicorp/vault/blob/main/CHANGELOG.md) - [Commits](hashicorp/vault@sdk/v0.2.1...v0.3.0) --- updated-dependencies: - dependency-name: github.com/hashicorp/vault/sdk dependency-type: direct:production update-type: version-update:semver-minor ... Signed-off-by: dependabot[bot] <[email protected]> Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com> * delete useless codes (#1551) * Update CHANGELOG.md * [#1559]: adjust the startup process of the configuration center (#1560) * [#1559]: 1. fix the startup process of the configuration center(remove error log) 2. refactor zk registry listener * delete redundant dynamic configuration center cache Co-authored-by: dongjianhui03 <[email protected]> * [#1276]modify the name of the receiver to a single character (#1561) Co-authored-by: dongjianhui03 <[email protected]> * style(common): rename constants in camel-case style (#1563) * style(common): rename constants in camel-case style * style(common): rename Ttl to TTL * fix: #1558: Set root config to global ptr in Init() function. (#1564) * fix: #1558 * fix: change the api to start the app by config-api * fix: in the Consumer service, the Reference config property ‘check’ use the Consumer config when omitted * Triple upgrade to grpc (#1566) * fix: init triple grpc * fix: delete replace * Fix: upgrade triple grpc * Fix: remove unused file * Fix decrease grpc go version to 1.14 * Fix: add server side serialization * register not use metadata configuration. * build(deps): bump github.com/go-resty/resty/v2 from 2.3.0 to 2.7.0 (#1579) Bumps [github.com/go-resty/resty/v2](https://github.com/go-resty/resty) from 2.3.0 to 2.7.0. - [Release notes](https://github.com/go-resty/resty/releases) - [Commits](go-resty/resty@v2.3.0...v2.7.0) --- updated-dependencies: - dependency-name: github.com/go-resty/resty/v2 dependency-type: direct:production update-type: version-update:semver-minor ... Signed-off-by: dependabot[bot] <[email protected]> Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com> * build(deps): bump google.golang.org/grpc from 1.41.0 to 1.42.0 (#1578) Bumps [google.golang.org/grpc](https://github.com/grpc/grpc-go) from 1.41.0 to 1.42.0. - [Release notes](https://github.com/grpc/grpc-go/releases) - [Commits](grpc/grpc-go@v1.41.0...v1.42.0) --- updated-dependencies: - dependency-name: google.golang.org/grpc dependency-type: direct:production update-type: version-update:semver-minor ... Signed-off-by: dependabot[bot] <[email protected]> Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com> * build(deps): bump github.com/knadh/koanf from 1.3.0 to 1.3.2 (#1577) Bumps [github.com/knadh/koanf](https://github.com/knadh/koanf) from 1.3.0 to 1.3.2. - [Release notes](https://github.com/knadh/koanf/releases) - [Commits](knadh/koanf@v1.3.0...v1.3.2) --- updated-dependencies: - dependency-name: github.com/knadh/koanf dependency-type: direct:production update-type: version-update:semver-patch ... Signed-off-by: dependabot[bot] <[email protected]> Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com> * fix: registry param * fix: fix ns and group * fix: integrate test * Fix: integrate test * fix: url fmt * fix: nacos servicediscovery group * Fix: enhance zk * start application (#1571) Co-authored-by: wangxiaowei14227 <[email protected]> Co-authored-by: lizhixin.lzx <[email protected]> Co-authored-by: LaurenceLiZhixin <[email protected]> * graceful shutdwon filter (#1585) Co-authored-by: wangxiaowei14227 <[email protected]> * Fix: some bugs and features for 3.0 (#1586) * fix: some bugs * Fix: remove file\k8s registry, k8s remote * refactor listenDirEvent 1. remove CLEAR label; 2. remove register/unregister logic for watch children; 3. reorder the zk dynamic configuration code 4. adapt the gost * hessian encode attachments return error (#1588) * upgrade gost dependency version * upgrade hessian to v1.9.5 (#1595) * build(deps): bump k8s.io/apimachinery from 0.16.9 to 0.22.4 (#1600) Bumps [k8s.io/apimachinery](https://github.com/kubernetes/apimachinery) from 0.16.9 to 0.22.4. - [Release notes](https://github.com/kubernetes/apimachinery/releases) - [Commits](kubernetes/apimachinery@v0.16.9...v0.22.4) --- updated-dependencies: - dependency-name: k8s.io/apimachinery dependency-type: direct:production update-type: version-update:semver-minor ... Signed-off-by: dependabot[bot] <[email protected]> Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com> * ftr: triple tracing (#1596) * ftr: triple tracing * fix: if-else to switch * [fix #1590] filter single instance (#1591) * filter single instance * filter single instance Co-authored-by: wangxiaowei14227 <[email protected]> * Ftr/triple reflect support (#1603) * ftr: triple-reflection * fix: comment and var name * Fix: comment typo * Fix: official website Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com> Co-authored-by: Xin.Zh <[email protected]> Co-authored-by: Mulavar <[email protected]> Co-authored-by: dongjianhui03 <[email protected]> Co-authored-by: Xuewei Niu <[email protected]> Co-authored-by: PhilYue <[email protected]> Co-authored-by: lizhixin.lzx <[email protected]> Co-authored-by: sanxun0325 <[email protected]> Co-authored-by: Xuewei Niu <[email protected]> Co-authored-by: zhaoyunxing <[email protected]> Co-authored-by: wangxw <[email protected]> Co-authored-by: wangxiaowei14227 <[email protected]> Co-authored-by: 望哥 <[email protected]>
AlexStocks
added a commit
that referenced
this pull request
Dec 1, 2021
* feat: try to support dubbo * Merge 3.0 to master (#1606) * build(deps): bump github.com/hashicorp/vault/sdk from 0.2.1 to 0.3.0 (#1552) Bumps [github.com/hashicorp/vault/sdk](https://github.com/hashicorp/vault) from 0.2.1 to 0.3.0. - [Release notes](https://github.com/hashicorp/vault/releases) - [Changelog](https://github.com/hashicorp/vault/blob/main/CHANGELOG.md) - [Commits](hashicorp/vault@sdk/v0.2.1...v0.3.0) --- updated-dependencies: - dependency-name: github.com/hashicorp/vault/sdk dependency-type: direct:production update-type: version-update:semver-minor ... Signed-off-by: dependabot[bot] <[email protected]> Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com> * delete useless codes (#1551) * Update CHANGELOG.md * [#1559]: adjust the startup process of the configuration center (#1560) * [#1559]: 1. fix the startup process of the configuration center(remove error log) 2. refactor zk registry listener * delete redundant dynamic configuration center cache Co-authored-by: dongjianhui03 <[email protected]> * [#1276]modify the name of the receiver to a single character (#1561) Co-authored-by: dongjianhui03 <[email protected]> * style(common): rename constants in camel-case style (#1563) * style(common): rename constants in camel-case style * style(common): rename Ttl to TTL * fix: #1558: Set root config to global ptr in Init() function. (#1564) * fix: #1558 * fix: change the api to start the app by config-api * fix: in the Consumer service, the Reference config property ‘check’ use the Consumer config when omitted * Triple upgrade to grpc (#1566) * fix: init triple grpc * fix: delete replace * Fix: upgrade triple grpc * Fix: remove unused file * Fix decrease grpc go version to 1.14 * Fix: add server side serialization * register not use metadata configuration. * build(deps): bump github.com/go-resty/resty/v2 from 2.3.0 to 2.7.0 (#1579) Bumps [github.com/go-resty/resty/v2](https://github.com/go-resty/resty) from 2.3.0 to 2.7.0. - [Release notes](https://github.com/go-resty/resty/releases) - [Commits](go-resty/resty@v2.3.0...v2.7.0) --- updated-dependencies: - dependency-name: github.com/go-resty/resty/v2 dependency-type: direct:production update-type: version-update:semver-minor ... Signed-off-by: dependabot[bot] <[email protected]> Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com> * build(deps): bump google.golang.org/grpc from 1.41.0 to 1.42.0 (#1578) Bumps [google.golang.org/grpc](https://github.com/grpc/grpc-go) from 1.41.0 to 1.42.0. - [Release notes](https://github.com/grpc/grpc-go/releases) - [Commits](grpc/grpc-go@v1.41.0...v1.42.0) --- updated-dependencies: - dependency-name: google.golang.org/grpc dependency-type: direct:production update-type: version-update:semver-minor ... Signed-off-by: dependabot[bot] <[email protected]> Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com> * build(deps): bump github.com/knadh/koanf from 1.3.0 to 1.3.2 (#1577) Bumps [github.com/knadh/koanf](https://github.com/knadh/koanf) from 1.3.0 to 1.3.2. - [Release notes](https://github.com/knadh/koanf/releases) - [Commits](knadh/koanf@v1.3.0...v1.3.2) --- updated-dependencies: - dependency-name: github.com/knadh/koanf dependency-type: direct:production update-type: version-update:semver-patch ... Signed-off-by: dependabot[bot] <[email protected]> Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com> * fix: registry param * fix: fix ns and group * fix: integrate test * Fix: integrate test * fix: url fmt * fix: nacos servicediscovery group * Fix: enhance zk * start application (#1571) Co-authored-by: wangxiaowei14227 <[email protected]> Co-authored-by: lizhixin.lzx <[email protected]> Co-authored-by: LaurenceLiZhixin <[email protected]> * graceful shutdwon filter (#1585) Co-authored-by: wangxiaowei14227 <[email protected]> * Fix: some bugs and features for 3.0 (#1586) * fix: some bugs * Fix: remove file\k8s registry, k8s remote * refactor listenDirEvent 1. remove CLEAR label; 2. remove register/unregister logic for watch children; 3. reorder the zk dynamic configuration code 4. adapt the gost * hessian encode attachments return error (#1588) * upgrade gost dependency version * upgrade hessian to v1.9.5 (#1595) * build(deps): bump k8s.io/apimachinery from 0.16.9 to 0.22.4 (#1600) Bumps [k8s.io/apimachinery](https://github.com/kubernetes/apimachinery) from 0.16.9 to 0.22.4. - [Release notes](https://github.com/kubernetes/apimachinery/releases) - [Commits](kubernetes/apimachinery@v0.16.9...v0.22.4) --- updated-dependencies: - dependency-name: k8s.io/apimachinery dependency-type: direct:production update-type: version-update:semver-minor ... Signed-off-by: dependabot[bot] <[email protected]> Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com> * ftr: triple tracing (#1596) * ftr: triple tracing * fix: if-else to switch * [fix #1590] filter single instance (#1591) * filter single instance * filter single instance Co-authored-by: wangxiaowei14227 <[email protected]> * Ftr/triple reflect support (#1603) * ftr: triple-reflection * fix: comment and var name * Fix: comment typo * Fix: official website Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com> Co-authored-by: Xin.Zh <[email protected]> Co-authored-by: Mulavar <[email protected]> Co-authored-by: dongjianhui03 <[email protected]> Co-authored-by: Xuewei Niu <[email protected]> Co-authored-by: PhilYue <[email protected]> Co-authored-by: lizhixin.lzx <[email protected]> Co-authored-by: sanxun0325 <[email protected]> Co-authored-by: Xuewei Niu <[email protected]> Co-authored-by: zhaoyunxing <[email protected]> Co-authored-by: wangxw <[email protected]> Co-authored-by: wangxiaowei14227 <[email protected]> Co-authored-by: 望哥 <[email protected]> * feat: support polaris * feat: support polarismesh registry * fix: fix cr issue * chore: run go mod tidy to fix ci-error * refactor: change polaris::ConsumerAPI.GetInstances => GetAllInstances * fix github action scan error * fix: fix go fmt error * fix: import formatter * fix: fix cr issue * fix: fix code-style check error * fix: fix cr issue * fix: fix maybe nil error * fix: fix lose event bug * style: fix code style Co-authored-by: springliao <[email protected]> Co-authored-by: Laurence <[email protected]> Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com> Co-authored-by: Xin.Zh <[email protected]> Co-authored-by: Mulavar <[email protected]> Co-authored-by: dongjianhui03 <[email protected]> Co-authored-by: Xuewei Niu <[email protected]> Co-authored-by: PhilYue <[email protected]> Co-authored-by: lizhixin.lzx <[email protected]> Co-authored-by: sanxun0325 <[email protected]> Co-authored-by: Xuewei Niu <[email protected]> Co-authored-by: zhaoyunxing <[email protected]> Co-authored-by: wangxw <[email protected]> Co-authored-by: wangxiaowei14227 <[email protected]> Co-authored-by: 望哥 <[email protected]> Co-authored-by: LaurenceLiZhixin <[email protected]>
AlexStocks
added a commit
that referenced
this pull request
Dec 21, 2021
* Merge 3.0 to master (#1606) * build(deps): bump github.com/hashicorp/vault/sdk from 0.2.1 to 0.3.0 (#1552) Bumps [github.com/hashicorp/vault/sdk](https://github.com/hashicorp/vault) from 0.2.1 to 0.3.0. - [Release notes](https://github.com/hashicorp/vault/releases) - [Changelog](https://github.com/hashicorp/vault/blob/main/CHANGELOG.md) - [Commits](hashicorp/vault@sdk/v0.2.1...v0.3.0) --- updated-dependencies: - dependency-name: github.com/hashicorp/vault/sdk dependency-type: direct:production update-type: version-update:semver-minor ... Signed-off-by: dependabot[bot] <[email protected]> Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com> * delete useless codes (#1551) * Update CHANGELOG.md * [#1559]: adjust the startup process of the configuration center (#1560) * [#1559]: 1. fix the startup process of the configuration center(remove error log) 2. refactor zk registry listener * delete redundant dynamic configuration center cache Co-authored-by: dongjianhui03 <[email protected]> * [#1276]modify the name of the receiver to a single character (#1561) Co-authored-by: dongjianhui03 <[email protected]> * style(common): rename constants in camel-case style (#1563) * style(common): rename constants in camel-case style * style(common): rename Ttl to TTL * fix: #1558: Set root config to global ptr in Init() function. (#1564) * fix: #1558 * fix: change the api to start the app by config-api * fix: in the Consumer service, the Reference config property ‘check’ use the Consumer config when omitted * Triple upgrade to grpc (#1566) * fix: init triple grpc * fix: delete replace * Fix: upgrade triple grpc * Fix: remove unused file * Fix decrease grpc go version to 1.14 * Fix: add server side serialization * register not use metadata configuration. * build(deps): bump github.com/go-resty/resty/v2 from 2.3.0 to 2.7.0 (#1579) Bumps [github.com/go-resty/resty/v2](https://github.com/go-resty/resty) from 2.3.0 to 2.7.0. - [Release notes](https://github.com/go-resty/resty/releases) - [Commits](go-resty/resty@v2.3.0...v2.7.0) --- updated-dependencies: - dependency-name: github.com/go-resty/resty/v2 dependency-type: direct:production update-type: version-update:semver-minor ... Signed-off-by: dependabot[bot] <[email protected]> Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com> * build(deps): bump google.golang.org/grpc from 1.41.0 to 1.42.0 (#1578) Bumps [google.golang.org/grpc](https://github.com/grpc/grpc-go) from 1.41.0 to 1.42.0. - [Release notes](https://github.com/grpc/grpc-go/releases) - [Commits](grpc/grpc-go@v1.41.0...v1.42.0) --- updated-dependencies: - dependency-name: google.golang.org/grpc dependency-type: direct:production update-type: version-update:semver-minor ... Signed-off-by: dependabot[bot] <[email protected]> Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com> * build(deps): bump github.com/knadh/koanf from 1.3.0 to 1.3.2 (#1577) Bumps [github.com/knadh/koanf](https://github.com/knadh/koanf) from 1.3.0 to 1.3.2. - [Release notes](https://github.com/knadh/koanf/releases) - [Commits](knadh/koanf@v1.3.0...v1.3.2) --- updated-dependencies: - dependency-name: github.com/knadh/koanf dependency-type: direct:production update-type: version-update:semver-patch ... Signed-off-by: dependabot[bot] <[email protected]> Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com> * fix: registry param * fix: fix ns and group * fix: integrate test * Fix: integrate test * fix: url fmt * fix: nacos servicediscovery group * Fix: enhance zk * start application (#1571) Co-authored-by: wangxiaowei14227 <[email protected]> Co-authored-by: lizhixin.lzx <[email protected]> Co-authored-by: LaurenceLiZhixin <[email protected]> * graceful shutdwon filter (#1585) Co-authored-by: wangxiaowei14227 <[email protected]> * Fix: some bugs and features for 3.0 (#1586) * fix: some bugs * Fix: remove file\k8s registry, k8s remote * refactor listenDirEvent 1. remove CLEAR label; 2. remove register/unregister logic for watch children; 3. reorder the zk dynamic configuration code 4. adapt the gost * hessian encode attachments return error (#1588) * upgrade gost dependency version * upgrade hessian to v1.9.5 (#1595) * build(deps): bump k8s.io/apimachinery from 0.16.9 to 0.22.4 (#1600) Bumps [k8s.io/apimachinery](https://github.com/kubernetes/apimachinery) from 0.16.9 to 0.22.4. - [Release notes](https://github.com/kubernetes/apimachinery/releases) - [Commits](kubernetes/apimachinery@v0.16.9...v0.22.4) --- updated-dependencies: - dependency-name: k8s.io/apimachinery dependency-type: direct:production update-type: version-update:semver-minor ... Signed-off-by: dependabot[bot] <[email protected]> Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com> * ftr: triple tracing (#1596) * ftr: triple tracing * fix: if-else to switch * [fix #1590] filter single instance (#1591) * filter single instance * filter single instance Co-authored-by: wangxiaowei14227 <[email protected]> * Ftr/triple reflect support (#1603) * ftr: triple-reflection * fix: comment and var name * Fix: comment typo * Fix: official website Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com> Co-authored-by: Xin.Zh <[email protected]> Co-authored-by: Mulavar <[email protected]> Co-authored-by: dongjianhui03 <[email protected]> Co-authored-by: Xuewei Niu <[email protected]> Co-authored-by: PhilYue <[email protected]> Co-authored-by: lizhixin.lzx <[email protected]> Co-authored-by: sanxun0325 <[email protected]> Co-authored-by: Xuewei Niu <[email protected]> Co-authored-by: zhaoyunxing <[email protected]> Co-authored-by: wangxw <[email protected]> Co-authored-by: wangxiaowei14227 <[email protected]> Co-authored-by: 望哥 <[email protected]> * fix unexported bug (#1634) * Revert "fix unexported bug (#1634)" (#1635) This reverts commit 5604ba5. * add OPay logo to user list Co-authored-by: Laurence <[email protected]> Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com> Co-authored-by: Xin.Zh <[email protected]> Co-authored-by: Mulavar <[email protected]> Co-authored-by: dongjianhui03 <[email protected]> Co-authored-by: Xuewei Niu <[email protected]> Co-authored-by: PhilYue <[email protected]> Co-authored-by: lizhixin.lzx <[email protected]> Co-authored-by: sanxun0325 <[email protected]> Co-authored-by: Xuewei Niu <[email protected]> Co-authored-by: zhaoyunxing <[email protected]> Co-authored-by: wangxw <[email protected]> Co-authored-by: wangxiaowei14227 <[email protected]> Co-authored-by: 望哥 <[email protected]> Co-authored-by: 刘月财 <[email protected]> Co-authored-by: AlexStocks <[email protected]>
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
Fix: #1558