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

feat: add etcd registry #14

Merged
merged 2 commits into from
Aug 28, 2022
Merged

Conversation

qiuyuyin
Copy link
Collaborator

What type of PR is this?

feat: add etcd registry

What this PR does / why we need it (English/Chinese):

en: add etcd registry
zh: 增加etcd注册中心

etcd/example/client/main.go Outdated Show resolved Hide resolved
etcd/go.mod Show resolved Hide resolved
etcd/readme.md Outdated Show resolved Hide resolved
etcd/register.go Outdated Show resolved Hide resolved
@li-jin-gou
Copy link
Collaborator

@qiuyuyin 可以参考 https://github.com/kitex-contrib/registry-etcd 改一下哈

etcd/readme.md Outdated Show resolved Hide resolved
etcd/resolver_test.go Outdated Show resolved Hide resolved
@qiuyuyin qiuyuyin force-pushed the yili-main branch 2 times, most recently from b90254c to 9603e38 Compare August 27, 2022 20:31
etcd/registry.go Outdated Show resolved Hide resolved
etcd/resolver.go Outdated Show resolved Hide resolved
etcd/resolver.go Outdated Show resolved Hide resolved
etcd/registry.go Outdated Show resolved Hide resolved
@qiuyuyin qiuyuyin force-pushed the yili-main branch 4 times, most recently from 8083211 to cc43818 Compare August 28, 2022 07:28
@li-jin-gou
Copy link
Collaborator

LGTM

1 similar comment
@zstone12
Copy link
Collaborator

LGTM

@hertz-contrib hertz-contrib deleted a comment from zstone12 Aug 28, 2022
@Skyenought Skyenought merged commit 49ba90f into hertz-contrib:main Aug 28, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants