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

使用运维API修改SwitchDomain部分开关无效. #12555

Open
XiaZhouxx opened this issue Aug 27, 2024 · 3 comments
Open

使用运维API修改SwitchDomain部分开关无效. #12555

XiaZhouxx opened this issue Aug 27, 2024 · 3 comments
Labels
area/Document Category issues or prs related to document. contribution welcome

Comments

@XiaZhouxx
Copy link
Contributor

Nacos 2.4.1根据官网最新文档运维API 开关参数说明
image
使用修改SwithDomain接口
image
根据开关名称修改部分开关无效(例如defaultPushCacheMillis, distroEnable, healthCheckEnabled 等
发现是entry参数内部匹配名称和/switches返回的开关名不一致导致, 这个是文档的遗漏还是代码问题吗?

@KomachiSion
Copy link
Collaborator

部分开关的显示名字和修改时的名字有一定区别, 这个属于历史遗留问题,我记得之前有个issue说打算处理这个问题,但是后来社区同学可能忙别的去了, 没有完成,文档应该是按照GET的参数名写的, 如果有空的话可以帮忙更新一下文档,加一列来表示更新该参数时需要的entry名。

@KomachiSion KomachiSion added area/Document Category issues or prs related to document. contribution welcome labels Aug 28, 2024
@XiaZhouxx
Copy link
Contributor Author

大佬我想尝试解决这个问题可以吗?我发现还有一些问题

  1. SwithDomain很多配置没有引用已被废弃的是否可以剔除?
  2. version 2.4.1 standalone启动模式非debug=true下已经无法调用修改,这个是设计不支持吗? 2.4.0之前支持修改但是无效, 只会持久化修改后的.
  3. 是否支持batch? 我看现有代码逻辑是支持entry=batch使用SwitchDomain Json值的方式修改(例如HttpHealthParams参数逻辑上只能通过batch) 但是会绕过部分开关的值校验逻辑, 以往的文档都是没有提到的.
  4. 遗漏pushCSharpVersion的处理.

@KomachiSion
Copy link
Collaborator

SwithDomain很多配置没有引用已被废弃的是否可以剔除?

这个我记得之前有一个issue在处理, 不过具体配置是否可以移除需要更多PMC进行确认

version 2.4.1 standalone启动模式非debug=true下已经无法调用修改,这个是设计不支持吗? 2.4.0之前支持修改但是无效, 只会持久化修改后的.

我测过可以, 如果不能修改, 请自查一下是否raft选主有问题。

是否支持batch? 我看现有代码逻辑是支持entry=batch使用SwitchDomain Json值的方式修改(例如HttpHealthParams参数逻辑上只能通过batch) 但是会绕过部分开关的值校验逻辑, 以往的文档都是没有提到的.

不支持,也没有计划支持, 开关接口理论上不会经常调用,如果确实有需要修改,可多次调用。

遗漏pushCSharpVersion的处理.

可以提交PR修复

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
area/Document Category issues or prs related to document. contribution welcome
Projects
None yet
Development

No branches or pull requests

2 participants