[ISSUE #14992] Fix v3 client instance distro routing#14993
[ISSUE #14992] Fix v3 client instance distro routing#14993sbinsun wants to merge 1 commit intoalibaba:developfrom
Conversation
Assisted-by: Claude Code
|
xujun102 seems not to be a GitHub user. You need a GitHub account to be able to sign the CLA. If you have already a GitHub account, please add the email address used for this commit to your account. You have signed the CLA already but the status is still pending? Let us recheck it. |
|
Thanks for your this PR. 🙏 感谢您提交的PR。 🙏 |
|
| Commit | Author | |
|---|---|---|
32a587c |
xujun102 | xujun102@midea.com |
How to fix:
- Add your commit email to your GitHub account: https://github.com/settings/emails
- Or update your local git config to use an email already linked to GitHub:
git config user.name "Your GitHub Username" git config user.email "your-github-email@example.com" - Amend your commits and force-push:
git rebase -i HEAD~1 # mark commits as "edit" and amend author git push --force-with-lease
This check will re-run automatically after you push.
Fixes #14992.
What is the purpose of the change
Fix v3 client naming instance register, heartbeat and deregister in cluster mode. Requests to
/v3/client/ns/instancenow participate in Distro routing like the legacy/v1/ns/instance/beatflow, so instance changes land on the responsible node and can be synchronized to the cluster.Brief changelog
DistroFilterfor/v3/client/ns/*.DistroIpPortTagGeneratorreadip,port, and legacybeatfrom form body when servlet parameters are unavailable.RsInfofor v3 heartbeat so missing ephemeral instances can be auto-registered byInstanceOperator#handleBeat.ephemeral=falseis explicitly provided.Verifying this change
mvn -pl naming -am -Dtest=InstanceOpenApiControllerTest,InstanceOperatorClientImplTest,DistroIpPortTagGeneratorTest,NamingConfigTest -DfailIfNoTests=false -Dsurefire.failIfNoSpecifiedTests=false testmvn -pl naming -am -DskipTests package apache-rat:check checkstyle:check spotbugs:checkmvn -B clean compile apache-rat:check checkstyle:check spotbugs:check -DskipTestsFollow this checklist to help us incorporate your contribution quickly and easily:
[ISSUE #123] Fix UnknownException when host config not exist.