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

Server start may fail and cause Register error #3480

Open
SuzukiHonoka opened this issue Dec 5, 2024 · 0 comments · May be fixed by #3481
Open

Server start may fail and cause Register error #3480

SuzukiHonoka opened this issue Dec 5, 2024 · 0 comments · May be fixed by #3481
Labels
bug Something isn't working

Comments

@SuzukiHonoka
Copy link

What happened:

If server failed to start, it will cause Register fail too.

What you expected to happen:

Return server error, instead register error if server error happens.
{"level":"warn","ts":"2024-12-05T19:02:16.607470+0800","logger":"etcd-client","caller":"[email protected]/retry_interceptor.go:63","msg":"retrying of unary invoker failed","target":"etcd-endpoints://0xc0003e4f00/127.0.0.1:2379","attempt":0,"error":"rpc error: code = Canceled desc = context canceled"}

How to reproduce it (as minimally and precisely as possible):

Start a program which listen port A, use kratos to listen port A too.

Anything else we need to know?:

The ctx will be cancled since the latst error, you guys did't handle it.

Environment:

  • Kratos version (use kratos -v): v2.8.2
  • Go version (use go version): go version go1.23.4 linux/amd64
  • OS (e.g: cat /etc/os-release):
NAME="Arch Linux"
PRETTY_NAME="Arch Linux"
ID=arch
BUILD_ID=rolling
ANSI_COLOR="38;2;23;147;209"
HOME_URL="https://archlinux.org/"
DOCUMENTATION_URL="https://wiki.archlinux.org/"
SUPPORT_URL="https://bbs.archlinux.org/"
BUG_REPORT_URL="https://gitlab.archlinux.org/groups/archlinux/-/issues"
PRIVACY_POLICY_URL="https://terms.archlinux.org/docs/privacy-policy/"
LOGO=archlinux-logo

  • Others:
    None
@SuzukiHonoka SuzukiHonoka added the bug Something isn't working label Dec 5, 2024
@SuzukiHonoka SuzukiHonoka linked a pull request Dec 5, 2024 that will close this issue
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working
Projects
None yet
Development

Successfully merging a pull request may close this issue.

1 participant