-
Notifications
You must be signed in to change notification settings - Fork 1.8k
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
[BUG] 1.0.x have issues BWC/rolling upgrade to 2.x #4130
Comments
Thanks @peterzhuamazon for the context. Trying to understand how to learn from this, couple of questions:
That said, I agree moving forward with |
Thanks. |
Thanks @peterzhuamazon Anyway I am ok with plugins testing with 1.1.0, but we need to document it in our docs explaining why OpenSearch 1.0 is not directly upgradable to 2.x. Let me know if you guys have any other suggestions. |
I think you got this right @saratvemulapalli. Care to open an issue in docs or just PR the caveat? |
Opened up an issue[1], if its not getting attention I'll PR it. |
I have talked to @qreshi @ohltyler on this extensively.
When using
opensearch 1.0.x
artifacts doing bwc test, asodfe
is deprecated right now, we see this:This exception getting thrown due to min compat version for OS 1.0.0 is ES 6.8.0 and it thinks 2.0.0 is ES 2.0.0 which is not on or after 6.8.0 as a number.
This issue was initially raised by @cliu123 here: #3001
It later get fixed by @nknize in this commit: 7f524c7
The issue is, when we release bundle
1.0.1
we did not update the opensearch-min code but rather usingmin 1.0.0 + 1.0.0.0/1.0.1.0 plugins
to produce the bundle1.0.1
release:https://opensearch.org/versions/opensearch-1-0-1.html
https://opensearch.org/docs/1.0/opensearch/install/plugins/
https://opensearch.org/docs/1.0/dashboards/install/plugins/
This means, no
1.0.1
core standalone tarball, so @nknize fix did not get released until1.1.0
.This also means the possible concern of
no 1.0.x version available for users to rolling upgrade from 1.0.x to 2.x
.I welcome more discussion on this so we can get more information on this issue.
As of now, the bwc tests we are switching are using
1.1.0
instead of1.0.0
.cc: @CEHENKLE @bbarani @saratvemulapalli @dblock
Thanks.
The text was updated successfully, but these errors were encountered: