-
Notifications
You must be signed in to change notification settings - Fork 1.7k
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
Add MainResponse version override as a cluster setting. #847
Conversation
✅ DCO Check Passed d694a20722e5fb96bfc5cadf814ff7c841b14a22 |
✅ Gradle Wrapper Validation success d694a20722e5fb96bfc5cadf814ff7c841b14a22 |
✅ Gradle Precommit success d694a20722e5fb96bfc5cadf814ff7c841b14a22 |
start gradle check |
✅ Gradle Check success d694a20722e5fb96bfc5cadf814ff7c841b14a22 |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM
Made an update here to improve the commit message |
✅ DCO Check Passed eaf0cf2645ebb1af43b3088aa923c0b1fcb9104e |
✅ Gradle Wrapper Validation success eaf0cf2645ebb1af43b3088aa923c0b1fcb9104e |
✅ Gradle Precommit success eaf0cf2645ebb1af43b3088aa923c0b1fcb9104e |
start gradle check |
✅ Gradle Check success eaf0cf2645ebb1af43b3088aa923c0b1fcb9104e |
@nknize thoughts on this? |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
this looks great. I just threw one small suggestion so we don't lose the task of removing this in 2.0.0 (which is currently the main branch)
server/src/main/java/org/opensearch/action/main/TransportMainAction.java
Outdated
Show resolved
Hide resolved
This change adds a new cluster setting "compatibility.override_main_response_version" that when enabled spoofs the version.number returned from MainResponse for REST clients expecting legacy version 7.10.2. Signed-off-by: Marc Handalian <handalm@amazon.com>
✅ Gradle Wrapper Validation success 1d959ba |
✅ DCO Check Passed 1d959ba |
✅ Gradle Precommit success 1d959ba |
start gradle check |
opensearch-project#847) This change adds a new cluster setting "compatibility.override_main_response_version" that when enabled spoofs the version.number returned from MainResponse for REST clients expecting legacy version 7.10.2. Signed-off-by: Marc Handalian <handalm@amazon.com>
opensearch-project#847) This change adds a new cluster setting "compatibility.override_main_response_version" that when enabled spoofs the version.number returned from MainResponse for REST clients expecting legacy version 7.10.2. Signed-off-by: Marc Handalian <handalm@amazon.com>
just tried using this to connect Kibana with opensearch and getting the following on kibana
Note the version v1.2.4 is still being "seen" by kibana, wasnt it supposed to show something like 7.10.xxx ? When calling a GET on / to opensearch directly i see "version" : { "number" : "7.10.2", When calling a GET on "/_nodes?filter_path=nodes..version%2Cnodes..http.publish_address%2Cnodes.*.ip" which seems to be the check that kibana is doing then i get the following (note the versions 1.2.4):
|
@gittygoo What version of Kibana are you trying to run? Generally, I wouldn't recommend running any version of Kibana on top of OpenSearch, even if we are able to change the version number returned here, this is an untested combination. |
@dblock I understand that but i still need to run kibana since there is currently no package of wazuh for opensearch. Hence why i'm trying this. I tried with kibana 7.9 and 7.10. Either way i guess these numbers should be changed so that this issue is correctly applied to all relevant endpoints |
@gittygoo Reopen a new issue? I suspect that changing the version in nodes response is going to do other bad things :( Note wazuh/wazuh-dashboard-plugins#3644 specifically re:Wazuh |
Signed-off-by: Marc Handalian handalm@amazon.com
Description
This PR addresses #693 by using a cluster setting instead of a system property suggested by 814
This setting is intended to be set after a rolling upgrade via the cluster settings API.
It can be turned off with the same request with a value of false.
A default value for this setting may also be configured via opensearch.yml. If not present, it defaults to false.
Issues Resolved
#693
Check List
By submitting this pull request, I confirm that my contribution is made under the terms of the Apache 2.0 license.
For more information on following Developer Certificate of Origin and signing off your commits, please check here.