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

修复2.2.x版本nacos 没有/nacos/v1/ns/operator/servers 接口的问题 #399

Open
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

Accelerator96
Copy link
Contributor

operator在check nacos状态的时候使用的是/nacos/v1/ns/operator/servers接口
但是在nacos 2.2.0版本的时候已经废弃这个接口了
所以改成/nacos/v1/core/cluster/nodes来check当前nacos的装备

@paderlol
Copy link
Collaborator

能否兼容老版本,或者在Readme进行说明,否则容易引起误会

@paderlol paderlol requested a review from shenkonghui April 11, 2023 11:41
@wilsonwu
Copy link
Contributor

Yes, let's pending this PR, and move on after align.

@wilsonwu
Copy link
Contributor

能否兼容老版本,或者在Readme进行说明,否则容易引起误会

已确认,这个PR修改可以兼容老版本,老版本具有新的接口,只是对老版本接口做了废弃操作,我们的产品中已经验证过,没问题的话可以合并。

@wilsonwu
Copy link
Contributor

@shenkonghui please help to review, thanks!

@CLAassistant
Copy link

CLA assistant check
Thank you for your submission! We really appreciate it. Like many open source projects, we ask that you sign our Contributor License Agreement before we can accept your contribution.


zhuohao.li 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.

@lihongmingming
Copy link

建议兼容/nacos/v1/ns/operator/servers接口

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.

5 participants